LightJobs - Rozbudowany skrypt na prace.

S

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

@MrJuliusz Zgadzam się z tobą

@lightereq Bez obrazy ale mógłbyś to umieścić w dziale nauka

Ten temat został przeniesiony do archiwum.