From 26e9854fb27cca8b8e4c5e930a11ecf811234355 Mon Sep 17 00:00:00 2001
From: 0nlineSam <uni@sam.tadjiky.de>
Date: Sat, 2 Nov 2024 14:33:45 +0100
Subject: [PATCH] LevelModel fixed too many fields PMD violation

---
 .idea/misc.xml                                |  3 ++
 .../enssat/BoulderDash/models/LevelModel.java | 45 +++++++++----------
 2 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/.idea/misc.xml b/.idea/misc.xml
index 6c8ad073..1b6e5a9f 100644
--- a/.idea/misc.xml
+++ b/.idea/misc.xml
@@ -1,5 +1,8 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project version="4">
+  <component name="PDMPlugin">
+    <option name="skipTestSources" value="false" />
+  </component>
   <component name="PWA">
     <option name="enabled" value="true" />
     <option name="wasEnabledAtLeastOnce" value="true" />
diff --git a/src/fr/enssat/BoulderDash/models/LevelModel.java b/src/fr/enssat/BoulderDash/models/LevelModel.java
index cbb5415a..46321108 100644
--- a/src/fr/enssat/BoulderDash/models/LevelModel.java
+++ b/src/fr/enssat/BoulderDash/models/LevelModel.java
@@ -16,6 +16,7 @@ import fr.enssat.BoulderDash.models.DirtModel;
 import fr.enssat.BoulderDash.models.ExpandingWallModel;
 import fr.enssat.BoulderDash.models.CursorModel;
 
+import java.awt.*;
 import java.awt.image.BufferedImage;
 import java.util.Observable;
 
@@ -32,18 +33,16 @@ import java.util.Observable;
  */
 public class LevelModel extends Observable implements Runnable {
 	private DisplayableElementModel[][] groundGrid;
-	private String levelName;
-	private AudioLoadHelper audioLoadHelper;
+    private AudioLoadHelper audioLoadHelper;
 	private int sizeWidth = 0;
 	private int sizeHeight = 0;
-	private int cursorXPosition = 0;
-	private int cursorYPosition = 0;
+	private final Point cursorPosition  = new Point(0, 0);
 	private boolean showCursor = false;
 	private CursorModel cursorModel;
 	private LevelLoadHelper levelLoadHelper;
 	private RockfordModel rockford;
 	private GameInformationModel gameInformationModel;
-	private int rockfordPositionX, rockfordPositionY;
+	private Point rockfordPosition;
 	private boolean gameRunning;
 	private boolean gamePaused;
 	// Are we in editor or game mode ?
@@ -67,13 +66,12 @@ public class LevelModel extends Observable implements Runnable {
      * @param  mode             Instance mode
 	 */
 	public LevelModel(String levelName, AudioLoadHelper audioLoadHelper, String mode) {
-		this.levelName = levelName;
-		this.audioLoadHelper = audioLoadHelper;
+        this.audioLoadHelper = audioLoadHelper;
 		this.gamePaused = false;
 		this.gameRunning = true;
 		this.mode = mode;
 
-		this.levelLoadHelper = new LevelLoadHelper(this.levelName);
+		this.levelLoadHelper = new LevelLoadHelper(levelName);
 
 		this.groundGrid = this.levelLoadHelper.getGroundGrid();
 		this.sizeWidth = this.levelLoadHelper.getWidthSizeValue();
@@ -137,8 +135,7 @@ public class LevelModel extends Observable implements Runnable {
 	 * Initializes the Rockford position attributes
 	 */
 	private void initRockford() {
-		this.rockfordPositionX = this.levelLoadHelper.getRockfordPositionX();
-		this.rockfordPositionY = this.levelLoadHelper.getRockfordPositionY();
+		this.rockfordPosition = new Point(this.levelLoadHelper.getRockfordPositionX(), this.levelLoadHelper.getRockfordPositionY());
 		this.rockford = this.levelLoadHelper.getRockfordInstance();
 	}
 
@@ -172,8 +169,8 @@ public class LevelModel extends Observable implements Runnable {
 	 * @param  posY  Vertical position of Rockford
 	 */
 	public void updateRockfordPosition(int posX, int posY) {
-		this.rockfordPositionY = posY;
-		this.rockfordPositionX = posX;
+		this.rockfordPosition.y = posY;
+		this.rockfordPosition.x = posX;
 	}
 
 	/**
@@ -219,7 +216,7 @@ public class LevelModel extends Observable implements Runnable {
 	 * @return  Horizontal position of Rockford
 	 */
 	public int getRockfordPositionX() {
-		return this.rockfordPositionX;
+		return this.rockfordPosition.x;
 	}
 
 	/**
@@ -310,7 +307,7 @@ public class LevelModel extends Observable implements Runnable {
 	 * @return  Vertical position of Rockford
 	 */
 	public int getRockfordPositionY() {
-		return this.rockfordPositionY;
+		return this.rockfordPosition.y;
 	}
 
 	/**
@@ -531,7 +528,7 @@ public class LevelModel extends Observable implements Runnable {
 	 * @return  Cursor position X value
 	 */
 	public int getCursorXPosition() {
-		return this.cursorXPosition;
+		return this.cursorPosition.x;
 	}
 
 	/**
@@ -540,7 +537,7 @@ public class LevelModel extends Observable implements Runnable {
 	 * @return  Cursor position Y value
 	 */
 	public int getCursorYPosition() {
-		return this.cursorYPosition;
+		return this.cursorPosition.y;
 	}
 
 	/**
@@ -549,8 +546,8 @@ public class LevelModel extends Observable implements Runnable {
 	 * @return  Cursor position new X value
 	 */
 	public int incrementCursorXPosition() {
-		if (this.cursorXPosition < (this.getSizeWidth() - 1 - 2)) {
-			this.cursorXPosition = this.cursorXPosition + 1;
+		if (this.cursorPosition.x < (this.getSizeWidth() - 1 - 2)) {
+			this.cursorPosition.x = this.cursorPosition.x + 1;
 		}
 
 		this.localNotifyObservers();
@@ -563,8 +560,8 @@ public class LevelModel extends Observable implements Runnable {
 	 * @return  Cursor position new X value
 	 */
 	public int decrementCursorXPosition() {
-		if (this.cursorXPosition > 0) {
-			this.cursorXPosition = this.cursorXPosition - 1;
+		if (this.cursorPosition.x > 0) {
+			this.cursorPosition.x = this.cursorPosition.x - 1;
 		}
 
 		this.localNotifyObservers();
@@ -577,8 +574,8 @@ public class LevelModel extends Observable implements Runnable {
 	 * @return  Cursor position new Y value
 	 */
 	public int incrementCursorYPosition() {
-		if (this.cursorYPosition < (this.getSizeHeight() - 1 - 2)) {
-			this.cursorYPosition = this.cursorYPosition + 1;
+		if (this.cursorPosition.y < (this.getSizeHeight() - 1 - 2)) {
+			this.cursorPosition.y = this.cursorPosition.y + 1;
 		}
 
 		this.localNotifyObservers();
@@ -591,8 +588,8 @@ public class LevelModel extends Observable implements Runnable {
 	 * @return  Cursor position new Y value
 	 */
 	public int decrementCursorYPosition() {
-		if (this.cursorYPosition > 0) {
-			this.cursorYPosition = this.cursorYPosition - 1;
+		if (this.cursorPosition.y > 0) {
+			this.cursorPosition.y = this.cursorPosition.y - 1;
 		}
 
 		this.localNotifyObservers();