Ranking

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.

jak masz advanced ban to troche ciezko zrobic taki skrypt bo advanced ban nie ma placeholderapi

Oni maja jeden wielki skrypt na to wszystko, lepszy jest plugin BedWars1058, mozesz go skonfigurowac jak chcesz i bedzie dzialac tak jakby byl z derpa.