Solved Help killing players and creating explosion

So, I'm trying to make a weird little plugin just for fun to test out some stuff, and I came across something. So, I'm trying to kill someone with an explosion in this code:

Code:java

package com.gmail.[Sensored].test;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
public final class Apples extends JavaPlugin {
@Override
public void onEnable(){
getLogger().info("OpMeFX started successfully!");
}
 
@Override
public void onDisable(){
getLogger().info("OpMeFX closed!");
 
}
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
if(cmd.getName().equalsIgnoreCase("opme")){
if (!(sender instanceof Player)){
sender.sendMessage("This command can only be run by a player.");
} else {
 
Player player = (Player) sender;
player.sendMessage("§4You really though that would work?");
float explosionPower = 4F;
Player target = sender.getWorld().getPlayer(args[0]);
target.getWorld().createExplosion(target.getLocation(), explosionPower);
target.setHealth(0);
return true;
}
}
return false;
}
}
 

but, in eclipse, getworld() is underlined in red, and it says The method getWorld() is undefined for the type CommandSender. I am kinda a newb to programming bukkit plugins, so any help would be appreciated! If you do help, then to you!

 

dragonfreak1000
CommandSender can be the Console, and the Console has no world I think you want to use player instead of sender there.

 

Thanks, but there's still one more problem. So, now that I have

Code:java

 public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
if(cmd.getName().equalsIgnoreCase("opme")){
if (!(sender instanceof Player)){
sender.sendMessage("This command can only be run by a player.");
} else {
 
Player player = (Player) sender;
player.sendMessage("§4You really though that would work?");
float explosionPower = 4F;
Player target = player.getWorld().getPlayer(args[0]);
target.getWorld().createExplosion(target.getLocation(), explosionPower);
target.setHealth(0);
return true;
}
}
return false;
}

Now getPlayer is underlined and it says The method getPlayer(String) is undefined for the type World.
I also tried

Code:java

 public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args){
if(cmd.getName().equalsIgnoreCase("opme")){
if (!(sender instanceof Player)){
sender.sendMessage("This command can only be run by a player.");
} else {
 
Player player = (Player) player;
player.sendMessage("§4You really though that would work?");
float explosionPower = 4F;
Player target = player.getWorld().getPlayer(args[0]);
target.getWorld().createExplosion(target.getLocation(), explosionPower);
target.setHealth(0);
return true;
}
}
return false;
}

 

dragonfreak1000
Yes the World dont gives you a specific Player, only a List of the Players that are on this World. The Server can give you the specific Player: Bukkit.getPlayer(args[0]);
If you want that the target is on the same World, you have to check this on your own.

 

Try creating multiple variables, one getting the location, another getting the player and another getting the world.

 

Ok, sorry I'm such a newb but now when I replace getPlayer() with Bukkit.getPlayer(), it underlines that in red. (And I imported the correct classes at the top.) but anyways, all I really want to do, is get where the player is and make an explosion and kill them.

 

dragonfreak1000

Code:java

public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
  if(cmd.getName().equalsIgnoreCase("opme")) {
    if (!(sender instanceof Player)) {
      sender.sendMessage("This command can only be run by a player.");
    } else {
      if (args.length == 1) {
        Player player = (Player) sender;
        player.sendMessage("§4You really though that would work?");
        Player target = Bukkit.getPlayer(args[0]);
        if (target != null) {
          target.getWorld().createExplosion(target.getLocation(), 4F);
          target.setHealth(0);
          return true;
        }
      }
    }
  }
  return false;
}

What I dont get is, that the command is opme, you send a message a to the commandsender, but you kill a player provided by the first argument. Why?

 

Yes, you should do

Code:java

Player target = (Player) sender;

 

I'd suggest looking in the docs before posting about an underlined error in Eclipse, eg, getPlayer(String), as it saves a lot of time and effort if you can fix most problems yourself

 

Ok I combined these 2 ideas and got it working with:

Code:

public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
    if(cmd.getName().equalsIgnoreCase("opme")) {
    if (!(sender instanceof Player)) {
    sender.sendMessage("This command can only be run by a player.");
    } else {
    if (args.length < 1) {
    Player target = (Player) sender;
    target.sendMessage("§4You really though that would work?");
    if (target != null) {
    target.getWorld().createExplosion(target.getLocation(), 4F);
    target.setHealth(0);
    return true;
    }
    }
    }
    }
    return false;
    }

so to you both!

Plugin now live with my other account (HaitherecreeperMC) on bukkitdev. Yes it says I'm 13, as I entered my age wrong on this one so it's right. It's currently awaiting approval.

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