Hmm? What am I doing wrong?

When I'm doing /tp bremdecoolste blabla it says " Unhandled exception executing command 'tp' in plugin TqCommands v1.0"

java file:

Code:java

  package me.bremdecoolste.TqCommands;
 
    import java.util.logging.Logger;
 
 
    import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
 
    public class TqCommands extends JavaPlugin{
        public final Logger logger = Logger.getLogger("Minecraft");
        public static TqCommands plugin;
 
 
        @Override
        public void onEnable()
        {
            PluginDescriptionFile pdfFile = this.getDescription();
            getLogger().info(pdfFile.getName() + " Version " + pdfFile.getVersion() + " TqCommands has been loaded! ");
        }
 
 
        @Override
        public void onDisable()
        {
            PluginDescriptionFile pdfFile = this.getDescription();
            getLogger().info(pdfFile.getName() + " Version " + pdfFile.getVersion() + " TqCommands has been stopped! ");
        }
 
 
        public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args)
        {
            Player player = (Player) sender;
 
 
//TP COMMAND           
 
            if(commandLabel.equalsIgnoreCase("tp"));
            {
                if(args.length == 0)
                {
                    player.sendMessage(ChatColor.DARK_RED + "/tp <target>");
                }
                else if(args.length == 1);
                {
                    Player targetPlayer = player.getServer().getPlayer(args[0]);
                    Location targetPlayerLocation = targetPlayer.getLocation();
                    player.teleport(targetPlayerLocation);
                    player.sendMessage(ChatColor.GRAY + "Teleporting to " + targetPlayer.getName());
                }
                if(args.length == 2)
                {
                    Player targetPlayer1 = player.getServer().getPlayer(args[1]);
                    Player targetPlayer2 = player.getServer().getPlayer(args[2]);
                    Location targetPlayer2Location = targetPlayer2.getLocation();
                    targetPlayer1.teleport(targetPlayer2Location);
                    player.sendMessage(ChatColor.GRAY + "Teleporting " + targetPlayer1.getName() + " to " + targetPlayer2.getName());
                    targetPlayer1.sendMessage(ChatColor.GRAY + "You have been teleported to " + targetPlayer2.getName());
                }
 
            }

plugin.yml:

Code:java

name: TqCommands
main: me.bremdecoolste.TqCommands.TqCommands
version: 1.0
description: >
            My FIRST plugin!
commands:
  heal:
    description: Heals a player
  feed:
    description: Heals a player
  tp:
    description: Heals a player

What am I doing wrong?

 

Stacktrace please

 

What does "Stacktrace" mean?
CeramicTitan

 

stacktrace = error, the whole error

 

Show Spoiler Hide Spoiler

Code:

[SEVERE] null
org.bukkit.command.CommandException: Unhandled exception executing command 'tp' in plugin TqCommands v1.0
at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46)
at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:192)
at org.bukkit.craftbukkit.v1_6_R2.CraftServer.dispatchCommand(CraftServer.java:523)
at net.minecraft.server.v1_6_R2.PlayerConnection.handleCommand(PlayerConnection.java:954)
at net.minecraft.server.v1_6_R2.PlayerConnection.chat(PlayerConnection.java:872)
at net.minecraft.server.v1_6_R2.PlayerConnection.a(PlayerConnection.java:829)
at net.minecraft.server.v1_6_R2.Packet3Chat.handle(SourceFile:49)
at net.minecraft.server.v1_6_R2.NetworkManager.b(NetworkManager.java:296)
at net.minecraft.server.v1_6_R2.PlayerConnection.e(PlayerConnection.java:116)
at net.minecraft.server.v1_6_R2.ServerConnection.b(SourceFile:37)
at net.minecraft.server.v1_6_R2.DedicatedServerConnection.b(SourceFile:30)
at net.minecraft.server.v1_6_R2.MinecraftServer.t(MinecraftServer.java:590)
at net.minecraft.server.v1_6_R2.DedicatedServer.t(DedicatedServer.java:226)
at net.minecraft.server.v1_6_R2.MinecraftServer.s(MinecraftServer.java:486)
at net.minecraft.server.v1_6_R2.MinecraftServer.run(MinecraftServer.java:419)
at net.minecraft.server.v1_6_R2.ThreadServerApplication.run(SourceFile:582)
Caused by: java.lang.ArrayIndexOutOfBoundsException: 2
at me.bremdecoolste.TqCommands.TqCommands.onCommand(TqCommands.java:125)
at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44)
... 15 more

CeramicTitan

 

at me.bremdecoolste.TqCommands.TqCommands.onCommand(TqCommands.java:125) What is on line 125 in your tq command class?

EDIT: Your problem is this:
Player targetPlayer1 = player.getServer().getPlayer(args[1]);
Player targetPlayer2 = player.getServer().getPlayer(args[2]);

change to this:
Player targetPlayer1 = player.getServer().getPlayer(args[0]);
Player targetPlayer2 = player.getServer().getPlayer(args[1]);

 

CeramicTitan

Player targetPlayer2 = player.getServer().getPlayer(args[2]);

 

and you should be adding null checks

 

THANKS! It works now! And what do you mean with adding null checks?

 

Instead, let me ask you a question, What do you do if args[0] or one of the target players are not online?

 

Aah, itll send an error message (supposed to do that). So I can check it with if player == null

 

check if the player is not null(!= null) after you declare your player variable