Refactoring.

[sixth-3d.git] / src / main / java / eu / svjatoslav / sixth / e3d / renderer / raster / RenderAggregator.java

diff --git a/src/main/java/eu/svjatoslav/sixth/e3d/renderer/raster/RenderAggregator.java b/src/main/java/eu/svjatoslav/sixth/e3d/renderer/raster/RenderAggregator.java
index 049938c..3384351 100644 (file)
--- a/src/main/java/eu/svjatoslav/sixth/e3d/renderer/raster/RenderAggregator.java
+++ b/src/main/java/eu/svjatoslav/sixth/e3d/renderer/raster/RenderAggregator.java
@@ -14,21 +14,16 @@ import eu.svjatoslav.sixth.e3d.renderer.raster.shapes.AbstractCoordinateShape;
 
 import java.io.Serializable;
 import java.util.ArrayList;
-import java.util.Collections;
 import java.util.Comparator;
 
 public class RenderAggregator {
 
-    ArrayList<AbstractCoordinateShape> shapes = new ArrayList<>();
-    ShapesComparator comparator = new ShapesComparator();
+    private ArrayList<AbstractCoordinateShape> shapes = new ArrayList<>();
+    private ShapesZIndexComparator comparator = new ShapesZIndexComparator();
 
     public void paint(final RenderingContext renderBuffer) {
-
-        Collections.sort(shapes, comparator);
-
-        for (final AbstractCoordinateShape shape : shapes)
-            shape.paint(renderBuffer);
-
+        shapes.sort(comparator);
+        shapes.forEach(shape -> shape.paint(renderBuffer));
     }
 
     public void queueShapeForRendering(final AbstractCoordinateShape shape) {
@@ -39,7 +34,7 @@ public class RenderAggregator {
         shapes.clear();
     }
 
-    static class ShapesComparator implements Comparator<AbstractCoordinateShape>, Serializable {
+    static class ShapesZIndexComparator implements Comparator<AbstractCoordinateShape>, Serializable {
 
         @Override
         public int compare(final AbstractCoordinateShape o1, final AbstractCoordinateShape o2) {