[⚙️ RPE - Roleplay Essentials | Economie, voitures, chat rp et plus !]

Bonsoir, ton skript me parait original et fonctionnel (Je ne vais pas pouvoir le tester pour le moment), cependant je rejoins l'idée de @KneaXy sur le fait que tu utilises beaucoup d'addon et que cela gâche un peu le skript (Mais bon on va pas en faire des caisses, les addons que tu utilises sont utilisés par beaucoup de gens, mis à part SkVault ...). Néanmoins le code est dans l'ensemble optimisé et propre, et je pense qu'avec quelques débug et mise à jours le skript pourrai arriver à un bon niveau.

Solutions alternatives aux addons (Un peu complèxe):

Je ne sais pas si tu connais l'addon skript-mirror...

C'est un addon permettant d'intégrer du Java dans tes skript, avec quoi tu pourrais supprimer WolvSK, SkVault etc...

Je te le donne au cas ou: https://forums.skunity.com/resources/skript-mirror.254/ Cela n'influe pas sur la note que je donne mais il pourrai donné un avantage a ton skript...(API par exemple)

 

[SkEmail]

Incroyable, grâce à ton outils, un nombre incroyable de possibilités s'ouvrent aux développeur!

Je te félicites

Cordialement,

FyRoOxX

[AntiDeath + Spawn]

Bonjour,

Je pense que tout a été dit, mais qu'aucune n'aide ne t'a été apporté, voilà une petit coup de pouce:

Syntax: force %player% to respawn

Cordialement,

FyRoOxX

[Gangar SkriptChatMenu Addon!]

Juste incroyable ! Bravo à se développeur, dommage qu'il n'est plus actif il fait du très bon travail !

Cordialement,

FyRoOxX

[SaymLog [LOG]]

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

[Proscrire une commande]

Bonjour,

Le code est très simpas et une bonne idée pour les personnes commencant à développer dans le monde du Skript (+1 pts), cependant je rejoinds l'avis de @Kilterra (Encore xD),

Il y a un soucis d'optimisation au niveau du "on command", je m'explique:

on command:
	loop {command.list::*}:
		if loop-value is command:
			if sender don't have permission "prc.admin":
				cancel event
				send "<red>Commande interdite" to sender
				stop

Tu pourrais simplement remplacer toutes ces lignes toutes moches par :

on command:
	if {command.list::%command%} is set:
		if sender don't have permission "prc.admin":
			cancel event
			send "<red>Commande interdite" to sender
			stop #Le stop est inutile ici, il ne sert qu'a arrèter le code, mais il s'arrète tout seul à la fin ...

Ensuite tu pourrais directement "integrer" le "/prcclear" dans le "/prc", tu gagnerais déjà 3 lignes

Et pour finir :

command /prc [<text>] [<text>]:
	permission: prc.admin
	trigger:
		if arg 1 is set: #Inutil car tu vérifie en dessous si c'est "add" donc il est forcement défini:
			if arg 1 is "add":
  				[...]
  
  #A la limite:
  command /prc [<text>] [<text>]:
	permission: prc.admin
	trigger:
		if arg 1 is not set: #Inutil car tu vérifie en dessous si c'est "add" donc il est forcement défini:
  			[CODE AIDE]
		else if arg 1 is "add":
  			[CODE AVEC ADD]
  		else:
  			make player execute "/prc"

Cordialement,

FyRoOxX

 

EDIT (1.1.0): Le code s'optimise petit à petit, et les options s'ajoutes, bravo, continue comme ça et je te promet de rajouter une étoile

[[1.10.2] ★ Minigames ★]

Bonjour,

L'idée est très originale (+1 pts)

Le code est très bien codé et l'ensemble est plutôt optimisé (+1 pts), La présention est claire et détaillée (+1 pts) et (quasiment) aucunes fautes d'orthographes  (+1 pts)

Et pour finir les jeux sont bien réalisé, bref comme uiytt, rien à dire ! (+1 pts)

 

[HydraLog]

Bonjour,

Pour moi le code est parfait et optimisé (+1 pts), en revanche le skript n'a pas de grande utilité du fait que les logs de base d'un serveur enregistre déjà toutes ces informations...

Il faut juste changer une options dans spigot.yml ...

Sinon, présentation claire et précise mais si un peu monotone ... (+1 pts)

Cordialement,

FyRoOxX

EDIT: Il est cependant vrai, que l'enregistrement personnel est intéréssant, tu pourrais essayer de la pousser un peu plus (+1 pts)

[Backpack]

Bonjour,

Rien à dire, le code est parfait, simple mais toujours utile, cependant tu pourrais mettre l'id du pack dans le lore...

Cordialement,

FyRoOxX

[SimpleWarp]

Bonjour,

Ce skript est très simple, mais toujours utiles, essaye peut-être de poussé un peu tes capacités, par exemple mettre des permissions par warp ...

En tout cas, je t'encourage pour la suite !

Cordialement,

FyRoOxX

EDIT: Essaye pour tes futur présentation, de mettre tes screenshot dans la présention et non dans le "Add screenshots".

[★ MinePay » Paiements en jeu ★]

Excelent skript, mais je pense qu'avec les prochaines conditions d'utilisations de DediPass, le skript ne sera malheureusement plus fonctionnel ...

Mais continue comme ca ! Tu fait de l'excelent travail ! Très belle présentation en plus de cela ( Très bien expliqué et simplement ) !

Voila 5 étoile pour toi !