Command not reading my if statement?

I'm making a 1v1 plugin and want to stop the null point exceptions even if the player types something null but even when I do that it's not sending them the message and I think i'm doing it right but it's not working can someone tell me why. Thank You!

Code: http://pastebin.com/8QANcjTh

 

Did you check if you can actually access args[1] and args[0] with an args.length check? Strings in args array cannot be normally null, because it is split from a message.

 

Necrodoom Yes,I have tried that and it still does not work but Thank you for the fast reply!

Bump

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

 

Code:java

 if(args[0].equalsIgnoreCase("join")){
                if(args[1].equalsIgnoreCase(args[1])){
 
                        if(args[1] != null){
                                Bukkit.broadcastMessage("§7[§41§cv§41§7]: §4 " + player.getName() + " §2Has Joined§4 " + arenaname);
                                final World w1 = Bukkit.getWorld((String) getConfig().get(arenaname + ".spawn1.world"));
                                final double x1 = getConfig().getDouble(arenaname + ".spawn1.x");
                                final double y1 = getConfig().getDouble(arenaname + ".spawn1.y");
                                final double z1 = getConfig().getDouble(arenaname + ".spawn1.z");
 
                                final World w2 = Bukkit.getWorld((String) getConfig().get(arenaname + ".spawn2.world"));
                                final double x2 = getConfig().getDouble(arenaname + ".spawn2.x");
                                final double y2 = getConfig().getDouble(arenaname + ".spawn2.y");
                                final double z2 = getConfig().getDouble(arenaname + ".spawn2.z”);
                    }else{
                                Bukkit.broadcastMessage("Working");
                                player.sendMessage("§7[§41§cv§41§7]: §4You must Type an Arena or the Arena doesn't exist!”);
                          }
                }
}

Try that. Mrcool234

 

maved145 ..............what. Your attempt makes even less sense.

Mrcool234 paste your edited file then.

 

Necrodoom How. Im simply making it check if args[1] doesn’t equal null then do something else(if it doe equal null) do something else.

 

maved145 1. An arg cannot ever equal null, it is done by splitting the command by spaces, you should make args.length checks like regular array logic.

2. if(args[1].equalsIgnoreCase(args[1])) should speak for it self. If not, you seriously need to learn Java.

 

Necrodoom Well that bit wasn’t my code.

 

maved145 if you are going to fix someone else's code for him and spoonfeed, at least do it right.

 

maved145

This is not going to work:

Code:java

if(args[1].equalsIgnoreCase(args[1])){

You can't check if args[1] equals args[1].

Code:java

if(args[1] == null){

Instead of checking if the argument is null (which is impossible), check if there's an arena which equals args[1].

 

Necrodoom Your honestly a jerk. Just leave me alone.

 

maved145 I'm sorry that I don't like it when people feed code without checking if its correct.

 

Necrodoom i understand that, but you could do it more politely. I know i should of checked the code, but i was kinda in a rush :L