ban

package me.Darrionat.easybans.commands;

import org.bukkit.Bukkit;
import org.bukkit.OfflinePlayer;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import me.Darrionat.easybans.Main;
import me.Darrionat.easybans.ui.BanUI;
import me.Darrionat.easybans.utils.Utils;

public class ban implements CommandExecutor {
	private Main plugin;

	public ban(Main plugin) {
		this.plugin = plugin;

		plugin.getCommand("banplayer").setExecutor(this);
	}

	@Override
	public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
		// Not even sure you need this but this is better than nesting all the code in an if statement
        if (!label.equalsIgnoreCase("banplayer")) return false;

        // Check and return false, looks nicer not to nest eveerything in if/else. Returning means no code after this is executed anyway
        if (!(sender instanceof Player)) {
			sender.sendMessage(Utils.chat("&cPlease use /consoleban to perm ban a user. If you want to do a temp ban please log in game."));
			return false;
        }

        Player p = (Player) sender;

        // Once again this stops nesting and makes it look a bit prettier
        if (!p.hasPermission("easybans.ban")) {
            p.sendMessage(Utils.chat("&cYou do not have permission to ban!"));
            return false;
        }

        // Same again, less nesting = better :D
        if (args.length < 1) {
            p.sendMessage(Utils.chat("&cFollow this format."));
            p.sendMessage(Utils.chat("      /banplayer [username]"));
            return false;
        }

        //get the offline player
        OfflinePlayer banPlayer = this.plugin.getOfflinePlayer(args[0]);

        if (banPlayer == null) {
            p.sendMessage(Utils.chat("&cCouldn't find that player? Try again maybe?"));  // probs want to change this message
            return false;
        }

        p.openInventory(BanUI.GUI(p));
        new BanUI(this.plugin, banPlayer);
        return true;
	}

}