Merge pull request #6 from BME-MIT-IET/manual-and-sonarcloud-review

Manual and sonarcloud review

.vscode/extensions.json

@@ -0,0 +1,6 @@
+{
+    "recommendations": [
+        "vscjava.vscode-java-pack",
+        "SonarSource.sonarlint-vscode"
+    ]
+}

.vscode/settings.json

@@ -2,5 +2,6 @@
     "sonarlint.connectedMode.project": {
         "connectionId": "bme-mit-iet-org",
         "projectKey": "BME-MIT-IET_iet-hf-2024-nomad-szabotorok"
-    }
+    },
+    "java.configuration.updateBuildConfiguration": "interactive"
 }

pom.xml

@@ -9,13 +9,11 @@
   <version>1.0-SNAPSHOT</version>
 
   <name>pipe_game</name>
-  <!-- FIXME change it to the project's website -->
-  <url>http://www.example.com</url>
 
   <properties>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-    <maven.compiler.source>1.7</maven.compiler.source>
-    <maven.compiler.target>1.7</maven.compiler.target>
+    <maven.compiler.source>1.8</maven.compiler.source>
+    <maven.compiler.target>1.8</maven.compiler.target>
     <sonar.organization>bme-mit-iet-org</sonar.organization>
 	<sonar.host.url>https://sonarcloud.io</sonar.host.url>
   </properties>

src/main/java/hu/bme/mit/iet/pipe_game/Cistern.java

@@ -26,6 +26,7 @@ public class Cistern extends SystemPart {
 	private boolean hasPipe = false;
 	private int pCount = 0;
 	private int timeLeft = 2;
+	private Random rand = new Random();
 
 	/**
 	 * Konstruktor, ami beállítja az id-t az osztály nevére és egy eltérő számra
@@ -39,13 +40,13 @@ public Cistern() {
 	 * A csőnek mindkét szomszégjának magát állítja be,
 	 * hogy ne legyen szabad és ne folyon ki belőle a víz
 	 */
-	public void CreatePipe() {
+	public void createPipe() {
 		Pipe p = new Pipe();
-		Control.AddPipe(p);
-		p.AddNeighbour(this);
-		p.AddNeighbour(this);
-		AddNeighbour(p);
-		AddNeighbour(p);
+		Control.addPipe(p);
+		p.addNeighbour(this);
+		p.addNeighbour(this);
+		addNeighbour(p);
+		addNeighbour(p);
 		pipe = p;
 		hasPipe = true;
 		pCount++;
@@ -54,17 +55,17 @@ public void CreatePipe() {
 	/**
 	 * Létrehoz egy új pumpát és eltárolja azt a tagváltozójába
 	 */
-	public void CreatePump() {
+	public void createPump() {
 		pump = new Pump();
-		Control.AddPump(pump);
+		Control.addPump(pump);
 	}
 
 	/**
 	 * A tagváltozóban eltárolt pumpa felvétele
 	 * @return visszadjuk a pumpát
 	 */
 	@Override
-	public Pump CarryPump() {
+	public Pump carryPump() {
 
 		Pump p = pump;
 		pump = null;
@@ -78,12 +79,12 @@ public Pump CarryPump() {
 	 * @return visszadja hogy mennyi vizet gyűjtött össze
 	 */
 	@Override
-	public int PullWater() {
+	public int pullWater() {
 
 		int points = 0;
-		for (SystemPart pipe: neighbours) {
-			points += pipe.getWater();
-			pipe.setWater(0);
+		for (SystemPart _pipe: neighbours) {
+			points += _pipe.getWater();
+			_pipe.setWater(0);
 		}
 		return points;
 	}
@@ -92,16 +93,15 @@ public int PullWater() {
 	 * A control minden körben meghívja,
 	 * ha letelik az idő akkor generál egy pumpát vagy csövet
 	 */
-	public void Generate() {
-		Random rand = new Random();
+	public void generate() {
 		int coin = rand.nextInt(2);
 
 		if (--timeLeft == 0) {
 			if (coin ==  1 ) {
-				CreatePump();
+				createPump();
 			}
 			else {
-				CreatePipe();
+				createPipe();
 			}
 			timeLeft = 2;
 		}

src/main/java/hu/bme/mit/iet/pipe_game/CisternView.java

@@ -31,7 +31,7 @@ public CisternView(View view, Cistern cistern) {
      * frissíti a part JButtont a jelenlegi állapotoknak megfelelően
      */
     @Override
-    public void Update() {
+    public void update() {
         button.setBounds(x,y,50,50);
         button.setBackground(Color.RED);
 
@@ -42,7 +42,7 @@ public void Update() {
          */
         view.getPanel().remove(button);
 
-        while (players.size() > 0){
+        while (!players.isEmpty()){
             view.getPanel().remove(players.get(0));
             players.remove(0);
         }
@@ -79,6 +79,7 @@ public void Update() {
      * getter fügvény ID-re
      * @return ID-t adja vissza
      */
+    @Override
     public String getID(){
         return cistern.getId();
     }