Aller au contenu

Classement


Contenu populaire

Affichage du contenu avec la meilleure réputation depuis le 01/19/20 dans Avis

  1. 2 points
    Excellent travail, c'est vraiment un super projet, bravo!
  2. 2 points
    Wow quel travail! En plus c'est configurable, je vais prendre exemple pour le skript-yaml:)
  3. 2 points
    Il n'y a rien à dire, c'est juste de l’excellent travail !
  4. 1 point
    Ne restaure pas l'inventaire quand on quitte le mod staff
  5. 1 point
  6. 1 point
    Super addon, avec des update régulière et toujours du nouveau contenu !
  7. 1 point
    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:.
  8. 1 point
    Ce Skript sera sûrement très utile aux débutants, celui-ci vous permettra de facilement protéger les comptes ayant un grade élevé. Je ne sais pas si celui est bypassable mais étant donné qu'il utilise le code A2F, cela m'étonnerais. Quelques petites choses sont tout de même à revoir : - Améliorer l'optimisation. -> Au lieu de créer 2 commandes distinctes, une pour les ADMINS et l'autre pour les JOUEURS, ce serait mieux de tout regrouper en 1 seule commande. -> Ou alors, tout mettre dans un GUI pour une meilleure gestion de son A2F. - Améliorer la clarté du code. -> Il n'est pas très facile d'aller faire des modifications dans ton code, rien n'est ordonné, c'est très brouillon enfaite. - Ajouter un système de configuration. -> Fais cela dans un fichier .YML ou directement dans les options du Skript. Cela nous permettra de modifier les messages, ... à notre guise. En conclusion, ce Skript est génial, c'est une superbe idée. Malgré que je le trouve un peu "vide", pas beaucoup de possibilité. -------------------------------------------------------------------------------- Voici quelques notes : - Originalité : 4.2/5 - Utilité : 4.6/5 - Propreté (Clarté) : 3.1/5 - Optimisation : 4.8/5 - Total : 16.7/20 (8.35/10)
  9. 1 point
    Très très bonne idée
  10. 1 point
    Le skript et tres bien mais se pendant je me retrouve avec beaucoup d'erreur car je suspose le script et pas apdates en 1.16 ! esce-que sa serait possible d'update le skript en 1.16 svp?
  11. 1 point
    GG, super addon !
  12. 1 point
    Le skript et très bien fait Bien Jouée a toi tu a fait des très bon effort j'ai trouvé aucun bug pour le moment
  13. 1 point
    Pixel perfect ! Congrats ! saucisson
  14. 1 point
    Merci pour votre script, l'idée est plutôt bonne. J'ai très rapidement jeté un œil au code et il est visiblement très améliorable. 1. Il est inutile d'ajouter un stop s'il n'y a plus aucun code après cela. 2. Vous semblez utiliser du YAML et des variables ? 3. Vous chargez plusieurs fois le même fichier, alors qu'il est déjà chargé. 4. Vous récupérez les données du YAML pour les transformer en variables normales. C'est une perte d'intérêt. 5. Je devine qu'il faut utiliser skript-mirror au vu du code ? Je vous conseillerai de l'ajouter dans votre présentation. 6. Certaines fonctions sont inutiles. Pour rappel, le temps d'exécution d'une fonction est bien plus élevé qu'en le faisant directement dans le code. Vous pouvez donner plus d'intérêt à ces fonctions ou les supprimer. Dans l'ensemble votre code est bien construit, mais il y a quelques remarques que je vous ai faites. Cette note sera modifiée en cas de changement de votre script, n'hésitez pas à me mentionner
  15. 1 point
    Très sympathique, bravo ! Au fait, les "while" s'arrêtent toutes seules si leur condition n'est plus vérifiée, pas besoin de check manuellement
  16. 1 point
    Le /reserve est pas mal ! Faudrait juste faire en sorte de pouvoir choisir si l'on veux que sa donne des minerais ou des lingot au joueur ! Et le hammer bug un peux et ne drops pas d'item en 1.15.2 !
  17. 1 point
    C'est un excellent skript globalement. Il y a cependant un petit élément fâcheux, lorsque qu'on mine à la limite d'une zone protégée, on peut casser les bords de celle-ci. Par exemple : Si je mine pile à la limite de la safezone, les blocs protégés se casseront quand même. Edit: étant donné que cela a été corrigé, je corrige également ma note, bonne continuation !
  18. 1 point
    Super support via discord, le script ne fonctionnait pas, le créateur du script est resté 1h avec moi sur discord pour m'aider à faire fonctionner le script sur mon serveur car j'avais des erreurs liées avec ma version ( 1.8 ) . Maintenant tout est bon
  19. 1 point
    Très sympathique, interface vraiment agréable ! Pour éviter de récupérer les plugins à chaque ouverture du menu, tu pourrais les stocker directement dans une variable fixe au chargement. Je te conseille aussi de rajouter un argument texte facultatif, i.e. [<text>] dans tes commandes pour éviter un message d'erreur provenant directement de Skript lorsqu'on ajoute des arguments ^^
  20. 1 point
    Franchement super, je vais justement faire un loup-garou sur mon serveur, j'aurai bien utilisé ton script mais nous avons des rôles inédits. Honnêtement, énorme gg à toi !
  21. 1 point
    Skript très sympa ! Merci de l'avoir partagé avec la commu skript
  22. 1 point
  23. 1 point
    Le skript est, de base, super, mais ayant un problème (via une mise a jour d'un addons), j'ai demandé de l'aide sur discord et mon problème a été réglé en quelques minutes. Staff réactif sur discord, skript génial, a téléchargé !
  24. 1 point
    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. ^^
  25. 1 point
    Wow ! C'est un skript super intéressant, autant dans l'écriture que dans le rendu ultra classe in-game. Tu utilises un fonction de manière ultra efficace. Je pense que ca peut aider beaucoup de débutants pour comprendre comment cela fonctionne. Quant au rendu, comme dit plus haut, il est très classe. Après je ne sais pas si ca tourne bien si 100 personnes parlent sur un serv avec tous les spawn d'armor stand, j'admire le fait d'avoir permis des personnalisations comme le délais de message et la "censure" du chat vanilla c'est un gros taff ! Gros gg à toi, merci d'avoir pris le temps et d'avoir partagé
  26. 1 point
    Très bon skript, très complet mais j'ai remarqué que tu utilisais les GUIs de Skquery qui sont parfois buggé ou qui obligent à mettre des wait après l'ouverture du GUI ou d'utiliser des loop pour formater des slots alors qu'avec tuske tu n 'aurais pas ce problème. J'aurai mis 4,5 si j'en avais eu la possibilité.
  27. 1 point
    Le script est juste parfait ! Rien à redire et je recommande !
  28. 1 point
    Salut, Merci beaucoup ! Cela fonctionne parfaitement.
  29. 1 point
    "je ne code pas en 1.8, car il y a trop de beugs" ou "skript en 1.8 n'est pas stable et trop vieux, manque de fonctionnalités.." En quelque sortes tu viens de prouver le contraire.
  30. 1 point
    Skript simple mais efficace. Quelques remarques cette fois-ci : On ne peut pas miner vers le bas si notre tête n'est pas inclinée, de même vers le haut (ne peut être résolu). On ne reçoit rien des minerais minés (expliqué dans le descriptif) L'usure de la pioche, elle n'est comptabilisée que comme si on minait 1 bloc, mais on en casse envrion 9 (= 9 points usure). ^^ (Voici comment je pensais faire : https://gist.github.com/Romitou/e9551fd77330917086439d9bb6f5f35f) Tout ça pour dire que c'est du bon travail, le Skript est fidèle à sa description. Pour ce qui est de l'usure, peut-être que l'intérêt d'un hammer est que la durabilité soit comptée comme un bloc, je ne sais pas. Dans ce cas je modifierai ma note. ^^
  31. 1 point
    Rien à redire, c'est parfait ?
  32. 1 point
    Très bon skript, fonctionne aucun error ?
  33. 1 point
    Très sympa ! Tu peux juste optimiser les longues lignes avec 36 nombres en "format gui slot (integers from 0 to 35) ..." ?
  34. 1 point
  35. 1 point
    Bonjour, Rien à dire, le code est parfait, simple mais toujours utile, cependant tu pourrais mettre l'id du pack dans le lore... Cordialement, FyRoOxX
  36. -1 points
    le code est simple et petit mais ne fonctionne pas en 1.15.2 gg quand meme
  37. -1 points
    Bonjour, j'ai un problème quand je tape /calendrier sa m'envoie : merci de m'aider au plus vite. Bonne journée a vous
×
×
  • 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.