Do poprawy tu jest sporo
1. Zmiennych z kropką się nie używa, chyba że zapisujesz pojedyncze wartości. Powinno być {sethome1::%player%}
{sethome1.%player%}
2. Taką zmienną to w ogóle widzę pierwszy raz, nie ma ona sensu. Powinno być {lokacjahome1::%player%}
{lokacjahome1.::.%player%}
3. Zamiast tworzyć zmienne z innymi nazwami mogłeś to zrobić w jednej, jak np poprzez {sethome::1 2 3 itd::%player%}
4. Permisje sprawdzasz po kilka razy, mogłeś zapisać to w funkcji przy uzyciu return i kod byłby krótszy
5. Nie używasz stop przy zakończeniu jednego z argumentów, aktualnie nawet jak gracz wpisze 1 to nadal skrypt bedzie sprawdzal czy arg 1 to nie jest przypadkiem 5, alternatywą dla tego jest też else if
6. Zamiast tworzyć zmienną sethome i ustawiać ją na true oraz lokacjahome i ustawiać ją na lokacje to czemu po prostu zmienną sethome nie ustawisz na lokacje i nie sprawdzisz czy zmienna jest zapelniona?
7. Tittle również można było wrzucić do funkcji, z okazji iż zawsze dodajesz pustą linijkę przed
8. Komenda home to jedna wielka betonowa sciana 5 razy powtarzasz kod (powinna być na to funkcja), nie używasz zmiennych czasowych
9. Powinno być of event-inventory zamiast player's current inventory, spami to warnem w konsoli
on inventory click:
name of player's current inventory is "Twoje homy"
clicked inventory is not player's inventory
cancel event
10. Coś takiego nie powinno występować, powinieneś tu wywoływać funkcje zamiast executowac komende
make player execute command "home 1"
11. Gui pozwala na kopiowanie/wyciaganie itemów. Powinno się je tworzyć poprzez zapisanie wszystkiego do zmiennej i na koncu jej otworzenie
open chest with 3 rows named "Czy na pewno usunąć home 1?" to player
12. Jak coś to mamy 2023 a nie 2014, format slot jest calkowicie przestarzaly
format slot 10 of player with 1 of red stained glass pane named "&cAnuluj" to run "sudo %player% home"
13. Te wszystkie gui od dołu mogły być zapisane w jednej funkcji. W jaki sposób? To ostatnie mogę pokazać.
function delhome(p: player, i: integer):
set {_} to chest inventory with 3 rows named "Czy na pewno usunąć home %{_i}%?"
set slot 10 of {_} to 1 of red stained glass pane named "&cAnuluj"
set slot 11 of {_} to 1 of red stained glass pane named "&cAnuluj"
set slot 12 of {_} to 1 of red stained glass pane named "&cAnuluj"
set slot 14 of {_} to 1 of lime stained glass pane named "&aPotwierdź"
set slot 15 of {_} to 1 of lime stained glass pane named "&aPotwierdź"
set slot 16 of {_} to 1 of lime stained glass pane named "&aPotwierdź"
open {_} to {_p}
A następnie wpisał byś jedynie delhome(player, 1), wtedy w inventory clicku trzeba dodać odczytywanie numeru z nazwy gui, kwestia 3-4 linijek