Ranking

Hej, stworzyłem stronę z egzaminami teoretycznymi na prawo jazdy. Strona jest stylizowana na oficjalny egzamin WORD oraz posiada oficjalne pytania WORD. Całość jest dostępna w pełni za darmo. W planach jest dodanie trybu "nauka". Chętnie usłyszę od was feedback link: https://test-na-prawo-jazdy.pl

wlasnie nie dziala ta komenda wpisuje w konsoli [18:31:23] [Server thread/INFO]: Unknown command. Type "/help" for help.

command /setspawn: permission : skript.spawn permission message : &4Nie masz dostepu do tej komendy kolego! trigger: set {spawn} to location of player send "&ePoprawnie ustawiono spawn na &a%location of player%" command /spawn: trigger: if {spawn} is set: set {_x} to x coordinate of player set {_y} to y coordinate of player set {_n} to 10 loop 10 times: wait 1 seconds remove 1 from {_n} if "%{_x}%" is not equal to "%x coordinate of player%": stop loop send "&4Teleportacja przerwana,poruszyles sie!" to player else if "%{_y}%" is not equal to "%y coordinate of player%": stop loop send "&4Teleportacja przerwana,poruszyles sie!" to player stop else: send "&6Teleportacja nastapi za &8[&a%{_n}%&8] &6sekund" teleport player to {spawn} send "&6Zostales przeteleportowany na spawn!" else: send "&4Spawn nie jest jeszcze ustawiony!" by RodgerDodg3r

Napisze ktoś skrypt na Spawn i setspawn Chodzi o to że ten co ma opa i wpisze /setspawn to tam jest Spawn i każdy co wpisze /Spawn dostaje efekt niewidzialności na 9 sekund i na środku ekranu pisze na zielono Teleportacja. I po 10 sekundach pisze Przeteleportowano na zielono w te miejsce gdzie ustawiliśmy setspawn a jeżeli się ruszumy to na czerwono pisze Ruszyłes się, teleport anukowany

Jako iż zostałem poproszony o ocenę projektu Mini Essentials, zrobię to w czym jestem najlepszy, czyli krytykowanie innych nie patrząc na siebie profesjonalnym ocenianiu skryptów innych użytkowników. A więc zaczynając, jednocześnie patrząc od góry: Według mnie mało przejrzysta dokumentacja zawarta w pierwszych linijkach skryptu. Proponował bym tutaj zawarcia większej ilości odstępów, użycia znaków imitujących listę (●, –) itp. Brak możliwości edytowania skryptu bez ingerencji w kod. Przydałoby się stworzyć takie magiczne coś jak opcję, pod składnią options: Brak stosowania prefiksu w nazwie zmiennej. Zmienna {spawn} powinna się nazywać {skdanielo::spawn} w celu niekolidowania z innymi skryptami. (W permisjach dodawałeś już ten prefiks.) Stosowanie tzw. "drabinki" w której skład wchodzi if oraz else. Dla przejrzystości kodu jak i dbania o samego siebie (w tym przypadku ciebie) lepszym zastosowaniem tam, gdzie jest to możliwe stosowanie składni if oraz stop, poprawi to czytelność kodu szczególnie w miejscach, gdzie wymaga się sprawdzenia dużej ilości warunków. Zastanawia mnie linia 326, jak i sam system banów. W jakim celu używasz składni kick i zmiennych, a nie preferowanej do tego składni skriptowej. (Link) W przypadku użycia składni odpowiedzialnej za banowanie, będzie istniała integracja z pluginami które np. zbierają listę zbanowanych użytkowników itp. Nie zauważyłem tu dużo rzeczy, które by się powtarzały, jednakże warto by było używać funkcji, np. podczas wykonywania komendy /gm. Więcej rzeczy się nie przypatrzyłem, co wynika z tego, że jest to dobrze napisany skrypt. Zastosowania widzę, szczególnie na hostingach z limitem pluginów. Dziękuje, dobranoc.