Getting the item in hand from a list

[M4T14SG4M3R]

I started to change from Skript to Java to increase my server performance so i basically want to check if the player has a certain item from a list, for example if the player has a stone, a diamond block or a glass block in the hand but i don't know how to do it
Code

if (player.getPlayer().getItemInHand().getType().equals(Material.WOOD)){
Actions goes here
}

 

[M4T14SG4M3R]

This is pretty basic, maybe you should learn more Java before you start working on plugins. Also why are you using

player.getPlayer()

?

final Player player = event.getPlayer();

 

[M4T14SG4M3R]

event.getPlayer().getPlayer()

???

Also are you sure you didn't use a decompiler?

As i said i'm new and i'm just trying to do the thing i said in the thread by using bukkit code examples

I searched on bukkit forums and google, and please if you will not help me don't reply to the thread. Thanks.

 

[Verang]

final Player player = event.getPlayer();

Not good with plugins but I think it should be:

public static Player event.getPlayer();

 

[M4T14SG4M3R]

Not good with plugins but I think it should be:

public static Player event.getPlayer();

Thanks, any ideas how to get the item in hand from a list?

 

[Kuzni]

final Player player = event.getPlayer();

event.getPlayer().getPlayer()

???

Also are you sure you didn't use a decompiler?

I'll explain it a tad softer, He's asking why you are getting player twice.

You've assigned it so when you type "player", eclipse or whatever will recognise it as event.getPlayer()

now you've written code thats "player.getPlayer();

So essentially, you're doing event.getPlayer().getPlayer();

 

[M4T14SG4M3R]

I'll explain it a tad softer, He's asking why you are getting player twice.

You've assigned it so when you type "player", eclipse or whatever will recognise it as event.getPlayer()

now you've written code thats "player.getPlayer();

So essentially, you're doing event.getPlayer().getPlayer();

Fixed that, please if you know how to do that tell me an example

 

[M4T14SG4M3R]

This.


I can spoon feed you right now but that doesn't help you in the future. I really recommend learning some Java. Also maybe show us all the code?

I think showing all the code will help nothing to do a basic thing as you say to get the item from a list

 

[Spoch]

Something along the lines of:

//(Your player object)
Player player = yourplayer

//Create a list and add the Material value STONE
List<Material> list  = new ArrayList<Material>();
list.add(Material.STONE);

//Check if the list contains the type (Material) of players itemInHand
if (list.contains(player.getItemInHand().getType())) {
    //Do your stuff here
}

Like others have stated above you don't need to append the extra getPlayer() as you already have the Player object, there is also no getPlayer() method in the Player interface.

 

[M4T14SG4M3R]

Something along the lines of:

//(Your player object)
Player player = yourplayer

//Create a list and add the Material value STONE
List<Material> list  = new ArrayList<Material>();
list.add(Material.STONE);

//Check if the list contains the type (Material) of players itemInHand
if (list.contains(player.getItemInHand().getType())) {
    //Do your stuff here
}

Thanks so much!

 

[Spoch]

You only have to create the list once.

True, the List could be initialized outside of the method, depending on use-case.