Solved Why is this giving me a NullPointerException

Let me just start off by saying that I do like the Bukkit configuration system, but only for simple things, like setting values of different properties and whatnot. However, I cannot stand it for complicated things like large series of lists and all that. I am making a kits plugin for my server, and I want to use a new kind of configuration system where you create folders to house different files that are read and parsed as items. I set this up just fine, but this line of code is causing some troubles...

Code:java

for(File f : new File(Main.dir + "Kits/" + name + "/Contents").listFiles()) contents.add(loadItemStack(f,false));

Now don't think I am stupid...loadKit is in fact a method in this class, but the error is not occurring in there because console would tell me what line of that method the error occurred in if that was the case. So it is something on this line that needs adjusting. Here is the rest of the code for my Kit class:

Code:java

package me.zDev.quantikits;
 
import java.io.File;
import java.nio.charset.Charset;
import java.nio.file.Files;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
 
import org.bukkit.Material;
import org.bukkit.enchantments.Enchantment;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.PlayerInventory;
import org.bukkit.inventory.meta.ItemMeta;
 
public class Kit
{
public String name;
public List<ItemStack> contents;
public ItemStack helmet, chestplate, leggings, boots;
 
public Kit(String key)
{
name = key;
contents = new ArrayList<ItemStack>();
try
{
for(File f : new File(Main.dir + "Kits/" + name + "/Contents").listFiles()) contents.add(loadItemStack(f,false));
helmet = loadItemStack(new File(Main.dir + "Kits/" + name + "/helmet"),true);
chestplate = loadItemStack(new File(Main.dir + "Kits/" + name + "/chestplate"),true);
leggings = loadItemStack(new File(Main.dir + "Kits/" + name + "/leggings"),true);
boots = loadItemStack(new File(Main.dir + "Kits/" + name + "/boots"),true);
 
} catch(Exception e) {
helmet = new ItemStack(Material.DIAMOND_HELMET);
chestplate = new ItemStack(Material.AIR);
leggings = new ItemStack(Material.AIR);
boots = new ItemStack(Material.AIR);
e.printStackTrace();
}
}
 
private ItemStack setName(ItemStack i, String s)
{
ItemMeta im = i.getItemMeta();
im.setDisplayName(s);
i.setItemMeta(im);
return i;
}
 
private ItemStack loadItemStack(File f, boolean armor) throws Exception
{
Material material = armor ? Material.AIR : Material.valueOf(f.getName().toUpperCase());
HashMap<Enchantment,Integer> enchantments = new HashMap<Enchantment,Integer>();
String itemName = "";
short dataValue = 0;
int amount = 1;
List<String> strings = Files.readAllLines(f.toPath(),Charset.forName("UTF-8"));
if(armor)
{
material = Material.valueOf(strings.get(0).toUpperCase());
strings.remove(0);
}
itemName = strings.get(0); strings.remove(0);
amount = Integer.parseInt(strings.get(0)); strings.remove(0);
dataValue = Short.parseShort(strings.get(0)); strings.remove(0);
if(!strings.isEmpty())
{
for(String s : strings)
{
String[] array = s.split(":");
enchantments.put(Enchantment.getByName(array[0].toUpperCase()),Integer.parseInt(array[1]));
}
}
ItemStack itemstack = setName(new ItemStack(material,amount,dataValue),itemName);
itemstack.addUnsafeEnchantments(enchantments);
return itemstack;
}
 
public void loadKit(Player p)
{
PlayerInventory inv = p.getInventory();
inv.setHelmet(helmet);
inv.setChestplate(chestplate);
inv.setLeggings(leggings);
inv.setBoots(boots);
for(ItemStack i : contents) inv.addItem(i);
Main.playerKits.put(p.getName(),this);
}
}

And this is where I create this instance:

Code:java

package me.zDev.quantikits;
 
import java.util.HashMap;
 
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener
{
    public static HashMap<String,Kit> playerKits;
    public static String dir;
 
    public void onEnable()
    {
        getServer().getPluginManager().registerEvents(this,this);
        playerKits = new HashMap<String,Kit>();
        dir = getDataFolder().getAbsolutePath();
    }
 
    @EventHandler
    public void onInteract(PlayerInteractEvent event)
    {
        new Kit("PvP").loadKit(event.getPlayer());
        event.getPlayer().sendMessage(dir);
    }
}

 

Maybe no such directory/no files?

 

Barinade

 

do null checks on every variable you have used

 

WOO HOO Barinade It turns out that I just needed a slash inside of one of my paths :/

SCREW YOU BUKKIT CONFIGURATION SYSTEM

 

The File class is not Bukkit's lol