Refactor: Rename AbstractVisitor

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/check/AbstractVisitor.java → CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/check/CheckableUtilsVisitor.java

@@ -37,11 +37,6 @@ import de.hft.stuttgart.citydoctor2.datastructure.TinObject;
 import de.hft.stuttgart.citydoctor2.datastructure.TransportationObject;
 import de.hft.stuttgart.citydoctor2.datastructure.Vegetation;
 import de.hft.stuttgart.citydoctor2.datastructure.WaterObject;
-import de.hft.stuttgart.citydoctor2.parser.ParserConfiguration;
-
-import java.util.Collections;
-import java.util.Map;
-import java.util.Set;
 
 /**
  *
@@ -52,7 +47,7 @@ import java.util.Set;
  * @author Matthias Betz
  *
  */
-public abstract non-sealed class AbstractVisitor implements CheckableVisitor {
+public abstract non-sealed class CheckableUtilsVisitor implements CheckableVisitor {
 
 	//Util Visitors have no dependencies
 	public boolean canExecute(Checkable c) {

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/check/CheckableVisitor.java

@@ -20,7 +20,7 @@ import de.hft.stuttgart.citydoctor2.datastructure.TransportationObject;
 import de.hft.stuttgart.citydoctor2.datastructure.Vegetation;
 import de.hft.stuttgart.citydoctor2.datastructure.WaterObject;
 
-public sealed interface CheckableVisitor permits Check, AbstractVisitor {
+public sealed interface CheckableVisitor permits Check, CheckableUtilsVisitor {
 
     boolean isValidator();
 

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/datastructure/CityDoctorModel.java

@@ -18,10 +18,8 @@
  */
 package de.hft.stuttgart.citydoctor2.datastructure;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
-import de.hft.stuttgart.citydoctor2.check.Check;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckError;
-import de.hft.stuttgart.citydoctor2.check.CheckableVisitor;
 import de.hft.stuttgart.citydoctor2.exceptions.CityDoctorWriteException;
 import de.hft.stuttgart.citydoctor2.parser.ParserConfiguration;
 import de.hft.stuttgart.citydoctor2.writer.CityGMLWriterUtils;
@@ -148,7 +146,7 @@ public class CityDoctorModel {
         folder.mkdirs();
         createFeatureStream().forEach(co -> {
             Map<Lod, Set<Polygon>> polygonMap = new EnumMap<>(Lod.class);
-            CheckableVisitor c = new AbstractVisitor() {
+            CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
 
                 @Override
                 public void check(Geometry geom) {

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/visitors/CheckableErrorCollector.java

 package de.hft.stuttgart.citydoctor2.utils.visitors;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckError;
 import de.hft.stuttgart.citydoctor2.check.Checkable;
 
@@ -9,7 +9,7 @@ import java.util.List;
 /**
  * This Visitor collects all CheckErrors in the datastructure of a checkable.
  */
-public class CheckableErrorCollector extends AbstractVisitor {
+public class CheckableErrorCollector extends CheckableUtilsVisitor {
 
     List<CheckError> errors;
 

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/visitors/CityObjectCollector.java

 package de.hft.stuttgart.citydoctor2.utils.visitors;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.datastructure.CityObject;
 
 import java.util.HashSet;
@@ -9,7 +9,7 @@ import java.util.Set;
 /**
  * This Visitor collects all sub-CityObjects that are in the datastructure of a CityObject (including itself) in a Set.
  */
-public class CityObjectCollector extends AbstractVisitor {
+public class CityObjectCollector extends CheckableUtilsVisitor {
 
     private final Set<CityObject> objects = new HashSet<>();
 

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/visitors/ClearMetaInformationVisitor.java

 package de.hft.stuttgart.citydoctor2.utils.visitors;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.datastructure.Geometry;
 
 /**
  * This Visitor removes the metadata created by the {@link PrepareForCheckingVisitor} for the low-memory consumption mode.
  */
-public class ClearMetaInformationVisitor extends AbstractVisitor {
+public class ClearMetaInformationVisitor extends CheckableUtilsVisitor {
 
     @Override
     public void check(Geometry geometry) {

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/visitors/ClearVisitorResultsVisitor.java

 package de.hft.stuttgart.citydoctor2.utils.visitors;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.Checkable;
 
 /**
  * This Visitor removes the check results from a CityObject and the objects in its datastructure.
  */
-public class ClearVisitorResultsVisitor extends AbstractVisitor {
+public class ClearVisitorResultsVisitor extends CheckableUtilsVisitor {
 
     @Override
     public void check(Checkable checkable) {

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/visitors/ContainsAnyErrorVisitor.java

 package de.hft.stuttgart.citydoctor2.utils.visitors;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.Checkable;
 import de.hft.stuttgart.citydoctor2.utils.CheckErrorFound;
 import de.hft.stuttgart.citydoctor2.utils.ThrowUtils;
@@ -11,7 +11,7 @@ import de.hft.stuttgart.citydoctor2.utils.ThrowUtils;
  * This visitor's check method terminates early by throwing an unchecked {@link CheckErrorFound},
  * use {@link #checkObject(Checkable) ContainsAnyErrorVisitor.checkObject(Checkable)} to handle the Throwable.
  */
-public class ContainsAnyErrorVisitor extends AbstractVisitor {
+public class ContainsAnyErrorVisitor extends CheckableUtilsVisitor {
 
     /**
      * Checks if a Checkable, or any Object in its datastructure, contains any error.

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/visitors/ContainsErrorVisitor.java

 package de.hft.stuttgart.citydoctor2.utils.visitors;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckId;
 import de.hft.stuttgart.citydoctor2.check.Checkable;
 import de.hft.stuttgart.citydoctor2.utils.CheckErrorFound;
@@ -12,7 +12,7 @@ import de.hft.stuttgart.citydoctor2.utils.ThrowUtils;
  * This visitor's check method terminates early by throwing an unchecked {@link CheckErrorFound},
  * use {@link #checkObject(Checkable, CheckId) ContainsAnyErrorVisitor.checkObject(Checkable, CheckId)} to handle the Throwable.
  */
-public class ContainsErrorVisitor extends AbstractVisitor {
+public class ContainsErrorVisitor extends CheckableUtilsVisitor {
 
     private CheckId checkId;
 

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/visitors/GmlGeometryRebuilder.java

 package de.hft.stuttgart.citydoctor2.utils.visitors;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.datastructure.CityObject;
 import de.hft.stuttgart.citydoctor2.parser.ParserConfiguration;
 import org.citygml4j.core.util.geometry.GeometryFactory;
@@ -11,7 +11,7 @@ import java.util.Objects;
  * This Visitor recreates the Geometries of the original CityGml object from the associated Geometries in CityDoctor's
  * internal model.
  */
-public class GmlGeometryRebuilder extends AbstractVisitor {
+public class GmlGeometryRebuilder extends CheckableUtilsVisitor {
 
     private final GeometryFactory factory;
     private final ParserConfiguration config;

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/visitors/PrepareForCheckingVisitor.java

 package de.hft.stuttgart.citydoctor2.utils.visitors;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.datastructure.Geometry;
 
 /**
  * This Visitor prepares the {@link Geometry Geometries} in a Checkable for low-memory consumption mode.
  */
-public class PrepareForCheckingVisitor extends AbstractVisitor {
+public class PrepareForCheckingVisitor extends CheckableUtilsVisitor {
 
     @Override
     public void check(Geometry geom) {

CityDoctorParent/CityDoctorModel/src/main/java/de/hft/stuttgart/citydoctor2/utils/visitors/UnsetGeometriesVisitor.java

 package de.hft.stuttgart.citydoctor2.utils.visitors;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.datastructure.CityObject;
 
 /**
  * This Visitor removes the GmlGeometries from the linked original GmlObjects in a CityObject's datastructure to reduce
  * memory bloat.
  */
-public class UnsetGeometriesVisitor extends AbstractVisitor {
+public class UnsetGeometriesVisitor extends CheckableUtilsVisitor {
 
 
     @Override

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/AbstractBuildingTest.java

@@ -40,7 +40,7 @@ import org.mockito.Mockito;
 import org.xmlobjects.gml.model.geometry.aggregates.MultiSurfaceProperty;
 import org.xmlobjects.gml.model.geometry.primitives.SolidProperty;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckError;
 import de.hft.stuttgart.citydoctor2.check.CheckId;
 import de.hft.stuttgart.citydoctor2.check.CheckResult;
@@ -59,7 +59,7 @@ public class AbstractBuildingTest {
 		AtomicInteger abCounter = new AtomicInteger(0);
 		AtomicInteger bsCounter = new AtomicInteger(0);
 		AtomicInteger biCounter = new AtomicInteger(0);
-		AbstractVisitor c = new AbstractVisitor() {
+		CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
 
 			@Override
 			public void check(AbstractBuilding ab) {
@@ -76,7 +76,7 @@ public class AbstractBuildingTest {
 		assertEquals(1, bsCounter.intValue());
 		assertEquals(0, biCounter.intValue());
 
-		AbstractVisitor c2 = new AbstractVisitor() {
+		CheckableUtilsVisitor c2 = new CheckableUtilsVisitor() {
 
 			@Override
 			public void check(Installation bi) {
@@ -106,7 +106,7 @@ public class AbstractBuildingTest {
 		ab.addBoundarySurface(mockBSurface);
 		ab.addGeometry(mockGeometry);
 
-		AbstractVisitor c = new AbstractVisitor() {
+		CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
 		};
 		ab.accept(c);
 

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/AbstractRoomTest.java

 package de.hft.stuttgart.citydoctor2.datastructure;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckError;
 import de.hft.stuttgart.citydoctor2.check.CheckId;
 import de.hft.stuttgart.citydoctor2.check.CheckResult;
@@ -50,7 +50,7 @@ public class AbstractRoomTest {
         abRoom.addBoundarySurface(mockBoundarySurface);
         abRoom.addGeometry(geometry);
 
-        AbstractVisitor c = new AbstractVisitor() {
+        CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
         };
         abRoom.accept(c);
 

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/AbstractTunnelTest.java

 package de.hft.stuttgart.citydoctor2.datastructure;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckError;
 import de.hft.stuttgart.citydoctor2.check.CheckId;
 import de.hft.stuttgart.citydoctor2.check.CheckResult;
@@ -52,7 +52,7 @@ public class AbstractTunnelTest {
         at.addBoundarySurface(mockBoundarySurface);
         at.addGeometry(mockGeometry);
 
-        AbstractVisitor c = new AbstractVisitor() {
+        CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
         };
         at.accept(c);
 

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/BoundarySurfaceTest.java

@@ -41,7 +41,7 @@ import org.junit.Test;
 import org.mockito.Mockito;
 import org.xmlobjects.gml.model.geometry.aggregates.MultiSurfaceProperty;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckError;
 import de.hft.stuttgart.citydoctor2.check.CheckId;
 import de.hft.stuttgart.citydoctor2.check.CheckResult;
@@ -58,7 +58,7 @@ public class BoundarySurfaceTest {
 		Opening oMock = Mockito.mock(Opening.class);
 		bs.addOpening(oMock);
 
-		AbstractVisitor c = new AbstractVisitor() {
+		CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
 		};
 		bs.accept(c);
 
@@ -71,7 +71,7 @@ public class BoundarySurfaceTest {
 		Opening oMock = Mockito.mock(Opening.class);
 		bs.addOpening(oMock);
 		AtomicInteger bsCounter = new AtomicInteger(0);
-		AbstractVisitor c = new AbstractVisitor() {
+		CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
 
 			@Override
 			public void check(BoundarySurface bs) {
@@ -89,7 +89,7 @@ public class BoundarySurfaceTest {
 		Opening oMock = Mockito.mock(Opening.class);
 		bs.addOpening(oMock);
 		AtomicInteger bsCounter = new AtomicInteger(0);
-		AbstractVisitor c = new AbstractVisitor() {
+		CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
 
 			@Override
 			public void check(BridgeObject bo) {

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/BridgeObjectTest.java

@@ -39,7 +39,7 @@ import org.citygml4j.core.util.geometry.GeometryFactory;
 import org.citygml4j.core.visitor.ObjectVisitor;
 import org.junit.Test;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckError;
 import de.hft.stuttgart.citydoctor2.check.CheckId;
 import de.hft.stuttgart.citydoctor2.check.CheckResult;
@@ -58,7 +58,7 @@ public class BridgeObjectTest {
 		bo.addBoundarySurface(new BoundarySurface(null));
 		AtomicInteger boCounter = new AtomicInteger(0);
 		AtomicInteger bsCounter = new AtomicInteger(0);
-		AbstractVisitor c = new AbstractVisitor() {
+		CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
 
 			@Override
 			public void check(BridgeObject bo) {
@@ -75,7 +75,7 @@ public class BridgeObjectTest {
 		assertEquals(1, bsCounter.intValue());
 
 		AtomicInteger boCounter2 = new AtomicInteger(0);
-		c = new AbstractVisitor() {
+		c = new CheckableUtilsVisitor() {
 		};
 		bo.accept(c);
 		assertEquals(0, boCounter2.intValue());
@@ -101,7 +101,7 @@ public class BridgeObjectTest {
 		bo.addBoundarySurface(bsMock);
 		bo.addGeometry(geomMock);
 
-		AbstractVisitor c = new AbstractVisitor() {
+		CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
 		};
 		bo.accept(c);
 

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/BuildingTest.java

@@ -26,7 +26,7 @@ import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import org.citygml4j.core.model.construction.RoofSurface;
 import org.junit.Test;
 import org.mockito.Mockito;
@@ -151,7 +151,7 @@ public class BuildingTest {
 		bp.addStorey(mockStorey);
 
 
-		AbstractVisitor c = new AbstractVisitor() {
+		CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
 		};
 		b.accept(c);
 

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/CityObjectTest.java

 package de.hft.stuttgart.citydoctor2.datastructure;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckError;
 import de.hft.stuttgart.citydoctor2.check.CheckId;
 import de.hft.stuttgart.citydoctor2.check.CheckResult;
@@ -31,7 +31,7 @@ public class CityObjectTest {
         Geometry mockGeometry = mock(Geometry.class);
         co.addGeometry(mockGeometry);
 
-        AbstractVisitor check = new AbstractVisitor() {
+        CheckableUtilsVisitor check = new CheckableUtilsVisitor() {
         };
 
         co.accept(check);

CityDoctorParent/CityDoctorModel/src/test/java/de/hft/stuttgart/citydoctor2/datastructure/GenericCityObjectTest.java

 package de.hft.stuttgart.citydoctor2.datastructure;
 
-import de.hft.stuttgart.citydoctor2.check.AbstractVisitor;
+import de.hft.stuttgart.citydoctor2.check.CheckableUtilsVisitor;
 import de.hft.stuttgart.citydoctor2.check.CheckError;
 import de.hft.stuttgart.citydoctor2.check.CheckId;
 import de.hft.stuttgart.citydoctor2.check.CheckResult;
@@ -30,7 +30,7 @@ public class GenericCityObjectTest {
         gco.addBoundarySurface(mockSurface);
         gco.addGeometry(mockGeometry);
 
-        AbstractVisitor c = new AbstractVisitor() {
+        CheckableUtilsVisitor c = new CheckableUtilsVisitor() {
         };
         gco.accept(c);