[Pytanie] Czy skrypt jest dobrze napisany?

[YashePL]

//

//

//

//

//

//

//

//

//

//

//

//

//

//

//

 

Witam.

 

Bo zrobiłem skrypt na wymiane i bym chciał się zapytac czy dobrze został wykonany. Jak jest zle to byscie go poprawili?.

 

Link:

http://pastebin.com/WXJk9Cua

Z gory dzienkuje .

[Nter]

Możesz dodać argument 1 is not set:

    send "Wybierz Pomoc / Gracz / Vip"

 

Lub coś takiego poza tym jest ok 

[CusT]

Możesz dodać argument 1 is not set:

    send "Wybierz Pomoc / Gracz / Vip"

 

Lub coś takiego poza tym jest ok

 

 

Mi wywala błędy, sprawdziłem i kilka jest.

[HAMMERxPL]

http://pastebin.com/0RP1BuXy

 

Możesz dać plusiki, czy coś.

[pablos11]

@Anestic szybszy byłeś

 

 

@ CusT

Ponieważ pobierając skrypty z neta musisz sam sobie wytabować, u góry jest info o złym wytabowaniu skryptu także resetujesz TAB i stawiasz na nowo

 

Co do skryptu:

Skull - nie jest to item MC, poprawnie trzeba dodać Zombie HEAD / Player HEAD / Skeleton HEAD

Skript nie odczytuje Skull nawet po dobrym wytabowaniu

 

Kolejny błąd 'is player' nie jest prawidłowe, poprawnie jest 'if player'< jeżeli Gracz, po poprawieniu tego wszystko dało się zresetować bez błędów.

 

I ostatni błąd, nie dodałeś warunku do pustego Argumentu, po wpisaniu /wymiana nic się nie dzieje, warto dodać na końcu else który bedzie zawierał send "Brak komendy ... komendy dostępne pod /wymiana pomoc

 

Oto poprawiony i w 100% działający kod :

options:
	glowa: 64 head
	glowa1: 5 diamond
	glowavip: 32 head
	glowavip1: 10 diamond
 
command /wymiana [<text>]:
	trigger:
		argument 1 is "pomoc":
			send "&2## &7&o&m------------------------- &2##"
			send " &8» &c/wymiana gracz - &6Wymienia glowy gracza na"
			send " &8» &65 Diamentow. &cWymaga: &8(&c64 glowy gracza&8)"
			send " &8» &c/wymiana vip - &6Wymienia glowy gracza na"
			send " &8» &610 Diamentow. &cWymaga: &8(&c32 glowy gracza)"
			send "&2## &7&o&m------------------------- &2##"
		argument 1 is "gracz":
			if player has {@glowa}:
				give {@glowa1} to the player
				send " &8» &6Wymieniles glowy gracza na 5 diamentow!"
				remove {@glowa} from player
			else:
				send " &8» &6Nie posiadasz 64 glow gracza!"
		argument 1 is "vip":
			if player has permission "EzSky.vip":
				if player has {@glowavip}:
					give {@glowavip1} to the player
					send " &8» &6Wymieniles glowy gracza na 10 diamentow!"
					remove {@glowavip} from player
				else:
					send " &8» &6Nie posiadasz 32 glow gracza!"
			else:
				send " &8» &6Nie Posiadasz rangi &cVIP!"
		else:
			send " &8» &6Komendy wyemiany dostepne pod &2/wymiana pomoc"

Sam pomysł nie jest głupi lecz znalazło się kilka błędów

[HAMMERxPL]

Poprawione: http://pastebin.com/T9Gj3BG9

[YashePL]

Dzienki @pablos11 . Temat do zamknięcie.