Solved Creating a freeze plugin

http://pastebin.com/LvJ2cmSW

I'm not sure what im doing wrong if you could help i'd really like that thanks.

 

I'm sorry, i'm really bad at coding, still learning, could you paste the new code with the fix so i can learn, i'm not sure quite what you mean, thanks for the fast reply!

 

You never cancel the event.

 

I don't know what that means, could you explain, thanks!

 

Your answer "probably returning false" is it actually doing that if it is how can i stop it doing that

 

Because reinventing the wheel is always better. Why use e.setCancelled(true) if you can store the location and set it back every time.

 

I'm really confused, in eclipse it says no errors but when im in game and try do /freeze it says an internal error occurred while attempting to perforn this command.

 

I think this is what you mean https://gyazo.com/dae4bfba288509cd11a14d5018ed52fb

 

That using l33tsp34k is annoying?

 

@timtower @Minecraft_GOLD

Looks like he's reading off the end of an array, if I do say so myself.

 

changed the first if statement in the onCommand from args.length ==0 to args.length < 1 still not working, my updated code http://pastebin.com/DEzLmtb9 @mythbusterman

 

@Minecraft_GOLD

It's because you never return early, like you think you are. You REALLY need to format that code.

 

@Minecraft_GOLD
Your spacing is awful, and probably what's causing your problems to be honest.

This is your trouble line:
Player target = Bukkit.getServer().getPlayer(args[0]);

The reason it's your trouble line is because you're only running that if:
if(args.length==0){

So basically what you're telling your plugin is that:

1. If I don't have any arguments;
2. Use the first argument to get a player

?????????

Also, wtf is this:
if(cmd.getName().equalsIgnoreCase("freeze"));{


@mythbusterma
His problem isn't that he's not returning early, his problem is that he has his args[0] call INSIDE his if (args.length == 0)

 

thanks for the reply what should i changed that line to? could you give me a hint as i said im still learning. thanks a ton!

 

Delete it, you do it twice

Code:

if (cmd.getName().equalsIgnoreCase("freeze")) {
                      if (args.length == 0) {
                               if (cmd.getName().equalsIgnoreCase("freeze")); {

Please press Ctrl+Shift+F in eclipse or Ctrl+Alt+L in IntelliJ, and repost the code

and change the args==0 line to args==1

 

Thanks for the reply! i deleted the trouble line and replaced the args==0 to args==1 But now every target. command is coming up red, thanks again ! new cocde - > http://pastebin.com/rxPCCzYH

 

Because in the hurry, you deleted the Player target = Bukkit.getServer().getPlayer() line. Use your head sometimes, and don't recline upon the plugin developement page.

 

Sorry, that was stupid of me and i know i shouldnt be so bad that this but i just needed help it's working now, thanks sir!

 

@Minecraft_GOLD
Edit the title of this thread and mark it as solved then please