Where can I put this method?

Hey there, I have this coins plugin which keeps track of players coins, adds coins, removes coins, does a lot of stuff with coins. Point is, I need to save these coins in a config file. I have a method to do this, but I don't know where to put it. Because it needs access to the config file, it throws and error when it is in a static method. I also need like it to save on a regular basis, like auto-saving...

Here is my whole plugin....
Thanks for your help guys!

Code:java

package me.jack.coins;
 
import java.util.HashMap;
import java.util.logging.Logger;
 
import org.bukkit.Bukkit;
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;
import org.bukkit.scheduler.BukkitRunnable;
 
public class Coins extends JavaPlugin {
 
    public final Logger logger = Logger.getLogger("Minecraft");
    public static Coins plugin;
 
    public final static HashMap<String, Integer> c = new HashMap<String, Integer>();
 
    public final MyApi api = new MyApi();
 
    public void onDisable() {
        plugin = null;
    }
 
    public void onEnable() {
        plugin = this;
        getConfig().options().copyDefaults(true);
        saveConfig();
        for(String s : getConfig().getKeys(true)) {
            Player p = Bukkit.getServer().getPlayer(s);
            c.put(p.getName(), getConfig().getInt(p.getName()));
        }
    }
 
 
    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) {
        Player p = (Player) sender;
        if(commandLabel.equalsIgnoreCase("coins")) {
            if(c.containsKey(p.getName())) {
                ChatUtils.message("You have " + ChatColor.BLUE + c.get(p.getName()) + ChatColor.WHITE + " coins!", p);
            } else {
                ChatUtils.message("Have no coins!", p);
            }
        } else if(commandLabel.equalsIgnoreCase("addcoins") && p.hasPermission("coins.add")) {
            if(args.length == 2) {
                Player targetPlayer = this.getServer().getPlayer(args[0]);
                int a = Integer.parseInt(args[1]);
                MyApi.addCoins(p, a);
                ChatUtils.message("You have given " + ChatColor.BLUE + a + ChatColor.WHITE + " coins to " + ChatColor.BLUE + targetPlayer.getName(), p);
                ChatUtils.message("You have recieved " + ChatColor.BLUE + a + ChatColor.WHITE + " coins!", targetPlayer);
            } else {
                ChatUtils.message("Not enough arguments!", p);
            }
        } else if(commandLabel.equalsIgnoreCase("removecoins") && p.hasPermission("coins.remove")) {
            if(args.length == 2) {
                Player targetPlayer = this.getServer().getPlayer(args[0]);
                int a = Integer.parseInt(args[1]);
                MyApi.removeCoins(p, a);
                ChatUtils.message("You have removed " + ChatColor.BLUE + a + ChatColor.WHITE + " coins from " + ChatColor.BLUE + targetPlayer.getName(), p);
                ChatUtils.message(ChatColor.BLUE + "" + a + ChatColor.WHITE + " coins have been removed from you!", targetPlayer);
            } else {
                ChatUtils.message("Not enough arguments!", p);
            }
        }
        return false;
 
    }
}
 
package me.jack.coins;
 
import org.bukkit.entity.Player;
 
 
public class MyApi {
 
    public static boolean hasCoins(Player p, int a) {
        if(Coins.c.get(p.getName()) >= a) {
            return true;
        } else {
            return false;
        }
    }
 
    public static void addCoins(Player p, int a) {
        if(Coins.c.containsKey(p.getName())) {
            Coins.c.put(p.getName(), Coins.c.get(p.getName()) + a);
        } else {
            Coins.c.put(p.getName(), a);
        }
    }
 
    public static void removeCoins(Player p, int a) {
        if(hasCoins(p, a)) {
            Coins.c.put(p.getName(), Coins.c.get(p.getName()) - a);
        } else {
            Coins.c.put(p.getName(), 0);
        }
    }
 
}

 

JTGaming2012 You should make the API class an enum singleton I don't really see a lot of bukkit programmers that use this, but it's a really easy way to create an easy-access utility class in my opinion. It will remove the need for your static methods as well(which are a lot of times good to avoid using anyways). It's also easy to setup:

Code:

public enum MyAPI
{
    INSTANCE;
 
    public void method1()
    {}
}

Now you can access any public, non-static methods in MyAPI by using this: MyAPI.INSTANCE.method1();

 

Just call MyApp something like CoinManager, make all the methods non-static and make it take your plugin in the constructor: public CoinManager(Coins plugin) { .. }
This way you can access the plugin with its configuration through a variable.

 

Zettelkasten It looks like he is doing that, but in the code he is just calling the static methods and not the instance of his API.

In your onCommand method, instead of using MyAPI.method(), use api.method(). Also, remove static from all of the methods in your api class.