Aller au contenu

1 Screenshot

Description de la ressource

Bonjours, ce skript est un skript d'annonce automatique entierement configurable !

Il utilise Skript-YAML 

Vous trouverez le fichiers Configs dans skript puis le fichier AutoAnnonce.yml toute la config et dedans

Pour modifier le fichier ouvrez le skript et modifier cette ligne: 

load yaml "plugins/Skript/Configs/AutoAnnonce.yml" as "AutoAnnonce" -->  load yaml "Votre chemin d'accès" as "AutoAnnonce"
 
/!\ Après avoir changez la ligne Time: rédémarrer le serveur ! /!\

Quoi de neuf dans la version 1.0.0   Voir le changelog

Publié le

Aucune liste des modifications n'est disponible pour cette version.

  • J'aime pas 1



Retour utilisateur

Vous ne pouvez donner votre avis qu'après avoir téléchargé cette ressource.


Romitou

   3 sur 3 membres a ou ont trouvé cet avis utile 3 / 3 membres

Salut, merci pour ta ressource !
J'ai quelques petites remarques le concernant.
 

# Ligne 2:
if file "plugins/Skript/Configs/AutoAnnonce.yml" doesn't exist:

Cette ligne est inutile, car skript-yaml vérifie lui-même l’existence ou non du fichier.
Dans le cas où tu veuilles écrire dans le fichier s'il n'est pas écrit, utilise une syntaxe propre à skript-yaml.
 

# Ligne 9:
save yaml "AutoAnnonce"

Tu enregistres déjà ton fichier plus bas, il est donc aussi inutile de l'enregistrer. Si tu enregistres les données de base pour pouvoir les réutiliser juste en dessous dans le script, sache que skript-yaml garde le fichier en cache et les données resteront lisibles même si tu ne l'enregistres pas.
 

# Ligne 18:
loop 800000 times:

Si je mets mon délai à une minute, ton script deviendra non fonctionnel au bout de 800 000 secondes (soit environ 9 jours). Pour contrer cela, utilise un while, beaucoup plus adéquat dans ce contexte.
 

# Ligne 25:
	else:
		stop

Dans ce cas, il est de même inutile d'indiquer un else, sachant qu'il ne va rien s'exécuter après. Ces deux lignes ne servent pas à grand chose.
 

    set {active} to skript-yaml value "AutoAnnonce" from "AutoAnnonce"
    set {prefixmessage} to skript-yaml value "PrefixMessage" from "AutoAnnonce"
    set {temps} to skript-yaml value "Time" from "AutoAnnonce"
    set {prefix} to skript-yaml value "Prefix" from "AutoAnnonce"
    set {message1} to skript-yaml value "Message1" from "AutoAnnonce"

Certaines valeurs ne sont pas utilisées dans le reste de ton script. Transforme-les en variable temporaire, qui semble être plus adapté présentement.


Dans l'ensemble, le script est bien réalisé et bien pensé, en se basant sur skript-yaml. Cependant, j'ai fait quelques remarques pour te permettre de l'améliorer. Ces remarques ne sont pas négatives, je tiens à préciser que je fais ceci pour améliorer ton script. De ce fait, je modifierai ma note ultérieurement si le script deviendra plus optimisé de mon point de vue. Si tu as des questions concernant mes remarques, n'hésite surtout pas à me demander, je t'aiderai avec plaisir. ^^
 

Réponse de l'auteur

Merci beaucoup avis très sympatique !

  • J'aime 2

Partager cet avis


Lien vers l’avis
×
×
  • Créer...

Information importante

Nous avons placé des cookies sur votre appareil pour aider à améliorer ce site. Vous pouvez choisir d’ajuster vos paramètres de cookie, sinon nous supposerons que vous êtes d’accord pour continuer.