Help with a NullPointer

While trying to create a plugin, I ran into a problem with my runnable class. When ever the runnable goes it says there is a nullpointer to my 26th line which is:

Code:java

Bukkit.broadcastMessage("" + plugin.getConfig().get("DrugDealer.PayAmount"));

But I know it isn't null as I am generating the config when it is enabled.
Any Ideas?

 

Maybe it's a lower/higher case problem, and then, you don't need to put an empty string before your plugin.getConfig()...

 

Webster56 herp derp it's supposed to be an int so ignore the

Code:java

plugin.getConfig().get("DrugDealer.PayAmount");

it is supposed to be

Code:java

plugin.getConfig().getInt("DrugDealer.PayAmount");

My bad (which still doesn't work)

 

boysnnoco
Show your config. The nullpointer is most likely referring to something null in the config.

 

AoH_Ruthless
This is my config:

Code:

Money:
  Start: 50
Gangsta:
  PayTime: 10
  PayAmount: 50
DrugDealer:
  PayTime: 10
  PayAmount: 50
Citizen:
  PayTime: 10
  PayAmount: 50
Hobo:
  PayTime: 10
  PayAmount: 50

and this is the code generating the config:

Code:java

final Configuration c = this.getConfig();
 
        c.addDefault("Money.Start", 50);
        /**
        * TODO: Add Support for differnt types of units (months, years,seconds,days,weeks, ect)
        */
        c.addDefault("Gangsta.PayTime", 10);
        c.addDefault("Gangsta.PayAmount", 50);
        c.addDefault("DrugDealer.PayTime", 10);
        c.addDefault("DrugDealer.PayAmount", 50);
        c.addDefault("Citizen.PayTime", 10);
        c.addDefault("Citizen.PayAmount", 50);
        c.addDefault("Hobo.PayTime", 10);
        c.addDefault("Hobo.PayAmount", 50);
        c.options().copyDefaults(true);
        saveConfig();

 

boysnnoco
Hmm, I don't see anything wrong with the config.
Can you post the runnable class?

 

AoH_Ruthless Sure here is my runnable class:

Code:

package me.boysnnoco.Jobs;
 
import java.io.File;
 
import me.boysnnoco.DarkRP.Main;
 
import org.bukkit.Bukkit;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
import org.bukkit.entity.Player;
 
public class DrugDealer implements Runnable {
 
    public Main plugin;
    public DrugDealer(Main instance) {
        plugin = instance;
    }
    public void run() {
        for(Player p : Bukkit.getOnlinePlayers()){
            FileConfiguration config = null;
            File file = new File("plugins"+File.separator +"DarkRP"+File.separator+"users" +File.separator+p.getName());
            if(file.exists()){
                config = YamlConfiguration.loadConfiguration(file);
                if(config.getString("Player.CurrentClass").equalsIgnoreCase("citizen")){
                 
                    Bukkit.broadcastMessage(""  + plugin.getConfig().getInt("DrugDealer.PayAmount"));
                    //plugin.econ.Add(p, plugin.getConfig().getInt("DrugDealer.PayAmount"));
                }
            }else{
                p.sendMessage("An error has occured while trying to pay you money for 'Drug Dealer' please report this to any online staff!");
            }
        }
 
    }
 
}

 

boysnnoco
Try this:

Code:java

private Main plugin;

instead of:

Code:java

public Main plugin;

 

AoH_Ruthless That doesn't seem to work, it is still generating a nullpointererror towards line 26.

 

Try doing a getString() ?

I don't see why you're getting an Int for a Broadcast unless you're trying to do an auto-broadcast, and if you are, do something like this:

Code:java

public void run() {
                                Bukkit.getServer().broadcastMessage(ChatColor.GREEN + settings.getConfig().getString("DrugDealer.PayAmount"));
                        }
                }, 60, 100);

 

iiHeroo Im getting an int to pay a person a certain amount of money using my economy system

 

boysnnoco

Is the even being created?

 

iiHeroo It is being created

 

So what do you want to broadcast? Like, what would the broadcast look like?
For ex:

Bukkit.broadcastMessage("hi, hi");

 

iiHeroo the broadcasting part of the plugin was just to track down what was null (which was the plugin.getConfig() part), what im trying to do is use my adding method to add money after a certain amount of time (as you can see commented out). btw, the broadcast should look like 50 (as that is the amount I specify in the configs).

 

If you want the broadcast to say 50, you need to get a string or simply put "50"

 

iiHeroo forget the whole broadcast thing, the point is that I put it there to get the problem of the nullpointer. the nullpointer is coming from plugin.getConfig().getInt("DrugDealer.PayAmount"); and I need that pay amount in order to pay a player using the job a certain amount of money.

bump
Need this fixed!

Ok after messing around a bit I seemed to get something that doesn't throw any errors but, it is being counted as 0.
This is what I have so far:

Code:java

package me.boysnnoco.Jobs;
 
import java.io.File;
 
 
import org.bukkit.Bukkit;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;
import org.bukkit.entity.Player;
 
public class DrugDealer implements Runnable {
    @Override
    public void run() {
        for(Player p : Bukkit.getOnlinePlayers()){
            FileConfiguration config = null;
            File file = new File("plugins"+File.separator +"DarkRP"+File.separator+"users" +File.separator+p.getName());
            File f = new File("plugins" + File.separator + "DarkRP" + File.separator + "config");
            FileConfiguration c = null;
            if(file.exists()){
                config = YamlConfiguration.loadConfiguration(file);
                c = YamlConfiguration.loadConfiguration(f);
                if(config.getString("Player.CurrentClass").toLowerCase().equalsIgnoreCase("citizen")){
                    Bukkit.broadcastMessage("" + c.getInt("DrugDealer.PayAmount"));
                }
            }else{
                p.sendMessage("An error has occured while trying to pay you money for 'Drug Dealer' please report this to any online staff!");
            }
        }
 
    }
 
}
 

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