Romitou

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. ^^

function itembuilder(v: integer, i: item, n: text, l: text, e: text, nbt: boolean) :: item: {_v} = 1: {_n} is empty: set {_itembuilder} to {_i} else if {_n} is not empty: set {_itembuilder} to {_i} named "%{_n}%" else if {_v} != 0: {_n} is empty: set {_itembuilder} to {_v} of {_i} else if {_n} is not empty: set {_itembuilder} to {_v} of {_i} named "%{_n}%" set {_itemlore::*} to {_l} split by "||" {_itemlore::*} is not empty: set {_line} to 1 loop {_itemlore::*}: set line {_line} of lore of {_itembuilder} to loop-value add 1 to {_line} set {_enchant::*} to {_e} split by "||" {_enchant::*} is not empty: loop {_enchant::*}: enchant {_itembuilder} with "%loop-value%" parsed as an enchantment type set {_itembuilder} to {_itembuilder} with no nbt if {_nbt} is true return {_itembuilder} Ce que ça va t'apporter c'est que tu vas désormais appeler ta fonction comme ça : set {_tonitem} to itembuilder(...) Et voici le code avec un retour ^^