Removed logger imports in SpawnWaveTask and used String builder in Le…

…velWaves as instructed by SonarCloud.
UQcsse3200 · Oct 16, 2023 · a1545af · a1545af
1 parent 45b7eec
commit a1545af
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/source/core/src/main/com/csse3200/game/components/tasks/SpawnWaveTask.java b/source/core/src/main/com/csse3200/game/components/tasks/SpawnWaveTask.java
@@ -4,8 +4,6 @@
 import com.csse3200.game.ai.tasks.PriorityTask;
 import com.csse3200.game.services.GameTime;
 import com.csse3200.game.services.ServiceLocator;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class SpawnWaveTask extends DefaultTask implements PriorityTask {
 

diff --git a/source/core/src/main/com/csse3200/game/components/tasks/waves/LevelWaves.java b/source/core/src/main/com/csse3200/game/components/tasks/waves/LevelWaves.java
@@ -124,11 +124,11 @@ public List<WaveClass> getWaves() {
 
     @Override
     public String toString() {
-      String result = "";
-      for (WaveClass wave : waves) {
-        result += wave.toString() + "\n";
-      }
-      return result;
+        StringBuilder result = new StringBuilder();
+        for (WaveClass wave : waves) {
+            result.append(wave.toString()).append("\n");
+        }
+        return result.toString();
     }
 
 }