różne ItemShop z poziomu gracza

Witam,

Chciałbym pokazać wam mój skrypt na itemshop z poziomu gracza, Gracz musi zakupic tylko monety na stronce
Komendy:
/sklep

/addmoney

/remmoney

/money

KOD:

https://code.skript.pl/jPc0RK3x

► Skrypt całkiem, całkiem, ale mam kilka ostrzeżeń.

► 1. Dlaczego nie zastosowałeś format slot w pętle?

► 2. Format slot jest zbugowane, i da się itemy wyciągać, set player's current inventory jest lepsze, poszukaj sobie linka i się naucz

► 3. Ocena końcowa: 7/10

~ Kiedy słyszysz, że jesteś zwykłym użytkownikiem, ~Shadow

4 minuty temu, ShadowEk napisał:

► Skrypt całkiem, całkiem, ale mam kilka ostrzeżeń.

► 1. Dlaczego nie zastosowałeś format slot w pętle?

► 2. Format slot jest zbugowane, i da się itemy wyciągać, set player's current inventory jest lepsze, poszukaj sobie linka i się naucz

► 3. Ocena końcowa: 7/10

 

~ Kiedy słyszysz, że jesteś zwykłym użytkownikiem, ~Shadow

Dzięki, Zapomniałem jak sie robi pętle format slot a z tym plauer's currnet inventory se poczytam

Skrypt naprawdę w porządku, tylko popraw błędy ortograficzne
(Sprzedarz itd.)

Ocena: 7/10  

No i dobra.

Skrypt nie jest najgorszy ALE JEST WIELE TEGO TYPU.

No i ok. Podam "wady i zalety"

Wady:

1. To jest bez sensu...

on first join:
    set {monety.%player%} to 0
on join:
    if {monety.%player%} is not set:
        set {monety.%player%} to 0

2. Nie stosujesz pętli.

3. Użyłeś funkcji "format slot", która jest zbugowana

4. Po co tyle komend zamiast argumentów? Te komendy dodają tylko bezużyteczne linijki.

5. Serio? Jeżeli ktoś nie ma świata "spawn" ?

execute player command "pex user %arg 1% group add %arg 2% spawn %arg 3%"

6. Brak jakiejkolwiek konfiguracji

7. Za dużo tego typu skryptów

8. ZA dużo linijek na taki mały skrypt.

Zalety:

1. Działanie skryptu jest ok, po prostu działa

2. Nie ma błędów

Masz ocenę 5/10.

3 godziny temu, HotAngel2K17 napisał:

Za dużo tego typu skryptów

No chyba nie XD, pozatym półowa tych "wad" które wymieniłeś, były już wcześniej wymienione xD.

Co do skryptu to 7/10

13 godzin temu, HotAngel2K17 napisał:

No i dobra.

Skrypt nie jest najgorszy ALE JEST WIELE TEGO TYPU.

No i ok. Podam "wady i zalety"

Wady:

1. To jest bez sensu...

on first join:
    set {monety.%player%} to 0
on join:
    if {monety.%player%} is not set:
        set {monety.%player%} to 0

2. Nie stosujesz pętli.

3. Użyłeś funkcji "format slot", która jest zbugowana

4. Po co tyle komend zamiast argumentów? Te komendy dodają tylko bezużyteczne linijki.

5. Serio? Jeżeli ktoś nie ma świata "spawn" ?

execute player command "pex user %arg 1% group add %arg 2% spawn %arg 3%"

6. Brak jakiejkolwiek konfiguracji

7. Za dużo tego typu skryptów

8. ZA dużo linijek na taki mały skrypt.

Zalety:

1. Działanie skryptu jest ok, po prostu działa

2. Nie ma błędów

 

Masz ocenę 5/10.

tyle komend poniewaz chcialem, aby sie jak najmniej bugowalo wyciaganie itemow z skrzynki. Kiedy robilem wszystko pod jedna komendą jako argumenty bugowalo sie owiele czesciej to wyciaganie skrzynek. Co do swiata to kazdy chyba moze sobie zmienic prawda ? P.S pisalem ze skrypt byl pisany na dany serwer, tylko i wylacznie z mysla o nim.

Trochę słabo kraść pomysł z sklepem z rangami ...

1 godzinę temu, UnFriendPL_ napisał:

tyle komend poniewaz chcialem, aby sie jak najmniej bugowalo wyciaganie itemow z skrzynki. Kiedy robilem wszystko pod jedna komendą jako argumenty bugowalo sie owiele czesciej to wyciaganie skrzynek. Co do swiata to kazdy chyba moze sobie zmienic prawda ? P.S pisalem ze skrypt byl pisany na dany serwer, tylko i wylacznie z mysla o nim.

Nie dziw się, że się bugowało gdyż robisz błędy przy robieniu inventory. Każdą funkcję format slot da się zbugować chyba, że jest bardzo dobrze zabezpieczona.

Zapoznaj się z moim poradnikiem. Napisałem go żeby nie popełniać tego typu błędów.

Pomysł w sam sobie nie jest zły, ale wiesz dopracuj go.