Solved Delhome not working

Hey Guys,

I made the commands "Home", "Sethome" and "Delhome" today, but delhome isn't working. Here's the code for sethome:

Code:java

if(args.length == 1) {
if(!cfg.isSet(p.getName() + "." + args[0] + ".world")) {
 
String World = p.getWorld().getName();
double X = p.getLocation().getX();
double Y = p.getLocation().getY();
double Z = p.getLocation().getZ();
double Yaw = p.getLocation().getYaw();
double Pitch = p.getLocation().getPitch();
 
cfg.set(p.getName() + "." + args[0] + ".world", World);
cfg.set(p.getName() + "." + args[0] + ".X", X);
cfg.set(p.getName() + "." + args[0] + ".Y", Y);
cfg.set(p.getName() + "." + args[0] + ".Z", Z);
cfg.set(p.getName() + "." + args[0] + ".Yaw", Yaw);
cfg.set(p.getName() + "." + args[0] + ".Pitch", Pitch);
try {
cfg.save(file);
}
catch (IOException e) {
e.printStackTrace();
}
p.sendMessage(plugin.prefix + "Das Home " + ChatColor.GOLD + args[0] + ChatColor.GREEN + " wurde gesetzt!");
}
else {
p.sendMessage("§cDas Home " + ChatColor.GOLD + args[0] + "§c existiert bereits!" );
}
}
else {
p.sendMessage(plugin.falsch + "sethome <Name>!");
}

And here for delhome:

Code:java

 if(args.length == 1) {
if(cfg.isSet(p.getName() + "." + args[0] + ".world")) {
 
cfg.set(p.getName() + args[0], null);
} else {
p.sendMessage("§cDas Home " + ChatColor.GOLD + args[0] + " §cexistiert nicht!");
}
} else {
p.sendMessage(plugin.falsch + "delhome <Name>!");
}

//Update: I don't get any errors, but nothing happens

 

nivek1212
You forgot the period (or ".") between the player name and the home name on line 4 in the second code box.

edit: also you forgot to save the config after modifying it.

 

You are probably missing a dot here: cfg.set(p.getName() + args[0], null);
Edit: ninja'd

 

Ok, now i have this:

Code:java

 public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
Player p = (Player) sender;
 
if(args.length == 1) {
 
try {
cfg.save(file);
} catch (IOException e) {
e.printStackTrace();
}
 
if(cfg.isSet(p.getName() + "." + args[0] + ".world")) {
 
cfg.set(p.getName() + "." + args[0], null);
 
} else {
p.sendMessage("§cDas Home " + ChatColor.GOLD + args[0] + " §cexistiert nicht!");
}
} else {
p.sendMessage(plugin.falsch + "delhome <Name>!");
}
 
return true;
}

And it works, but when i set a home, and then want to delete if without reloading, it says that it doesn't exists, but deletes it

//For the english guys: When it says, it doesnt exists, it's this :

Code:java

p.sendMessage("§cDas Home " + ChatColor.GOLD + args[0] + " §cexistiert nicht!");

 

nivek1212
Sorry, I don't quite understand you. Try to save the file after deleting the home, the same way you save it when you add a home. You always have to save the file in order for the changes to take place.

 

Ah i got it: The problem was, that I saved the config and then deleted it. I wanted to load the config and delete it, but I messed something up. So the full code is now

Code:java

    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
        Player p = (Player) sender;
 
        if(args.length == 1) {
 
            try {
                cfg.load(file);
            } catch (IOException e) {
                e.printStackTrace();
            } catch (InvalidConfigurationException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }
 
            if(cfg.isSet(p.getName() + "." + args[0] + ".world")) {
 
                cfg.set(p.getName() + "." + args[0], null);
 
                p.sendMessage(plugin.prefix + "Das Home " + ChatColor.GOLD + args[0] + " §awurde gelöscht!");
 
                try {
                    cfg.save(file);
                } catch (IOException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
 
            } else {
                p.sendMessage("§cDas Home " + ChatColor.GOLD + args[0] + " §cexistiert nicht!");
            }
        } else {
            p.sendMessage(plugin.falsch + "delhome <Name>!");
        }
 
        return true;
    }

Thanks for your help