Adding spawns using Config and Ints

Hi!

I am trying to figure out how to add special Team spawns to my minigame using a config file.

Current Code

Code:java

if (cmd.getName().equalsIgnoreCase("CoreDefense")) {
if (args.length == 0) {
helpMessage(p);
 
} else if (args[0].equalsIgnoreCase("addred")) {
if(!(m.getConfig().contains("RedSpawn"))){
m.getConfig().set("RedSpawn.1.X", x);
m.getConfig().set("RedSpawn.1.Y", y);
m.getConfig().set("RedSpawn.1.Z", z);
m.saveConfig();
return true;
}
int i = m.getConfig().getInt("RedSpawn."); // Doesn't work :)
 
} else if (args[0].equalsIgnoreCase("addblue")) {
}
return true;
}
 
return false;
}

- When the player types the command for the first time it saves the location in a list like so:

Code:

RedSpawn
    '1':
        X: x
        Y: y
        Z: z

It works fine.
But when the player types it again there must be a second location saved with, as parent '2'.

I tried getting the int out of the path and then doing ' i++ ' but that didn't work, or I did it wrong.

Can someone help me?

What it should be:

First time command:

Code:

RedSpawn
    '1':
        X: x
        Y: y
        Z: z

Second time command:

Code:

RedSpawn
  '1':
        X: x
        Y: y
        Z: z
    '2':
        X: x
        Y: y
        Z: z

third etc.

Code:

RedSpawn
  '1':
        X: x
        Y: y
        Z: z
  '2':
        X: x
        Y: y
        Z: z
    '3':
        X: x
        Y: y
        Z: z

 

Dubehh
Wheres the i++?

 

Mrawesomecookie
I removed it.
Since it didn't work..

 

Dubehh use ConfigurationSection#getKeys() (getting your ConfigurationSection using FileConfiguration#getConfigurationSection)
Call it (pass in false to make sure you don't get children keys), then check the Collection's size that it returns, and then add one to that, and it'll give you the next number you want to set for

 

xTigerRebornx

Im stuck ,

I've read the jd.bukkit thing, but I can't figure this out.

Code:java

            } else if (args[0].equalsIgnoreCase("addred")) {
                if(!(m.getConfig().contains("RedSpawn"))){
                    m.getConfig().set("RedSpawn.1.X", x);
                    m.getConfig().set("RedSpawn.1.Y", y);
                    m.getConfig().set("RedSpawn.1.Z", z);
                    m.saveConfig();
                    return true;
                }
                    int i = m.getConfig().getConfigurationSection("RedSpawn.").getInt("") + 1;
                    p.sendMessage(i + " is a nummber"); /*    Debug message        */

 

Dubehh What are you trying to do in more specific terms?

 

TheWolfBadger

Ill just recreate it for understanding

- Player types /coredefense addred
- The players location (x,y,z) gets saved into a config (so I can load them in a respawnEvent)
- Once a player types /coredefense addred again, the location gets added into the config again.
So if I would spam /coredefense addred there would be a infinite list of locations.


I was trying to do it with ints but if there is a easier / better way to do it, please reply

 

Dubehh So you don't want them to repeat the command if it's set already or you want it to overwrite the last one if there already is one?

 

I don't have specific demands, I just want to save locations into a config (in a list)
like so:

Code:

RedSpawn:
    - Location 1: //first time  '/Coredefense addred '
        - X: x
        - Y: y
        - Z: z
    - Location 2: //second time  '/Coredefense addred '
        - X: x
        - Y: y
        - Z: z
    - Location 3: //third time  '/Coredefense addred '
        - X: x
        - Y: y
        - Z: z
    - Location 4: //fourth time  '/Coredefense addred '
        - X: x
        - Y: y
        - Z: z

TheWolfBadger

See it as adding warps. But forget the naming of warps.

So if I would compare this to a warp plugin
it would be /warp instead of /warp <name>

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Dubehh Ahh so I recommend you save the locations as strings.

Code:java

String locSerialized = loc.getWorld().getName() + "," + loc.getX() + "," + loc.getY() + "," + loc.getZ() + "," + loc.getPitch() + "," + loc.getYaw());
String[] locString = locSerialized.split(",");
 
Location fromString = new Location(Bukkit.getWorld(locString[0]), Double.parseDouble(locString[1) ... etc );

Credit to ZeusAllMighty11 for this code on an old post! . Didn't feel like writing my own out .

 

TheWolfBadger
But if you have a easier / better method, feel free to share

TheWolfBadger

could u show me how to use this in my code?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Code:java

public String locToString(Location loc) {
String locSerialized = loc.getWorld().getName() + "," + loc.getX() + "," + loc.getY() + "," + loc.getZ() + "," + loc.getPitch() + "," + loc.getYaw());
return locSerialized;
}
public Location stringToLoc(String s) {
String[] locString = locSerialized.split(",");
 
Location fromString = new Location(Bukkit.getWorld(locString[0]), Double.parseDouble(locString[1), Double.parseDouble(locString[2]), Double.parseDouble(locString[3]), Float.parseFloat(locString[4]), Float.parseFloat(locString[5]));
return fromString;
}

Dubehh

 

TheWolfBadger

I might sound like a total noob, but I have no clue how to use these methods with my command.
Could u show it how to use it in my current code? (onCommand method)

Currently tried;

Code:java

if (cmd.getName().equalsIgnoreCase("CoreDefense")) {
            if (args.length == 0) {
                helpMessage(p);
 
            } else if (args[0].equalsIgnoreCase("addred")) {
                if(!(m.getConfig().contains("RedSpawn"))){
                    m.getConfig().set("RedSpawn.1.X", x);
                    m.getConfig().set("RedSpawn.1.Y", y);
                    m.getConfig().set("RedSpawn.1.Z", z);
                    m.saveConfig();
                    return true;
                }
                List<String> redSpawn = m.getConfig().getStringList("RedSpawn");
                    String loc = m.spawnloc.locToString(p.getLocation());
                    redSpawn.add(loc);
                    m.getConfig().set("RedSpawn", redSpawn);
 

I have your methods in a seperate class btw.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Dubehh

Code:java

 
if (cmd.getName().equalsIgnoreCase("CoreDefense")) {
            if (args.length == 0) {
                helpMessage(p);
 
            } else if (args[0].equalsIgnoreCase("addred")) {
                if(!(m.getConfig().contains("RedSpawn"))){
                    m.getConfig().set("RedSpawn.1", locToString(p.getLocation())); //You should be able to figure out how to //make this a list and how to utilize getting the different locations.  
                    m.saveConfig();
                    return true;
                }
//This should work: 
                List<String> redSpawn = m.getConfig().getStringList("RedSpawn");
                    String loc = m.spawnloc.locToString(p.getLocation());
                    redSpawn.add(loc);
                    m.getConfig().set("RedSpawn", redSpawn);
public String locToString(Location loc) {
String locSerialized = loc.getWorld().getName() + "," + loc.getX() + "," + loc.getY() + "," + loc.getZ() + "," + loc.getPitch() + "," + loc.getYaw());
return locSerialized;
}
public Location stringToLoc(String s) {
String[] locString = locSerialized.split(",");
 
Location fromString = new Location(Bukkit.getWorld(locString[0]), Double.parseDouble(locString[1), Double.parseDouble(locString[2]), Double.parseDouble(locString[3]), Float.parseFloat(locString[4]), Float.parseFloat(locString[5]));
return fromString;
}

Dubehh This is what you should use for the addRed command now:

Code:java

List<String> redSpawn = m.getConfig().getStringList("RedSpawn");
String loc = m.spawnloc.locToString(p.getLocation());
redSpawn.add(loc);
m.getConfig().set("RedSpawn", redSpawn);
m.saveConfig();
 

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

TheWolfBadger

Code:java

public Location stringToLoc(String s) {
        String[] locString = locSerialized.split(","); //here
 
        Location fromString = new Location(Bukkit.getWorld(locString[0]),
                Double.parseDouble(locString[1]),
                Double.parseDouble(locString[2]),
                Double.parseDouble(locString[3]),
                Float.parseFloat(locString[4]), Float.parseFloat(locString[5]));
        return fromString;
    }

locSerialized cannot be resolved

TheWolfBadger

Code:java

if (cmd.getName().equalsIgnoreCase("CoreDefense")) {
            if (args.length == 0) {
                helpMessage(p);
 
            } else if (args[0].equalsIgnoreCase("addred")) {
                List<String> redSpawn = m.getConfig().getStringList("RedSpawn");
                String loc = m.spawnloc.locToString(p.getLocation());
                redSpawn.add(loc);
                m.getConfig().set("RedSpawn", redSpawn);
                m.saveConfig();
 
            } else if (args[0].equalsIgnoreCase("addblue")) {
            }
            return true;
        }

Current code.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Dubehh

No offence but do you know java?

Where was locSerialized declared as a variable?

 

Dubehh Looks good to me. What's the problem?
Change locSerialized to 's'

 

TheWolfBadger
Thanks man! Sorry for all of those questions, but I've been rewriting my plugin the entire day, and I couldn't think straight
Like for u!

TheWolfBadger

(Hopefully) last question. When I am calling a location out of it,
I use this:

Code:java

Location loc2 = m.spawnloc.stringToLoc(m.getConfig().getString("RedSpawns."));

But that doesn't work.

Could u tell me how to use stringToLoc properly?
This is seriously my last question about this topic.
You gave me a ton of information about saving it, and it works. Only answer if you want it, since u did alot for me already, thank you. Seriously, I am grateful for the time you've put into me.

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Dubehh You are saving it as RedSpawn, not "RedSpawns.", so why are you fetching that instead of what you are saving?

 

TheWolfBadger

Hi, it is not working/ or im doin' it totally wrong.

Command:

Code:java

} else {
                Location loc = m.spawnloc.stringToLoc(m.getConfig().getString("RedSpawn"));
                if(p.getLocation().equals(loc)){
                    p.sendMessage("Confirmed");
                }
                else{
                    p.sendMessage("denied");
                }

Error:

Code:

org.bukkit.command.CommandException: Unhandled exception executing command 'coredefense' in plugin C
oreDefense v0.1
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46) ~[craftbukkit.jar:git-Buk
kit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:199) ~[craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchCommand(CraftServer.java:544) ~[craftb
ukkit.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.PlayerConnection.handleCommand(PlayerConnection.java:932) [c
raftbukkit.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.PlayerConnection.a(PlayerConnection.java:814) [craftbukkit.j
ar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.PacketPlayInChat.a(PacketPlayInChat.java:28) [craftbukkit.ja
r:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.PacketPlayInChat.handle(PacketPlayInChat.java:47) [craftbukk
it.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.NetworkManager.a(NetworkManager.java:146) [craftbukkit.jar:g
it-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.ServerConnection.c(SourceFile:134) [craftbukkit.jar:git-Bukk
it-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.u(MinecraftServer.java:655) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:250) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:545) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java:457) [craftbukkit.j
ar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:617) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
Caused by: java.lang.NullPointerException
        at com.dubehh.Locations.SpawnLocations.stringToLoc(SpawnLocations.java:27) ~[?:?]
        at com.dubehh.CoreDefense.MainCmd.onCommand(MainCmd.java:48) ~[?:?]
        at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44) ~[craftbukkit.jar:git-Buk
kit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
        ... 13 more

SpawnLocations.java:27

Code:java

public Location stringToLoc(String s) {
        String[] locString = s.split(","); // This one
 
        Location fromString = new Location(Bukkit.getWorld(locString[0]),
                Double.parseDouble(locString[1]),
                Double.parseDouble(locString[2]),
                Double.parseDouble(locString[3]),
                Float.parseFloat(locString[4]), Float.parseFloat(locString[5]));
        return fromString;
    }

Thanks

 

Bump
TheWolfBadger (Sorry for adding you, but you gave me the methods, so you might see what im doin wrong)

TheWolfBadger

Bump

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

TheWolfBadger
Methods (only x, y z and World)

Code:java

public String locToString(Location loc) {
        String locSerialized = loc.getWorld().getName() + "," + loc.getX()
                + "," + loc.getY() + "," + loc.getZ();
        return locSerialized;
    }
 
    public Location stringToLoc(String s) {
        String[] locString = s.split(",");
 
        Location fromString = new Location(Bukkit.getWorld(locString[0]),
                Double.parseDouble(locString[1]),
                Double.parseDouble(locString[2]), Double.parseDouble(locString[3]));
        return fromString;
    }
 

Trying code:

Code:java

Location loc = m.spawnloc.stringToLoc(m.getConfig().getString("RedSpawns"));
                if(p.getLocation().equals(loc)){
                    p.sendMessage("works");
                }
                else{
                    p.sendMessage("doesnt work");
                }

- How can I call a location if there are multiple locations saved, since they aren't numbered (or if I could call them randomly, which is even better)
- Can't get a location out of a string

Error

Code:text

[23:04:32 ERROR]: null
org.bukkit.command.CommandException: Unhandled exception executing command 'coredefense' in plugin C
oreDefense v0.1
at org.bukkit.command.PluginCommand.execute(PluginCommand.java:46) ~[craftbukkit.jar:git-Buk
kit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at org.bukkit.command.SimpleCommandMap.dispatch(SimpleCommandMap.java:199) ~[craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at org.bukkit.craftbukkit.v1_7_R1.CraftServer.dispatchCommand(CraftServer.java:544) ~[craftb
ukkit.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.PlayerConnection.handleCommand(PlayerConnection.java:932) [c
raftbukkit.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.PlayerConnection.a(PlayerConnection.java:814) [craftbukkit.j
ar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.PacketPlayInChat.a(PacketPlayInChat.java:28) [craftbukkit.ja
r:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.PacketPlayInChat.handle(PacketPlayInChat.java:47) [craftbukk
it.jar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.NetworkManager.a(NetworkManager.java:146) [craftbukkit.jar:g
it-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.ServerConnection.c(SourceFile:134) [craftbukkit.jar:git-Bukk
it-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.MinecraftServer.u(MinecraftServer.java:655) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.DedicatedServer.u(DedicatedServer.java:250) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.MinecraftServer.t(MinecraftServer.java:545) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.MinecraftServer.run(MinecraftServer.java:457) [craftbukkit.j
ar:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
at net.minecraft.server.v1_7_R1.ThreadServerApplication.run(SourceFile:617) [craftbukkit.jar
:git-Bukkit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
Caused by: java.lang.NumberFormatException: For input string: "-14650.596521360496]"
at sun.misc.FloatingDecimal.readJavaFormatString(Unknown Source) ~[?:1.7.0_55]
at java.lang.Double.parseDouble(Unknown Source) ~[?:1.7.0_55]
at com.dubehh.Locations.SpawnLocations.stringToLoc(SpawnLocations.java:30) ~[?:?]
at com.dubehh.CoreDefense.MainCmd.onCommand(MainCmd.java:72) ~[?:?]
at org.bukkit.command.PluginCommand.execute(PluginCommand.java:44) ~[craftbukkit.jar:git-Buk
kit-1.7.2-R0.2-14-g15b04d8-b2991jnks]
... 13 more
>
 
 
 

EDIT by Moderator: merged posts, please use the edit button instead of double posting.