Solved Stats plugin no errors in eclipse or server console.

I created a stats plugin today where a player types /stats and it would show them there Deaths and kills aswell as there XP. I also added a /stats (Player) command but for some reason when I type /stats in minecraft nothing pops up Here is my code.


Code removed.

 

DamnHippo Debug your code. Print out checks, find out where it is stopping, and try and figure out why

 

Make sure you commands are registered, your command executor is set in your onEnable() etc. Also I'm sure your way of using a config works for storing information, but let me present a maybe easier and cleaner way to do it. Create a class like a settings manager and for kills, deaths etc create a file and file config like;

Code:java

 private FileConfiguration killsconfig;
private File killsfile;
private FileConfiguration deathsconfig;
private File deathsfile;
private Plugin p;
 

Then create a setup method and in it put this;

Code:java

 if(!p.getDataFolder().exists()) p.getDataFolder().mkdir();
killsfile = new File(p.getDataFolder(), "kills.yml");
if(!killsfile.exists()){
try{ killsfile.createNewFile(); }
catch(Exception e) { e.printStackTrace(); }
}
if(!p.getDataFolder().exists()) p.getDataFolder().mkdir();
deathsfile = new File(p.getDataFolder(), "deaths.yml");
if(!deathsfile.exists()){
try{ deathsfile.createNewFile(); }
catch(Exception e) { e.printStackTrace(); }
}

This basically creates the database folder if it already hasn't, then creates the file if it already hasn't.
Then say below;

Code:java

 killsconfig = YamlConfiguration.loadConfiguration(killsfile);
deathsconfig = YamlConfiguration.loadConfiguration(deathsfile);

Now call this setup method in the onEnable(), on a side note it should be something like this;

Code:java

 public void setup(Plugin p){
this.p = p;
}

Then create methods for getting and setting paths in the file like this;

Code:java

 
public int getKills(String p){
return killsconfig.getInt("kills." + p);
}
public void addKills(String p){
setKills(p, getKills(p) + 1);
}
public void setKills(String p, int amount){
killsconfig.set("kills." + p, amount);
saveKills();
}
public int getDeaths(String p){
return deathsconfig.getInt("deaths." + p);
}
public void addDeaths(String p){
setDeaths(p, getDeaths(p) + 1);
}
public void setDeaths(String p, int amount){
deathsconfig.set("deaths." + p, amount);
saveDeaths();
}
public void saveKills(){
try{ killsconfig.save(killsfile); }
catch(Exception e) { e.printStackTrace(); }
}
public void saveDeaths(){
try{ deathsconfig.save(deathsfile); }
catch(Exception e) { e.printStackTrace(); }
}

 

DamnHippo Debugging your code is a basic you must know how to do and it will help you fix a vast amount of problems you may have in the future, that is probably the best thing for you to do.
Another thing I'd say is that we need your class that extends JavaPlugin aswell as the one provided

 

That is because using saveConfig() or saveDefaultConfig() does that for you, however my way is more, hmm controllable in a sense.

Code:java

getCommand("help").setExecutor(new Help());
getCommand("promote").setExecutor(new Promote());
getCommand("me").setExecutor(new Me());
getCommand("msg").setExecutor(new Msg());
getCommand("killme").setExecutor(new killme());
getCommand("kill").setExecutor(new kill());
getCommand("slap").setExecutor(new Slap());
getCommand("rules").setExecutor(new rules());
getCommand("banish").setExecutor(new banish());
getCommand("kit").setExecutor(new kit());
getCommand("potion").setExecutor(new potion());
getCommand("stats").setExecutor(new stats(this));

Maybe because it is given "this" and all the others aren't?

Normally when you use "this" it refers to the body of text for example in the main class it could refer to a plugin or a listener, is that what you mean? What is the actual parameter it takes?

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

 

itzrobotix 'this' refers to the current object it is in, which in the OPs case, his class that extends JavaPlugin. Since his constructor takes in the same object that he is referring to with this, it is a valid use. The keyword 'this' is not the problem.
DamnHippo Like I've said before, you need to debug your code to see where it gets stuck. Whether this be a simple Bukkit.broadcastMessage() or a System.out.println(), you need something to indicate each step of the program and those messages will tell you where it gets stuck

 

Thanks for clearing that up. On a side note that is the first thing he should have done. To see where the code stops, very useful and obvious.

 

itzrobotix It is indeed the first thing that he should've done, and the first thing I suggested (which he may or may not have completely ignored my suggestion)

 

Then he wonders why he isn't getting anywhere.

 

That's an issue. No curly bracket.

 

Are you trying to imply that I do? Also you asked us for help, if you think we are bugging you trying to teach you how to help yourself, sling your hook.

 

Hippo again, I'm on 24/7 most of the time.~ I can always help.