Mała pomoc

Witam mam do was pytanie czy ten skrypt jest dobrze napisany ?
 

variables:
    {ochrona::%player%} = 30
every minute:
    loop all players:
        {ochrona::%loop-player%} is not false:
            remove 1 from {ochrona::%loop-player%}
        {ochrona::%loop-player%} is 0:
            send "Koniec ochony" to loop-player
            set {ochrona::%loop-player%} to false
 
on damage:
    {ochrona::%victim%} isn't false:
        cancel event
    {ochrona::%attacker%} isn't false:
        set {ochrona::%attacker%} to false
        send "Koniec ochrony" to attacker
on join:
    {ochrona::%player%} isn't false:
        execute player command "/ochrona"
 
command /ochrona:
    trigger:
        {ochrona::%player%} is false:
            send "&cNie masz juz ochrony!"
        else:
            send "&aPozostalo ci %{ochrona::%player%}% minut ochrony!"
command /ochronaadmin <player> <text>:
    permission: Altros.*
    trigger:
        arg 2 is number:
            set {ochrona::%player-arg%} to arg 2
        else:
            set {ochroba::%player-arg"%} to false