Compiler-Fehler if Bedingung fehlerhaft

[Playermagier]

Hallo,
folgendes Problem besteht. In der Methode 'handleMouseClicked' in der verschachtelten if-Bedingung

if(kreis.klickanzahl == 1 )
               {
                   kreis.setFill(Color.BLUE);
                   kreis.setFarbe(Color.BLUE);
                   if( letzterKreis.getColor() == vorletzterKreis.getColor())
                   kreiseVerbinden(vorletzterKreis, kreis);
               }

verbindet er die Kreise nicht miteinander. Kann mir evtl. jemand weiterhelfen? Ich habe im weiteren, zusätzlichen Code hinzugefügt für das bessere Verständnis.

private class Kreis extends Circle
{
       private Color farbe;
       private int klickanzahl=0;
       private int kantenanzahl =0;
       public Kreis ( double xpos, double ypos, int CN , int KN, Color f)
       {
           // Von Superklasse Circle geerbt
           this.setCenterX(xpos);
           this.setCenterY(ypos);
           this.setRadius(RADIUS);
           this.setFill(Color.GREEN);
           farbe = f;
           klickanzahl = CN;
           kantenanzahl = KN;
       }
       public Color setFarbe(Color Farbe) 
       {
           return farbe = Farbe;
       }

       public Color getColor()
       {
           return farbe;
       }
        void erhoeheCN()
        {
            klickanzahl = klickanzahl + 1;
        }

        void erhoeheKN(){
            kantenanzahl = kantenanzahl + 1;
        }
}

@FXML
   public void handleMouseClicked(MouseEvent event) 
   {
       // Anzahl der Klicks pro Knoten sollen gemerkt werden
       for(Kreis kreis:kreise)
       {
           letzterKreis = null;
           vorletzterKreis = null;
           if(kreis.isClicked(event.getX(), event.getY()))
           {
               vorletzterKreis = letzterKreis;
               letzterKreis = kreis;
               kreis.erhoeheCN();
               System.out.println(kreis.klickanzahl);

               if(kreis.klickanzahl == 1 )
               {
                   kreis.setFill(Color.BLUE);
                   kreis.setFarbe(Color.BLUE);
                   if( letzterKreis.getColor() == vorletzterKreis.getColor())
                   kreiseVerbinden(vorletzterKreis, kreis);
               }
               else 
               {
                   kreis.klickanzahl = kreis.klickanzahl - 2;
                   kreis.setFill(Color.GREEN);
                   System.out.println(kreis.klickanzahl);
               }
           }
       }

@FXML
   public void handleNeuesProblemButton(ActionEvent event) 
   {
       paneCircles.setDisable(false);   // die Zeichenfläche wird (wieder) aktiv

       // Kreise neu generieren und platzieren
       if(start)
       {
           start = false;
           neuesProblem();
       }
       else
       {
           kreise.clear();
           //zahlen.clear();
           letzterKreis = null;
           KnotenAnzahlAendern();

       // da nur die Kreise der Pane gehören, können wir alle Elemente entfernen
       paneCircles.getChildren().removeAll(paneCircles.getChildren());
       }
       // platziere neue Kreise mit Zahlen
       kreiseZeichnen();
   }

public void neuesProblem() 
{
   TextInputDialog dialog = new TextInputDialog("");
   dialog.setTitle ("Neues Problem");
   dialog.setHeaderText("Spielhinweis: Grüner Punkt ist Start- und Zielknoten.");
   dialog.setContentText("Wieviele Knoten möchten Sie verbinden?");

   Optional<String> result = dialog.showAndWait();
   result.ifPresent(anzahlKnoten -> System.out.println("Anzahl der Knoten " + anzahlKnoten));

   anzahlKnoten = Integer.parseInt(result.get());
   }
   public void KnotenAnzahlAendern()
   {
       Alert alert = new Alert(AlertType.CONFIRMATION);
       alert.setTitle("Auswahl von Anzahl der Knoten");
       alert.setContentText("Möchten Sie die Anzahl der Knoten ändern?");

       ButtonType buttonTypeJa = new ButtonType("Ja");
       ButtonType buttonTypeNein = new ButtonType("Nein",ButtonData.CANCEL_CLOSE);

       Optional<ButtonType> result = alert.showAndWait();

       if(result.get() == buttonTypeJa)
       {
           neuesProblem();
       }
   }

public void kreiseVerbinden(Kreis letzterKreis, Kreis vorletzterKreis) {
Line path = new Line(letzterKreis.getCenterX(),letzterKreis.getCenterY(),vorletzterKreis.getCenterX();
vorletzterKreis.getCenterY());
       path.setStrokeWidth(5.);
       path.setStroke(Color.BLACK);
       paneCircles.getChildren().add(path);

       // Kreise auf den Linien zeichnen
       // Zuerst alle entfernen, dann alle hinzufuegen
       paneCircles.getChildren().removeAll(kreise);
       paneCircles.getChildren().addAll(kreise);
   }

Vielen Dank im Voraus.

 

[MarkusM]

Probiere mal:

if( letzterKreis.getColor().equals(vorletzterKreis.getColor())){....

 

[Fulumbler]

Was MarkusM gesagt hat:
Durch '==' vergleichst du, ob es sich bei der Farbe um dasselbe Objekt handelt, mit equals schaust du, ob die Werte gleich sind.

 

[Playermagier]

if(kreis.klickanzahl == 1 )
               {
                   kreis.setFill(Color.BLUE);
                   kreis.setFarbe(Color.BLUE);
                   System.out.println("Blue___");
                   if( kreis.getColor().equals(vorletzterKreis.getColor() )){
                       System.out.println("es geht nicht :-( ");
                   kreiseVerbinden(vorletzterKreis, kreis);
                   }
               }

System.out.println("Blue___"); wird hier noch ausgeführt, jedoch funktioniert nicht die innere If Bedingung nicht!
Zusätzlich spuckt der Compiler folgende Meldung aus:
Caused by: java.lang.NullPointerException
at application.view.Handlungsreisender_Controller.kreiseVerbinden(Handlungsreisender_Controller.java:344)
at application.view.Handlungsreisender_Controller.handleMouseClicked(Handlungsreisender_Controller.java:158)

 

[Libentis]

if( letzterKreis.getColor().equals(vorletzterKreis.getColor()) )
     kreiseVerbinden(vorletzterKreis, kreis);

oder

if( kreis.getColor().equals(vorletzterKreis.getColor()) )
     kreiseVerbinden(vorletzterKreis, kreis);

?
(Unterschied ist in der if, einmal vergleichst du mit dem letzten Kreis und einmal mit dem aktuellen Kreis)

 

[Playermagier]

Kann es sein das wir im if vergleich, ein vergleich auf ein objekt machen was nicht vorhanden ist im Speicher?

 

[thecain]

Schau doch einfach mal mit dem Debugger nach. Wir können nur raten

 

[Playermagier]

Fehler gefunden der ist wirklich dämlich!
in handleMouseClicked(MouseEvent event)......

der Ausdruck : letzterKreis = null;
vorletzterKreis = null;
hat immer letzterKreis und vor letzter Kreis immer auf null !

Danke für eure Hilfe