Skrypt na czat | Skrypty

[pan_krewetka]

Siemka, mam do was pewne pytanko, dopiero uczę pisać się skrypty ale jakoś daje sobie radę. Moje pytanie brzmi:
1. Jakie poradniki polecacie dla początkujących ?

I drugie:
2. Co mógłbym poprawić w tym skrypcie:

 

command /chat [<text>]:
    permission: chat.admin
    permission message:&8>> &4Nie masz dostepu do ustawien czatu !!
    trigger:
        if arg 1 isn't "on" or "off" or "clear":
            send "&8&l>> &5Poprawne uzycie to &d/chat <on/off/clear>"
            stop
        if arg is "on":
            send "&3Czat zostal pomyslnie &awlaczony &3!!"
            broadcast "&8&l>> &3Chat zostal &awlaczony, &3przez &b%player%&8&l<<"
            set {chat.status} to true
        if arg is "off":
            send "&3Czat zostal pomyslnie &cwylaczony &3!!"
            broadcast "&8&l>> &3Chat zostal &cwylaczony, &3przez &b%player%&8&l<<"
            set {chat.status} to false
        if arg 1 is "clear":
            loop all players:
                loop 100 times:
                    send "" to loop-player
            broadcast "&8&l>> &3Chat zostal wyczyszczony przez &b%player% &8&l<<"
            
on chat:
    if player doesn't have permission "chat.pisanie":
        if {chat.status} is false:
            send "&8&l>> &7Chat jest aktualnie zablokowany."
            cancel event
            stop

I zanim zaczniecie hejtować, tak wiem, że pewien poradnik opisuje jak zrobić dokładnie taki skrypt. Ale ja go trochę pozmieniałem

[AlexsanderARG]

Żeby nie było, od razu zaznaczam, że nie będę hejtować, ale wyrażać szczerą i w miarę możliwości merytoryczną opinię. Będę też wyrażać tylko i wyłącznie swoje zdanie, więc jeżeli Ci to nie pasuje, nie czytaj.

 

Więc tak, skrypt niezwykle prosty oraz (moim zdaniem) lekko niepotrzebny. Jak dla mnie skrypt jest napisany źle, nie w kontekście działania, a w kontekście składni. Nie pasuje mi jedna rzecz, a dokładnie to

permission: chat.admin
permission message:&8>> &4Nie masz dostepu do ustawien czatu !!

Jakbym ja to pisał, zrobiłbym to tak

if player has permission "chat.admin":
	- kod jeżeli ma uprawnienie
else:
	- kod jeżeli uprawnienia nie ma

Moim zdaniem jest to ładniejsze, oraz bardziej intuicyjne.

 

Drugą kwestią jest to, że używasz brodcast, zamiast send.

 

No i teraz to, nigdy bym nie wpadł na to, że można to tak zrobić, no ale w moim mniemaniu jest to totalnie błędne

loop all players:
	loop 100 times:
		send "" to loop-player

Zamiast tego, lepiej użyć

loop 100 times:
	send " " to all players
send "Chat został wyczyszczony" to all players

 

 

A co do tego, czy polecam jakieś poradniki, no to nie, nie polecam. Jedyne co mogę polecić, to zbiór eventów oraz różnych takich rzeczy http://en.njol.ch/projects/skript/doc