Syntax error, insert "}" to complete class body. Help Please

I've recently ran into a very annoying problem in my code. I don't know what to do about it, and i really need help. By looking at my code, you will see I am still a beginner. I started taking lessons 2 days ago, and I am only 11 years old. So, i will need a really good explanation in the replies if you could. This is the code: http://pastebin.com/L74ppbex
The error is shown on line 31 which is the last one, and the error displays the following text when i hover my mouse over it: "Syntax error, insert "}" to complete class body"

I would really appreciate good replies, Thank you

 

@samyounes03 Your missing a } also they are really badly indented.

 

@samyounes03
It would help if you learn more of java before coding plugins. Its suprising how 5 mins looking at java books would help you help solve your problem

Fix ..

Code:java

 
 
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if (sender instanceof Player) {
            Player player = (Player) sender;
            //return false; < dont return here
 
            if (cmd.getName().equalsIgnoreCase("hello")) {
                player.sendMessage(ChatColor.AQUA + "Hello, " + ChatColor.RED + player.getName() + ChatColor.AQUA + "!");
                return true;
            }
            if (cmd.getName().equalsIgnoreCase("hi")) {
                player.sendMessage(ChatColor.AQUA + "Hi, " + ChatColor.RED + player.getName() + ChatColor.AQUA + "!");
                return true;
            }
 
        }
        return true;
    }
 
 
 

The problem was the syntax
A method starts with a '{' and must end with a '}' nothing more to it. You had one to many brackets
Again look into java before starting plugins


EDIT: onCommand needs to turn a boolean, if the commandsender isnt an instance of player, then it would still the ifs, but it needs to return true. so at the end just add a return true. Like above
======================================
If i helped set tag prefix as [Solved]
Support: [Like]
Ruptur -

 

+1 op!
You have to indent nicelly your code, or else you will be really fast lost.
here http://pastebin.com/XCYTzhAL is your code, indented and working.

Just for you to know : For every '{' there hase to be an '}'
And i can assure you that this will not working.
You will NEVER have acces to your command!
But I will let you search why

 

This didn't fix it, it still has the exact same error on the last line, and when i add "}" , an error appears next to public boolean and the first "if' statement saying: Unreachable code.

 

Haha, this error is for the one i said you.
I will give you help :
When you have a "Return" in your function, it leave the current funtion and do not execute what is behind.
So if I have this :

Code:

boolean hello()
{
return (true);
print("hello world!\n");
}

It will never EVER print hello, because it quit the function hello() before he can print "hello"

 

Code:

package me.samyounes03.MyFirstPlugin;

import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;

public class Main extends JavaPlugin {
   
    public void onEnable() {
        getLogger().info("Plugin Enabled");
    }
   
    public void onDisable() {
        getLogger().info("Plugin Disbaled");
    }
   
    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        if (sender instanceof Player) {
            Player player = (Player) sender;
        return false;
            if (cmd.getName().equalsIgnoreCase("hello")) {
                player.sendMessage(ChatColor.AQUA + "Hello, " + player.getName() + "!");
               
        return false;
            }
            if (cmd.getName().equalsIgnoreCase("hi")) {
                player.sendMessage(ChatColor.AQUA + "Hi, " + player.getName() + "!");
            }

        return false; }
        return false; } }

Line 23 -> 27 unreachable code.

 

@samyounes03
Like a said above,

Code:

// som code
return false;
player.sendMessage("This wont work");

If you return, then you are telling the program to leave that specify 'method' and anything after it will NOT run.

You simply need to remove the

Code:

return false;

And it will work

======================================
If i helped set tag prefix as [Solved]
Support: [Like]
Ruptur -

 

Also, use if you are using Eclipse, do Ctrl+Shift+F to format your code nicely for you,

 

@samyounes03 PLEASE learn Java before trying to code with the Bukkif API. Even if you learn the basics, it'll make so much more sense. Trying to code with Bukkit before learning Java will get you nowhere and will leave you confused. Again, please learn Java before coding Bukkit.