nothing is working...

Hey Guys,

i have some problems with these two classes and i wanted to ask you guys if u know what i did wrong?

PHP:

1.
 
package me.jeykoplays.Commands;

import org.bukkit.Bukkit;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;

public class FlyCommand implements CommandExecutor {

    @Override
    public boolean onCommand(CommandSender cs, Command cmd, String label, String[] args) {
        if(!(cs instanceof Player)) {
            cs.sendMessage("You have to be a Player to perform this Command.");
            return true;
        }
        Player p = (Player) cs;
        if (!p.hasPermission("ServerCore.fly")) {
            p.sendMessage("§cYou have don't have the Permissions to perform this Command.");
            return true;
        }
        if (args.length == (0)) {
            if (p.getAllowFlight() == (true)) {
                p.setAllowFlight(false);
                p.setFlying(false);
                p.sendMessage("§aYou can't fly anymore.");
                return true;
            } else {
                if (p.getAllowFlight() == (false)) {
                    p.setAllowFlight(true);
                    p.setFlying(true);
                    p.sendMessage("§aYou can fly now.");
                    return true;
                }
            }
        }
        if (args.length == (1)) {
            if (args[1].equalsIgnoreCase("on")) {
                p.setAllowFlight(true);
                p.setFlying(true);
                p.sendMessage("§aYou can fly now.");
                return true;
            }
            if (args[1].equalsIgnoreCase("off")) {
                p.setAllowFlight(false);
                p.setFlying(false);
                p.sendMessage("§aYou can't fly anymore.");
                return true;
            }
            if (args[1].equalsIgnoreCase("toggle")) {
                if (p.getAllowFlight() == (true)) {
                    p.setAllowFlight(false);
                    p.setFlying(false);
                    p.sendMessage("§aYou can't fly anymore.");
                    return true;
                } else {
                    if (p.getAllowFlight() == (false)) {
                        p.setAllowFlight(true);
                        p.setFlying(true);
                        p.sendMessage("§aYou can fly now.");
                        return true;
                    }
                }
            }
            if (args.length == (2)) {
                Player pl = Bukkit.getPlayerExact(args[2]);
                if (!pl.isOnline()) {
                    p.sendMessage("§cThis Player is not Online.");
                    return true;
                }
                if (pl.isDead()) {
                    p.sendMessage("§cThis Player is dead.");
                    return true;
                }
                if (args[1].equalsIgnoreCase("on")) {
                    pl.setAllowFlight(true);
                    pl.setFlying(true);
                    pl.sendMessage("§aYou can fly now.");
                    return true;
                }
                if (args[1].equalsIgnoreCase("off")) {
                    pl.setAllowFlight(false);
                    pl.setFlying(false);
                    pl.sendMessage("§aYou can't fly anymore.");
                    return true;
                }
                if (args[1].equals(null)) {
                    p.sendMessage("§cUsage: /fly <ON|OFF|TOGGLE> <PlayerName> ");
                    return true;
                }
                if (args[1].equalsIgnoreCase("toggle")) {
                    if (pl.getAllowFlight() == (true)) {
                        pl.setAllowFlight(false);
                        pl.setFlying(false);
                        pl.sendMessage("§aYou can't fly anymore.");
                        return true;
                    } else {
                        if (pl.getAllowFlight() == (false)) {
                            pl.setAllowFlight(true);
                            pl.setFlying(true);
                            pl.sendMessage("§aYou can fly now.");
                            return true;
                        }
                    }
                }
                if ((args.length == (3)) || (args.length >= (3)) || (args[1].equalsIgnoreCase("help"))) {
                    p.sendMessage("§cUsage: /fly <ON|OFF|TOGGLE> <PlayerName> ");
                    return true;
                }
            }
        }
        return false;
    }
}


and

PHP:

2.
 
package me.jeykoplays.Commands;

import org.bukkit.Bukkit;
import org.bukkit.GameMode;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;

public class GamemodeCommand implements CommandExecutor {

    @Override
    public boolean onCommand(CommandSender cs, Command cmd, String label, String[] args) {
        if(!(cs instanceof Player)) {
            cs.sendMessage("You have to be a Player to perform this Command.");
            return true;
        }
        Player p = (Player) cs;
        if (!p.hasPermission("ServerCore.gamemode")) {
            p.sendMessage("§cYou have don't have the Permissions to perform this Command.");
            return true;
        }
        if (args.length == (0)) {
            if(p.getGameMode() == (GameMode.CREATIVE)) {
                p.setGameMode(GameMode.SURVIVAL);
                p.sendMessage("§aYou have changed your Gamemode to "+p.getGameMode()+".");
                return true;
            } else {
                if(p.getGameMode() == (GameMode.SURVIVAL)) {
                    p.setGameMode(GameMode.CREATIVE);
                    p.sendMessage("§aYou have changed your Gamemode to "+p.getGameMode()+".");
                    return true;
                } else {
                    if (p.getGameMode() == (GameMode.ADVENTURE)) {
                        p.setGameMode(GameMode.CREATIVE);
                        p.sendMessage("§aYou have changed your Gamemode to "+p.getGameMode()+".");
                        return true;
                    }
                }
            }
                
        }
        if (args.length == (1)) {
            if ((args[1].equalsIgnoreCase("survival")) || (args[1].equalsIgnoreCase("0"))) {
                p.setGameMode(GameMode.SURVIVAL);
                p.sendMessage("§aYou have changed your Gamemode to "+p.getGameMode()+".");
                return true;
            }
            if ((args[1].equalsIgnoreCase("creative")) || (args[1].equalsIgnoreCase("1"))) {
                p.setGameMode(GameMode.CREATIVE);
                p.sendMessage("§aYou have changed your Gamemode to "+p.getGameMode()+".");
                return true;
            }
            if ((args[1].equalsIgnoreCase("adventure")) || (args[1].equalsIgnoreCase("2"))) {
                p.setGameMode(GameMode.ADVENTURE);
                p.sendMessage("§aYou have changed your Gamemode to "+p.getGameMode()+".");
                return true;
            }
        }
        if (args.length == (2)) {
            Player pl = Bukkit.getPlayerExact(args[2]);
            if (!pl.isOnline()) {
                p.sendMessage("§cThis Player is not Online.");
                return true;
            }
            if (pl.isDead()) {
                p.sendMessage("§cThis Player is dead.");
                return true;
            }
            if ((args[1].equalsIgnoreCase("survival")) || (args[1].equalsIgnoreCase("0"))) {
                pl.setGameMode(GameMode.SURVIVAL);
                pl.sendMessage("§a"+p.getName()+"has changed your Gamemode to "+p.getGameMode()+".");
                p.sendMessage("§aYou have changed"+pl+"'s Gamemode to "+p.getGameMode()+".");
                return true;
            }
            if ((args[1].equalsIgnoreCase("creative")) || (args[1].equalsIgnoreCase("1"))) {
                pl.setGameMode(GameMode.CREATIVE);
                pl.sendMessage("§a"+p.getName()+"has changed your Gamemode to "+p.getGameMode()+".");
                p.sendMessage("§aYou have changed"+pl+"'s Gamemode to "+p.getGameMode()+".");
                return true;
            }
            if ((args[1].equalsIgnoreCase("adventure")) || (args[1].equalsIgnoreCase("2"))) {
                pl.setGameMode(GameMode.ADVENTURE);
                pl.sendMessage("§a"+p.getName()+"has changed your Gamemode to "+p.getGameMode()+".");
                p.sendMessage("§aYou have changed"+pl+"'s Gamemode to "+p.getGameMode()+".");
                return true;
            }
            if (args[1].equals(null)) {
                p.sendMessage("§cUsage: /gamemode <SURVIVAL|CREATIVE|ADVENTURE> <Name>");
                return true;
            }
        }
        if ((args.length == (3)) || (args.length >= (3)) || (args[1].equalsIgnoreCase("help"))) {
            p.sendMessage("§cUsage: /gamemode <SURVIVAL|CREATIVE|ADVENTURE> <Name>");
            return true;
        }
        return false;
    }
}



THANKS A LOT!

 

What exactly is your problem...? Post your Stacktrace if you have any.

 

No there is no Stacktrace ... Both Plugins are not working only /fly or /gamemode Command is working but not the /gamemode 1 or something and a Playername ... same at the fly command

 

JeykoExample
I see what you did, and it should be throwing an error to Console. Indexes in Java start with 0, but in your commands, you check for args[1] for the first argument. It should be args[0].