Solved Getting a bukkit player help

I've done this before, I just can't remember how to do it. this is my code:

Code:java

}else if(args.length == 3){
              Bukkit.broadcastMessage("1");
              if(speak.equalsIgnoreCase("kit")){
                  Bukkit.broadcastMessage("2");
                  if(args[0].equalsIgnoreCase("give")){
                      Bukkit.broadcastMessage("3");
 
                          Bukkit.broadcastMessage("4");
 
                          Player getter = Bukkit.getPlayer(args[1]);
                          Bukkit.broadcastMessage("5");
                          Bukkit.broadcastMessage(args[1]);
                            if(args[1].equalsIgnoreCase("pvp")){
                                pvpkit.add(getter.getName());
                                  getter.sendMessage(ChatColor.GOLD + "You have been given the pvp kit");
                                  pvpkit(getter);
                            }else if(args[1].equalsIgnoreCase("archer")){
                                archkit.add(getter.getName());
                                  getter.sendMessage(ChatColor.GOLD + "You have been given the archer kit");
                                  archerkit(getter);
                            }else if(args[1].equalsIgnoreCase("kangaroo")){
                                kanga.add(getter.getName());
                                  getter.sendMessage(ChatColor.GOLD + "You have been given the kangaroo kit");
                                  kangakit(getter);
                            }else if(args[1].equalsIgnoreCase("stomper")){
                                stomper.add(getter.getName());
                                  getter.sendMessage(ChatColor.GOLD + "You have been given the stomper kit");
                                  stomperkit(getter);
                            }else if(args[1].equalsIgnoreCase("zues")){
                                zues.add(getter.getName());
                                  getter.sendMessage(ChatColor.GOLD + "You have been given the zues kit");
                                  zues(getter);
                            }else if(args[1].equalsIgnoreCase("shadow")){
                                shadow.add(getter.getName());
                                  getter.sendMessage(ChatColor.GOLD + "You have been given the shadow kit");
                                  shadow(getter);
                            }else if(args[1].equalsIgnoreCase("squid")){
                                squid.add(getter.getName());
                                  getter.sendMessage(ChatColor.GOLD + "You have been given the squid kit");
                                  squid(getter);
                            }else if(args[1].equalsIgnoreCase("viper")){
                                viper.add(getter.getName());
                                  getter.sendMessage(ChatColor.GOLD + "You have been given the viper kit");
                                  viper(getter);
                            }else{
                                sender.sendMessage(ChatColor.RED + args[0] + args[1] + args[2] + args[3]);
                                sender.sendMessage("invalid kit");
                            }
 
 
                  }
              }
          }
 

It will reach number 5, then give an error. The error is "null" on the command /kit give <player> <kit>

Am I getting the player wrong? Should I get it from the bukkit online players?

 

Code:java

Bukkit.getServer().getPlayer(args[1]);

 

jthort Is args[1] the player name or the kit name, because you seem to be using it for both.

Both are correct, it's redundant but it's there.

 

EnderTroll68 Thats what I thought, but it's still giving me the same error

Alshain01 I printed out args[1] and it's the players exact name

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

 

I understand, but then your checking to see if it is "pvp". Should that be args[2]?

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

 

Alshain01 Your right, it should be args[2] when checking which kit, thanks a lot. Solved

 

Also (most likely your NPE) there is no args[3]. You checked that above if args.length == 3 (meaning 0 to 2)

sender.sendMessage(ChatColor.RED + args[0] + args[1] + args[2] + args[3]);

 

Alshain01 Yeah that is just a left over debugging thing I used when I used a different command, i'll remove it

 

Yeah, but that's what was throwing the error. Your debugging was actually working against you. args[1] was never equal to a kit name so it fell through to that line and since args[3] was null... Null Pointer Exception.