[Skrypt] Szybkie pytanie

[ivall]

Tym razem jakiś lepszy skrypt, aktualnie mój największy (poświęciłem na niego ok. półtora godziny)

Jest to skrypt umożliwiający zrobienie szybkiego pytania

Użycie:

/spstworz <poprawna odpowiedź> <pytanie> - tworzy nowe szybkie pytanie, wymagana permisja "sp.use"

/sp pomoc - pokazuje dostępne komendy

/sp <odpowiedź> - odpowiada na szybkie pytanie

Jak to działa: tworzycie szybkie pytanie, pytanie jest wyświetlane na czacie i gracze mają 15 sekund na odpowiedzenie poprzez /sp <odpowiedź>, po 15 sekundach szybkie głosowanie się kończy, a gracze którzy odpowiedzieli poprawnie otrzymują item.

Aby zmienić dostawany item za poprawną odpowiedź trzeba w options( ) zmienić te golden apple na co tam chcecie 

Wymagania: tylko skript, ale to raczej logiczne

Kod: https://pastebin.com/bMsZ5QfP

Proszę o code-review i o informację jak można polepszyć optymalizację ?

 

[AlexsanderARG]

No to powiem tak, skrypt banalny i nie wyobrażam sobie, jak mogłeś pisać go przez półtorej godziny (może po prostu ja za szybko robię skrypty).

 

Według mnie składnia ma błędy, otóż nie musisz używać

else if arg 2 is not set:

Mały przykład używania if i else

if player has permission "":
	//Kod który ma się wykonać jeżeli gracz ma uprawnienia
else:
	//Kod który ma się wykonać jeżeli gracz nie ma uprawnień

 

I teraz tak, czemu używasz 

permission: sp.use
permission message: &8SP » &c&oBrak permisji

Skoro możesz użyć przykładu z powyżej

if player has permission "sp.use":
	//kod
else:
	send "&8SP » &c&oBrak permisji"
	stop

 

 

Co do optymalizacji, jeżeli zastosujesz się do porad wyżej, to będzie dobrze. Natomiast, nie wiem, czy ten skrypt działa, ponieważ nie podoba mi się ten fragment

if {odpowiedzi::player} is arg 1:
                send "&8SP » &a&oGratulacje, odpowiedziales poprawnie!" to player
                give {@item} to player

Zaraz sprawdzę działanie ? 

 

Skrypt działa

[KaPeGie]

3 godziny temu, AlexsanderARG napisał:

No to powiem tak, skrypt banalny i nie wyobrażam sobie, jak mogłeś pisać go przez półtorej godziny (może po prostu ja za szybko robię skrypty).

 

Według mnie składnia ma błędy, otóż nie musisz używać

else if arg 2 is not set:

Mały przykład używania if i else

if player has permission "":
	//Kod który ma się wykonać jeżeli gracz ma uprawnienia
else:
	//Kod który ma się wykonać jeżeli gracz nie ma uprawnień

 

I teraz tak, czemu używasz 

permission: sp.use
permission message: &8SP » &c&oBrak permisji

Skoro możesz użyć przykładu z powyżej

if player has permission "sp.use":
	//kod
else:
	send "&8SP » &c&oBrak permisji"
	stop

 

 

Co do optymalizacji, jeżeli zastosujesz się do porad wyżej, to będzie dobrze. Natomiast, nie wiem, czy ten skrypt działa, ponieważ nie podoba mi się ten fragment

if {odpowiedzi::player} is arg 1:
                send "&8SP » &a&oGratulacje, odpowiedziales poprawnie!" to player
                give {@item} to player

Zaraz sprawdzę działanie ? 

 

Skrypt działa

Wszystko okej, tylko nie rozumiem jednej rzeczy. Dlaczego mowisz, ze uzywanie "if player has permission" jest lepsze od "permission" skoro to pierwsze robi się tylko wtedy kiedy gracz ma permisje, a to drugie gdy gracz ma permisje lub consola jest senderem.

[AlexsanderARG]

@KaPeGie

 

Nigdzie nie napisałem, że jest lepsze, ale według mnie jest bardziej intuicyjne.