Aller au contenu
  • J'aime 1

Review Averages

3.0 out of 5 stars
0%
0%
100%
0%
0%

User Feedback

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


FyRoOxX

   2 sur 6 membres a ou ont trouvé cet avis utile 2 / 6 membres

Bonjour,

Ton code m'a l'air claire mais si néanmoins il n'est pas optimisé : (-1 pts)

- Tu as mis 5 "on load" alors que tu peux tout mettre en un seul,

- Les lignes (100 - 108) et (111 - 119) sont les mêmes, pourquoi ne pas les combinés ?

- Tu as également mis plusieurs "on command"

- Le "on skript load" = "on load"

- Ensuite en mettant "on death of player:" au lieu de "on death" tu précises que c'est uniquement à la mort d'un joueur, tu économises une ligne "if victim is a player:"

Ensuite il y a quelques erreurs de codes qui peuvent nuir au fonctionnement de ton skript dans certaines versions (Au dela de Skript2.2-Dev27) :

- "if attacker is player" > "if attacker is a player": pareil pour "if victim is player" > "if victim is a player"

Et pour éviter quelques petites choses inutiles: (-1 pts)

Ceci:

if arg 1 is "global":
if arg 1 is "console":
if arg 1 is "joueur":
if arg 1 is not "joueur" or "global" or "console":

Remplace le par ceci:
if arg 1 is "global":
else if arg 1 is "console":
else if arg 1 is "joueur":
else:
Ca t'évite de mettre "joueur or global or console" surtout quand tu as beaucoup d'arguments.

PS: Le code est intéréssant, mais pas très utile, comme je l'ai dit très récémment à une personnes ayant fait un code similaire, les logs de ce genre peuvent ètre activé dans les options du serveur (spigot.yml).

Cordialement,

FyRoOxX


Lien vers l’avis



Autres ressources de _FireMan23_


Retour utilisateur

×
×
  • 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.