Skrypt na inventory click nie dziala

1 minutę temu, Misio12320 napisał:

The expression 'clicked slot' may only be used in an inventory click event (GuanYu.sk, line 257: if clicked slot is 43:')

Skrypt przeładowywuje się bardzo długo. Czy to wina tego, że pobiera on główkę z internetu czy to wina długości kodu?

Może być z długości kodu.

Popraw sobie 'clicked slot' na 'event.getRawSlot()'.

Polecam użyć Skripta do formatowania GUI, Helios zrobił o tym poradnik na forum.

reflect ci do tego zbędny - wystarczy dać 'index of clicked slot' 

@edit @Misio12320

a nie, wait bo ja nie zauważyłem kodu... więc mam pytanie - po kiego tam reflect skoro wszystko jest w skript dostępne - inventory click event, inventory name i tak dalej? po co sobie utrudniać życie skoro efekt jest ten sam ale kod czytelniejszy w skript?

5 minut temu, Kormic napisał:

Popraw sobie 'clicked slot' na 'event.getRawSlot()'.

There's no player in an on bukkit event event (GuanYu.sk, line 258: play sound "minecraft:custom.gjbc" {@Dzwiek} {Pitch} at player')
[10:56:07 INFO]: [Skript] Encountered 28 errors while reloading GuanYu.sk! - zamieniłem clicked slot, tak jak kazałeś.

5 minut temu, Kormic napisał:

Polecam użyć Skripta do formatowania GUI, Helios zrobił o tym poradnik na forum.

Dobrze. dziękuję.

2 minuty temu, PanMaruda napisał:

reflect ci do tego zbędny - wystarczy dać 'index of clicked slot'

Jedni piszą żebym zamienił SkQuery na Skript-Reflect, teraz na jakieś index. to w końcu nie wiem który jest lepszy i który ma lepszą wydajność

1 minutę temu, Misio12320 napisał:

Jedni piszą żebym zamienił SkQuery na Skript-Reflect, teraz na jakieś index. to w końcu nie wiem który jest lepszy i który ma lepszą wydajność

napisz do mnie na discordzie wieczorem to ci wytłumaczę o co chodzi z tym wszystkim.

44 minuty temu, Kormic napisał:

Polecam użyć Skripta do formatowania GUI, Helios zrobił o tym poradnik na forum.

Użyłem jego poradnika i teraz mam tak:
 

[11:34:49 ERROR]: a slot can't be set to 'Light Gray Stained Glass Pane named "Następna strona" to close then run player command "/GuanYu1" to permission "*"' because the latter is neither an item type nor an item stack (GuanYu.sk, line 61: set slot 44 of {_gui} to {@LGGP} named "Następna strona" to close then run player command "/GuanYu1" to permission "*"')
[11:34:50 ERROR]: a slot can't be set to 'Green Stained Glass Pane named "Następna strona" to close then run player command "/GuanYu2" with permission "*"' because the latter is neither an item type nor an item stack (GuanYu.sk, line 112: set slot 44 of {_gui} to Green Stained Glass Pane named "Następna strona" to close then run player command "/GuanYu2" with permission "*"')
[11:34:50 ERROR]: red stained glass pane named "Poprzednia strona" is not a number (GuanYu.sk, line 113: set slot 36 of {_gui} with Red Stained Glass Pane named "Poprzednia strona" to close then run player command "/GuanYu" with permission "*"')
[11:34:50 ERROR]: a slot can't be set to 'Green Stained Glass Pane named "Powrót do strony 1-szej" to run player command "/guanyu" to permission "*"' because the latter is neither an item type nor an item stack (GuanYu.sk, line 154: set slot 44 of {_gui} to Green Stained Glass Pane named "Powrót do strony 1-szej" to run player command "/guanyu" to permission "*"')
[11:34:50 ERROR]: a slot can't be set to 'Red Stained Glass Pane named "Poprzednia strona" to run player command "/guanyu" to permission "*"' because the latter is neither an item type nor an item stack (GuanYu.sk, line 155: set slot 36 of {_gui} to Red Stained Glass Pane named "Poprzednia strona" to run player command "/guanyu" to permission "*"')

Potrzebowałbym otwarcia strony "Następnej strony" oraz "Poprzedniej strony"...

Godzinę temu, Misio12320 napisał:

There's no player in an on bukkit event event (GuanYu.sk, line 258: play sound "minecraft:custom.gjbc" {@Dzwiek} {Pitch} at player')
[10:56:07 INFO]: [Skript] Encountered 28 errors while reloading GuanYu.sk! - zamieniłem clicked slot, tak jak kazałeś.

Godzinę temu, Kormic napisał:

W takim wypadku musisz również zamienić 'player' z 'event.getWhoClicked()'.

Godzinę temu, Misio12320 napisał:

Jedni piszą żebym zamienił SkQuery na Skript-Reflect, teraz na jakieś index. to w końcu nie wiem który jest lepszy i który ma lepszą wydajność

Tu nie chodzi o wydajność, a o to, że SkQuery powoduje problemy z wykrywaniem nazwy ekwipunku na wersjach 1.13+. Dlatego też trzeba użyć skript-reflecta, aby ominąć ten problem.

Jeszcze jedna uwaga co do ustawiania szyb w twoim skrypcie.

Możesz to zrobić o wiele prościej.

set slot (integers between 0 and 53) of {_gui} to white stained glass pane named " "

Taka linijka ustawi wszystkie 54 sloty w GUI z sześcioma rzędami jako białe szyby.

46 minut temu, Misio12320 napisał:

Użyłem jego poradnika

Na pewno?

set slot 44 of {_gui} to {@LGGP} named "Następna strona" to close then run player command "/GuanYu1" to permission "*"

To wygląda na pomieszanie SkQuery z Vanilla Skriptem, nie ma możliwości wykonania czegoś takiego, upewnij się, że na pewno dobrze przeczytałeś poradnik

25 minut temu, Kormic napisał:

set slot (integers between 0 and 53) of {_gui} to white stained glass pane named " "

 

Tak, ale w każdym z tych GUI jest coś w środku, te główki..

26 minut temu, Kormic napisał:

W takim wypadku musisz również zamienić 'player' z 'event.getWhoClicked()'.

1 godzinę temu, Misio12320 napisał:

Myślę, że Skript-Reflect usunę i skorzystam z Vanilla Skript.

19 minut temu, CoFFeIN04 napisał:

To wygląda na pomieszanie SkQuery z Vanilla Skriptem, nie ma możliwości wykonania czegoś takiego, upewnij się, że na pewno dobrze przeczytałeś poradnik

Wiem, tam sa jakieś ukryte komentarze z "W SKRIPT VANILLA GUI MOŻNA WYCIĄGAĆ SLOT0 SIĘ URUCHOMI" itd. nie widze jednak by zrobić "stronę 2".

1 minutę temu, Misio12320 napisał:

Tak, ale w każdym z tych GUI jest coś w środku, te główki..

Ja wiem, ja wiem. Dlatego też napisałem jedynie o ustawianiu szyb. Sam widzisz, że można to znacznie skrócić.

2 minuty temu, Misio12320 napisał:

Myślę, że Skript-Reflect usunę i skorzystam z Vanilla Skript.

Nie usuwaj tego dodatku, jest niezwykle przydatny. Sądzę, że jest najlepszym dodatkiem do Skripta.

Jeżeli masz serwer na wersji 1.13 lub nowszej i chcesz zostawić SkQuery, tym bardziej nie powinieneś rezygnować z tego dodatku. Nie będziesz w stanie sprawdzać nazwy GUI.

4 minuty temu, Misio12320 napisał:

Wiem, tam sa jakieś ukryte komentarze z "W SKRIPT VANILLA GUI MOŻNA WYCIĄGAĆ SLOT0 SIĘ URUCHOMI" itd. nie widze jednak by zrobić "stronę 2".

Komentarze w poradniku Heliosa?

Akcje dla tych główek (po ich kliknięciu) zdefiniuj w evencie 'on inventory click'.

Mógłbyś jeszcze wyjaśnić kwestię ze stronami w GUI? Z tym również masz problem i potrzebujesz naprowadzenia jak to zrobić?

9 minut temu, Kormic napisał:

Ja wiem, ja wiem. Dlatego też napisałem jedynie o ustawianiu szyb. Sam widzisz, że można to znacznie skrócić.

14 minut temu, Misio12320 napisał:

Rzeczywiście, przepraszam że taki nie miły najazd zrobiłem. Dziękuje przyda się.

10 minut temu, Kormic napisał:

Mógłbyś jeszcze wyjaśnić kwestię ze stronami w GUI? Z tym również masz problem i potrzebujesz naprowadzenia jak to zrobić?

Poprosiłbym tylko o to. Chodzi mi o następną stronę tak jak na filmiku...

8 minut temu, Misio12320 napisał:

Rzeczywiście, przepraszam że taki nie miły najazd zrobiłem. Dziękuje przyda się.

Nie uznałem tego nawet za najazd, więc nie ma sprawy.

17 minut temu, Misio12320 napisał:

Poprosiłbym tylko o to. Chodzi mi o następną stronę tak jak na filmiku...

No to knugi to napisał od samego początku, więc też powinieneś się w tym w miarę połapać.

Jak czegoś nie wiesz, pytaj śmiało. Nie spodziewaj się jednak, że system stron w GUI to będzie kaszka z mlekiem.

Problem został rozwiązany.