A "}" bug.

main.Survivalgames.java

Code:java

package main;
 
import java.io.File;
import java.io.IOException;
import java.util.ArrayList;
import java.util.HashMap;
 
import org.bukkit.Location;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
import org.bukkit.entity.Player;
import org.bukkit.inventory.Inventory;
import org.bukkit.plugin.java.JavaPlugin;
 
 
public class SurvivalGames extends JavaPlugin {
 
    public String prefix ="§7[§bSurvivalGames§7] ";
 
    public ArrayList<Player> dead = new ArrayList<Player>();
    public ArrayList<Player> online = new ArrayList<Player>();
 
    public HashMap<Location, Inventory> sgchest = new HashMap<Location, Inventory>();
 
    public File file = new File("plugins/SurvivalGames", "config.yml");
    public FileConfiguration cfg = YamlConfiguration.loadConfiguration(this.file);
 
    public boolean joinable;
    public boolean onspawn;
    public boolean friendly;
 
    public int expid;
    public int exp;
 
    public int startid;
    public int start;
    public int friede;
    public int friedeid;
 
    public void onEnable() {
 
        System.out.println("[SurvivalGames Asacs] Aktiviert!");
 
        this.joinable = true;
        this.friendly = true;
        this.onspawn = false;
 
        this.exp = 120;
        this.start = 15;
        this.friede = 180;
 
    }
 
    public boolean onCommand(CommandSender sender, Command cmd, String label,
                String[] args) {
 
 
        Player p = (Player)sender;
 
        if(cmd.getName().equalsIgnoreCase("sg")) {
 
        if(args.length == 0) {
            p.sendMessage("§7Das ist ein Survivalgames Plguin.");
        }
 
        if(args.length == 1) {
            if(args[0].equalsIgnoreCase("start")) {
                if(p.hasPermission("sg.start")) {
                    this.exp = 11;
                    }
                }
            }
 
            if(args.length == 2) {
 
            if(args[0].equalsIgnoreCase("setlobby")) {
                if(p.hasPermission("sg.setlobby")) {
                    this.cfg.set("SurvivalGames.Spawn.World", p.getWorld().getName());
                    this.cfg.set("SurvivalGames.Spawn.X", Double.valueOf(p.getLocation().getX()));
                    this.cfg.set("SurvivalGames.Spawn.Y", Double.valueOf(p.getLocation().getY()));
                    this.cfg.set("SurvivalGames.Spawn.Z", Double.valueOf(p.getLocation().getZ()));
                    this.cfg.set("SurvivalGames.Spawn.Yaw", Double.valueOf(p.getLocation().getYaw()));
                    this.cfg.set("SurvivalGames.Spawn.Pitch", Double.valueOf(p.getLocation().getPitch()));
                try {
                    this.cfg.save(this.file);
                    p.sendMessage(this.prefix + "§aDie Map Spawn wurde gesetzt.");   
                    } catch (IOException e) {
                    e.printStackTrace();
                    }       
                }
            }   
        }
    }
        return false;
 
    }
 
 

The last one "}" shows "Syntax error, insert "}" to complete ClassBodySurvivalGames.java"


plugin.yml

Code:

name: SurvivalGames
version: 1.0
author: Asacs
website: www.asacsplay.me
main: main.SurvivalGames

Any stuff i forgett?

 

To me the return from command looks like its in wrong place... try moving it one } up. Im on my phone right now so I might be wrong.

 

You need one more } at the end of your file to close your class's body.

 

nope nothink change.

DxDy

Yep i had that too, but when I ingame not one command works. Any idea why?

 

Return statement is outside the method

EDIT: Never mind. I counted wrong

 

I think you really need to look at your brace-placement. Try to have a proper intendation, that goes a long way in solving your problems here Each brace has to have one closing brace to it. And where they are defines what is executed.

 

Dydy im new at java coding & not english.. so i dont get anythink what you said.

 

The awesome thing about using proper indentation is that you will never run into these kinds of problems. What DxDy is talking about is the different between this:

Code:

public void someMethod() {
    if (somethingIsTrue) {
        doSomething();
    } else {
        doSomethingElse();
    }
}

And this:

Code:

public void someMethod()
                {    if (somethingIsTrue)
{           doSomething();
    }
else                {
        doSomethingElse();    }
}

Do it the right way the first time (it's not that hard, especially considering all the help you get from modern IDEs and/or text editors), and you'll never have to worry about not being able to read your own code, which is exactly your problem here.

Think of indentation and proper formatting of your code as the equivalent of formatting in your typical newspaper article. Which do you prefer? This:

Code:

   This is a load of crap that I just
came up with, just to give an example
of proper formatting in article-type
texts. The purpose of this text is to
demonstrate just how important proper
formatting is for the readability of
a written work.
 
   Clearly, there is no doubt that if
code (as well as any written work) is
formatted properly, it is very simple
to spot syntactical errors, because
they will stand out from the rest.

Or this:

Code:

                    This is a
load of       crap that I just came  
up       with,just to give        an
example      of proper        formatt
ing in article - typetexts.The purpos
e of this text is todemonstrate    ju
st how important properform
atting is for the read  ability ofa w
ritten work. Clearly
 
  ,there is         no    doubt    th
at if code (as well as any writ  ten
 work) isforma tted properly, it    i
s very      simpleto spot syntacti cal 
errors, beca use..they will stand out
from the rest      .

Edit: And just FYI, the Java compiler is very, very stable. If it says there is an error, it's most likely because there is. This is not a bug, but an invalid Java source file. There are times when the static analysis phase of a compiler fails (and it usually means there is a better/simpler/more logic way of doing what you're trying to do), but chances are very small that you'll run into them, especially at this stage.