Tutorial Create a Navigator!

Hey guys! My name is Bacon, also known as Jordan!
Today I'm going to be showing you how to create a Navigator!

So first we're gonna do this in your class!

Code:java

public class Main extends JavaPlugin implements Listener /* In your class implements Listener! (If you use a seperate class do not use extends JavaPlugin in it! */ {
    @Override
    public void onEnable(){
        getServer().getPluginManager().registerEvents(this, this);
        //Or if you wanna use a seperate class
        getServer().getPluginManager().registerEvents(new EventClass(), this);
    }
}

Then we're going to create an inventory!

Code:java

public Inventory navigator = Bukkit.createInventory(null, 9 /*Size <-*/, "This is the name");

Now lets add our items!

Code:java

//In our onEnable
    ItemStack navigatoritem = new ItemStack(Material.DIAMOND_AXE);
    ItemMeta navigatoritemmeta = navigatoritem.getItemMeta();
    navigatoritemmeta.setDisplayName("§6Survival Games");
    navigatoritemmeta.setLore(Arrays.asList("§fBattle it out against 23 others!", "§fThere will be only one survivor.."));
    navigatoritem.setItemMeta(navigatoritemmeta);
 
    navigator.setItem(0, navigatoritem);

Now lets let them open the navigator!

Code:java

@EventHandler
    public void onPlayerInteract(PlayerInteractEvent e){
        Player p = e.getPlayer();
        Action a = e.getAction();
        Material m = p.getItemInHand().getType();
        if(a == Action.RIGHT_CLICK_AIR || a == Action.RIGHT_CLICK_BLOCK){
            if(m == null){
                   return;
            } else if(m == Material.COMPASS && p.getItemInHand().getItemMeta().getDisplayName().equalsIgnoreCase("§6Navigator")){
                p.openInventory(navigator);
            }
        }
    }

Now let's setup the clicking item!

Code:java

@EventHandler
    public void onPlayerClick(InventoryClickEvent e){
        Material m = e.getCurrentItem().getType();
        if(e.getInventory().equals(navigator)){
            e.setCancelled(true);
            if(m == Material.DIAMOND_AXE && e.getCurrentItem().getItemMeta().getDisplayName().equalsIgnoreCase("§6Survival Games")){
                //Do stuff here
            }
        }
    }

And there you go! You now have a navigator! Thank you for reading this, if you have any issues or need help just comment!

Pastebin: http://pastebin.com/HEYxwaYA

 

And I now have a NullPointerException because I didn't have an item in my hand.

 

Well, what does it matter?

 

@BaconStripzMan Everything.

 

Huh?

 

@BaconStripzMan It matters everything.

 

Well, it gives no errors. So what does it matter, the code still works!

 

@BaconStripzMan NPE, not an error? That's rad man

 

why

 

If you don't set the items in a method it will not let you use navigatoritemmeta.whatever

 

@BaconStripzMan Why the static? That was the big, bolder thing... Did you not see it?

 

Try using itemmetas in a non-static method @nverdier

 

@BaconStripzMan What, is it difficult or something? I don't know why your telling me to do that... Statics aren't good in this situation, you realize.

 

Does it work? Yes..

 

@BaconStripzMan Does it cause memory leaks? Yes.

 

it's very bad coding conventions to use 'static' when not needed. In most situations, (when using the Bukkit API) you would only use static when messing with NMS. You do not need to use static on this.

 

You know what stuff the forums, I never get any helpful feedback or support, I'm outta here

 

@BaconStripzMan What we are saying is very helpful. Are you trying to make us feel bad or something?

 

No, maybe you could tell me how to fix it and stop arguing with something that isn't true. I've said if you don't use your item setting in a method you have got to use the static method and to set the item in a static method you have to use public "static" Inventory...

EDIT: Fixed the NullPointerException

 

@BaconStripzMan What you have is all in the same class, yes? Why do you need statics at all? Just remove them.

 

Image 1 (No Statics):

Image 2:

@nverdier

 

@BaconStripzMan Don't remove the brackets, just the statics.

 

Ok thank you !

 

Really.
Im so excited when I click on an empty part of my inventory, and Im scanning the console (READING A SUPER IMPORTANT MESSAGE) and well, Then I get a tsunami of errors
NPE AT LINE 15! ERROR ERROR!

'Static{'
really? Do you need to risk using static?

Also, in the image..
you have errors l.l

 

If you actually read my post you would of seen I fixed the errors.. So yeah?

 

So yeah.

 

I no longer see any errors, do you?

 

I fixed it Read the post! @ChipDev

 

@BaconStripzMan how should that work? You should create a method for filling the inventory, not just some brackets around it. Take a look here, I've done it for you http://paste.minidigger.me:7777/axopefohab.avrasm