Classement

Bonjour, un add-on très bien réalisé et qui apporte de belles fonctionnalités ! Je souhaite si possible aider à l'améliorer selon mes connaissances acquises lors de la réalisation d'un add-on : Utiliser des expressions propriétaires lorsqu'il est possible Par exemple, pour l'expression ExprClientIP, je pense qu'il serait préférable d'utiliser une propriété, c'est-à-dire : [the] <value> of <owner> <owner>'s <value> Ici, le propriétaire étant clientsocket et la valeur l'IP. Pour cela tu peux utiliser une SimplePropertyExpression ainsi que la méthode register. Améliorer les méthodes toString Afin de donner plus d'informations aux utilisateurs lorsque Skript utilisera cette méthode, je pense qu'il faudrait inclure les valeurs de l'expression directement dans le toString. Par exemple avec ExprServerPort : private Expression<AdaptServerSocket> server; # ... </> @Override public @NotNull String toString(final @Nullable Event e, final boolean debug) { return "get server's port"; } # Pourrait devenir : @Override public @NotNull String toString(final @Nullable Event e, final boolean debug) { return "get " + server.toString(e, debug) + "'s port"; } Possibles problèmes avec les patterns Par exemple avec le pattern de CondSocketConnected, il est de : %socket%[ is|'s] connect[ed] Les parenthèses signifient qu'il faut faire un choix entre plusieurs propositions, et les crochets représentent l'optionnalité de certaines parties de la syntaxe. Si tu souhaites donner le choix entre is et 's aux utilisateurs mais que ces choix soient optionnels, tu devrais utiliser : [(is|'s)]. De plus, Skript gère les choix optionnels et les espaces, il n'est pas nécessaire d'inclure les espaces dans les optionnalités. Encore une fois ça n'engage que moi et ma façon de voir les choses, mais la syntaxe finale ressemblerait plutôt à : %socket% [(is|'s)] connect[ed] L'utilisateur sera toujours dans la possibilité d'utiliser if %socket% connected:.