[EconomySK] - Seria Skryptów która stworzy ekonomię serwera od podstaw!

Witajcie!

Wraz z @Hiri12 postanowiliśmy na bazie jego RynekSK stworzyć dla was całą serię skryptów!
Celem EconomySK jest Stworzenie od podstaw całej ekonomii serwera w schludnej jednakowo wyglądającej postaci.

Na start Serię rozpoczynają 4 skrypty:

EconomyCoreSk:
Skrypt dodający komendy typu dodawanie/odejmowanie kasy, przelewy itp oraz topke hajsu!
Kod zawiera wszelkie rzeczy które łączą całą serię i będzie na bieżąco aktualizowany,

aby zapewnić jak najlepsze osiągi oraz schludny wygląd wszystkich skryptów z serii.

RynekSK:
Skrypt który doskonale znacie w odświeżonej wersji z dodatkowymi opcjami oraz usuniętymi bugami :)

AnvilGuiSearch:
Dodatek do RynekSK dodający opcje szukania w GUI kowadła

SklepSK:

Skrypt dodający AdminShop w którym Admin może dodać przedmioty wpisując
/sklep dodaj <cena> <cena> trzymając item w ręku


Linki do poszczególnych Skryptów:
EconomyCoreSk:

kod zawiera placeholder %economy_balance% który jest obsługiwany przez inne pluginy
jeżeli go nie potrzebujesz i nie posiadasz addona Skript-placeholders usuń ostatnią sekcję kodu!

RynekSK oraz AnvilGuiSearch:

SklepSK:

Jakie funkcje posiada EconomyCoreSk?
Komendy:
/mybalance - pokazuje stan konta gracza
/skeconomy <gracz> <add/set/remove> <ilość> - komenda pozwalająca dodoać/ustawić/odjąć kwote graczowi w niej podanemu
/transfermoney - komenda pozwalająca na przelewy kasy między graczami
/topmoney - topka hajsu - w opcjach jest możliwość włączenia topki w GUI

Jakie funkcje będą jescze dostępne?
W kolejnej aktualizacji dodam Gui które będzie otwierało Rynek SKlep oraz inne dodane skrypty z Serii

Miałem iść spać, bo do roboty trzeba wstać, ale zdaje się, że noc nieco mi się przeciągnie.

Po pierwsze, komenda /skeconomy jest bardzo brzydko napisana, bo zostały w niej powielone warunki sprawdzające czy argumenty zostały podane. Można przecież ją skrócić do następującej postaci:

if arg 1 or arg 2 or arg 3 is not set:
	send "..."
	stop
if arg 2 is "add":
	# [...]
else if arg 2 is "set":
	# [...]
else if arg 2 is "remove":
	# [...]

O wiele ładniej, nieprawdaż? Zaskoczę Was, bo można jeszcze prościej to załatwić za pomocą pola 'usage' w komendzie i usunięcia opcjonalności argumentów (nawiasy kwadratowe). Wtedy pierwszy warunek staje się całkowicie zbędny.

Po drugie, komenda /transfermoney posiada ten sam problem z niepotrzebnie opcjonalnymi argumentami, których istnienie i tak dalej jest sprawdzane w kodzie. Co więcej, warunki w linijkach 46 i 59 nie mają żadnego sensu, ponieważ same typy argumentów wymagają aby podany gracz był na serwerze (typ 'player' oznacza gracza obecnego na serwerze) i żeby podany drugi argument był liczbą.

Po trzecie, sortowanie potencjalnie dużej listy na każde zawołanie gracza jest bardzo nieodpowiedzialne i generuje niepotrzebne obciążenie dla serwera. Lepszym rozwiązaniem będzie skorzystanie z jakiegoś periodicala, który okresowo sortowałby tę listę ze stanami kont graczy i tworzył na ich podstawie toplistę zapisywaną do jakiejś listy globalnej. Jej zawartość byłaby następnie pobierana przez komendę kiedy to tylko potrzebne.
Zalecam sprawdzić skrypt PyraTop REBORN (autorstwa mojego dobrego przyjaciela @PanMaruda), który można znaleźć na tym forum. Służy on właśnie do tworzenia toplisty na podstawie dowolnej listy zmiennych.

Pozdrawiam.