Solved Simple } error,

Hey, i have a error at the bottom } is red underline.

Code:java

package com.saicogaming;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.enchantments.Enchantment;
import org.bukkit.entity.EntityType;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
public class StartCommandExecutor implements CommandExecutor {
 
    start plugin;
 
    public StartCommandExecutor(start instance){
        plugin = instance;
    }
 
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label,
            String[] args) {
        Player p = (Player) sender;
        if (cmd.getName().equalsIgnoreCase("enchant")) {
            // Make sure that the player specified exactly one argument (the
            // name of the player to give enchanted item too).
            if (args.length != 1) {
                // For The Player himself
                ItemStack item = new ItemStack(Material.DIAMOND_SWORD);
                item.addEnchantment(Enchantment.DAMAGE_ALL, 5);
                p.getInventory().addItem(item);
                sender.sendMessage(ChatColor.BLUE
                        + "[Enchantment]"
                        + ChatColor.RED
                        + p.getName()
                        + " Here is a Diamond Sword, Enchanted with Sharpness 5");
                return true;
            }
 
            // Make sure the send is a player.
            if (!(sender instanceof Player)) {
                sender.sendMessage("Only players another player an item");
                sender.sendMessage("This is an arbitrary requirement for demonstration purposes only.");
                return true;
            }
 
            // Get the player who should be set on fire. Remember that indecies
            // start with 0, not 1.
            Player target = Bukkit.getServer().getPlayer(args[0]);
            // Make sure the player is online.
            if (target == null) {
                sender.sendMessage(args[0] + " is not currently online.");
                return true;
            }
            ItemStack item = new ItemStack(Material.DIAMOND_SWORD);
            item.addEnchantment(Enchantment.DAMAGE_ALL, 5);
            target.getInventory().addItem(item);
            sender.sendMessage(ChatColor.BLUE + "[Enchantment] "
                    + ChatColor.RED + args[0]
                    + " Here is a Diamond Sword, Enchanted with Sharpness 100");
            return true;
 
            // Ignite Area
 
        } else if (cmd.getName().equalsIgnoreCase("ignite")) {
            // Make sure that the player specified exactly one argument (the
            // name of the player to ignite).
            if (args.length != 1) {
                // When onCommand() returns false, the help message associated
                // with that command is displayed.
                return false;
            }
 
            // Make sure the sender is a player.
            if (!(sender instanceof Player)) {
                sender.sendMessage("Only players can set other players on fire.");
                sender.sendMessage("This is an arbitrary requirement for demonstration purposes only.");
                return true;
            }
 
            // Get the player who should be set on fire. Remember that indecies
            // start with 0, not 1.
            Player target = Bukkit.getServer().getPlayer(args[0]);
 
            // Make sure the player is online.
            if (target == null) {
                sender.sendMessage(args[0] + " is not currently online.");
                return true;
            }
 
            // Sets the player on fire for 1,000 ticks (there are ~20 ticks in
            // second, so 50 seconds total).
            target.setFireTicks(1000);
            target.setAllowFlight(true);
            return true;
 
 
            // This is for the Heal Command
 
 
 
        } else if (cmd.getName().equalsIgnoreCase("heal")) {
            if (args.length != 1) {
                // if no player is added
                p.setFireTicks(0);
                p.setFoodLevel(20);
                ItemStack item = new ItemStack(Material.GOLDEN_APPLE);
                p.getInventory().addItem(item);
                p.sendMessage((p.getName() + " §6You have been healed."));
                return true;
            }
 
            Player target = Bukkit.getServer().getPlayer(args[0]);
            // Make sure the player is online.
            if (target == null) {
                sender.sendMessage(args[0] + " is not currently online.");
                // find the player, if he/she is online
                return true;
            }
            target.addPotionEffect(new PotionEffect(
                    PotionEffectType.REGENERATION, 1000, 10));
            // gives player regen 10
            target.setFireTicks(0);
            target.setFoodLevel(20);
            target.sendMessage((args[0] + " §6You have been healed."));
            // Takes fire off and also gives full food and displays msg
            return true;
        } else if (cmd.getName().equalsIgnoreCase("sex")) {
            // Make sure that the player specified exactly one argument (the
            // name of the player to ignite).
            if (args.length == 0) {
                p.sendMessage(ChatColor.AQUA + "</sex> <targetPlayer> ");
            }
            if (args.length == 1) {
                Player targetPlayer = p.getServer().getPlayer(args[0]);
                Player p2 = targetPlayer;
                Location targetlocation = targetPlayer.getLocation();
                targetPlayer.getWorld().spawnEntity(targetlocation,
                        EntityType.PIG);
                targetPlayer.getWorld().spawnEntity(targetlocation,
                        EntityType.PIG);
                ItemStack item = new ItemStack(Material.CARROT_ITEM);
                p2.getInventory().addItem(item);
                p2.getInventory().addItem(item);
                sender.sendMessage(ChatColor.BLUE + "[Sex] " + ChatColor.RED
                        + args[0] + " Have some wheat and cause some Lovin");
                return true;
                //HighJmp Command to jump high!
            }else if (cmd.getName().equalsIgnoreCase("highjmp")) {
                if (args.length == 0) {
                    p.sendMessage( ChatColor.AQUA + "</highjmp> <on/off> " );
                }if (args[0].equalsIgnoreCase("on"));{
                    p.addPotionEffect(new PotionEffect(PotionEffectType.JUMP, Integer.MAX_VALUE, 10));
                    p.setAllowFlight(true);
                    p.sendMessage("§5[HighJmp] §6You have been given Jump 10, to turn off do /highjmp off");
                    return true;
 
                }if (args[0].equalsIgnoreCase("off")) {
                    for(PotionEffect pe : p.getActivePotionEffects()){
                        if(pe.getType() == PotionEffectType.JUMP){
                            p.removePotionEffect(pe.getType());
                            p.setAllowFlight(false);
                            p.sendMessage("§5[HighJmp] §6Your Jump is gone :( do /highjmp on to get it back!");
                        }
 
                    }
                    return true;       
                }
                return false;
            }
 
        }
        }

can you see why?

 

You are missing a return statement.

 

Does it say one of those messages when you hover it?

1. Syntax error, insert "}" to complete ClassBody
2. Syntax error on token "}", delete this token

 

nirajm34 Because you need 1 more bracket. Here is a fixed class:

PHP:

package com.saicogaming;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.enchantments.Enchantment;
import org.bukkit.entity.EntityType;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
public class StartCommandExecutor implements CommandExecutor {
 
    start plugin;
 
    public StartCommandExecutor(start instance){
        plugin = instance;
    }
 
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String label,
            String[] args) {
        Player p = (Player) sender;
        if (cmd.getName().equalsIgnoreCase("enchant")) {
            // Make sure that the player specified exactly one argument (the
            // name of the player to give enchanted item too).
            if (args.length != 1) {
                // For The Player himself
                ItemStack item = new ItemStack(Material.DIAMOND_SWORD);
                item.addEnchantment(Enchantment.DAMAGE_ALL, 5);
                p.getInventory().addItem(item);
                sender.sendMessage(ChatColor.BLUE
                        + "[Enchantment]"
                        + ChatColor.RED
                        + p.getName()
                        + " Here is a Diamond Sword, Enchanted with Sharpness 5");
                return true;
            }
 
            // Make sure the send is a player.
            if (!(sender instanceof Player)) {
                sender.sendMessage("Only players another player an item");
                sender.sendMessage("This is an arbitrary requirement for demonstration purposes only.");
                return true;
            }
 
            // Get the player who should be set on fire. Remember that indecies
            // start with 0, not 1.
            Player target = Bukkit.getServer().getPlayer(args[0]);
            // Make sure the player is online.
            if (target == null) {
                sender.sendMessage(args[0] + " is not currently online.");
                return true;
            }
            ItemStack item = new ItemStack(Material.DIAMOND_SWORD);
            item.addEnchantment(Enchantment.DAMAGE_ALL, 5);
            target.getInventory().addItem(item);
            sender.sendMessage(ChatColor.BLUE + "[Enchantment] "
                    + ChatColor.RED + args[0]
                            + " Here is a Diamond Sword, Enchanted with Sharpness 100");
            return true;
 
            // Ignite Area
 
        } else if (cmd.getName().equalsIgnoreCase("ignite")) {
            // Make sure that the player specified exactly one argument (the
            // name of the player to ignite).
            if (args.length != 1) {
                // When onCommand() returns false, the help message associated
                // with that command is displayed.
                return false;
            }
 
            // Make sure the sender is a player.
            if (!(sender instanceof Player)) {
                sender.sendMessage("Only players can set other players on fire.");
                sender.sendMessage("This is an arbitrary requirement for demonstration purposes only.");
                return true;
            }
 
            // Get the player who should be set on fire. Remember that indecies
            // start with 0, not 1.
            Player target = Bukkit.getServer().getPlayer(args[0]);
 
            // Make sure the player is online.
            if (target == null) {
                sender.sendMessage(args[0] + " is not currently online.");
                return true;
            }
 
            // Sets the player on fire for 1,000 ticks (there are ~20 ticks in
            // second, so 50 seconds total).
            target.setFireTicks(1000);
            target.setAllowFlight(true);
            return true;
 
 
            // This is for the Heal Command
 
 
 
        } else if (cmd.getName().equalsIgnoreCase("heal")) {
            if (args.length != 1) {
                // if no player is added
                p.setFireTicks(0);
                p.setFoodLevel(20);
                ItemStack item = new ItemStack(Material.GOLDEN_APPLE);
                p.getInventory().addItem(item);
                p.sendMessage((p.getName() + " §6You have been healed."));
                return true;
            }
 
            Player target = Bukkit.getServer().getPlayer(args[0]);
            // Make sure the player is online.
            if (target == null) {
                sender.sendMessage(args[0] + " is not currently online.");
                // find the player, if he/she is online
                return true;
            }
            target.addPotionEffect(new PotionEffect(
                    PotionEffectType.REGENERATION, 1000, 10));
            // gives player regen 10
            target.setFireTicks(0);
            target.setFoodLevel(20);
            target.sendMessage((args[0] + " §6You have been healed."));
            // Takes fire off and also gives full food and displays msg
            return true;
        } else if (cmd.getName().equalsIgnoreCase("sex")) {
            // Make sure that the player specified exactly one argument (the
            // name of the player to ignite).
            if (args.length == 0) {
                p.sendMessage(ChatColor.AQUA + "</sex> <targetPlayer> ");
            }
            if (args.length == 1) {
                Player targetPlayer = p.getServer().getPlayer(args[0]);
                Player p2 = targetPlayer;
                Location targetlocation = targetPlayer.getLocation();
                targetPlayer.getWorld().spawnEntity(targetlocation,
                        EntityType.PIG);
                targetPlayer.getWorld().spawnEntity(targetlocation,
                        EntityType.PIG);
                ItemStack item = new ItemStack(Material.CARROT_ITEM);
                p2.getInventory().addItem(item);
                p2.getInventory().addItem(item);
                sender.sendMessage(ChatColor.BLUE + "[Sex] " + ChatColor.RED
                        + args[0] + " Have some wheat and cause some Lovin");
                return true;
                //HighJmp Command to jump high!
            }else if (cmd.getName().equalsIgnoreCase("highjmp")) {
                if (args.length == 0) {
                    p.sendMessage( ChatColor.AQUA + "</highjmp> <on/off> " );
                }if (args[0].equalsIgnoreCase("on"));{
                    p.addPotionEffect(new PotionEffect(PotionEffectType.JUMP, Integer.MAX_VALUE, 10));
                    p.setAllowFlight(true);
                    p.sendMessage("§5[HighJmp] §6You have been given Jump 10, to turn off do /highjmp off");
                    return true;
 
                }if (args[0].equalsIgnoreCase("off")) {
                    for(PotionEffect pe : p.getActivePotionEffects()){
                        if(pe.getType() == PotionEffectType.JUMP){
                            p.removePotionEffect(pe.getType());
                            p.setAllowFlight(false);
                            p.sendMessage("§5[HighJmp] §6Your Jump is gone :( do /highjmp on to get it back!");
                        }
 
                    }
                    return true;     
                }
            }
 
        }
        return false;
    }
}

 

yes

 

I believe you need a close brace } after line 153.
Doing this would return true if on /XXX they use 1 parameter. CUrrently you have /XXX enclosing /highjump.

Oh sorry this was posted before I saw the other posts

 

and thanks, but this is what happened when pasted in

http://gyazo.com/66902830af1a26bd777b6f5dc1464055.png

 

nirajm34 Just remove the return true.

 

That code still needs a curly brace after return true at:
return true;
//HighJmp Command to jump high!
}else if (cmd.getName().equalsIgnoreCase("highjmp")) {