Did I do Something Wrong? - Schematics and Objects

If anyone could help, that would be great. I am making a custom "plots" plugin for my server and I am getting nothing from my code. Zero console output and everything. First, my object class:

Code:java

public class Plot {
private int width;
private int length;
private int height;
private String owner;
private Location centerpoint;
private boolean occupied;
 
public Plot(Location centerpoint, String owner, int width, int length, int height){
this.width = width;
this.length = length;
this.height = height;
this.owner = owner;
this.centerpoint = centerpoint;
}
 
public int getWidth(){
return width;
}
 
public int getLength(){
return length;
}
 
public int getHeight(){
return height;
}
 
public String getOwner(){
return owner;
}
 
public void setOwner(String name){
this.owner = name;
}
 
public Location getCenter(){
return centerpoint;
}
 
public void setCenter(Location name){
this.centerpoint = name;
}
 
public boolean isOccupied(){
return occupied;
}
 
public void setOccupied(boolean b){
this.occupied = b;
}
 
 
}

Pretty simple, basic object programming. My Manager:

Code:java

public class PlotManager{
 
List<Plot> plots;
 
public PlotManager(){
this.plots = new ArrayList<Plot>();
}
 
public List<Plot> getPlots(){
return plots;
}
}

The Methods in my main class (might be where the problem is but I dont know, there is no output from the console):

Code:java

 public static void spawnBuilding(String fileLoc, Plot plot){
File schem = new File(fileLoc);
try {
SchematicUtils.pasteSchematic(Bukkit.getWorld("plots"), new Location(Bukkit.getWorld("plots"), plot.getCenter().getBlockX(), plot.getCenter().getBlockY(), plot.getCenter().getBlockZ()), SchematicUtils.loadSchematic(schem));
} catch (IOException e) {
e.printStackTrace();
}
}
 
public static void generatePlot(Plot plot, Player player, Location loc){
plot = new Plot(loc, player.getName(), 30, 256, 30);
int xlength = plot.getWidth();
int ylength = plot.getHeight();
int zlength = plot.getLength();
 
int x_start = plot.getCenter().getBlockX();
int y_start = plot.getCenter().getBlockY();
int z_start = plot.getCenter().getBlockZ();
 
 
int x_length = x_start + xlength;
int y_length = y_start + ylength;
int z_length = z_start + zlength;
 
for (int x_operate = x_start; x_operate <= x_length; x_operate++) {
for (int y_operate = y_start; y_operate <= y_length; y_operate++) {
for (int z_operate = z_start; z_operate <= z_length; z_operate++) {
loc = new Location(Bukkit.getWorld("plots"), x_operate, y_operate, z_operate);
spawnBuilding("plots.schematic", plot);
plot.setOccupied(true);
}
}
}
}
 
public static boolean previousPlot(Plot plot){
if(plot.isOccupied()){
return true;
}
return false;
}

And finally, the commands:

Code:java

if(commandLabel.equalsIgnoreCase("plot") || commandLabel.equalsIgnoreCase("plots")){
if(sender instanceof Player){
Player player = (Player) sender;
if(args.length == 0){
player.sendMessage(ChatColor.DARK_PURPLE + "§l<><><>< " + ChatColor.GOLD + "§lPLOTS " + ChatColor.DARK_PURPLE + "§l><><><>\n" + ChatColor.YELLOW + "\nOwn Plot: " + ChatColor.BLUE + plotVal(player.getName()));
} else if(args.length == 1){
if(args[0].equalsIgnoreCase("add")){
if(hasPlot.contains(player.getName())){
player.sendMessage(ChatColor.AQUA + "[TheParkMC] " + ChatColor.DARK_RED + "You Already Own A Plot!");
return true;
} else {
if(manager == null){
manager = new PlotManager();
}
for(Plot pl : manager.getPlots()){
if(previousPlot(pl)){
Plot newPlot = new Plot(new Location(Bukkit.getWorld("plots"), pl.getCenter().getBlockX() + 33, pl.getCenter().getBlockY(), pl.getCenter().getBlockZ()), player.getName(), 30, 256, 30);
generatePlot(newPlot, player, newPlot.getCenter());
stringGivePlots(player.getName());
player.teleport(newPlot.getCenter());
player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "WELCOME TO YOUR OWN PERSONAL HUB!");
player.setGameMode(GameMode.CREATIVE);
player.getInventory().clear();
} else if(!previousPlot(pl)){
Plot newPlot = new Plot(new Location(Bukkit.getWorld("plots"), 65.5, 61, 11.5), player.getName(), 30, 256, 30);
generatePlot(newPlot, player, newPlot.getCenter());
stringGivePlots(player.getName());
player.teleport(newPlot.getCenter());
player.sendMessage(ChatColor.GOLD + "" + ChatColor.BOLD + "WELCOME TO YOUR OWN PERSONAL HUB!");
player.setGameMode(GameMode.CREATIVE);
player.getInventory().clear();
}
}
}
} else if(args[0].equalsIgnoreCase("home")){
for(Plot pl : manager.getPlots()){
if(pl.getOwner().contains(player.getName()));
player.teleport(pl.getCenter());
player.getInventory().clear();
player.setGameMode(GameMode.CREATIVE);
}
}
}
} else {
sender.sendMessage("You Can Only Use This Command As A Player!");
}
} 

And yes, in my main class I do have the PlotManager Instance:

Code:java

PlotManager manager;

The command /plot is fine; however, the command /plot add is not. Am I doing something wrong here? The schematic works because I tested it with another plugin and it is the same method. I got the method from this Thread in case any one is wondering: http://forums.bukkit.org/threads/pasting-loading-schematics.87129/

Thank you for your help!

 

Right after "if(args.length == 1)" add a test output. Example: "System.out.println("TEST");" If that doesn't get printed, the error occurs somewhere before the if. Otherwise, the error is within the if. Just move that test print around until you narrow down the field as to where your error occurs.

Chances are, you have an if statement wrong and don't have any chat output before returning.

 

The for-statement probably doesn't get executed, as the list is empty. I don't see anywhere in your code that you add plots to the list...

 

zeeveener Rocoty
I took both of your advises, I checked for the List being null, and I think it is. I added a new method in my PlotManager class to check if the List is null:

Code:java

 public void addDefaultPlot(){
if(plots == null){
Plot plot = new Plot(new Location(Bukkit.getWorld("plots"), 65.5, 61, 11.5), "sdfag3asg", 30, 256, 30);
plots.add(plot);
}
}

Although, I do not know if that works because I am also getting nothing besides the phrase "SHOULD SHOW UP" when I put the test logger in the command.

Code:java

if(manager == null){
manager = new PlotManager();
}
logger.info("SHOULD SHOW UP");
if(manager.getPlots() == null){
logger.info("HERE");
manager.addDefaultPlot();
logger.info("AFTER");
}

I am just checking if the list is null, and nothing happens after that. Is something wrong here? Thanks!

 

A better solution would be to check if

Code:java

manager.getPlots().isEmpty();

 

Why don't you try actually adding stuff to the list?............