AdisRepair - Skrypt na naprawę przedmiotu.

AdisRepair

Siemanko, od jakiegoś czasu bawie się pluginem skript, napisałem pare różnych skryptów, również z pomocą społeczności, ale dziś przychodzę ze skryptem na naprawę przedmiotu.
 

Jak działa skrypt?

Po wpisaniu komendy /napraw gracz może naprawić przedmiot, który aktualnie trzyma. Musi posiadać przedmiot do naprawy, czyli gwiazde netheru.
Komenda /dajnaprawke daje jedną naprawkę dla danego gracza.

Prosty skrypt, ale mam nadzieje że się komuś przyda.

Wersja 1.2
Kod: https://code.skript.pl/5qeT1egE

1.1 - Zamiast nazwy i lore w skrypcie dodałem config oraz uprościłem skrypt (dzięki Kormic)

1.2 - Dodany config przedmiotu oraz naprawiona 13 linijka kodu (aktualnego)
25.03 - Naprawiłem skrypt, bo były błędy

1. Ani jedno 'stop' w kodzie nie jest potrzebne.

2. W komendzie napraw powinieneś dodać parametr 'executable by: players'. Nie jest to wymagane, ale lepiej to zrobić dla spokoju duszy.

3. Zalecam coś zrobić z tymi zagnieżdżonymi warunkami. Kod przez to traci na czytelności.

# Jak ty to robisz:

if arg-1 is set:
	if arg-2 is set:
		if arg-3 is set:
			# Dalszy kod
		else:
			send "Podaj argument nr 3!"
	else:
		send "Podaj argument nr 2!"
else:
	send "Podaj argument nr 1!"		

# Jak ja bym to zrobił:

if arg-1 is not set:
	send "Podaj argument nr 1!"
	stop
if arg-2 is not set:
	send "Podaj argument nr 2!"
	stop
if arg-3 is not set:
	send "Podaj argument nr 3!"
	stop
# Dalszy kod

O wiele lepiej, prawda?

4. Nether gwiazdę wraz z nazwą i lore wypadałoby zapisać w sekcji 'options', aby nie trzeba było wszystkiego ręcznie zmieniać.

7 minut temu, Kormic napisał:

1. Ani jedno 'stop' w kodzie nie jest potrzebne.

2. W komendzie napraw powinieneś dodać parametr 'executable by: players'. Nie jest to wymagane, ale lepiej to zrobić dla spokoju duszy.

3. Zalecam coś zrobić z tymi zagnieżdżonymi warunkami. Kod przez to traci na czytelności.

# Jak ty to robisz:

if arg-1 is set:
	if arg-2 is set:
		if arg-3 is set:
			# Dalszy kod
		else:
			send "Podaj argument nr 3!"
	else:
		send "Podaj argument nr 2!"
else:
	send "Podaj argument nr 1!"		

# Jak ja bym to zrobił:

if arg-1 is not set:
	send "Podaj argument nr 1!"
	stop
if arg-2 is not set:
	send "Podaj argument nr 2!"
	stop
if arg-3 is not set:
	send "Podaj argument nr 3!"
	stop
# Dalszy kod

O wiele lepiej, prawda?

Okej, wydaje mi się, że poprawiłem według twoich zaleceń

5 minut temu, Warpedex napisał:

Okej, wydaje mi się, że poprawiłem według twoich zaleceń

Świetna robota, tylko zapomniałeś o 'stop' po 22. linijce. Chodzi o to, że nawet jeżeli argument nie będzie ustawiony, kod i tak zostanie wykonany.

Poza tym wydaje mi się, że to już będzie wersja 1.0.1. 

1 minutę temu, Kormic napisał:

Świetna robota, tylko zapomniałeś o 'stop' po 22. linijce. Chodzi o to, że nawet jeżeli argument nie będzie ustawiony, kod i tak zostanie wykonany.

Poza tym wydaje mi się, że to już będzie wersja 1.0.1. 

Dobra, wszystko zrobiłem wersja 1.1 razem z configiem

Dodałem jeszcze powiadomienie o tym do kogo wysłałeś naprawkę oraz powiadomienie dla gracza, który otrzymał tą naprawkę.
EDIT: config dla drugiej linijki lore dodany

Ja dodałbym do configu przedmiot, czyli ta gwiazda netheru tutaj, by nie trzeba było się zagłębiać w skrypt, wiem może i jest krótki, ale to od ciebie zależy jak go rozwiniesz

W dniu 22.02.2021 o 14:39, Shateq napisał:

Ja dodałbym do configu przedmiot, czyli ta gwiazda netheru tutaj, by nie trzeba było się zagłębiać w skrypt, wiem może i jest krótki, ale to od ciebie zależy jak go rozwiniesz

Zrobione

Ej a przerobisz mi tak żeby był /napraw ale trzeba mieć 10 lvl żeby go użyc

23 minuty temu, IgiBoy09 napisał:

Ej a przerobisz mi tak żeby był /napraw ale trzeba mieć 10 lvl żeby go użyc

dodaj sobie

if player has 1000 xp: