Button Label lesen

[kaper28]

Hallo Freunde, ich möchte die labels von den Button lesen, dafür habe ich die getLabel() Methode benutzt aber klappt irdenwie nicht. Hier ist mein Code:

package calculator.pack;

import java.awt.Button;
import java.awt.Frame;
import java.awt.GridLayout;
import java.awt.Label;
import java.awt.Panel;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;



public class AwtFenster extends Frame {
	private Panel panelZahlen, panelausgabe;
	private Button button1, button2, button3, button4, button5, button6;
	private Button button7, button8, button9, button0, buttonx, buttonxx;
    private Label ergebnis;
	// Konstruktor
	public AwtFenster() {
		super("Calculator");

		//LayoutMannager für das Hauptfenster 
		this.setLayout(new GridLayout(2,1));
		
		//Aufbau des Panels panelzahlen
		panelZahlen=new Panel();
	    panelZahlen.setLayout(new GridLayout(4, 3, 10, 10));
	    Button button1 = new Button(" 1 ");
		Button button2 = new Button(" 2 ");
		Button button3 = new Button(" 3 ");
		Button button4 = new Button(" 4 ");
		Button button5 = new Button(" 5 ");
		Button button6 = new Button(" 6 ");
		Button button7 = new Button(" 7 ");
		Button button8 = new Button(" 8 ");
		Button button9 = new Button(" 9 ");
		Button button0 = new Button(" 0 ");
		Button buttonx = new Button(" * ");
		Button buttonxx = new Button(" # ");
	    panelZahlen.add(button1);
	    panelZahlen.add(button2);
	    panelZahlen.add(button3);
	    panelZahlen.add(button4);
	    panelZahlen.add(button5);
	    panelZahlen.add(button6);
	    panelZahlen.add(button7);
	    panelZahlen.add(button8);
	    panelZahlen.add(button9);
	    panelZahlen.add(buttonx);
	    panelZahlen.add(button0);
	    panelZahlen.add(buttonxx);
	 
	    button1.addActionListener(new Aktion());	
	    //Aufbau des Ausgabe Panels
	    panelausgabe= new Panel();
	    ergebnis= new Label("Bitte Warten !");
	    panelausgabe.add(ergebnis);
	    
	    //zusammensetzen der Panels
	    this.add(panelZahlen);
	    this.add(panelausgabe);
	    

        // Event-Handling für`s Hauptfenster
        this.addWindowListener(new WindowAdapter() {
            public void windowClosing(WindowEvent e) {
                System.exit(0);
            }
        });

    }
	    
		//Ein Handling für die Action Klasse Aktion 
	class Aktion implements ActionListener{
		public void actionPerformed(ActionEvent e){
			berechnen(e);
		}
		
	}
	
	public void berechnen(ActionEvent e){
		
	
	
		 // Welcher Knopf wurde gedrückt
	 	String z1=(button1.getLabel());
        if (e.getActionCommand() == button1.getLabel()) {
       
        	// Ergebnis ausgeben
            ergebnis.setText(String.valueOf(z1));
	
	
	}
	}
}

 

[SlaterB]

Strings mit equals vergleichen,
und was läßt dich vermuten, dass etwas nicht klappt?
der Vergleich mit e.getActionCommand() bringt nichts, wenn kein ActionCommand gesetzt ist
(oder das Label des Buttons ist tatsächlich automatisch auch das ActionCommand, weiß ich gar nicht genau)

du kannst aber die Source vergleichen:
if (e.Source() == button1) {
das geht dann sogar mit ==, da es ein und dasselbe Objekt sein muss, während man sich bei Strings normalerweise mit Gleichtheit begnügen sollte
(equals bei Buttons ist auch nicht verboten, macht aber intern eh einen == Vergleich)


Problem dürfte noch sein, dass button1 null ist,
im Konstruktor setzt du nicht das Klassenattribut, sondern definierst eine gleichnamige lokale Variable

Button button1 = new Button(" 1 ");

->

button1 = new Button(" 1 ");

 

[kaper28]

package calculator.pack;

import java.awt.Button;
import java.awt.Frame;
import java.awt.GridLayout;
import java.awt.Label;
import java.awt.Panel;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;



public class AwtFenster extends Frame {
	private Panel panelZahlen, panelausgabe;
	private Button button1, button2, button3, button4, button5, button6;
	private Button button7, button8, button9, button0, buttonx, buttonxx;
    private Label ergebnis;
	// Konstruktor
	public AwtFenster() {
		super("Calculator");

		//LayoutMannager für das Hauptfenster 
		this.setLayout(new GridLayout(2,1));
		
		//Aufbau des Panels panelzahlen
		panelZahlen=new Panel();
	    panelZahlen.setLayout(new GridLayout(4, 3, 10, 10));
	    button1 = new Button("1"); 
	    //Button button1 = new Button("1");
		Button button2 = new Button(" 2 ");
		Button button3 = new Button(" 3 ");
		Button button4 = new Button(" 4 ");
		Button button5 = new Button(" 5 ");
		Button button6 = new Button(" 6 ");
		Button button7 = new Button(" 7 ");
		Button button8 = new Button(" 8 ");
		Button button9 = new Button(" 9 ");
		Button button0 = new Button(" 0 ");
		Button buttonx = new Button(" * ");
		Button buttonxx = new Button(" # ");
	    panelZahlen.add(button1);
	    panelZahlen.add(button2);
	    panelZahlen.add(button3);
	    panelZahlen.add(button4);
	    panelZahlen.add(button5);
	    panelZahlen.add(button6);
	    panelZahlen.add(button7);
	    panelZahlen.add(button8);
	    panelZahlen.add(button9);
	    panelZahlen.add(buttonx);
	    panelZahlen.add(button0);
	    panelZahlen.add(buttonxx);
	 
	    button1.addActionListener(new Aktion());	
	    //Aufbau des Ausgabe Panels
	    panelausgabe= new Panel();
	    ergebnis= new Label("Bitte Warten !");
	    panelausgabe.add(ergebnis);
	    
	    //zusammensetzen der Panels
	    this.add(panelZahlen);
	    this.add(panelausgabe);
	    

        // Event-Handling für`s Hauptfenster
        this.addWindowListener(new WindowAdapter() {
            public void windowClosing(WindowEvent e) {
                System.exit(0);
            }
        });

    }
	    
		//Ein Handling für die Action Klasse Aktion 
	class Aktion implements ActionListener{
		public void actionPerformed(ActionEvent e){
			
		
		
			if (e.getSource() == button1)
	        	// Ergebnis ausgeben
	            ergebnis.setText(String.valueOf(button1));
		
		}
		
	}
	
}

so jetzt kommt las erbegniss so was raus

java.awt.button

 

[SlaterB]

nur weil ich die if-Bedingung geändert habe, muss das nicht auch für ergebnis.setText() gelten,
dort kannst du es machen wie zuvor, falls das geht, oder was immer du vorhast


> button1 = new Button("1");
> //Button button1 = new Button("1");
> Button button2 = new Button(" 2 ");
> Button button3 = new Button(" 3 ");

ich hoffe, dir ist aufgefallen, dass für die anderen Buttons das gleiche Problem besteht,
ein Array von Buttons wäre auch nicht schlecht

 

[Schandro]

ergebnis.setText(String.valueOf(button1));

Du gibts den Button ANSICH aus. Du willst aber das Label des Buttons ausgeben:

ergebnis.setText(button1.getLabel());

( Das String.valueOf ist an dieser Stelle sinnlos da getLabel() eh ein String returnt )