Merge pull request #15 from BME-MIT-IET/FixedSonarLintIssues

fixed Sonarlint issues

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));
             }
         }

src/main/java/jatek/Cso.java

@@ -109,7 +109,7 @@ public boolean targyLerakas(Pumpa p) {
      */
     @Override
     public List<? extends Mezo> getNeighbours() {
-        return szomszedosCsucs;
+        return getSzomszedosCsucs();
     }
 
     /**

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<>();
     }
 
     /**
@@ -153,7 +153,7 @@ public void csoCsere(Cso cs1, Cso cs2){
      */
     @Override
     public List<? extends Mezo> getNeighbours(){
-        return szomszedosCso;
+        return getSzomszedosCso();
     }
 
     /**

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));
             }
         }

src/main/java/jatek/GameFrame.java

@@ -14,12 +14,25 @@ public class GameFrame extends JFrame {
     /**
      * TextFieldek, amikbe a felhasználó paramétereket adhat meg egy gomblenyomáshoz.
      */
-    private JTextField hova, ki, be, hanyadik;
+    private JTextField hova;
+    private JTextField ki;
+    private JTextField be;
+    private JTextField hanyadik;
 
     /**
      *  Gombok a játék irányításához
      */
-    private JButton mozog, atallit, lyukaszt, ragaszt, javit, csFelvesz, csLerak, pFelvesz, pLerak, csuszosit, jatekVege;
+    private JButton mozog;
+    private JButton atallit;
+    private JButton lyukaszt;
+    private JButton ragaszt;
+    private JButton javit;
+    private JButton csFelvesz;
+    private JButton csLerak;
+    private JButton pFelvesz;
+    private JButton pLerak;
+    private JButton csuszosit;
+    private JButton jatekVege;
 
     /**
      *  GameFrame LayoutManagere, hogy váltani lehessen menü és játék között
@@ -29,12 +42,11 @@ public class GameFrame extends JFrame {
     /**
      *  Menühöz tartozó textFieldek, a szerelő és szabotőrszám megadására
      */
-    private JTextField szereloszam, szabotorszam;
+    private JTextField szereloszam;
+    private JTextField szabotorszam;
 
-    /**
-     * Menühöz tartozó JPanel
-     */
-    private JPanel menu;
+
+
 
     /**
      * GameFrame konstruktora, meghívja az inicializáló függvényeket és megjeleníti a menüt.
@@ -43,7 +55,7 @@ public class GameFrame extends JFrame {
     public GameFrame() {
         super("CsipCsap Jateka woooohoo");
 
-        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
+        this.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
         this.setSize(500, 500);
         this.setResizable(false);
         this.setLayout(card);
@@ -56,6 +68,10 @@ public GameFrame() {
      *  Inicializálja a menüt és a gombját, valamint a hozzá tartozó textfieldeket és labeleket.
      */
     private void initMenu(){
+        /**
+        * Menühöz tartozó JPanel
+        */
+        JPanel menu;
         menu = new JPanel();
         JButton newGame = new JButton("Játék indítása");
         newGame.addActionListener((ActionEvent e) -> {
@@ -106,41 +122,41 @@ private void initComponents(){
         //kozos
         JLabel kozos = new JLabel("Közös akciók");
         mozog = new JButton("Mozog");
-        JLabel hova_ = new JLabel("Hova:");
+        JLabel hovaLabel = new JLabel("Hova:");
         hova = new JTextField("", 2);
         atallit = new JButton("Pumpát átállít");
-        JLabel ki_ = new JLabel("Ki:");
+        JLabel kiLabel = new JLabel("Ki:");
         ki = new JTextField("", 2);
-        JLabel be_ = new JLabel("Be:");
+        JLabel beLabel = new JLabel("Be:");
         be = new JTextField("", 2);
         lyukaszt = new JButton("Csövet lyukaszt");
         ragaszt = new JButton("Csövet ragasztóz");
         jatekVege = new JButton("Játék vége");
 
         //szerelo
         JLabel szerelo = new JLabel("Szerelő akciói");
-        szerelo.setAlignmentX(JLabel.CENTER_ALIGNMENT);
+        szerelo.setAlignmentX(Component.CENTER_ALIGNMENT);
         javit = new JButton("Mezőt javít");
-        javit.setAlignmentX(JButton.CENTER_ALIGNMENT);
+        javit.setAlignmentX(Component.CENTER_ALIGNMENT);
         csFelvesz = new JButton("Csövet felvesz");
-        csFelvesz.setAlignmentX(JButton.CENTER_ALIGNMENT);
-        JLabel hanyadik_ = new JLabel("Hanyadik:");
-        hanyadik_.setAlignmentX(JLabel.CENTER_ALIGNMENT);
+        csFelvesz.setAlignmentX(Component.CENTER_ALIGNMENT);
+        JLabel hanyadikLabel = new JLabel("Hanyadik:");
+        hanyadikLabel.setAlignmentX(Component.CENTER_ALIGNMENT);
         hanyadik = new JTextField("", 2);
         hanyadik.setMaximumSize(hanyadik.getPreferredSize());
-        hanyadik_.setAlignmentX(JTextField.CENTER_ALIGNMENT);
+        hanyadikLabel.setAlignmentX(Component.CENTER_ALIGNMENT);
         csLerak = new JButton("Csövet lerak");
-        csLerak.setAlignmentX(JButton.CENTER_ALIGNMENT);
+        csLerak.setAlignmentX(Component.CENTER_ALIGNMENT);
         pFelvesz = new JButton("Pumpát felvesz");
-        pFelvesz.setAlignmentX(JButton.CENTER_ALIGNMENT);
+        pFelvesz.setAlignmentX(Component.CENTER_ALIGNMENT);
         pLerak = new JButton("Pumpát lerak");
-        pLerak.setAlignmentX(JButton.CENTER_ALIGNMENT);
+        pLerak.setAlignmentX(Component.CENTER_ALIGNMENT);
 
         //szabotor
         JLabel szabotor = new JLabel("Szabotőr akciói");
-        szabotor.setAlignmentX(JLabel.CENTER_ALIGNMENT);
+        szabotor.setAlignmentX(Component.CENTER_ALIGNMENT);
         csuszosit = new JButton("Csövet csuszósít");
-        csuszosit.setAlignmentX(JButton.CENTER_ALIGNMENT);
+        csuszosit.setAlignmentX(Component.CENTER_ALIGNMENT);
 
         //gombok lenyomásának lekezelése
         addLambdas();
@@ -151,12 +167,12 @@ private void initComponents(){
         JPanel pan = new JPanel();
         pan.add(kozos);
         pan.add(mozog);
-        pan.add(hova_);
+        pan.add(hovaLabel);
         pan.add(hova);
         pan.add(atallit);
-        pan.add(ki_);
+        pan.add(kiLabel);
         pan.add(ki);
-        pan.add(be_);
+        pan.add(beLabel);
         pan.add(be);
         pan.add(lyukaszt);
         pan.add(ragaszt);
@@ -170,7 +186,7 @@ private void initComponents(){
         leftPanel.add(javit);
         leftPanel.add(Box.createVerticalStrut(15));
         leftPanel.add(csFelvesz);
-        leftPanel.add(hanyadik_);
+        leftPanel.add(hanyadikLabel);
         leftPanel.add(hanyadik);
         leftPanel.add(Box.createVerticalStrut(15));
         leftPanel.add(csLerak);
@@ -211,13 +227,14 @@ public void showGame() {
         card.show(getContentPane(), "game");
     }
 
-
     /**
-     *  Minden gombhoz hozzárendel egy actionListenert
+     * Megadja a mozog gomb lenyomására történő eseményeket.
+     * @param jatekos
+     * @param value
      */
-    public void addLambdas() {
-        addActionListenerToButton(mozog, hova, (jatekos, value) -> {
-            if(!value.isEmpty())
+
+    private void mozgasAction(Jatekos jatekos, String value){
+        if(!value.isEmpty())
                 try {
                     jatekos.mozgas(Integer.parseInt(value) - 1);
                 }
@@ -227,7 +244,12 @@ public void addLambdas() {
             else
                 jatekos.mozgas(0);
             hova.setText("");
-        });
+    }
+    /**
+     *  Minden gombhoz hozzárendel egy actionListenert
+     */
+    public void addLambdas() {
+        addActionListenerToButton(mozog, hova, this::mozgasAction);
         atallit.addActionListener((ActionEvent e) -> {
                     int akt = Kontroller.getInstance().getAktualisJatekos();
                     List<Jatekos> jatekosok = Kontroller.getInstance().getJatekosok();

src/main/java/jatek/GamePanel.java

@@ -45,7 +45,7 @@ public static synchronized GamePanel getInstance()
     /**
      * A pontokat szemléltető címke: "szabotőrok pontja - szerelők pontja" formátumban
      */
-    JLabel score_;
+    JLabel score;
 
     /**
      * Default Konstruktora a Game Panelnek.
@@ -55,9 +55,9 @@ private GamePanel(){
         jatekosok = new ArrayList<>();
         csovek = new ArrayList<>();
         csucsok = new ArrayList<>();
-        score_ = new JLabel("Score: "+Kontroller.getInstance().getSzabotorPontok()+"-"+Kontroller.getInstance().getSzereloPontok());
-        score_.setAlignmentY(JLabel.BOTTOM_ALIGNMENT);
-        this.add(score_);
+        score = new JLabel("Score: "+Kontroller.getInstance().getSzabotorPontok()+"-"+Kontroller.getInstance().getSzereloPontok());
+        score.setAlignmentY(Component.BOTTOM_ALIGNMENT);
+        this.add(score);
     }
 
     /**
@@ -77,7 +77,7 @@ public void paintComponent(Graphics g) {
         for (JatekosView j : jatekosok){
             j.draw(g);
         }
-        score_.setText("Score: "+Kontroller.getInstance().getSzabotorPontok()+"-"+Kontroller.getInstance().getSzereloPontok());
+        score.setText("Score: "+Kontroller.getInstance().getSzabotorPontok()+"-"+Kontroller.getInstance().getSzereloPontok());
     }
 
     /**
@@ -145,13 +145,13 @@ public CsucsView getCsucsViewFromCsucs(Csucs c){
      * @return A csövet cipelő játékosok listája.
      */
     public List<JatekosView> getPlayerViewFromCso(Cso c){
-        List<JatekosView> jatekosok = new ArrayList<>();
+        List<JatekosView> cipeloJatekosok = new ArrayList<>();
         for (JatekosView jv : this.jatekosok){
             if (jv.getJatekos().getCso() == c){
-                jatekosok.add(jv);
+                cipeloJatekosok.add(jv);
             }
         }
-        return jatekosok;
+        return cipeloJatekosok;
     }
 
     /**

src/main/java/jatek/Jatekos.java

@@ -113,11 +113,11 @@ public void pumpaAtallitasa(int bemeneti, int kimeneti){
         Kontroller.getInstance().ujraRajzol();
     }
 
-    public void csuszosit(){};
-    public void mezotJavit(){};
-    public void csovegFelvetele(int cs){};
-    public void csovegetLerak(){};
-    public void pumpaFelvetele(){};
-    public void pumpatLerak(){};
-    public Cso getCso(){return null;};
+    public void csuszosit(){}
+    public void mezotJavit(){}
+    public void csovegFelvetele(int cs){}
+    public void csovegetLerak(){}
+    public void pumpaFelvetele(){}
+    public void pumpatLerak(){}
+    public Cso getCso(){return null;}
 }

src/main/java/jatek/Kontroller.java

@@ -285,10 +285,13 @@ public void initJatekosok(int szerelokSz, int szabotorokSz){
      * @param csapat a csapat neve, lehet "szerelok" vagy "szabotorok".
      */
     public void pontNovel(String csapat){
-        switch (csapat) {
-            case "szerelok" -> szereloPontok++;
-            case "szabotorok" -> szabotorPontok++;
+        if (csapat.equals("szerelok")) {
+             szereloPontok++;
         }
+        else{
+            szabotorPontok++;
+        }
+
         Kontroller.getInstance().ujraRajzol();
     }
 

src/main/java/jatek/Mezo.java

@@ -21,8 +21,8 @@ public abstract class Mezo {
 	/**
 	 * Létrehoz egy mezőt, és inicializálja a jatekosRajta listát.
 	 */
-	public Mezo(){
-		jatekosRajta = new ArrayList<Jatekos>();
+	protected Mezo(){
+		jatekosRajta = new ArrayList<>();
 	}
 
 	/**

src/main/java/jatek/PumpaView.java

@@ -58,12 +58,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));
             }
         }
@@ -72,28 +72,28 @@ public void draw(Graphics g){
         g.setColor(Color.BLACK);
         Cso kimeneti = szomszedoscsovek.get(pumpa.getKimenetiCso());
         CsoView kimenetiView = GamePanel.getInstance().getCsoViewFromCso(kimeneti);
-        Vec2 direction = new Vec2(kimenetiView.getX1()-getX(),kimenetiView.getY1()-getY());
+        Vec2 direction = new Vec2(kimenetiView.getX1()-(double)getX(),kimenetiView.getY1()-(double)getY());
         if (kimenetiView.getX1() == getX() && kimenetiView.getY1() == getY()) {
-            direction = new Vec2(kimenetiView.getX2() - getX(), kimenetiView.getY2() - getY());
+            direction = new Vec2(kimenetiView.getX2() - (double)getX(), kimenetiView.getY2() - (double)getY());
         }
         direction = direction.normalize();
 
-        GamePanel.getInstance().drawTriangle(g,new Vec2(getX(),getY()).add(direction.multiply(radius+10)),direction.multiply(3), 8);
+        GamePanel.getInstance().drawTriangle(g,new Vec2(getX(),getY()).add(direction.multiply(radius+10.0)),direction.multiply(3), 8);
         g.setColor(Color.WHITE);
-        GamePanel.getInstance().drawTriangle(g,new Vec2(getX(),getY()).add(direction.multiply(radius+12)),direction.multiply(3), 5);
+        GamePanel.getInstance().drawTriangle(g,new Vec2(getX(),getY()).add(direction.multiply(radius+12.0)),direction.multiply(3), 5);
 
         //kirajzolja a bemeneti cso iranyat
         Cso bemeneti = szomszedoscsovek.get(pumpa.getBemenetiCso());
         CsoView bemenetiView = GamePanel.getInstance().getCsoViewFromCso(bemeneti);
-        Vec2 direction2 = new Vec2(bemenetiView.getX1()-getX(),bemenetiView.getY1()-getY());
+        Vec2 direction2 = new Vec2(bemenetiView.getX1()-(double)getX(),bemenetiView.getY1()-(double)getY());
         if (bemenetiView.getX1() == getX() && bemenetiView.getY1() == getY()) {
-            direction2 = new Vec2(bemenetiView.getX2() - getX(), bemenetiView.getY2() - getY());
+            direction2 = new Vec2(bemenetiView.getX2() - (double)getX(), bemenetiView.getY2() - (double)getY());
         }
         direction2 = direction2.normalize();
         g.setColor(Color.BLACK);
-        GamePanel.getInstance().drawTriangle(g,new Vec2(getX(),getY()).add(direction2.multiply(radius+40)),direction2.multiply(-3), 8);
+        GamePanel.getInstance().drawTriangle(g,new Vec2(getX(),getY()).add(direction2.multiply(radius+40.0)),direction2.multiply(-3), 8);
         g.setColor(Color.WHITE);
-        GamePanel.getInstance().drawTriangle(g,new Vec2(getX(),getY()).add(direction2.multiply(radius+38)),direction2.multiply(-3), 5);
+        GamePanel.getInstance().drawTriangle(g,new Vec2(getX(),getY()).add(direction2.multiply(radius+38.0)),direction2.multiply(-3), 5);
     }
 
     /**

src/main/java/jatek/SzereloView.java

@@ -33,7 +33,7 @@ public void draw(Graphics g){
         if(aktJatekos == szerelo){
             //ha a szerelőnél van tárgy, akkor kiírja a képernyő aljára, ha nincs nála semmi, nem ír ki semmit
             if(szerelo.getCsoveg()!=null || szerelo.getPumpa()!=null){
-                String itemString = new String("Item: ");
+                String itemString = "Item: ";
                 if(szerelo.getCsoveg()!=null){
                     itemString+="Csővég";
                 }