Problem with onPlayerInteract and onInventoryClick

Hi, so I recently gotten into coding Minecraft plugins and I'm having troubles with both onPlayerInteract and onInventoryClick and I noticed I wasn't the only person having troubles with this plugin I was hoping the plugin dev that helped with the server I play on to help, but he hasn't been on in a very long time (probably because he is working on his own server). here is the code

Code:java

        @EventHandler
        public void onInventoryClick;
        InventoryClickEvent event;
        {

And

Code:java

                @EventHandler
                public void onPlayerInteract;
                PlayerInteractEvent e;
                {

I'm hoping the dev can help Garris0n

 

pinkitty1 So.....exactly what is the problem?

 

Not really sure what the problem is but just going to guess you forgot to register events...

 

I am getting this whenever I hover my mouse over the errors "void is an invalid type for the variable onInventoryClick" and the same thing for onPlayerInteract

 

pinkitty1 I suggest you go and learn basic Java if you want to understand the problem, and try and begin coding plugins. You'll have a hard time understanding most things if you don't

 

pinkitty1
IDE thinks you are creating a variable and void is not a type for making variables. Instead you want to make a void method for events. Refer to here: http://wiki.bukkit.org/Event_API_Reference

 

You didn't set up the event right.

 

@EventHandler
public void interact(PlayerInteractEvent e) {
}
Job done

 

I think it might have to deal with something I did wrong (most likely case, and I'm hoping that is the case) here is a look at a little amount of the code and the picture with the errors in it

 

What is the error if you hover over it?

 

Sadly the same thing :/

 

I think it is a problem with your syntax.

 

pinkitty1
can you post you're full class here so we can look over it? it would be easier

 

Lol alright but keep in mind I recently got into coding plugins and let alone I really don't know java well, because so far the most complex plugin I made was you do /freeze and it freezes every player including you

Code:java

package com.pinkitty1.CustomInv;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.inventory.InventoryClickEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.Inventory;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.ItemMeta;
 
public class CustomInv extends org.bukkit.plugin.java.JavaPlugin implements
Listener {
 
public void onEnable() {
getServer().getPluginManager().registerEvents(this, this);
 
}
 
private void openGUI(Player player) {
Inventory inv = Bukkit.createInventory(null, 9, ChatColor.DARK_GREEN
+ "Chest Selector");
 
Inventory chest1inv = Bukkit.createInventory(null, 54,
ChatColor.DARK_GREEN + "Double Chest 1");
ItemStack chest1 = new ItemStack(Material.CHEST);
ItemMeta chest1Meta = chest1.getItemMeta();
 
ItemStack chest2 = new ItemStack(Material.ACACIA_STAIRS);
ItemMeta chest2Meta = chest2.getItemMeta();
Inventory chest2inv = Bukkit.createInventory(null, 54,
ChatColor.DARK_GREEN + "Double Chest 2");
 
ItemStack chest3 = new ItemStack(Material.APPLE);
ItemMeta chest3Meta = chest3.getItemMeta();
Inventory chest3inv = Bukkit.createInventory(null, 54,
ChatColor.DARK_GREEN + "Double Chest 3");
 
ItemStack chest4 = new ItemStack(Material.BAKED_POTATO);
ItemMeta chest4Meta = chest4.getItemMeta();
Inventory chest4inv = Bukkit.createInventory(null, 54,
ChatColor.DARK_GREEN + "Double Chest 4");
 
ItemStack chest5 = new ItemStack(Material.BIRCH_WOOD_STAIRS);
ItemMeta chest5Meta = chest5.getItemMeta();
Inventory chest5inv = Bukkit.createInventory(null, 54,
ChatColor.DARK_GREEN + "Double Chest 5");
 
ItemStack chest6 = new ItemStack(Material.BONE);
ItemMeta chest6Meta = chest6.getItemMeta();
Inventory chest6inv = Bukkit.createInventory(null, 54,
ChatColor.DARK_GREEN + "Double Chest 6");
 
ItemStack chest7 = new ItemStack(Material.BOOK);
ItemMeta chest7Meta = chest7.getItemMeta();
Inventory chest7inv = Bukkit.createInventory(null, 54,
ChatColor.DARK_GREEN + "Double Chest 7");
 
ItemStack chest8 = new ItemStack(Material.BOWL);
ItemMeta chest8Meta = chest8.getItemMeta();
Inventory chest8inv = Bukkit.createInventory(null, 54,
ChatColor.DARK_GREEN + "Double Chest 8");
 
ItemStack chest9 = new ItemStack(Material.BUCKET);
ItemMeta chest9Meta = chest9.getItemMeta();
Inventory chest9inv = Bukkit.createInventory(null, 54,
ChatColor.DARK_GREEN + "Double Chest 9");
 
chest1Meta.setDisplayName(ChatColor.DARK_RED + "Chest 1");
chest1.setItemMeta(chest1Meta);
 
chest2Meta.setDisplayName(ChatColor.DARK_RED + "Chest 2");
chest2.setItemMeta(chest2Meta);
 
chest1Meta.setDisplayName(ChatColor.DARK_RED + "Chest 3");
chest3.setItemMeta(chest3Meta);
 
chest4Meta.setDisplayName(ChatColor.DARK_RED + "Chest 4");
chest4.setItemMeta(chest4Meta);
 
chest5Meta.setDisplayName(ChatColor.DARK_RED + "Chest 5");
chest5.setItemMeta(chest5Meta);
 
chest6Meta.setDisplayName(ChatColor.DARK_RED + "Chest 6");
chest6.setItemMeta(chest6Meta);
 
chest7Meta.setDisplayName(ChatColor.DARK_RED + "Chest 7");
chest7.setItemMeta(chest7Meta);
 
chest8Meta.setDisplayName(ChatColor.DARK_RED + "Chest 8");
chest8.setItemMeta(chest8Meta);
 
chest9Meta.setDisplayName(ChatColor.DARK_RED + "Chest 9");
chest9.setItemMeta(chest9Meta);
 
inv.setItem(0, chest1);
inv.setItem(1, chest2);
inv.setItem(2, chest3);
inv.setItem(3, chest4);
inv.setItem(4, chest5);
inv.setItem(5, chest6);
inv.setItem(6, chest7);
inv.setItem(7, chest8);
inv.setItem(8, chest9);
 
player.openInventory(inv);
{
@EventHandler
public void invclick(
InventoryClickEvent event){
 
if (ChatColor.stripColor(event.getInventory().getName())
.equalsIgnoreCase("Chest Selector"))
return;
Player player1 = (Player) event.getWhoClicked();
event.setCancelled(true);
 
if (event.getCurrentItem() == null
|| event.getCurrentItem().getType() == Material.AIR
|| !event.getCurrentItem().hasItemMeta()) {
player1.closeInventory();
return;
 
switch (event.getCurrentItem().getType()) {
case CHEST:
player1.openInventory(chest1inv);
break;
case ACACIA_STAIRS:
player1.openInventory(chest2inv);
break;
case APPLE:
player1.openInventory(chest3inv);
break;
case BAKED_POTATO:
player1.openInventory(chest4inv);
break;
case BIRCH_WOOD_STAIRS:
player1.openInventory(chest5inv);
break;
case BONE:
player1.openInventory(chest6inv);
break;
case BOOK:
player1.openInventory(chest7inv);
break;
case BOWL:
player1.openInventory(chest8inv);
break;
case BUCKET:
player1.openInventory(chest9inv);
break;
 
default:
player.closeInventory();
break;
 
}
 
@EventHandler
public void interact(PlayerInteractEvent e) {
 
Action a = e.getAction();
ItemStack is = e.getItem();
 
if (a == Action.PHYSICAL || is == null
|| is.getType() == Material.AIR)
return;
 
if (is.getType() == Material.FEATHER)
openGUI(e.getPlayer());
}
}
}
}
}
}
 

and yes I know the } brackets are in the wrong places but I didn't want to go over the video I used again for the custom gui I was in fact lazy

 

pinkitty1
you really should keep learning java and the bukki api tell a friend to help you out on it or look on TheBCBros tutorials they are really helpful

 

I used TheBCBros video on custom gui's but out of all of my friends none of them even know how to code D: but I guess I will go over the video again

 

EnchantedMiners
So I tried to fix what was wrong but now I am getting errors here

Code:java

                    switch (event.getCurrentItem().getType()) {
                    case CHEST:
                        player1.openInventory(chest1inv);
                        break;
                    case ACACIA_STAIRS:
                        player1.openInventory(chest2inv);
                        break;
                    case APPLE:
                        player1.openInventory(chest3inv);
                        break;
                    case BAKED_POTATO:
                        player1.openInventory(chest4inv);
                        break;
                    case BIRCH_WOOD_STAIRS:
                        player1.openInventory(chest5inv);
                        break;
                    case BONE:
                        player1.openInventory(chest6inv);
                        break;
                    case BOOK:
                        player1.openInventory(chest7inv);
                        break;
                    case BOWL:
                        player1.openInventory(chest8inv);
                        break;
                    case BUCKET:
                        player1.openInventory(chest9inv);
                        break

and also here

Code:java

        Inventory chest1inv = Bukkit.createInventory(null, 54,
                ChatColor.DARK_GREEN + "Double Chest 1");
        ItemStack chest1 = new ItemStack(Material.CHEST);
        ItemMeta chest1Meta = chest1.getItemMeta();
 
        ItemStack chest2 = new ItemStack(Material.ACACIA_STAIRS);
        ItemMeta chest2Meta = chest2.getItemMeta();
        Inventory chest2inv = Bukkit.createInventory(null, 54,
                ChatColor.DARK_GREEN + "Double Chest 2");
 
        ItemStack chest3 = new ItemStack(Material.APPLE);
        ItemMeta chest3Meta = chest3.getItemMeta();
        Inventory chest3inv = Bukkit.createInventory(null, 54,
                ChatColor.DARK_GREEN + "Double Chest 3");
 
        ItemStack chest4 = new ItemStack(Material.BAKED_POTATO);
        ItemMeta chest4Meta = chest4.getItemMeta();
        Inventory chest4inv = Bukkit.createInventory(null, 54,
                ChatColor.DARK_GREEN + "Double Chest 4");
 
        ItemStack chest5 = new ItemStack(Material.BIRCH_WOOD_STAIRS);
        ItemMeta chest5Meta = chest5.getItemMeta();
        Inventory chest5inv = Bukkit.createInventory(null, 54,
                ChatColor.DARK_GREEN + "Double Chest 5");
 
        ItemStack chest6 = new ItemStack(Material.BONE);
        ItemMeta chest6Meta = chest6.getItemMeta();
        Inventory chest6inv = Bukkit.createInventory(null, 54,
                ChatColor.DARK_GREEN + "Double Chest 6");
 
        ItemStack chest7 = new ItemStack(Material.BOOK);
        ItemMeta chest7Meta = chest7.getItemMeta();
        Inventory chest7inv = Bukkit.createInventory(null, 54,
                ChatColor.DARK_GREEN + "Double Chest 7");
 
        ItemStack chest8 = new ItemStack(Material.BOWL);
        ItemMeta chest8Meta = chest8.getItemMeta();
        Inventory chest8inv = Bukkit.createInventory(null, 54,
                ChatColor.DARK_GREEN + "Double Chest 8");
 
        ItemStack chest9 = new ItemStack(Material.BUCKET);
        ItemMeta chest9Meta = chest9.getItemMeta();
        Inventory chest9inv = Bukkit.createInventory(null, 54,
                ChatColor.DARK_GREEN + "Double Chest 9");

The error messages are "chest1inv cannot be resolved to a variable" and "The value of the local variable chest1inv is not used" if you think you can fix this I will be very happy

 

"The value of the local variable chest1inv is not used" is not an error ,is just a 'tip'.
All errors you are/were getting could be easily fixed with a bit knowledge about Java. I'm not saying you need to stop asking for help, what I am saying is that you must at least know java to code a plugin. Bukkit has nothing to do with the errors that you are getting, actually you don't really need to learn java to know what "chest1inv cannot be resolved to a variable" means (you must know English for that). So I really recommend either watching more videos or try to learn java (again, I guess you tried once). Good luck.

 

Yea I know I'm just a bit hesitant I guess since I already coded a minecraft mod without knowing java

 

pinkitty1 can you tell me what you actually want to do with you're plugin ? so i can make a quick code for you