Ändern von Text in JLabel mit einem Button

[Mojo7310]

Guten Tag,

ich fass mich kurz. Ich möchte den Text von isPalindrom beim Drücken vom Butten umdrehen ändern. Aber anstatt sich zu ändern, verschwiendet er einfach.

Ich hoffe auf schnelle Hilfe und anfängerfreundliche Antwort.

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.awt.Graphics.*;

public class PalindromPrueferGUI extends JFrame
{
    private JTextField eingabe;
    private JTextField ausgabe;
    private JButton umdrehen;
    private JPanel pEingabe;
    private JPanel pAusgabe;
    private JPanel pUmdrehen;
    private JLabel isPalindrom;
    private Palindrompruefer wow;

   
    public PalindromPrueferGUI()
    {
        super("Palindromprüfer");
       
       
        eingabe = new JTextField(20);
        ausgabe = new JTextField(20);
        ausgabe.setEditable(false);
        umdrehen = new JButton("Umdrehen");
       
       
        String palindrom = eingabe.getText();
       
        wow = new Palindrompruefer(palindrom);
       

        pEingabe = new JPanel();
        pAusgabe = new JPanel();
        pUmdrehen = new JPanel();
       
        isPalindrom = new JLabel("Noch nicht getestet!");
       
       
        pEingabe.add(new JLabel("Eingabe: "));
        pEingabe.add(eingabe);
       
        pAusgabe.add(new JLabel("Ausgabe: "));
        pAusgabe.add(ausgabe);
        umdrehen.addActionListener(new MyActionListener());
       
       
       
        pUmdrehen.add(umdrehen);
        pUmdrehen.add(isPalindrom);
       
   
        Container contentPane = getContentPane();
        contentPane.setLayout(new BorderLayout());
       
        contentPane.add(pEingabe,BorderLayout.NORTH);
        contentPane.add(pUmdrehen,BorderLayout.CENTER);
        contentPane.add(pAusgabe,BorderLayout.SOUTH);
       
       
       
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        pack();
        setVisible(true);
       
    }
   
   
    public void setEingabe(String text){
        eingabe.setText(text);
    }

   
    public class MyActionListener implements ActionListener{
        public void actionPerformed(ActionEvent e){
            boolean g = wow.isso();
           
            if(g == true){
                isPalindrom.setText("Es handelt sich um ein Palindrom");
               
            }
            else{
               
                isPalindrom.setText("Es handelt sich nicht um ein Palindrom");
               
                isPalindrom.updateUI();
             
            }
        }
    }

}

 

[KonradN]

Die Fehlerbeschreibung verstehe ich gerade nicht. Evtl. kannst Du das etwas näher beschreiben.

Ein Problem, das aber auffällt: Du erstellst den Palindromprüfer einmalig direkt beim erstellen der UI:
wow = new Palindrompruefer(palindrom);

Das ist also damit der leere String weil ja noch keine Eingabe erfolgt ist. Du willst es ja erst prüfen, wenn der Knopf gedrückt ist, daher musst Du da dann die Eingabe auslesen und auswerten.

 

[Mojo7310]

Danke für die schnelle Antwort.

Der angesprochene Fehler ist mir gerade auch aufgefallen, sollte aber für das angesprochene Problem erstmal uninteressant sein.

Beim ausführen/klichen auf den Button ändert sich der angezeigte Text "Noch nicht getestet!" nicht sondern verschwindet komplett. Dies ist das Problem. Ich hab jegliche Methode online ausprobiert, aber leider funktioniert nichts.

Ich hoffe so wurde es etwas klarer

 

[KonradN]

Danke für die schnelle Antwort.

Der angesprochene Fehler ist mir gerade auch aufgefallen, sollte aber für das angesprochene Problem erstmal uninteressant sein.

Beim ausführen/klichen auf den Button ändert sich der angezeigte Text "Noch nicht getestet!" nicht sondern verschwindet komplett. Dies ist das Problem. Ich hab jegliche Methode online ausprobiert, aber leider funktioniert nichts.

Ich hoffe so wurde es etwas klarer

Der Code, den Du gezeigt hast, führt aber nicht zu diesem Problem. Es reicht, den Text eines Labels neu zu setzen. (Der updateUI Aufruf wird nicht gebraucht - sollte aber auch kein Problem darstellen.)

Dann wäre die Frage, was Du uns an Code nicht gezeigt hast.

 

[Mojo7310]

Tatsächlich ist dies der gesamte Code. Ich programmiere in BlueJ. Mehr habe ich nicht genutzt.

 

[KonradN]

Es muss noch mindestens die Klasse Palindrompruefer geben. Bei BlueJ wäre evtl. noch die Frage, wie Du das Programm startest.

Erstellst Du einfach eine Instanz von PalindromPrueferGUI? Ich kenne mich mit BlueJ nicht wirklich aus - gut möglich, dass BlueJ hier diverse Probleme hat mit den so erzeugten Objekten, wenn man das nicht richtig bedient.

 

[Mojo7310]

Achso. Ok. Da hatte ich ein Verständnisproblem.

Nachfolgend finden Sie den Quellcode des Palindrompruefer.

public class Palindrompruefer
{
    private Stack <Character> carsten;
    private String umgedreht, krass;
    
    public Palindrompruefer(String krass)
    {
        // Instanzvariable initialisieren
        carsten = new Stack();
        for(int l = 0 ;l< krass.length();l++){
            carsten.push(krass.charAt(l));
            }
        pruefen();
    }
    public void pruefen(){
        umgedreht = null;
        while(!carsten.isEmpty()){
            umgedreht = umgedreht + carsten.pop();
              
        }
          
    }
    public boolean isso(){
        boolean c = false;
          if(umgedreht == krass){
            c = true;
        }
        else{
            c = false;
        }
         return c;
    }   
}

Ich erstelle eine Instanz der PalindromPrueferGui, was wie folgt aussieht.

 

[MarvinsDepression]

String palindrom = eingabe.getText();
wow = new Palindrompruefer(palindrom);

Wieso schreibst Du diese Zeilen in den Konstruktor. Wann soll dieser Code ausgeführt werden? Doch erst dann, wenn der Button gedrückt wird.

Diese Zeilen müssen in die Methode actionPerformed()

 

[Mojo7310]

Ich habe es gerade verändert. Etwas Ändern am ursprünglichen Fehler tut es aber leider immer noch nicht.

 

[MarvinsDepression]

...
    isPalindrom.setText("Es handelt sich um ein Palindrom");
} else {
    isPalindrom.setText("Es handelt sich nicht um ein Palindrom");
...

sollte doch warscheinlich

...
    ausgabe.setText("Es handelt sich um ein Palindrom");
} else {
    ausgabe.setText("Es handelt sich nicht um ein Palindrom");
...

heißen??

 

[KonradN]

Ok, das ist ein Bildschirmfoto vom Start. Und wenn Du den Knopf drückst, dann dürfte es so aussehen:


Das ist das Ergebnis von genau Deinem Code - nach Erstellen einer Instanz von der GUI Klasse und Drücken des Umdrehen Knopfes.

Ansonsten ist auch in der Klasse Palindrompruefer an vielen Stellen der Wurm drin.

Angefangen, dass der Parameter krass nicht gemerkt wird. Dann ist die Berechnung des umgedrehten Wortes falsch. (Aus "123" macht er "null321"). Und zuletzt: Instanzen vergleicht man nicht per == sondern mit equals.

Aber das sind halt alles andere Probleme und ich kann kein Problem mit der Aktualisierung der UI erkennen.

 

[Mojo7310]

Ok, trotzdem vielen vielen Dank für die Hilfe. Vermutlich wäre ich ohne diese verzweifelt.