More refactorings

api/src/main/java/com/github/skjolber/packing/api/Box.java

@@ -52,8 +52,13 @@ protected BoxStackValue newStackValue(int dx, int dy, int dz, StackConstraint co
 	protected final long minimumArea;
 	protected final long maximumArea;
 
-	public Box(String id, String name, long volume, int weight, BoxStackValue[] stackValues) {
-		super(id, name);
+	protected final String id;
+	protected final String description;
+
+	public Box(String id, String description, long volume, int weight, BoxStackValue[] stackValues) {
+		this.id = id;
+		this.description = description;
+
 		this.volume = volume;
 		this.weight = weight;
 		this.stackValues = stackValues;
@@ -65,6 +70,16 @@ public Box(String id, String name, long volume, int weight, BoxStackValue[] stac
 			boxStackValue.setStackable(this);
 		}
 	}
+
+	@Override
+	public String getDescription() {
+		return description;
+	}
+
+	@Override
+	public String getId() {
+		return id;
+	}
 
 	@Override
 	public int getWeight() {

api/src/main/java/com/github/skjolber/packing/api/Container.java

@@ -147,9 +147,13 @@ protected ContainerStackValue[] getStackValues() {
 	protected final long minArea;
 	protected final long maxArea;
 
-	public Container(String id, String name, long volume, int emptyWeight, long maxLoadVolume, int maxLoadWeight, long minArea, long maxArea) {
-		super(id, name);
+	protected final String id;
+	protected final String description;
 
+	public Container(String id, String description, long volume, int emptyWeight, long maxLoadVolume, int maxLoadWeight, long minArea, long maxArea) {
+		this.id = id;
+		this.description = description;
+
 		this.emptyWeight = emptyWeight;
 		this.maxLoadVolume = maxLoadVolume;
 		this.maxLoadWeight = maxLoadWeight;
@@ -159,6 +163,16 @@ public Container(String id, String name, long volume, int emptyWeight, long maxL
 		this.maxArea = maxArea;
 	}
 
+	@Override
+	public String getDescription() {
+		return description;
+	}
+
+	@Override
+	public String getId() {
+		return id;
+	}
+
 	@Override
 	public int getWeight() {
 		return emptyWeight + getStack().getWeight();

api/src/main/java/com/github/skjolber/packing/api/ContainerConstraint.java

@@ -1,6 +1,6 @@
 package com.github.skjolber.packing.api;
 
-import com.github.skjolber.packing.api.packager.LoadableItemFilterBuilder;
+import com.github.skjolber.packing.api.packager.StackableItemsFilterBuilder;
 
 /**
  * 
@@ -10,6 +10,6 @@
 
 public interface ContainerConstraint {
 
-	LoadableItemFilterBuilder<?> newLoadableFilterBuilder();
+	StackableItemsFilterBuilder<?> newLoadableFilterBuilder();
 
 }

api/src/main/java/com/github/skjolber/packing/api/PlacementSupportBuilder.java

@@ -13,7 +13,6 @@ public abstract class PlacementSupportBuilder<B extends PlacementSupportBuilder<
 	protected Container container;
 	protected ContainerStackValue stackValue;
 	protected Stack stack;
-	protected StackPlacement stackPlacement;
 
 	public B withContainer(Container container) {
 		this.container = container;

api/src/main/java/com/github/skjolber/packing/api/Stackable.java

@@ -8,28 +8,15 @@ public abstract class Stackable implements Serializable {
 
 	private static final long serialVersionUID = 1L;
 
-	protected final String id;
-	protected final String description;
-
-	public Stackable(String id, String description) {
-		super();
-		this.id = id;
-		this.description = description;
-	}
-
 	public abstract long getVolume();
 
 	public abstract int getWeight();
 
 	public abstract StackValue[] getStackValues();
 
-	public String getDescription() {
-		return description;
-	}
+	public abstract String getDescription();
 
-	public String getId() {
-		return id;
-	}
+	public abstract String getId();
 
 	public List<StackValue> fitsInside(Dimension bound) {
 		List<StackValue> list = new ArrayList<>();

api/src/main/java/com/github/skjolber/packing/api/packager/BoundedStackable.java

@@ -0,0 +1,77 @@
+package com.github.skjolber.packing.api.packager;
+
+import com.github.skjolber.packing.api.StackValue;
+import com.github.skjolber.packing.api.Stackable;
+
+/**
+ * 
+ * Stackable item which fit within certain bounds, i.e. load dimensions of a container.
+ * 
+ */
+
+public class BoundedStackable extends Stackable {
+
+	private static final long serialVersionUID = 1L;
+
+	protected final StackValue[] values;
+	protected final Stackable stackable;
+
+	protected final long minimumArea;
+	protected final long maximumArea;
+
+	public BoundedStackable(Stackable stackable, StackValue[] stackValues) {
+		this.values = stackValues;
+		this.stackable = stackable;
+
+		this.minimumArea = getMinimumArea(stackValues);
+		this.maximumArea = getMinimumArea(stackValues);
+	}
+
+	public Stackable getStackable() {
+		return stackable;
+	}
+
+	public StackValue getStackValue(int index) {
+		return values[index];
+	}
+
+	@Override
+	public long getVolume() {
+		return stackable.getVolume();
+	}
+
+	@Override
+	public int getWeight() {
+		return stackable.getWeight();
+	}
+
+	@Override
+	public StackValue[] getStackValues() {
+		return values;
+	}
+
+	@Override
+	public String getDescription() {
+		return stackable.getDescription();
+	}
+
+	@Override
+	public String getId() {
+		return stackable.getId();
+	}
+
+	@Override
+	public Stackable clone() {
+		return new BoundedStackable(stackable.clone(), values);
+	}
+
+	@Override
+	public long getMinimumArea() {
+		return minimumArea;
+	}
+
+	@Override
+	public long getMaximumArea() {
+		return maximumArea;
+	}
+}

api/src/main/java/com/github/skjolber/packing/api/packager/DefaultStackableItemsFilter.java

@@ -0,0 +1,16 @@
+package com.github.skjolber.packing.api.packager;
+
+public class DefaultStackableItemsFilter implements StackableItemsFilter {
+
+	protected final StackableItems loadableItems;
+
+	public DefaultStackableItemsFilter(StackableItems loadableItems) {
+		this.loadableItems = loadableItems;
+	}
+
+	@Override
+	public void loaded(int index) {
+		// do nothing
+	}
+
+}