Solved Causing an event with a material

I am trying to make a plugin to cause lightning to strike where ever a player right clicks, but only if they are holding a gold axe. This is my code:

Code:

@EventHandler
    public void onPlayerInteract(PlayerInteractEvent event){
        Player player = event.getPlayer();
        World world = player.getWorld();
        if(event.getAction() == (Action.RIGHT_CLICK_BLOCK)){
            if(player.getItemInHand().getType() == Material.GOLD_AXE){
                Block block = event.getClickedBlock();
                world.strikeLightning(block.getLocation());
            }
        }
    }

I also added a listener, but it still doesn't work:

Code:

public class superaxe extends JavaPlugin implements Listener{
 
    public final Logger logger = Logger.getLogger("Minecraft");
 
    public static superaxe plugin;
 
    @Override
    public void onDisable(){
        PluginDescriptionFile pdfFile = this.getDescription();
        this.logger.info(pdfFile.getName() + " Has Been Disabled");
        getServer().getPluginManager().registerEvents(this, this);
    }

Everytime I do this, whether I change it to right of left clicking, the lightning will not strike. I have changed the code a few times, but nothing works. Am I doing something wrong? Can someone please tell me the write code?
Also, please use my tag so that I can come back easily, thanks!

 

Where you check what the action is you must also check if you right click air. So

Code:java

 
if(event.getAction() == Action.RIGHT_CLICK_BLOCK || event.getAction() == Action.RIGHT_CLICK_AIR)
 

 

TheYamsMan you register your listener in on disable, while you should register it in onEnable.

A general note:
- don't use Logger.getLogger("Minecraft"), use the one provided by JavaPlugin#getLogger()
- remove that static plugin instance, since you won't need it.

Bram0101 do note that when you use RIGHT_CLICK_AIR, getClickedBlock() will return null.

 

fireblast709
So...um...ok just to recap.
Remove the static plugin. Totally or should I replace it with something?
What logger should I use? Can you show me the code? I don't know what you are talking about.
Do you recommend something else other than right_click_air?

Bram0101
Thanks for the help!

 

TheYamsMan If you so desperately need a static instance, you can always use

Code:java

JavaPlugin.getPlugin(<yourmainclass>.class)

Or pass the instance via constructors

Code:java

public class SomeRandomClass
{
    private final YourMainClass plugin;
 
    public SomeRandomClass(YourMainClass plugin)
    {
        this.plugin = plugin;
    }
 
    // Other code
}
 
// Now from your main class, it would look like this
new SomeRandomClass(this);

The logger I am talking about resides in JavaPlugin, which is the class that your main class extends. So in your main class, just call

Code:java

this.getLogger();

which gives you a Logger instance specially created for your plugin.

You can still use RIGHT_CLICK_AIR, especially if you would like a greater range than just the blocks around the player. In that case you would have to manually find the target block, since event.getClickedBlock() would return null. Here I recommend a BlockIterator, a class found in the util package of Bukkit.

Example usage:

Code:java

public Block getTarget(Player player, int range)
{
    BlockIterator bi = new BlockIterator(player, range);
    Block target = null;
    while(bi.hasNext())
    {
        target = bi.next();
        // In this example we only consider AIR blocks as
        // transparent.
        if(target.getType() != Material.AIR)
            break; // Since we found our block, no need to continue looking
        target = null;
        // Make sure that if we don't find a block in the loop, that we set it to null
        // This is used to indicate that we didn't find the block when we return
    }
    return target;
}

 

fireblast709 Thank you soo much!

fireblast709 When I put the other logger in, tons of errors pop up. Lots of misplaced constructs and also it says "The field JavaPlugin.logger is not visible." DO I need to change my onEnable and onDisable things? EDIT: Where do I put in the right clicking part into the blockIterator? When I simply add the rightclick.AIR thing, I strike lightnig, but on myself. ???
Code:

Code:java

public void onPlayerInteract(PlayerInteractEvent event){
Player p = event.getPlayer();
World world = p.getWorld();
if(event.getAction() == Action.RIGHT_CLICK_BLOCK || event.getAction == Action.RIGHT_CLICK_AIR){
If(p.getItemInHand().getType() == Material.GOLD_AXE){
Location loc = p.getEyeLocation();
world.strikeLightning(loc);
}
}
}

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

 

Or Player.getTargetBlock(null to ignore air, range as int);

 

itzrobotix deprecated. My code would pretty much do the same, but evades the 'magic values' (material IDs)

TheYamsMan You use this.getLogger(), for example in onEnable(). Also, the right clicking part goes into the event handling method (the one annotated with @EventHandler and with the first (and only) parameter PlayerInteractEvent). Then you basically check for event.getAction() == Action.RIGHT_CLICK_AIR, and use the method I provided (and I do suggest to read the documentation to make sure you fully understand it), which will either return the target Block, or null if there was no in the specified radius.

 

fireblast709
It still tells me that "The field JavaPlugin.logger is not visible." Do you know why this is?
This is my onEnable code right now:

Code:java

@Override
Public void onEnable(){
PluginDescriptionFile pdfFile = this.getDescription();
this.getLogger();
this.logger.info(pdfFile.getName() + "Version " + pdfFile.getVersion() + " has been enabled");
//this is where my problem is, it has the error on logger in this.logger.info
getServer().getPluginManager().registerEvents(this, this);
//this is for my listener, it works just fine
}

The same thing happens for my onDisable:

Code:java

@Override
public void onDisable(){
PluginDescriptionFile pdfFile = this.getDescription();
this.logger.info(pdfFile.getName() + " Version " + pdfFile.getVersion() + " has been enabled.");
}

The logger fit in the right place though. Thanks.

 

TheYamsMan instead of this.logger, use this.getLogger()

 

fireblast709
no .info after?

fireblast709 When I change it to this.getLogger.info(...) or this.getLogger(...) It says it can't be resolved or is not a field.

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

 

TheYamsMan this.getLogger().info("Hello World")

 

fireblast709
Thank you so much for your help!

fireblast709
Sorry, one more thing.
Can you show me how it fits together? My player interact event wont fit with the publick Block getTarget(....... stuff.

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

 

Doesn't mean it won't work. I'm pretty sure it has been deprecated for a while now.

 

fireblast709
Can you show me how to fit it together? I cant fit them because they are both public and stuff.

 

@fireblast709
Can you please respond?

This is what I have for the event check, but I still get errors.

Code:java

@EventHandler
    public void onClick(PlayerInteractEvent event){
        if(event.getAction()==Action.RIGHT_CLICK_AIR){
            public Block getTarget(Player player, int range){
            //get target gives me an error, so does the comma and last parentheses before the curly bracket   
                        BlockIterator bi = new BlockIterator(player, range);
                Block target = null;
                World world = player.getWorld();
                while(bi.hasNext()){
                    target = bi.next();
                    if(target.getType() != Material.AIR){
                        Location loc = target.getLocation();
                        world.strikeLightning(loc);
                    }
                    target = null;
                }
                return target;
                                //This give me an error, change to
                        }
        }
    }

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

 

TheYamsMan that is because it isn't even valid Java, you cannot just paste a function inside another function.

Code:java

@EventHandler
public void onClick(PlayerInteractEvent event)
{
    if(event.getAction() == Action.RIGHT_CLICK_AIR)
    {
        BlockIterator bi = new BlockIterator(player, range);
        Block target = null;
        World world = player.getWorld();
        while(bi.hasNext())
        {
            target = bi.next();
            if(target.getType() != Material.AIR)
            {
                Location loc = target.getLocation();
                world.strikeLightning(loc);
                break;
            }
        }
    }
}

 

fireblast709 OK thanks, sorry for sending you messages and whatnot. i was just a bit anxious...XD

ALSO, I know that isn't valid, I just wasn't sure what to take out or what to move and whatever else...

EDIT: I added the player as event.getPlayer(); ya know, normal, but it still gives me an error on range. Since the declaring the block getTarget(...) thing was taken out, range is not defined as anything. Could you tell me how to get it back in?

 

TheYamsMan it should be defined by you. Like, what do you want the range to be

 

fireblast709 I dont know. I am unfamiliar with the block iterator. If I wanted it to be the same as before (with public in it) it would be nested functions. What do you think it should be?

 

TheYamsMan just replace range with any number you want it to be

 

fireblast709 I knew that wouldn't be what it sounded like. XD Thank you so much for all your help! I am officially done with this thread!