Ist an diesem Befehl irgendwas falsch?

[leclerc16]

if(!nameItaTextFeld.getText().equals("Info"));
{
nameItaTextFeld.setForeground(Color.red);
}


Wenn im Textfeld etwas anderes als "Info" eingegeben wird, soll sich die Farbe in rot verändern, wenn man auf den Button klickt, ansonsten soll die Farbe so bleiben. Stattdesse verändert er die Farbe immer in rot, egal ob die Bedinung erfüllt wird oder nicht. Woran könnte das liegen?

Hier ist der kmplette Code. Die if-Klammer befindet sich ganz unten in der kleinen privaten Klasse

import java.awt.Color;
import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JTextField;

public class Frame extends JFrame{
   
   
       
        JLabel stadtDeuLabel     = new JLabel("(DEU) Stadt");
        JLabel stadtItaLabel     = new JLabel("(ITA) Stadt");
        JLabel stadtEngLabel    = new JLabel("(ENG) Stadt");
        JLabel nameItaLabel        = new JLabel("(ITA) Land/Name");
        JLabel nameEngLabel     = new JLabel("(ENG) Land/Name");
       
        JLabel korrekturStadtDeu = new JLabel("Bosnien und Herzigowina");
        JLabel korrekturStadtIta = new JLabel();
        JLabel korrekturStadtEng = new JLabel();
        JLabel korrekturNameIta  = new JLabel();
        JLabel korrekturNameEng  = new JLabel();
       
        JLabel anzeige             = new JLabel("Karibik");
        JLabel erklärung         = new JLabel("Westindien bz. westindische Inseln");
       
       
        JTextField nameItaTextFeld     = new JTextField();
        JTextField nameEngTextFeld    = new JTextField();
        JTextField stadtDeuTextFeld    = new JTextField();
        JTextField stadtItaTextFeld    = new JTextField();
        JTextField stadtEngTextFeld    = new JTextField();
       
        JButton go            = new JButton("GO!");
        JButton weiter         = new JButton("Weiter");
       
        Font korrekturFont    = new Font("Tahoma", Font.PLAIN, 16);
        Font textFeldSchrift  = new Font("Tahoma", Font.PLAIN,16);
        Font anzeigeSchrift   = new Font("Tahoma", Font.PLAIN, 40);
        Font erklärungSchrift = new Font("Tahoma", Font.PLAIN, 17);
       
        Vokabeln [] a;
        static int nr = 0;
       
       
       

       
    Frame(Vokabeln [] a){
        this.a=a;
        setTitle("GEO-QUIZ");
       
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setVisible(true);
        setSize(800,650);
        setLocation(500,200);
        setLayout(null);
       
        nameItaTextFeld.setBounds(50,400,250,50);
        nameEngTextFeld.setBounds(50,500,250,50);
        stadtDeuTextFeld.setBounds(350,300,250,50);
        stadtItaTextFeld.setBounds(350,400,250,50);
        stadtEngTextFeld.setBounds(350,500,250,50);
           
        nameItaTextFeld.setFont(textFeldSchrift);
        nameEngTextFeld.setFont(textFeldSchrift);
        stadtDeuTextFeld.setFont(textFeldSchrift);
        stadtItaTextFeld.setFont(textFeldSchrift);
        stadtEngTextFeld.setFont(textFeldSchrift);
       
        add(nameItaTextFeld);
        add(nameEngTextFeld);
        add(stadtDeuTextFeld);
        add(stadtItaTextFeld);
        add(stadtEngTextFeld);
       
        korrekturStadtDeu.setBounds(356,335,250,50);
        korrekturStadtIta.setBounds(50,400,250,50);
        korrekturStadtEng.setBounds(50,400,250,50);
        korrekturNameIta.setBounds(50,400,250,50);
        korrekturNameEng.setBounds(50,400,250,50);
       
        korrekturStadtDeu.setFont(korrekturFont);
        korrekturStadtIta.setFont(korrekturFont);
        korrekturStadtEng.setFont(korrekturFont);
        korrekturNameIta.setFont(korrekturFont);
        korrekturNameEng.setFont(korrekturFont);
       
        korrekturStadtDeu.setForeground(Color.blue);
        korrekturStadtIta.setForeground(Color.blue);
        korrekturStadtEng.setForeground(Color.blue);
        korrekturNameIta.setForeground(Color.blue);
        korrekturNameEng.setForeground(Color.blue);
               
        add(korrekturStadtDeu);
        add(korrekturStadtIta);
        add(korrekturStadtEng);
        add(korrekturNameIta);
        add(korrekturNameEng);
       
        go.setBounds(650,400,100,50);
        go.addActionListener(new GoListener());
        weiter.setBounds(650,500,100,50);
        weiter.addActionListener(new WeiterListener());
       
        add(weiter);
        add(go);
       
        nameItaLabel.setBounds(60,370,250,25);
        nameEngLabel.setBounds(60,470,250,25);
        stadtDeuLabel.setBounds(360,270,250,25);
        stadtItaLabel.setBounds(360,370,250,25);
        stadtEngLabel.setBounds(360,470,250,25);
       
        add(nameItaLabel);
        add(nameEngLabel);
        add(stadtDeuLabel);
        add(stadtItaLabel);
        add(stadtEngLabel);
       
        anzeige.setBounds(50,70,700,60);
        anzeige.setFont(anzeigeSchrift);
        add(anzeige);
       
        erklärung.setBounds(60,130,300,50);
        erklärung.setFont(erklärungSchrift);
        add(erklärung);
        }
   
   
   
   
    public void neueVokabelAnzeigen() {
       
        nameItaTextFeld.setForeground(Color.black);
        nameEngTextFeld.setForeground(Color.black);
        stadtDeuTextFeld.setForeground(Color.black);
        stadtItaTextFeld.setForeground(Color.black);
        stadtEngTextFeld.setForeground(Color.black);
       
        stadtDeuTextFeld.setText("");
        stadtItaTextFeld.setText("");
        stadtEngTextFeld.setText("");
        nameItaTextFeld.setText("");
        nameEngTextFeld.setText("");
       
       
        stadtDeuTextFeld.setVisible(true);
        stadtItaTextFeld.setVisible(true);
        stadtEngTextFeld.setVisible(true);
       
        anzeige.setText(a[nr].nameDeu);
        erklärung.setText(a[nr].erklärung);
        if(a[nr].ausblenden) {
            stadtDeuTextFeld.setVisible(false);
            stadtItaTextFeld.setVisible(false);
            stadtEngTextFeld.setVisible(false);
            nameItaTextFeld.requestFocus();
        }else
       
        stadtDeuTextFeld.requestFocus();
    }

   
   

    private class WeiterListener implements ActionListener{
        @Override
        public void actionPerformed(ActionEvent arg0) {
           
           
            nr++;
            neueVokabelAnzeigen();   
        }
    }




    private class GoListener implements ActionListener{
        @Override
        public void actionPerformed(ActionEvent arg0) {
           
            if(!nameItaTextFeld.getText().equals("Info");
            {
                nameItaTextFeld.setForeground(Color.red);   
            }
        }
    }
   
}

Für das Beispiel habe ich "Info" genommen, eigentlich fragt das Programm Ländernamen ab.

 

[httpdigest]

Entferne das Semikolon am Ende von if(!nameItaTextFeld.getText().equals("Info");
Was das effektiv bewirkte war, dass das empty Statement ausgeführt wurde, wenn die if Bedingung zutraf. Und danach wurde der davon unabhängige Block danach ausgeführt

 

[leclerc16]

Stimmt, habe ich übersehen. Herzelichen Dank!