Checkstyle: Remove unused imports and make formatting changes

Author: flTobi

engine/src/main/java/org/destinationsol/FactionDisplay.java

@@ -19,12 +19,11 @@
 import com.badlogic.gdx.Input;
 import com.badlogic.gdx.graphics.Color;
 import com.badlogic.gdx.math.Vector2;
-
-import org.destinationsol.game.SolGame;
+import org.destinationsol.game.FactionInfo;
 import org.destinationsol.game.ObjectManager;
 import org.destinationsol.game.SolCam;
+import org.destinationsol.game.SolGame;
 import org.destinationsol.game.SolObject;
-import org.destinationsol.game.FactionInfo;
 import org.destinationsol.game.ship.SolShip;
 import org.destinationsol.ui.SolInputManager;
 import org.destinationsol.ui.UiDrawer;
@@ -48,12 +47,12 @@ public void drawFactionNames(SolGame game, UiDrawer uiDrawer, SolInputManager in
         }
         // angle must be zero as the camera angles on planets mess up the text display
         if (isPressed && cam.getAngle() == 0 && !inputManager.isScreenOn(game.getScreens().mapScreen)) {
-            for(SolObject obj: objManager.getObjects()) {
+            for (SolObject obj : objManager.getObjects()) {
                 if (obj instanceof SolShip) {
                     SolShip ship = (SolShip) obj;
                     Vector2 drawPosition = cam.worldToScreen(ship);
                     uiDrawer.drawString(ship.getFactionName(), drawPosition.x * SolApplication.displayDimensions.getRatio(),
-                            drawPosition.y -.1f, 1, false, Color.valueOf(info.getFactionColors().get(ship.getFactionID()).toString()));
+                            drawPosition.y - .1f, 1, false, Color.valueOf(info.getFactionColors().get(ship.getFactionID()).toString()));
                 }
             }
         }

engine/src/main/java/org/destinationsol/assets/Assets.java

@@ -38,8 +38,6 @@
 import java.util.Optional;
 import java.util.Set;
 
-import static java.lang.System.exit;
-
 /**
  * A high-level wrapper over the AssetHelper class.
  *

engine/src/main/java/org/destinationsol/files/HullConfigManager.java

@@ -16,23 +16,20 @@
 package org.destinationsol.files;
 
 import com.badlogic.gdx.math.Vector2;
-import org.destinationsol.assets.json.Validator;
-import org.destinationsol.game.AbilityCommonConfig;
-import org.destinationsol.modules.ModuleManager;
-import org.json.JSONArray;
-import org.json.JSONObject;
 import org.destinationsol.assets.Assets;
-import org.destinationsol.assets.json.Json;
+import org.destinationsol.assets.json.Validator;
 import org.destinationsol.common.SolMath;
+import org.destinationsol.game.AbilityCommonConfig;
 import org.destinationsol.game.AbilityCommonConfigs;
 import org.destinationsol.game.item.Engine;
 import org.destinationsol.game.item.ItemManager;
 import org.destinationsol.game.particle.DSParticleEmitter;
 import org.destinationsol.game.ship.AbilityConfig;
 import org.destinationsol.game.ship.hulls.GunSlot;
 import org.destinationsol.game.ship.hulls.HullConfig;
-import org.terasology.gestalt.module.Module;
-import org.terasology.gestalt.naming.Name;
+import org.destinationsol.modules.ModuleManager;
+import org.json.JSONArray;
+import org.json.JSONObject;
 
 import java.lang.reflect.Method;
 import java.util.ArrayList;

engine/src/main/java/org/destinationsol/game/FactionInfo.java

@@ -54,7 +54,7 @@ private void createFactionList() {
     private static Set<String> getModuleSet() {
         Set<String> moduleSet = new HashSet<String>();
         Set<ResourceUrn> moduleUrn = Assets.getAssetHelper().list(Json.class, "[a-zA-Z0-9]*:factions");
-        for(ResourceUrn module : moduleUrn) {
+        for (ResourceUrn module : moduleUrn) {
             moduleSet.add(module.toString());
         }
         return moduleSet;
@@ -76,14 +76,14 @@ public static ArrayList getFactionColors() {
 
     public static int getFactionID(SolShip ship) {
         String shipName = ship.getHull().getHullConfig().getInternalName();
-        for(String modulePath: getModuleSet()) {
+        for (String modulePath : getModuleSet()) {
             Json factionJson = Assets.getJson(modulePath);
             Validator.getValidatedJSON(modulePath, "engine:schemaFactions");
             JSONArray factionJsonArray = factionJson.getJsonValue().getJSONArray("factions");
             shipName = shipName.replaceAll(".*:", "");
-            for(int n = 0; n < factionJsonArray.length(); n++) {
-                for(int z = 0; z < factionJsonArray.getJSONObject(n).getJSONArray("ships").length(); z++) {
-                    if(shipName.equals(factionJsonArray.getJSONObject(n).getJSONArray("ships").get(z))) {
+            for (int n = 0; n < factionJsonArray.length(); n++) {
+                for (int z = 0; z < factionJsonArray.getJSONObject(n).getJSONArray("ships").length(); z++) {
+                    if (shipName.equals(factionJsonArray.getJSONObject(n).getJSONArray("ships").get(z))) {
                         return n;
                     }
                 }

engine/src/main/java/org/destinationsol/game/GalaxyFiller.java

@@ -16,12 +16,9 @@
 package org.destinationsol.game;
 
 import com.badlogic.gdx.math.Vector2;
-import org.destinationsol.assets.json.Validator;
-import org.json.JSONObject;
-import com.google.gson.JsonParseException;
 import org.destinationsol.Const;
-import org.destinationsol.assets.Assets;
 import org.destinationsol.assets.json.Json;
+import org.destinationsol.assets.json.Validator;
 import org.destinationsol.common.SolMath;
 import org.destinationsol.common.SolRandom;
 import org.destinationsol.files.HullConfigManager;
@@ -41,6 +38,7 @@
 import org.destinationsol.game.planet.SysConfig;
 import org.destinationsol.game.ship.FarShip;
 import org.destinationsol.game.ship.hulls.HullConfig;
+import org.json.JSONObject;
 
 import java.util.ArrayList;
 

engine/src/main/java/org/destinationsol/game/GameColors.java

@@ -17,10 +17,8 @@
 
 import com.badlogic.gdx.graphics.Color;
 import org.destinationsol.assets.json.Validator;
-import org.json.JSONObject;
-import org.destinationsol.assets.Assets;
-import org.destinationsol.assets.json.Json;
 import org.destinationsol.common.SolColorUtil;
+import org.json.JSONObject;
 
 import java.util.HashMap;
 import java.util.Map;

engine/src/main/java/org/destinationsol/game/MountDetectDrawer.java

@@ -21,7 +21,7 @@
 import org.destinationsol.common.SolColor;
 import org.destinationsol.game.ship.SolShip;
 
-public class MountDetectDrawer implements UpdateAwareSystem{
+public class MountDetectDrawer implements UpdateAwareSystem {
     private final Vector2 myNePos;
     private final TextureAtlas.AtlasRegion myTexture;
 

engine/src/main/java/org/destinationsol/game/ObjectManager.java

@@ -287,12 +287,12 @@ public List<SolObject> getObjects() {
      *
      * @param squaredDistance If the distance between any object and {@code fromObject} is less than this,
      *                        the {@code action} will be executed with that object as the argument.
-     * @param fromObject The point of reference. This is <b>not</b> exempt from the {@code action}.
-     * @param action The action to commit to {@code fromObject}.
+     * @param fromObject      The point of reference. This is <b>not</b> exempt from the {@code action}.
+     * @param action          The action to commit to {@code fromObject}.
      */
     public void doToAllCloserThan(float squaredDistance, SolObject fromObject, Consumer<SolObject> action) {
-        for(SolObject obj : myObjs) {
-            if(fromObject.getPosition().dst2(obj.getPosition()) < squaredDistance) {
+        for (SolObject obj : myObjs) {
+            if (fromObject.getPosition().dst2(obj.getPosition()) < squaredDistance) {
                 action.accept(obj);
             }
         }

engine/src/main/java/org/destinationsol/game/PlayerSpawnConfig.java

@@ -16,11 +16,9 @@
 package org.destinationsol.game;
 
 import org.destinationsol.assets.json.Validator;
-import org.json.JSONObject;
-import org.destinationsol.assets.Assets;
-import org.destinationsol.assets.json.Json;
 import org.destinationsol.files.HullConfigManager;
 import org.destinationsol.game.item.ItemManager;
+import org.json.JSONObject;
 
 public class PlayerSpawnConfig {
     final ShipConfig mainStation;

engine/src/main/java/org/destinationsol/game/ShipConfig.java

@@ -16,14 +16,14 @@
 package org.destinationsol.game;
 
 import com.badlogic.gdx.math.Vector2;
-import org.destinationsol.assets.json.Validator;
-import org.json.JSONArray;
-import org.json.JSONObject;
 import org.destinationsol.assets.Assets;
 import org.destinationsol.assets.json.Json;
+import org.destinationsol.assets.json.Validator;
 import org.destinationsol.files.HullConfigManager;
 import org.destinationsol.game.item.ItemManager;
 import org.destinationsol.game.ship.hulls.HullConfig;
+import org.json.JSONArray;
+import org.json.JSONObject;
 import org.terasology.gestalt.assets.ResourceUrn;
 
 import java.util.ArrayList;
@@ -58,15 +58,15 @@ public Vector2 getSpawnPos() {
         return spawnPos;
     }
 
-    public int getMoney(){
+    public int getMoney() {
         return money;
     }
 
-    public HullConfig getHull(){
+    public HullConfig getHull() {
         return hull;
     }
 
-    public String getItems(){
+    public String getItems() {
         return items;
     }