luckyblocks [Skrypt] Skrypt na Lucky Blocki!

Cześć! Tak jak w tytule zrobiłem skrypt na lucky bloki! 
Nigdzie nie widziałem tak bardzo rozbudowanego skryptu na Lucky Blocki.

Wymiagania!
∘ Najlepiej najnowszy skript - skUnity - skUnity Downloads
∘ Najlepiej najnowszy SkQuery - Releases · SkQuery/SkQuery · GitHub

W skrypcie znajduje się:
∘ Komenda na dodawanie graczom lucky blocka! - /luckyblock give i /luckyblock give all
∘ Możliwość zmiany dropu i dodatkowego dropu w configu.
∘ Permisja do lucky blocka! - luckyblock

Wersja 1.0.0

Cytat

Link do wersji 1.0.0 na pastebin:
 Link niedostępny!

Wersja 1.8.0

Cytat

Link do wersji 1.8.0 na code.skript.pl:
 Link niedostępny!
Dodano:
∘ Nic nie dodano, ale zaszły zmiany w configu.

Wersja 2.0.0

Cytat

Link do wersji 2.0.0 na code.skript.pl:
 Link niedostępny!
Dodano:
∘ Naprawiono błąd z wiadomościami przy otwieraniu lucky blocka!
∘ Dodano komendę na dawanie wszystkim graczom na serwerze lucky blocka - /luckyblockall
 - Permisja - give.luckyblock.all

Wersja 2.0.1

Cytat

Link do wersji 2.0.1 na code.skript.pl:
 Link niedostępny!
Dodano:
∘ Dodano małą łatkę do wersji 2.0.0

Wersja 2.5.0

Cytat

Link do wersji 2.5.0:
 Link niedostępny!
Dodano:
∘ Możliwość wyboru ilości dawanych lucky blocków dla danego gracza. - /luckyblock <nick> <ilość>
∘ Dodano tez kilka małych poprawek!

Wersja 3.0.0

Cytat

Link do wersji 3.0.0:
 Link niedostępny!
Dodano:
∘ Help gui! - /luckyblock help
∘ Zmieniono wszystkie komendy!
 - Aby dać wszystkim graczom na serwerze lucky blocka wpisujemy
  /luckyblock give all
 - Aby dać danemu graczu lucky blocka wpisujemy
  /luckyblock give [gracz] [ilosc]

Wersja 4.0.0

Cytat

Link do wersji 4.0.0:
 Link niedostępny!
Dodano:
∘ Zmieniono blok lucky blocka na gąbkę.
∘ Dodano dźwięk przy stawianiu lucky blocka
∘ Usunięto błędy ortograficzne oraz błędy stylistyczne (nadal nie ma polskich znaków)
∘ Naprawiono help gui.
∘ Uprościłem kod.
∘ Zmieniono niszczenie lucky blocków (break) na zamienianie je w powietrze (set to air).

Wersja 4.3.0

Cytat

Link do wersji 4.3.0:
 Link niedostępny!
Dodano:
∘ Zmieniono muzykę przy otwiieraniu lucky blocka
∘ Naprawiono błąd ze zwykłymi gąbkami
∘ Zmieniono wiadomości z chatu na wiadomości action bar'u.
∘ Dodano prefixy wiadomości.

Wersja 4.3.2

Cytat

Link do wersji 4.3.0:
 Link niedostępny!
Dodano:
∘ Uproszczono kod.

Wersja 5.0.0

Cytat

Link do wersji 5.0.0:
 Link niedostępny!
Dodano:
∘ Dodano komendę "/luckyblock kod" - działa to w ten sposób, że na chacie pokazuje
się 10 cyfrowy kod który należy przepisać aby dostać lucky blocka!

Wersja 6.0.0

Cytat

Link do wersji 6.0.0:
 code.skript.pl - kod PcPiO9qt
Dodano:
∘ Przy otwieraniu lucky blocka pojawiają się particle.
∘ Zmieniono lore(opis) lucky blocka.
∘ Dodano wysyłanie się kodów na lucky blocki co 1h.

Jestem otwarty na propozycje!
Jeżeli znajdziecie błąd piszcie w odp!

Ten temat został przeniesiony.

1. Dłuższe kody wstawiamy na serwis code.skript.pl.

2. Sekcja 'on load' jest zdecydowanie zbyt długa. Zamiast czyścić i na nowo dodawać przedmioty do listy, lepiej jest ją po prostu ustawić.

set {drop::*} to iron ingot, gold ingot and obsidian

3. Nie twórz tak prostych nazw zmiennych, gdyż jest szansa na to, iż będą kolidować z innymi zmiennymi na serwerze osoby korzystającej z twojego skryptu. Już o wiele lepiej byłoby użyć listy:

{luckyBlock::drop::*}

4. Linijka 20. w skrypcie jest zbędna, gdyż i tak w linijce 21. sprawdzasz czy narzędziem gracza jest bedrock.

5. W linijce 28. zamiast ustawiać blok jako powietrze, lepiej jest anulować event. Ustawianie bloku jako powietrze potrafi niejednokrotnie bugować graczy i same bloki.

6. Napisałeś, aby nie zmieniać nic w komendzie, a przecież tam jest ustawiona permisja komendy, wiadomości itd.

7. Ze względu na to, że typem argumentu nr 1 jest 'player', nie ma sensu sprawdzać czy jest offline, gdyż ten typ wymaga gracza obecnego na serwerze. Jeżeli chcesz sprawdzić, czy podany gracz jest offline, jako typ argumentu nr 1 użyj 'offline player'.

Co do samego skryptu, przydałaby się jakaś mała sekcja konfiguracyjna pozwalająca chociażby ustalić czym ma być Lucky Block czy też jakie mają być wiadomości wysyłane w skrypcie.

Napisałeś, że nie widziałeś zbyt wielu skryptów na Lucky Block, a ja ci powiem, że poszukałem 2-3 minuty i znalazłem z 30 takich skryptów.

Wersja 1.8.0 już jest!

Kolejna wersja 2.0.0 już wyszła!!

Wersja 3.0.0 już powstała!

Wersja 4.0.0 weszła na rejony XD!

W dniu 30.12.2020 o 20:31, pation napisał:

nie dostępny

jak już to niedostępny

1 minutę temu, CoFFeIN04 napisał:

jak już to niedostępny

Co już nie dostępne?

Jeżeli chodzi ci o starsze wersje to nie ma do nich linków ponieważ wszystko robie na jednym, po prostu edytuje stary

Teraz, pation napisał:

nie dostępne

piszę przecież, że pisze się niedostępny

17 godzin temu, CoFFeIN04 napisał:

piszę przecież, że pisze się niedostępny

A okej  Już poprawiam!

Wersja 4.3.0 już została dodana i naprawiono błędy!

1. Nie lepiej options:? zamiast

    set {prefix} to "&9&l[&b&lLuckyBlocki&9&l] &9&l>>"
    set {errorprefix} to "&9&l[&b&lError&9&l] &9&l>>"
    set {helpprefix} to "&9&l[&b&lHelp&9&l] &9&l>>"

2. 

    add iron ingot to {Lucky::drop::*}
    add gold ingot to {Lucky::drop::*}
    add obsidian to {Lucky::drop::*}
    add emerald to {Lucky::drop::*}
    add diamond to {Lucky::drop::*}

łatwiej:

set {Lucky::drop::*} to iron ingot, gold ingot, obsidian, emerald and diamond

zobacz ile mniej to zajmuje

3.

on place:
   if event-block is sponge named "&6&lLucky Block":
        if player's tool is sponge named "&6&lLucky Block":

nie łatwiej?

on place of sponge:
	if name of player's tool is "&6&lLucky Block"

4. Tutaj nie powinieneś użyć może zmiennej tymczasowej?

            set {luckydrop} to random element of {Lucky::drop::*}
            drop {luckydrop} at event-block

takiej z _ na początku nazwy 

5. Po co pętla,

            loop 9 times:
                set player's current inventory's slot ((loop-number)-1) to {@Gui-BackGround}

skoro można to zrobić tak:

            set player's current inventory's slot (integers between 0 and 8) to {@Gui-BackGround}

6. Zamiast 3 razy anulować event, zrób to raz na początku eventu po spełnieniu warunków.

        else:
            cancel event

Powyższe rady ulepszą twój kod i będzie on bardziej optymalny

27 minut temu, CoFFeIN04 napisał:

1. Nie lepiej options:? zamiast

    set {prefix} to "&9&l[&b&lLuckyBlocki&9&l] &9&l>>"
    set {errorprefix} to "&9&l[&b&lError&9&l] &9&l>>"
    set {helpprefix} to "&9&l[&b&lHelp&9&l] &9&l>>"

2. 

    add iron ingot to {Lucky::drop::*}
    add gold ingot to {Lucky::drop::*}
    add obsidian to {Lucky::drop::*}
    add emerald to {Lucky::drop::*}
    add diamond to {Lucky::drop::*}

łatwiej:

set {Lucky::drop::*} to iron ingot, gold ingot, obsidian, emerald and diamond

zobacz ile mniej to zajmuje

3.

on place:
   if event-block is sponge named "&6&lLucky Block":
        if player's tool is sponge named "&6&lLucky Block":

nie łatwiej?

on place of sponge:
	if name of player's tool is "&6&lLucky Block"

4. Tutaj nie powinieneś użyć może zmiennej tymczasowej?

            set {luckydrop} to random element of {Lucky::drop::*}
            drop {luckydrop} at event-block

takiej z _ na początku nazwy 

5. Po co pętla,

            loop 9 times:
                set player's current inventory's slot ((loop-number)-1) to {@Gui-BackGround}

skoro można to zrobić tak:

            set player's current inventory's slot (integers between 0 and 8) to {@Gui-BackGround}

6. Zamiast 3 razy anulować event, zrób to raz na początku eventu po spełnieniu warunków.

        else:
            cancel event

 

Powyższe rady ulepszą twój kod i będzie on bardziej optymalny

Dzięki za pomoc już wprowadzone zmiany {:

Zauważyłem, że options źle używasz https://wiki.skript.pl/Opcje (tzn nie zmieniłeś zmiennych na options)

4 godziny temu, CoFFeIN04 napisał:

Zauważyłem, że options źle używasz https://wiki.skript.pl/Opcje (tzn nie zmieniłeś zmiennych na options)

W nowej wersji tego nie zmieniłem, ale postaram się trochę poduczyć i to zrobić.

Nowa wersja 5.0.0 już dostępna