Skocz do zawartości
  • 0

Gui


FluXiak

Pytanie

11 odpowiedzi na to pytanie

Rekomendowane odpowiedzi

  • 0

1. Kompas to nie blok.

2. KOD:

Spoiler

@EventHandler
public void handle(PlayerInteractEvent event)
{

    if(event.getAction() == Action.RIGHT_CLICK_BLOCK || event.getAction() == Action.RIGHT_CLICK_AIR)

    {

        if(event.getPlayer().getInventory().getItemInHand() != null && event.getInventory.getItemInHand().getType() == Material.COMPASS)

        {

            final int slots = 54; /* Ilosc slot */

            final String name = "Nazwa GUI";

            event.getPlayer().openInventory(Bukkit.createInventory(null, slots, name));

        }
    }
}

 

Pisane z pamięci, więc nw czy wszystko git.

Daj jakieś serduszko czy coś za pomoc xd

Odnośnik do komentarza
https://skript.pl/temat/31745-gui/#findComment-205693
Udostępnij na innych stronach

  • 0
37 minut temu, STEINER napisał:

1. Kompas to nie blok.

2. KOD:

  Ukryj zawartość


@EventHandler
public void handle(PlayerInteractEvent event)
{

    if(event.getAction() == Action.RIGHT_CLICK_BLOCK || event.getAction() == Action.RIGHT_CLICK_AIR)

    {

        if(event.getPlayer().getInventory().getItemInHand() != null && event.getInventory.getItemInHand().getType() == Material.COMPASS)

        {

            final int slots = 54; /* Ilosc slot */

            final String name = "Nazwa GUI";

            event.getPlayer().openInventory(Bukkit.createInventory(null, slots, name));

        }
    }
}

 

Pisane z pamięci, więc nw czy wszystko git.

Daj jakieś serduszko czy coś za pomoc xd

czym to się rózni od mojego ?

Jaki problem zmienić kompas na coś innego 😛

Odnośnik do komentarza
https://skript.pl/temat/31745-gui/#findComment-205702
Udostępnij na innych stronach

  • 0
42 minuty temu, STEINER napisał:

1. Kompas to nie blok.

2. KOD:

  Ukryj zawartość


@EventHandler
public void handle(PlayerInteractEvent event)
{

    if(event.getAction() == Action.RIGHT_CLICK_BLOCK || event.getAction() == Action.RIGHT_CLICK_AIR)

    {

        if(event.getPlayer().getInventory().getItemInHand() != null && event.getInventory.getItemInHand().getType() == Material.COMPASS)

        {

            final int slots = 54; /* Ilosc slot */

            final String name = "Nazwa GUI";

            event.getPlayer().openInventory(Bukkit.createInventory(null, slots, name));

        }
    }
}

 

Pisane z pamięci, więc nw czy wszystko git.

Daj jakieś serduszko czy coś za pomoc xd

przecież pan anioł też sprawdza po materiale a nie po bloku, więc nie wiem ocb xD

Odnośnik do komentarza
https://skript.pl/temat/31745-gui/#findComment-205703
Udostępnij na innych stronach

  • 0
3 minuty temu, HotAngel2K17 napisał:

czym to się rózni od mojego ?

Jaki problem zmienić kompas na coś innego 😛

On napisał "jakiegoś bloku np. kompasu" to poprawiłem, że kompas to nie blok.

Priority HIGH jest niepotrzebne w jego pluginie na lobby (prawdopodobnie).

Twój style code to brzydki :rak: (rak) chociaż o tym się nie dyskutuje. 

Poza tym z 🖕wyssana metoda openGui, tym bardziej, że pytanie brzmi JAK OTWORZYĆ GUI a nie jak wykonać metodę trzymając kompas w łapie. 

Odnośnik do komentarza
https://skript.pl/temat/31745-gui/#findComment-205704
Udostępnij na innych stronach

  • 0
7 minut temu, Queito napisał:

przecież pan anioł też sprawdza po materiale a nie po bloku, więc nie wiem ocb xD

 

NrMTcXG.png

Chodzi o realne życie nie o klocki. 

Życie - https://pl.wikipedia.org/wiki/Życie

Kompas to PRZEDMIOT nie BLOK i nie chodzi o enum Material!!!

Odnośnik do komentarza
https://skript.pl/temat/31745-gui/#findComment-205706
Udostępnij na innych stronach

  • 0
27 minut temu, STEINER napisał:

On napisał "jakiegoś bloku np. kompasu" to poprawiłem, że kompas to nie blok.

Priority HIGH jest niepotrzebne w jego pluginie na lobby (prawdopodobnie).

Twój style code to brzydki :rak: (rak) chociaż o tym się nie dyskutuje. 

Poza tym z 🖕wyssana metoda openGui, tym bardziej, że pytanie brzmi JAK OTWORZYĆ GUI a nie jak wykonać metodę trzymając kompas w łapie. 

Czytaj ze zrozumieniem, otwierać gui można na 1000 sposobów, twórca tematu nie pisał o konkretnych sposobach.

"Dodam jeszcze, że ten sposób openGui to tylko przykład" - napisałem, że to tylko przykład

Priority HIGH jest w moim pluginie, zapomniałem usunąć dawając część kodu tutaj.

 

Nie rozumiem, po co się czepiać, kolega chciał tylko przypomnienie, a nie jak zrobić to w sposób idealny

 

Odnośnik do komentarza
https://skript.pl/temat/31745-gui/#findComment-205707
Udostępnij na innych stronach

  • 0
1 godzinę temu, STEINER napisał:

1. Kompas to nie blok.

2. KOD:

  Pokaż ukrytą zawartość


@EventHandler
public void handle(PlayerInteractEvent event)
{

    if(event.getAction() == Action.RIGHT_CLICK_BLOCK || event.getAction() == Action.RIGHT_CLICK_AIR)

    {

        if(event.getPlayer().getInventory().getItemInHand() != null && event.getInventory.getItemInHand().getType() == Material.COMPASS)

        {

            final int slots = 54; /* Ilosc slot */

            final String name = "Nazwa GUI";

            event.getPlayer().openInventory(Bukkit.createInventory(null, slots, name));

        }
    }
}

 

Pisane z pamięci, więc nw czy wszystko git.

Daj jakieś serduszko czy coś za pomoc xd

może mu chodziło o blok z inventory -,-

Odnośnik do komentarza
https://skript.pl/temat/31745-gui/#findComment-205711
Udostępnij na innych stronach

  • 0

Nie porównujemy enumów metodą equals, od tego jest ==

takie priority (high) jest niepotrzebne, zamiast 3 ify z returnami - wystarczy jeden z użyciem || (lub)

action nigdy nie bedzie nullem, wiec taki if useless

Cytat

warunek nizej i tak sprawdza material, wiec ten if niepotrzebny

no i jest od dawna metoda e.getItem() wiec pobieranie itemu z reki gracza jest bez sensu (tym bardziej chyba z tej glownej, bo przeciez gracz moze uzyc tez drugiej reki?)

zastostuj sie do tych wszystkich rad do kodów które ci podesłali ;) 

Małe wytłumaczenie co do samego tematu, jak chcesz zrobić sam:

tworzysz klase implementująca listener, tworzysz metode z adnotacja @EventHandler, w parametrach dajesz PlayerInteractEvent (bo to go chcesz nasluchiwac)
w metodzie sprawdzasz czy PlayerInteractEvent#getItem().getType() to Material.COMPASS -> otwierasz inventory, które najlepiej jest stworzyć w konstruktorze (tak zeby bylo tylko raz tworzone, a nie za kazdym razem przy evencie), inventory tworzysz uzywajac Bukkit.createInventory(inventoryHolder (najczesciej null), inventorySize, inventoryName);

Edytowane przez yooniks
Odnośnik do komentarza
https://skript.pl/temat/31745-gui/#findComment-205721
Udostępnij na innych stronach

  • 0
9 godzin temu, Queito napisał:

może mu chodziło o blok z inventory -,-

Mi chodzi o to, że kompas to nie blok!

https://minecraft-pl.gamepedia.com/Bloki - widzisz tu kompas? 

Czepiam się do tego co napisał. Jakby napisał "jakiegoś przedmiotu np. kompasu" albo "jakiegoś bloku np. dirtu" to bym nie napisał, a ty drążysz temat i jeszcze mówisz o inventory jak ja mówię, że nie chodzi o np. kod tylko o to co napisał....

Odnośnik do komentarza
https://skript.pl/temat/31745-gui/#findComment-205723
Udostępnij na innych stronach

Nieaktywny
Ten temat został zamknięty. Brak możliwości dodania odpowiedzi.
  • Ostatnio przeglądający   0 użytkowników

    • Brak zarejestrowanych użytkowników przeglądających tę stronę.
×
×
  • Dodaj nową pozycję...