Merge branch 'main' into bdd-tests

BME-MIT-IET · May 22, 2024 · 0d7d5cb · 0d7d5cb
2 parents 865220d + a69795f
commit 0d7d5cb
Show file tree

Hide file tree

Showing 25 changed files with 432 additions and 182 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -0,0 +1,37 @@
+name: SonarCloud
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    types: [opened, synchronize, reopened]
+jobs:
+  build:
+    name: Build and analyze
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          fetch-depth: 0  # Shallow clones should be disabled for a better relevancy of analysis
+      - name: Set up JDK 17
+        uses: actions/setup-java@v3
+        with:
+          java-version: 17
+          distribution: 'zulu' # Alternative distribution options are available.
+      - name: Cache SonarCloud packages
+        uses: actions/cache@v3
+        with:
+          path: ~/.sonar/cache
+          key: ${{ runner.os }}-sonar
+          restore-keys: ${{ runner.os }}-sonar
+      - name: Cache Maven packages
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2
+          key: ${{ runner.os }}-m2-${{ hashFiles('**/pom.xml') }}
+          restore-keys: ${{ runner.os }}-m2
+      - name: Build and analyze
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}  # Needed to get PR information, if any
+          SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+        run: mvn -B verify org.sonarsource.scanner.maven:sonar-maven-plugin:sonar -Dsonar.projectKey=BME-MIT-IET_iet-hf-2024-mit-knock-off
diff --git a/pom.xml b/pom.xml
@@ -11,6 +11,8 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
          <maven.compiler.source>14</maven.compiler.source>
          <maven.compiler.target>14</maven.compiler.target>
+         <sonar.organization>bme-mit-iet-org</sonar.organization>
+         <sonar.host.url>https://sonarcloud.io</sonar.host.url>
     </properties>
 
     <dependencyManagement>

diff --git a/src/main/java/jatek/CiszternaView.java b/src/main/java/jatek/CiszternaView.java
@@ -53,12 +53,12 @@ public void draw(Graphics g){
         for (int i=0; i<szomszedoscsovek.size(); i++) {
             if (!szomszedoscsovek.subList(0, i).contains(szomszedoscsovek.get(i))) {
                 CsoView cv = GamePanel.getInstance().getCsoViewFromCso(szomszedoscsovek.get(i));
-                Vec2 direction = new Vec2(cv.getX1() - getX(), cv.getY1() - getY());
+                Vec2 direction = new Vec2(cv.getX1() - ((double)getX()), cv.getY1() - ((double)getY()));
                 if (cv.getX1() == getX() && cv.getY1() == getY()) {
-                    direction = new Vec2(cv.getX2() - getX(), cv.getY2() - getY());
+                    direction = new Vec2(cv.getX2() - ((double)getX()), cv.getY2() - ((double)getY()));
                 }
                 direction = direction.normalize();
-                direction = direction.multiply(radius - 5);
+                direction = direction.multiply(radius - 5.0);
                 g.drawString(Integer.toString(i + 1), (int) (getX() + direction.getX() - 5), (int) (getY() + direction.getY() + 5));
             }
         }

diff --git a/src/main/java/jatek/Cso.java b/src/main/java/jatek/Cso.java
@@ -10,6 +10,8 @@
  * Passzív elem tehát víz úgy folyhat rajta, ha egy aktív elem szivattyúz belőle vagy bele pumpál.
  */
 public class Cso extends Mezo {
+
+    private Random random = new Random();
 
     /**
      * A szomszédos csúcsok listája
@@ -59,7 +61,7 @@ public void removeNeighbour(Csucs csucs){
      * @return Igazat ad vissza ha a játékos ezen a mezőn marad mozgása végén.
      */
     public boolean jatekostElfogad(Jatekos j) {
-        if(getJatekosRajta().size() == 0 && szomszedosCsucs.size() == 2){
+        if(getJatekosRajta().isEmpty() && szomszedosCsucs.size() == 2){
             if(allapot == Allapot.NORMALIS){
                 getJatekosRajta().add(j);
                 return true;
@@ -75,7 +77,6 @@ public boolean jatekostElfogad(Jatekos j) {
                     j.setAktMezo(det);
                 }
                 else {
-                    Random random = new Random();
                     int irany = random.nextInt(2);
                     szomszedosCsucs.get(irany).jatekostElfogad(j);
                     j.setAktMezo(szomszedosCsucs.get(irany));
@@ -95,7 +96,6 @@ public boolean targyLerakas(Pumpa p) {
         Cso fele = new Cso();
         Csucs szCsucs = this.szomszedosCsucs.get(0);
         szCsucs.csoCsere(this, fele);
-        //szCsucs.felcsatol(fele);
         p.felcsatol(fele);
         p.felcsatol(this);
         Kontroller.getInstance().addCso(fele);
@@ -109,7 +109,7 @@ public boolean targyLerakas(Pumpa p) {
      */
     @Override
     public List<? extends Mezo> getNeighbours() {
-        return szomszedosCsucs;
+        return getSzomszedosCsucs();
     }
 
     /**
@@ -128,7 +128,6 @@ public void jatekosElront() {
     public void szereloJavit() {
         if(rossz){
             setRossz(false);
-            Random random = new Random();
             foltozasiGarancia = random.nextInt(3) + 2;
         }
     }
@@ -185,7 +184,7 @@ public void allapotValtozas(Allapot allapot) {
     public void stepTime(){
         if(timeToNormal == 1){
             allapotValtozas(Allapot.NORMALIS);
-            if(getJatekosRajta().size() != 0){
+            if(!getJatekosRajta().isEmpty()){
                 getJatekosRajta().get(0).leragad(false);
             }
         }
@@ -270,6 +269,7 @@ public List<Csucs> getSzomszedosCsucs() {
      * Beallitja a determinisztikus mukodeshez a csucsot amire a jatekos csuszik
      * @param csucs A csucs, amire csuszhat.
      */
+    @Override
     public void setDet(Csucs csucs){
         det = csucs;
     }

diff --git a/src/main/java/jatek/CsoView.java b/src/main/java/jatek/CsoView.java
@@ -85,10 +85,9 @@ private Polygon getPolygon(int size){
         Vec2 vonalCsucs2 = vektor1.add(n.multiply(-size));
         Vec2 vonalCsucs3 = vektor2.add(n.multiply(size));
         Vec2 vonalCsucs4 = vektor2.add(n.multiply(-size));
-        int[] x_ek = {(int) vonalCsucs1.getX(), (int) vonalCsucs3.getX(), (int) vonalCsucs4.getX(), (int) vonalCsucs2.getX()};
-        int[] y_ok = {(int) vonalCsucs1.getY(), (int) vonalCsucs3.getY(), (int) vonalCsucs4.getY(), (int) vonalCsucs2.getY()};
-        Polygon polygon = new Polygon(x_ek, y_ok, 4);
-        return polygon;
+        int[] xEk = {(int) vonalCsucs1.getX(), (int) vonalCsucs3.getX(), (int) vonalCsucs4.getX(), (int) vonalCsucs2.getX()};
+        int[] yOk = {(int) vonalCsucs1.getY(), (int) vonalCsucs3.getY(), (int) vonalCsucs4.getY(), (int) vonalCsucs2.getY()};
+        return new Polygon(xEk, yOk, 4);
     }
 
     /**
@@ -119,7 +118,7 @@ public void draw(Graphics g){
         List<Csucs> szomszedosCsucsok  =cso.getSzomszedosCsucs();
         CsucsView csucsv1  =null;
         CsucsView csucsv2 = null;
-        if(szomszedosCsucsok.size() !=0)
+        if(!szomszedosCsucsok.isEmpty())
             csucsv1 = GamePanel.getInstance().getCsucsViewFromCsucs(szomszedosCsucsok.get(0));
         if(szomszedosCsucsok.size() ==2)
             csucsv2 = GamePanel.getInstance().getCsucsViewFromCsucs(szomszedosCsucsok.get(1));

diff --git a/src/main/java/jatek/Csucs.java b/src/main/java/jatek/Csucs.java
@@ -15,9 +15,9 @@ public abstract class Csucs extends Mezo{
     /**
      * Létrehoz egy Csucsot, és inicializálja a szomszedosCso listát.
      */
-    public Csucs(){
+    protected Csucs(){
         super();
-        szomszedosCso = new ArrayList<Cso>();
+        szomszedosCso = new ArrayList<>();
     }
 
     /**
@@ -62,6 +62,7 @@ public boolean jatekostElfogad(Jatekos j) {
      * A kapott játékost eltávolítja a jatekosRajta listából.
      * @param j A játékos, akit el akarunk távolítani a listaból.
      */
+    @Override
     public void jatekostEltavolit(Jatekos j){
         getJatekosRajta().remove(j);
     }
@@ -97,7 +98,7 @@ public Cso adjCsovet(int cs){
         if(szomszedosCso.size() > 1){
             int k = cs % szomszedosCso.size();
             Cso temp = szomszedosCso.get(k);
-            if(temp.getJatekosRajta().size() == 0){
+            if(temp.getJatekosRajta().isEmpty()){
                 lecsatol(temp);
                 return temp;
             }
@@ -152,7 +153,7 @@ public void csoCsere(Cso cs1, Cso cs2){
      */
     @Override
     public List<? extends Mezo> getNeighbours(){
-        return szomszedosCso;
+        return getSzomszedosCso();
     }
 
     /**

diff --git a/src/main/java/jatek/ForrasView.java b/src/main/java/jatek/ForrasView.java
@@ -48,12 +48,12 @@ public void draw(Graphics g){
         for (int i=0; i<szomszedoscsovek.size(); i++) {
             if (!szomszedoscsovek.subList(0, i).contains(szomszedoscsovek.get(i))) {
                 CsoView cv = GamePanel.getInstance().getCsoViewFromCso(szomszedoscsovek.get(i));
-                Vec2 direction = new Vec2(cv.getX1() - getX(), cv.getY1() - getY());
+                Vec2 direction = new Vec2(cv.getX1() - ((double)getX()), cv.getY1() - ((double)getY()));
                 if (cv.getX1() == getX() && cv.getY1() == getY()) {
-                    direction = new Vec2(cv.getX2() - getX(), cv.getY2() - getY());
+                    direction = new Vec2(cv.getX2() - ((double)getX()), cv.getY2() - ((double)getY()));
                 }
                 direction = direction.normalize();
-                direction = direction.multiply(radius - 5);
+                direction = direction.multiply(radius - 5.0);
                 g.drawString(Integer.toString(i + 1), (int) (getX() + direction.getX() - 5), (int) (getY() + direction.getY() + 5));
             }
         }