Checking if the block a player is looking at is something

So i want to make it so on the command it checks if the block is something, but I can't seem to find out how to do it? can anyone help me?

Code:java

public boolean onCommand(CommandSender sender, Command command, String name, String[] args) {
        Player player = (Player) sender; {
            if(command.getName().equalsIgnoreCase("mb")) {
                if(args.length == 0) {
                    player.sendMessage(Prefix + ChatColor.BLUE + "Commands:");
                    return true;
 
                } else if(args.length == 1) {
                    if(args[0].equalsIgnoreCase("create")) {
 
                    }
 
                }
 
 
            }
 
 
        }
 
        return true;
    }
 

what "if" statement would I use?

 

http://jd.bukkit.org/dev/doxygen/d4...Entity.html#a889b8d9fb7d4a9ad042392a1cdd7e284

 

AdamQpzm I have seen that ¬_¬ but it doesn't seem to work?

Please could you just tell me how I would do what i want to do?

 

yewtree8 Well, what didn't work? How did you try and use it? I'm not going to just tell you how to do it.

 

AdamQpzm

It keeps on coming up as depricated with the error that it doesn't compute with the type living entity

 

yewtree8 Deprecation is a warning, it should still work. Also post the code with your attempt to use it please.

 

AdamQpzm

Code:java

public boolean onCommand(CommandSender sender, Command command, String name, String[] args) {
        Player player = (Player) sender; {
            if(command.getName().equalsIgnoreCase("mb")) {
                if(args.length == 0) {
                    player.sendMessage(Prefix + ChatColor.BLUE + "Commands:");
                    return true;
 
                } else if(args.length == 1) {
                    if(args[0].equalsIgnoreCase("create")) {
                        if(player.getTargetBlock(34, 100)) { // IT HAS A RED LINE UNDERNEATH
 
 
 
                        }
                    }
 
                }
 
 
            }
 
 
        }
 
        return true;
    }
 
 
 
 
}
 

 

Mabe you could get the target block location and use

Code:

player.getWorld().getBlockAt(x,y,z)

 

BlazingBroGamer

How would I implement this into my code?

 

yewtree8
Nevermind... I found an easier way.

Code:

Player p = (Player) sender;
p.getTargetBlock(null,100).setType(Material.WOOL);

I used wool for example purposes, and usually when I do null for the target block, I get it to work. Hope it works!

 

BlazingBroGamer

It says that you can't convert void to boolean

BlazingBroGamer

I want to check IF the block is something

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

yewtree8
Well I think I didn't read the question right. I think what your looking for is this:

Code:

Player p = (Player) sender;
                if(p.getTargetBlock(null, 100).getType() == Material.WOOL && p.getTargetBlock(null, 100).getType  != null){
                    //Code here!
                }

Again, I used wool for example purposes. I think it is important for a null check, since there is a high chance it might be null.

yewtree8
Oh and also, you can check if the block is air or not, then return a boolean. So it might go something like this:

Code:

public boolean blockCheck(Player p){
if(p.getTargetBlock(null,100).getType != Material.AIR && p.getTargetBlock(null,100).getType != null){
return true;
}else{
return false;

That should work I think

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

BlazingBroGamer

Thankyou good sir you are a life saver

 

yewtree8
Welcome, and also remember to mark the problem as solved!

 

BlazingBroGamer I can't even begin to imagine why you check whether it's a wool block and then check whether it's a null type.

yewtree8 Was 34 meant to mean anything?

 

AdamQpzm
Hmm... True... But the question is to check if a player is looking at a block, and I guess I didn't read the question properly, therefore I used a wool block, as an example. My last code is working, I think, according to yewtree8

 

BlazingBroGamer


By any chance would you know how I could add this target block to an arraylist?

 

yewtree8
For a block arraylist, it would be like this:

Code:

List<Block> block = new ArrayList<Block>();
Block b = p.getTargetBlock(null, 100).getType();
block.add(b);

 

BlazingBroGamer That wouldn't work, you're getting the type.

 

AdamQpzm
Right.... Forgot to remove the type.. I got so used to using Materials... Well for block, you don't need the getType(), but if you want a material, u getType()