Skocz do zawartości

Mój skrypt na sprawdzanie gracza


GrabekYT
 Udostępnij

Rekomendowane odpowiedzi

Witam
Tak, jak w temacie napisałem skrypt na sprawdzanie graczy, prosiłbym o ocenkę :)

Wymagane dodatki: skRayFall

Przygotowany na wersje: 1.16.1
Co zawiera ten skrypt ?

  1. automatyczne banowanie gracza na ip
  2. wysyłane automatycznie wiadomości
  3. zablokowanie komend podczas sprawdzania
  4. otwierany ekwipunek z możliwością przyznania się
  5. napis na cały ekran, że jest się sprawdzanym

Skrypt: 

 

 

Edytowane przez xNightPower_
Nieaktywny

Umieszczam długi kod w serwisie code.skript.pl

Odnośnik do komentarza
Udostępnij na innych stronach

1. Większość 'stop' w kodzie jest niepotrzebnych.

2. Kod zawiera okropnie zagnieżdżone conditionale. Zalecam jak najszybciej pozbyć się tego nawyku.

# Zamiast robić coś takiego:

if arg 1 is set:
	if arg 2 is set:
		if arg 3 is set:
			# Dalszy kod
		else:
	else:
else:

# Lepiej robić to w ten sposób:

if arg 1 is not set:
	stop
if arg 2 is not set:
	stop
if arg 3 is not set:
	stop
# Dalszy kod	

3. Ustawiasz sloty w zbyt obszerny sposób.

set {_i} to 0
loop 27 times:
	set player's current inventory's slot {_i} to black stained glass pane named " "
	add 1 to {_i}

Możesz to zrobić o wiele prościej:

set slot (integers between 0 and 26) of player's current inventory to black stained glass pane named " "

 

Odnośnik do komentarza
Udostępnij na innych stronach

3 minuty temu, Kormic napisał:

1. Większość 'stop' w kodzie jest niepotrzebnych.

2. Kod zawiera okropnie zagnieżdżone conditionale. Zalecam jak najszybciej pozbyć się tego nawyku.


# Zamiast robić coś takiego:

if arg 1 is set:
	if arg 2 is set:
		if arg 3 is set:
			# Dalszy kod
		else:
	else:
else:

# Lepiej robić to w ten sposób:

if arg 1 is not set:
	stop
if arg 2 is not set:
	stop
if arg 3 is not set:
	stop
# Dalszy kod	

3. Ustawiasz sloty w zbyt obszerny sposób.


set {_i} to 0
loop 27 times:
	set player's current inventory's slot {_i} to black stained glass pane named " "
	add 1 to {_i}

Możesz to zrobić o wiele prościej:


set slot (integers between 0 and 26) of player's current inventory to black stained glass pane named " "

 

Dzięki :)

Odnośnik do komentarza
Udostępnij na innych stronach

Według opinii wielu osób (w tym mnie), ten skrypt jest jedynie zmodyfikowaną wersją tego przedstawionym we wspomnianym wcześniej filmiku:

Godzinę temu, Skillek napisał:

Witam, skrypt nie należy do ciebie. 

 

Nie można nie wspomnieć o wartości dodanej w modyfikacjach (np. komenda /przyznanie). @GrabekYT, nie ma w tym nic złego - zważywszy na fakt, że szkodliwość tego "plagiatu" nie jest duża, dotyczy drobnego skryptu, a ty jesteś początkującym skrypterem. Życzę miłej przygody ze Skriptem i czekam na więcej!

Złe jest jednak to, że podkreślasz swoje rzekome autorstwo ("Mój skrypt", "napisałem skrypt"). Jeśli tego nie zmienisz, może to skutkować konsekwencjami (moderatorskie ostrzeżenia czy tymczasowa blokada dostępu do forum).

Odnośnik do komentarza
Udostępnij na innych stronach

Nieaktywny
Ten temat został zamknięty. Brak możliwości dodania odpowiedzi.
 Udostępnij

  • Ostatnio przeglądający   0 użytkowników

    • Brak zarejestrowanych użytkowników przeglądających tę stronę.
×
×
  • Dodaj nową pozycję...