Skocz do zawartości
lightereq

LightJobs - Rozbudowany skrypt na prace.

Rekomendowane odpowiedzi

Użytkownik

Pierwsza sekunda przeglądania skryptu i już zawód!

- Zmienne globalne! DLACZEGO! DLACZEGO! Jeżeli już na siłe chcesz ich używać to po co "setować" odrazu(powoduję to lagi i syf w variables)?

 

- Em okej? 

every tick in "Spawn" or "Plots" or "Surowcowa":
    loop all players:
        if {wantedlvl::%loop-player%} is more than {@maxwantedlvl}:
            set {wantedlvl::%loop-player%} to 10

Przy dodawaniu wantedlvl skrypt może przecierz sprawdzać czy wantedlvl jest większy niż maxwantedlvl, i przy okazji jeżeli ktoś już zmieni maksymalny poziom poszukiwania np na 50 to zmienna "setuje" się na 10.

 

- Po co mieć milion zmiennych np "pracagórnik" jeżeli wystarczy jedna np. "praca" i potem sprawdzać "if {praca::%player%} is "Górnik"? 

- on any move  O NIE! POPROSTU NO NIE. 

Już dalej nie czytam tego skryptu bo jest on optymalizacyjną katastrofą. Na plus jest to że używasz "set player's current inventory's slot" zamiast "format slot"

Na przyszłość, zacznij używać yaml a tagże pomyśl na skróceniu kodu.

 

Skrypt 3/10

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Losowy awatar
Ten temat został zamknięty. Brak możliwości dodania odpowiedzi.

  • Przeglądający   0 użytkowników

    Brak zarejestrowanych użytkowników przeglądających tę stronę.

×