Czy ktos pomoze naprawic blad?

48 minut temu, _SirSmarr napisał:

Błędy są co prawda tylko dwa, jednakże mam również kilka uwag, które pozwolą na napisanie Panu lepszego (to znaczy schludniejszego) kodu.

Zacznę od błędów.

1. Pierwszy błąd dotyczy braku możliwości interpretacji przez Skripta listenera zdarzenia 'on tab complete'.
   Zdarzenie te nie jest wspierane przez samego Skripta, należy więc skorzystać z jednego z dodatków. Osobiście mogę polecić dodatek SkBee, który obsługuje zdarzenie 'on tab complete', jednakże zaznaczam, że kod będzie wymagał niewielkich modyfikacji.
   Fragment dokumentacji z przykładem dla SkBee i tego zdarzenia znajdzie Pan na tej stronie.
2. Drugi błąd jest związany z niepoprawnym użyciem składni wyrażenia 'parsed as'. Proszę zauważyć, że może Pan je stosować jedynie do tekstu - w definicji komendy jest już wymóg, iż drugi jej argument musi być dowolną liczbą (<number> w linijce 5.), a więc Pan tak właściwie zmusza Skript do interpretowania liczby rzeczywistej jako liczbę rzeczywistą (co nie ma sensu i jest całkowicie zbędne). Gdy gracz poda drugi argument niebędący liczbą, komenda nie zadziała i na czacie wyświetli się komunikat zawarty w polu konfiguracji komendy 'usage' (linijka 7.).

A teraz przejdźmy do moich uwag.

1. Pierwsza uwaga odnosi się do pierwszego warunku w kodzie komendy i jest również powiązana z ustawieniem typu drugiego argumentu jako 'number'. Nietrudno spostrzec, że w takiej sytuacji warunek ten (linijka 11.) jest niepotrzebny, ponieważ nigdy nie zostanie spełniony. Sprawdzanie typu drugiego argumentu wykona wewnętrznie sama komenda, co objawi się (jak w 2. błędzie) pokazaniem graczowi komunikatu z pola 'usage'.
2. W mojej ocenie typem drugiego argumentu powinien być 'integer' (liczba całkowita), nie 'number' (liczba rzeczywista). Mało sensownym wydaje się możliwość ocenienia czyjejś działki na przykładowo 4.3827193. Ograniczenie ocen do zbioru {1,2,3,4,5} ma ręce i nogi, no chyba że Panu zależy na ocenach takich jak chociażby 4,5 czy też 4,8.
   Samo ograniczenie zakresu możliwych liczb w argumencie 2. wynika z warunku w linijce 15.

Gdyby miał Pan jakieś pytania, proszę śmiało je zadawać.

Pozdrawiam.