Jak napisać GUI

[MieciuXD1234]

Witam chciałbym się dowiedzieć jak dodać gui np. drop bo zrobiłem coś takiego:

 

public class DropGui implements CommandExecutor {

 

@Override

public boolean onCommand(CommandSender sender, Command cmd, String label,

String[] args) {

if (cmd.getName().equalsIgnoreCase("drop")) {

if ((sender instanceof Player)) {

}

Player player = (Player) sender;

openGUI(player);

}

 

return true;

}

 

private void openGUI(Player player) {

Inventory inv = Bukkit.createInventory(null, 9, ChatColor.RED

+ "Drop ze stone:");

inv.setItem(0, new ItemStack(Material.COBBLESTONE, 1));

inv.setItem(1, new ItemStack(Material.COBBLESTONE, 1));

inv.setItem(6, new ItemStack(Material.COBBLESTONE, 1));

inv.setItem(7, new ItemStack(Material.COBBLESTONE, 1));

 

}

 

}

 

 

Serwer tak jakby nie odczytuje tego, brak błędów w konsoli itp. komenda tak jakby działa ale nic nie wyświetla ani nic nie robi po prostu /drop i nic

[AdamGrzegorz]

1.  

if ((sender instanceof Player)) {
}
Player player = (Player) sender;
openGUI(player);

masz źle zrobione warunki.   (nie musisz dodawać sprawdzenia czy komenda jest "drop", to dziwny nawyk ludzi używających eklajpse na youtube. plugin.yml się tym zajmie.)

if (sender instanceof Player) {
    Player player = (Player) sender;
    openGUI(player);
}

2. Nie twórz za każdym razem nowego itemu

inv.setItem(0, new ItemStack(Material.COBBLESTONE, 1));
inv.setItem(1, new ItemStack(Material.COBBLESTONE, 1));
inv.setItem(6, new ItemStack(Material.COBBLESTONE, 1));
inv.setItem(7, new ItemStack(Material.COBBLESTONE, 1));

zamień na

ItemStack i = new ItemStack(Material.COBBLESTONE);
inv.setItem(0, i);
inv.setItem(1, i);
inv.setItem(6, i);
inv.setItem(7, i);

A już najlepiej to jakoś inaczej rozwiązać pętlą.

 

3. Nie wyświetlasz tego inventory graczowi

dodaj

player.openInventory(inv);

4. Nie twórz nowego inventory za każdym razem

private static Inventory inv = Bukkit.createInventory(null, 9, ChatColor.RED + "Drop ze stone:");

public void openGUI(Player player) {
   ItemStack i = new ItemStack(Material.COBBLESTONE);
   inv.setItem(0, i);
   inv.setItem(1, i);
   inv.setItem(6, i);
   inv.setItem(7, i);

   player.openInventory(inv);
}

[MieciuXD1234]

 

1.  

if ((sender instanceof Player)) {
}
Player player = (Player) sender;
openGUI(player);

masz źle zrobione warunki.   (nie musisz dodawać sprawdzenia czy komenda jest "drop", to dziwny nawyk ludzi używających eklajpse na youtube. plugin.yml się tym zajmie.)

if (sender instanceof Player) {
    Player player = (Player) sender;
    openGUI(player);
}

2. Nie twórz za każdym razem nowego itemu

inv.setItem(0, new ItemStack(Material.COBBLESTONE, 1));
inv.setItem(1, new ItemStack(Material.COBBLESTONE, 1));
inv.setItem(6, new ItemStack(Material.COBBLESTONE, 1));
inv.setItem(7, new ItemStack(Material.COBBLESTONE, 1));

zamień na

ItemStack i = new ItemStack(Material.COBBLESTONE);
inv.setItem(0, i);
inv.setItem(1, i);
inv.setItem(6, i);
inv.setItem(7, i);

A już najlepiej to jakoś inaczej rozwiązać pętlą.

 

3. Nie wyświetlasz tego inventory graczowi

dodaj

player.openInventory(inv);

4. Nie twórz nowego inventory za każdym razem

private static Inventory inv = Bukkit.createInventory(null, 9, ChatColor.RED + "Drop ze stone:");

public void openGUI(Player player) {
   ItemStack i = new ItemStack(Material.COBBLESTONE);
   inv.setItem(0, i);
   inv.setItem(1, i);
   inv.setItem(6, i);
   inv.setItem(7, i);

   player.openInventory(inv);
}

Dzięki wielkie!

Ale jeszcze mam pytanie np. jak zamiast nazwy "Cobblestone" pisało np. "szansa"

[MarcinWieczorek]

meta = i.getItemMeta();
meta.setDisplayName("Szansa");
i.setItemMeta(meta);

Jeśli interesuje Cię już takie naprawdę zaawansowane API to masz tutaj:

https://github.com/MarcinWieczorek/NovaGuilds/blob/master/src/main/java/co/marcin/novaguilds/impl/util/guiinventory/GUIInventoryGuildMenu.java

[MieciuXD1234]

meta = i.getItemMeta();
meta.setDisplayName("Szansa");
i.setItemMeta(meta);

Jeśli interesuje Cię już takie naprawdę zaawansowane API to masz tutaj:

https://github.com/MarcinWieczorek/NovaGuilds/blob/master/src/main/java/co/marcin/novaguilds/impl/util/guiinventory/GUIInventoryGuildMenu.java

 

Słuchaj nie chce mi to działać zrobiłem tak

 

private void openGUI(Player player) {

Inventory inv = Bukkit.createInventory(null, 4 * 9,

"§4§lDrop ze stone:");

ItemStack i = new ItemStack(Material.COBBLESTONE);

inv.setItem(0, i);

ItemMeta meta = i.getItemMeta();

meta.setDisplayName("Szansa");

i.setItemMeta(meta);

 

 

player.openInventory(inv);

[MarcinWieczorek]

A daj setItem niżej, przed openInventory

[MieciuXD1234]

A daj setItem niżej, przed openInventory

A no działa! a mogę zrobic \n i dalej pisać czy nie za działa?

[MarcinWieczorek]

To co masz niżej to lore które jest List<String>. Tworzysz nowy ArrayList, dodajesz tekst i robisz meta.setLore(lore)

[AdamGrzegorz]

item.setLore(Arrays.asList("pierwsza linijka", "druga linijka", "trzecia linijka"));

I nie rób § bo to żenada.

Napisz sobie coś takiego: 

    public static String fixColors(String text) {
        return ChatColor.translateAlternateColorCodes('&', text);
    }

    public static String[] fixColors(String... texts) {
        String[] strings = new String[texts.length];
        for (int i = 0; i < strings.length; i++)
            strings[i] = fixColors(texts[i]);
        return strings;
    }

    public static List<String> fixColors(List<String> texts) {
        return texts.stream().map(Util::fixColors).collect(Collectors.toList());
    }

i rób 

item.setLore(fixColors(Arrays.asList("1", "2")));

albo

p.sendMessage(fixColors("&6Elo"));

Można by się tam spierać o szybkość takiego rozwiązania, ale przynajmniej mniej rakowe niż § a różnicę w szybkości zauważy jedynie @GotoFinal ! (bez obrazy gotuś, loffciam cię)

[MieciuXD1234]

 

    public static String[] fixColors(String... texts) {

        String[] strings = new String[texts.length];

        for (int i = 0; i < strings.length; i++)

            strings = fixColors(texts);

        return strings;

to nie działa

[MieciuXD1234]

item.setLore(Arrays.asList("pierwsza linijka", "druga linijka", "trzecia linijka"));

I nie rób § bo to żenada.

Napisz sobie coś takiego: 

    public static String fixColors(String text) {
        return ChatColor.translateAlternateColorCodes('&', text);
    }

    public static String[] fixColors(String... texts) {
        String[] strings = new String[texts.length];
        for (int i = 0; i < strings.length; i++)
            strings[i] = fixColors(texts[i]);
        return strings;
    }

    public static List<String> fixColors(List<String> texts) {
        return texts.stream().map(Util::fixColors).collect(Collectors.toList());
    }

i rób 

item.setLore(fixColors(Arrays.asList("1", "2")));

albo

p.sendMessage(fixColors("&6Elo"));

Można by się tam spierać o szybkość takiego rozwiązania, ale przynajmniej mniej rakowe niż § a różnicę w szybkości zauważy jedynie @GotoFinal ! (bez obrazy gotuś, loffciam cię)

 

+ to 

 

    public static List<String> fixColors(List<String> texts) {

        return texts.stream().map(Util::fixColors).collect(Collectors.toList());

[MieciuXD1234]

Zamykam!