Swing JButton int im Event

[Andi_CH]

So langsam möchte ich Swing kennen lernen und bin aber nicht mit der erst besten Lösung zufrieden ;-)
Ich hab als Übungsumfeld einen Rechner der funktioniert, allerdings werden die numerischen Keys mit String.compare identifiziert, ich möchte aber lieber einen int erhalten.

Die Buttons werden in einem loop instanziert

buttons[i] = new JButton(String.valueOf(i));

Ein int als id lässt sich nicht setzen.

Meine Versuche:

private static class KeyAction implements Action {
	private final int mId;

	public KeyAction(int id) {
		mId = id;
	}

	@Override
	public void actionPerformed(ActionEvent e) {
		// ja da käme was anders hin :-)
		System.out.println("Id = " + this.mId);
	}
// und noch einige Dummyimplementationen
}

// Die Buttons werden nun wie folgt instanziert
buttons[i] = new JButton(new KeyAction(i));
// leider haben die Buttons jetzt keinen Text mehr drin und setText gibt es nicht.

buttons[i] = new JButton(String.valueOf(i));
buttons[i].setAction(a);
// hat denselben Effekt -> kein Text

Wie macht man so etwas richtig?

 

[SlaterB]

wenn du das Interface javax.swing.Action implementierst, dann müssen es doch noch ein paar Methoden mehr sein,
so wie bisher gepostet kompiliert die KeyAction-Klasse gar nicht, erweitert lautet sie mindestens:

private static class KeyAction
    implements Action
{
    private final int mId;

    public KeyAction(int id)
    {
        mId = id;
    }

    public void actionPerformed(ActionEvent e)
    {
        // ja da käme was anders hin :-)
        System.out.println("Id = " + this.mId);
    }
    // und noch einige Dummyimplementationen

    public Object getValue(String key)
    {
        return null;
    }

    public void putValue(String key, Object value)
    {
    }

    public void setEnabled(boolean b)
    {
    }

    public boolean isEnabled()
    {
        return false;
    }

    public void addPropertyChangeListener(PropertyChangeListener listener)
    {
    }

    public void removePropertyChangeListener(PropertyChangeListener listener)
    {
    }
}

(edit: ach, jetzt sehe ich 'Dummyimplementationen'..)

getValue() ist dann unter anderem für den Text zuständig, siehe Bedienunganleitung (API) :
Action (Java Platform SE 6)

wegen PropertyChangeListener usw. ist ein reines Implementieren des Interfaces aber aufwendig,
erbe lieber von AbstractAction, da ist das meiste dann schon da,

oder gehts auch einfacher ohne Action, wofür nochmal genau benötigt?

 

[Andi_CH]

Ich hab da einen Taschenrechner geschrieben (hätte Guttenberg wohl behauptet - nö, ich hab den gefunden und spiele damit rum - Ich habe ja auch den Header unverändert gelassen, da ich mich nicht mit fremden Federn schmücken will)

Was mich stört ist, dass Calculator.actionPerformed einen String aus dem Event holt und die vergleicht - Strings sind mir einfach unsympatisch in den Vergleichen (Zeile 121)

Das ist die noch am wenigsten verbastelte Version
So erscheint der Text auf dem Knopf aber im ActionListener kommt ein String als Identifikation

Ich möchte einfach, dass der Actionlistener einen int als Id erhält
(switch-case, schneller Vergleiche, direktes rechnen ...)

/*
Name: Hemanth. B
Website: [url=http://www.java-swing-tutorial.com]java-swing-tutorial.com[/url]

Topic: A basic Java Swing Calculator

Conventions Used in Source code
---------------------------------
	1. All JLabel components start with jlb*
	2. All JPanel components start with jpl*
	3. All JMenu components start with jmenu*
	4. All JMenuItem components start with jmenuItem*
	5. All JDialog components start with jdlg*
	6. All JButton components start with jbn*
*/

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.FlowLayout;
import java.awt.Font;
import java.awt.GridLayout;
import java.awt.Window;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;

import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.KeyStroke;

@SuppressWarnings("serial")
public class Calculator extends JFrame implements ActionListener{
	// Variables
	final int MAX_INPUT_LENGTH = 20;
	final int INPUT_MODE = 0;
	final int RESULT_MODE = 1;
	final int ERROR_MODE = 2;
	int displayMode;

	boolean clearOnNextDigit, percent;
	double lastNumber;
	String lastOperator;

	private JMenu jmenuFile, jmenuHelp;
	private JMenuItem jmenuitemExit, jmenuitemAbout;
	
	private JLabel jlbOutput;
	private JButton jbnButtons[];
	private JPanel jplMaster, jplBackSpace, jplControl;
	
	/*
	 * Font(String name, int style, int size)
      Creates a new Font from the specified name, style and point size.
	 */
	
	Font f12 = new Font("Times New Roman", 0, 12);
	Font f121 = new Font("Times New Roman", 1, 12);
	
	// Constructor 
	public Calculator() 
	{
		/* Set Up the JMenuBar.
		 * Have Provided All JMenu's with Mnemonics
		 * Have Provided some JMenuItem components with Keyboard Accelerators
		 */ 
		
		jmenuFile = new JMenu("File");
		jmenuFile.setFont(f121);
		jmenuFile.setMnemonic(KeyEvent.VK_F);
		
		jmenuitemExit = new JMenuItem("Exit");
		jmenuitemExit.setFont(f12);
		jmenuitemExit.setAccelerator(KeyStroke.getKeyStroke( KeyEvent.VK_X, ActionEvent.CTRL_MASK));
		jmenuFile.add(jmenuitemExit);

		jmenuHelp = new JMenu("Help");
		jmenuHelp.setFont(f121);
		jmenuHelp.setMnemonic(KeyEvent.VK_H);

		jmenuitemAbout = new JMenuItem("About Calculator");
		jmenuitemAbout.setFont(f12);
		jmenuHelp.add(jmenuitemAbout);
		
		JMenuBar mb = new JMenuBar();
		mb.add(jmenuFile);
		mb.add(jmenuHelp);
		setJMenuBar(mb);
		
		//Set frame layout manager

		setBackground(Color.gray);

		jplMaster = new JPanel();

		jlbOutput = new JLabel("0");
		jlbOutput.setHorizontalTextPosition(JLabel.RIGHT);
		jlbOutput.setBackground(Color.WHITE);
		jlbOutput.setOpaque(true);
		
		// Add components to frame
		getContentPane().add(jlbOutput, BorderLayout.NORTH);

		jbnButtons = new JButton[23];
//		GridLayout(int rows, int cols, int hgap, int vgap) 

		JPanel jplButtons = new JPanel();	// container for Jbuttons

		// Create numeric Jbuttons
		for (int i=0; i<=9; i++)
		{
			// set each Jbutton label to the value of index
			jbnButtons[i] = new JButton(String.valueOf(i));
		}

		// Create operator Jbuttons
		jbnButtons[10] = new JButton("+/-");
		jbnButtons[11] = new JButton(".");
		jbnButtons[12] = new JButton("=");
		jbnButtons[13] = new JButton("/");
		jbnButtons[14] = new JButton("*");
		jbnButtons[15] = new JButton("-");
		jbnButtons[16] = new JButton("+");
		jbnButtons[17] = new JButton("sqrt");
		jbnButtons[18] = new JButton("1/x");
		jbnButtons[19] = new JButton("%");
		
		jplBackSpace = new JPanel();
		jplBackSpace.setLayout(new GridLayout(1, 1, 2, 2));

		jbnButtons[20] = new JButton("Backspace");
		jplBackSpace.add(jbnButtons[20]);

		jplControl = new JPanel();
		jplControl.setLayout(new GridLayout(1, 2, 2 ,2));

		jbnButtons[21] = new JButton(" CE ");
		jbnButtons[22] = new JButton("C");

		jplControl.add(jbnButtons[21]);
		jplControl.add(jbnButtons[22]);

//		Setting all Numbered JButton's to Blue. The rest to Red
		for (int i=0; i<jbnButtons.length; i++)	{
			jbnButtons[i].setFont(f12);

			if (i<10)
				jbnButtons[i].setForeground(Color.blue);
				
			else
				jbnButtons[i].setForeground(Color.red);
		}
	
		// Set panel layout manager for a 4 by 5 grid
		jplButtons.setLayout(new GridLayout(4, 5, 2, 2));
		
		//Add buttons to keypad panel starting at top left
		// First row
		for(int i=7; i<=9; i++)		{
			jplButtons.add(jbnButtons[i]);
		}
		
		// add button / and sqrt
		jplButtons.add(jbnButtons[13]);
		jplButtons.add(jbnButtons[17]);
		
		// Second row
		for(int i=4; i<=6; i++)
		{
			jplButtons.add(jbnButtons[i]);
		}
		
		// add button * and x^2
		jplButtons.add(jbnButtons[14]);
		jplButtons.add(jbnButtons[18]);

		// Third row
		for( int i=1; i<=3; i++)
		{
			jplButtons.add(jbnButtons[i]);
		}
		
		//adds button - and %
		jplButtons.add(jbnButtons[15]);
		jplButtons.add(jbnButtons[19]);
		
		//Fourth Row
		// add 0, +/-, ., +, and =
		jplButtons.add(jbnButtons[0]);
		jplButtons.add(jbnButtons[10]);
		jplButtons.add(jbnButtons[11]);
		jplButtons.add(jbnButtons[16]);
		jplButtons.add(jbnButtons[12]);
		
		jplMaster.setLayout(new BorderLayout());
		jplMaster.add(jplBackSpace, BorderLayout.WEST);
		jplMaster.add(jplControl, BorderLayout.EAST);
		jplMaster.add(jplButtons, BorderLayout.SOUTH);

		// Add components to frame
		getContentPane().add(jplMaster, BorderLayout.SOUTH);
		requestFocus();
		
		//activate ActionListener
		for (int i=0; i<jbnButtons.length; i++){
			jbnButtons[i].addActionListener(this);
		}
		
		jmenuitemAbout.addActionListener(this);
		jmenuitemExit.addActionListener(this);

		clearAll();

		//add WindowListener for closing frame and ending program
		addWindowListener(new WindowAdapter() {

				public void windowClosed(WindowEvent e)
				{
					System.exit(0);
				}
			}
		);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	}	//End of Contructor Calculator

	// Perform action
	public void actionPerformed(ActionEvent e){
		double result = 0;
	   
		if(e.getSource() == jmenuitemAbout){
			JDialog dlgAbout = new CustomABOUTDialog(this, "About Java Swing Calculator", true);
			dlgAbout.setVisible(true);
		}else if(e.getSource() == jmenuitemExit){
			System.exit(0);
		}	

		// Search for the button pressed until end of array or key found
		for (int i=0; i<jbnButtons.length; i++)
		{
			if(e.getSource() == jbnButtons[i])
			{
				switch(i)
				{
					case 0:
						addDigitToDisplay(i);
						break;

					case 1:
						addDigitToDisplay(i);
						break;

					case 2:
						addDigitToDisplay(i);
						break;

					case 3:
						addDigitToDisplay(i);
						break;

					case 4:
						addDigitToDisplay(i);
						break;

					case 5:
						addDigitToDisplay(i);
						break;

					case 6:
						addDigitToDisplay(i);
						break;

					case 7:
						addDigitToDisplay(i);
						break;

					case 8:
						addDigitToDisplay(i);
						break;

					case 9:
						addDigitToDisplay(i);
						break;

					case 10:	// +/-
						processSignChange();
						break;

					case 11:	// decimal point
						addDecimalPoint();
						break;

					case 12:	// =
						processEquals();
						break;

					case 13:	// divide
						processOperator("/");
						break;

					case 14:	// *
						processOperator("*");
						break;

					case 15:	// -
						processOperator("-");
						break;

					case 16:	// +
						processOperator("+");
						break;

					case 17:	// sqrt
						if (displayMode != ERROR_MODE)
						{
							try
							{
								if (getDisplayString().indexOf("-") == 0)
									displayError("Invalid input for function!");

								result = Math.sqrt(getNumberInDisplay());
								displayResult(result);
							}

							catch(Exception ex)
							{
								displayError("Invalid input for function!");
								displayMode = ERROR_MODE;
							}
						}
						break;

					case 18:	// 1/x
						if (displayMode != ERROR_MODE){
							try
							{
								if (getNumberInDisplay() == 0)
									displayError("Cannot divide by zero!");
	
								result = 1 / getNumberInDisplay();
								displayResult(result);
							}
							
							catch(Exception ex)	{
								displayError("Cannot divide by zero!");
								displayMode = ERROR_MODE;
							}
						}
						break;

					case 19:	// %
						if (displayMode != ERROR_MODE){
							try	{
								result = getNumberInDisplay() / 100;
								displayResult(result);
							}
	
							catch(Exception ex)	{
								displayError("Invalid input for function!");
								displayMode = ERROR_MODE;
							}
						}
						break;

					case 20:	// backspace
						if (displayMode != ERROR_MODE){
							setDisplayString(getDisplayString().substring(0,
										getDisplayString().length() - 1));
							
							if (getDisplayString().length() < 1)
								setDisplayString("0");
						}
						break;

					case 21:	// CE
						clearExisting();
						break;

					case 22:	// C
						clearAll();
						break;
				}
			}
		}
	}

	void setDisplayString(String s){
		jlbOutput.setText(s);
	}

	String getDisplayString (){
		return jlbOutput.getText();
	}

	void addDigitToDisplay(int digit){
		if (clearOnNextDigit)
			setDisplayString("");

		String inputString = getDisplayString();
		
		if (inputString.indexOf("0") == 0){
			inputString = inputString.substring(1);
		}

		if ((!inputString.equals("0") || digit > 0)  && inputString.length() < MAX_INPUT_LENGTH){
			setDisplayString(inputString + digit);
		}
		

		displayMode = INPUT_MODE;
		clearOnNextDigit = false;
	}

	void addDecimalPoint(){
		displayMode = INPUT_MODE;

		if (clearOnNextDigit)
			setDisplayString("");

		String inputString = getDisplayString();
	
		// If the input string already contains a decimal point, don't
		//  do anything to it.
		if (inputString.indexOf(".") < 0)
			setDisplayString(new String(inputString + "."));
	}

	void processSignChange(){
		if (displayMode == INPUT_MODE)
		{
			String input = getDisplayString();

			if (input.length() > 0 && !input.equals("0"))
			{
				if (input.indexOf("-") == 0)
					setDisplayString(input.substring(1));

				else
					setDisplayString("-" + input);
			}
			
		}

		else if (displayMode == RESULT_MODE)
		{
			double numberInDisplay = getNumberInDisplay();
		
			if (numberInDisplay != 0)
				displayResult(-numberInDisplay);
		}
	}

	void clearAll()	{
		setDisplayString("0");
		lastOperator = "0";
		lastNumber = 0;
		displayMode = INPUT_MODE;
		clearOnNextDigit = true;
	}

	void clearExisting(){
		setDisplayString("0");
		clearOnNextDigit = true;
		displayMode = INPUT_MODE;
	}

	double getNumberInDisplay()	{
		String input = jlbOutput.getText();
		return Double.parseDouble(input);
	}

	void processOperator(String op) {
		if (displayMode != ERROR_MODE)
		{
			double numberInDisplay = getNumberInDisplay();

			if (!lastOperator.equals("0"))	
			{
				try
				{
					double result = processLastOperator();
					displayResult(result);
					lastNumber = result;
				}

				catch (DivideByZeroException e)
				{
				}
			}
		
			else
			{
				lastNumber = numberInDisplay;
			}
			
			clearOnNextDigit = true;
			lastOperator = op;
		}
	}

	void processEquals(){
		double result = 0;

		if (displayMode != ERROR_MODE){
			try			
			{
				result = processLastOperator();
				displayResult(result);
			}
			
			catch (DivideByZeroException e)	{
				displayError("Cannot divide by zero!");
			}

			lastOperator = "0";
		}
	}

	double processLastOperator() throws DivideByZeroException {
		double result = 0;
		double numberInDisplay = getNumberInDisplay();

		if (lastOperator.equals("/"))
		{
			if (numberInDisplay == 0)
				throw (new DivideByZeroException());

			result = lastNumber / numberInDisplay;
		}
			
		if (lastOperator.equals("*"))
			result = lastNumber * numberInDisplay;

		if (lastOperator.equals("-"))
			result = lastNumber - numberInDisplay;

		if (lastOperator.equals("+"))
			result = lastNumber + numberInDisplay;

		return result;
	}

	void displayResult(double result){
		setDisplayString(Double.toString(result));
		lastNumber = result;
		displayMode = RESULT_MODE;
		clearOnNextDigit = true;
	}

	void displayError(String errorMessage){
		setDisplayString(errorMessage);
		lastNumber = 0;
		displayMode = ERROR_MODE;
		clearOnNextDigit = true;
	}

	public static void main(String args[]) {
		Calculator calci = new Calculator();
//		Container contentPane = calci.getContentPane();
//		contentPane.setLayout(new BorderLayout());
		calci.setTitle("Java Swing Calculator");
		calci.setSize(241, 217);
		calci.pack();
		calci.setLocation(400, 250);
		calci.setVisible(true);
		calci.setResizable(false);
	}
	
}		//End of Swing Calculator Class.

@SuppressWarnings("serial")
class DivideByZeroException extends Exception{
	public DivideByZeroException()
	{
		super();
	}
	
	public DivideByZeroException(String s)
	{
		super(s);
	}
}

@SuppressWarnings("serial")
class CustomABOUTDialog extends JDialog implements ActionListener {
	JButton jbnOk;

	CustomABOUTDialog(JFrame parent, String title, boolean modal){
		super(parent, title, modal);
		setBackground(Color.black);
		
		JPanel p1 = new JPanel(new FlowLayout(FlowLayout.CENTER));

		StringBuffer text = new StringBuffer();
		text.append("Calculator Information\n\n");
		text.append("Developer:	Hemanth\n");
		text.append("Version:	1.0");
		
		JTextArea jtAreaAbout = new JTextArea(5, 21);
		jtAreaAbout.setText(text.toString());
		jtAreaAbout.setFont(new Font("Times New Roman", 1, 13));
		jtAreaAbout.setEditable(false);

		p1.add(jtAreaAbout);
		p1.setBackground(Color.red);
		getContentPane().add(p1, BorderLayout.CENTER);

		JPanel p2 = new JPanel(new FlowLayout(FlowLayout.CENTER));
		jbnOk = new JButton(" OK ");
		jbnOk.addActionListener(this);

		p2.add(jbnOk);
		getContentPane().add(p2, BorderLayout.SOUTH);

		setLocation(408, 270);
		setResizable(false);

		addWindowListener(new WindowAdapter() {
				public void windowClosing(WindowEvent e)
				{
					Window aboutDialog = e.getWindow();
					aboutDialog.dispose();
				}
			}
		);

		pack();
	}

	public void actionPerformed(ActionEvent e)
	{
		if(e.getSource() == jbnOk)	{
			this.dispose();
		}
	}
	
}

 

[Andi_CH]

Mal PUSH.

Weiss wirklich niemand was ich machen muss damit ich im Eventhandler die gedrückte Taste anhand eines int identifizieren kann und auf der trotzdem ein Text zu sehen ist?

Der Hinweis kam per PN - hab gestern wohl was übersehen.

Instanzieren der Buttons

	jbnButtons[i] = new JButton(new MyAction(i));
	jbnButtons[i].setText(Integer.toString(i));

-> Um Platz zu sparen hab ich die leeren Funktionen gelöscht

private static class MyAction implements Action {

	final private int mId;

	public MyAction(int id) { mId = id;	}

	@Override
	public void actionPerformed(ActionEvent e) {
		System.out.println(mId);
		switch (value()) {
		case 1:
			// blabla
			break;
		}
	}
}

 

[Ironzwerg]

So wie ich das sehen konnte, wird in Calculator.actionPerformed kein String verglichen, sondern ein Objekt.
Und zwar ob e.getSource gleich das JMenuitem jmenuitemAbout ist.

Diese Variante halte ich für nicht gut. Da hätte man eher mit instanceOf arbeiten sollen.

Aber nun zu deiner Frage.
Direkt scheint es nicht möglich zu sein.
Da würde mir nur der Name des Button einfallen, den kannst du unabhängig vom Text setzen.
Als Name könntest du dann deine ID als String setzen und beim Auslesen wieder in einen int umwandeln.
Dabei musst du natürlich aufpassen, nicht dass du wirklich mal einen Text als Namen verwendest.

Was anderes würde mir so auch nicht einfallen

 

[SlaterB]

@Andi_CH
oh, ich hatte überhaupt nicht gesehen dass das dein Thema ist, dachte du postest auch als Antworter,

all die langen Code habe ich mir nicht in Detail angeschaut, wie wärs noch mit einer Alternative Map<JButton,Integer> ?

 

[Andi_CH]

Ok, stimmt an eine Map hab ich nicht gedacht -> Effizienz unbekannt

Was mich störte ist, dass in einem for-loop der button gesucht wird und der index dann in einem switch case ausgewertet wird - grausam kompliziert.
Strings als Id's stören mich auch - egal in welchem Zusammenhang - ein kleiner Tippfehler und nichts geht und die Effizient? na ja.

Kann man wirklich keine int - Id vergeben welche im Event wieder mitgeschickt wird und nicht erst über for loop oder eine Map gesucht werden muss?

[Vorsicht Vielosofie ]
Java ist schon eine komische Sprache - kaum etwas ist einfach einfach - warum werden oft so komplizierte Wege beschritten?
[/PHILOSOPHIE]

 

[SlaterB]

in diesem Fall ist nur das schwerfällige Swing Schuld,
ein JButton bzw. jede JComponent bräuchte für dich eine int-Id

dass das allgemein so selbstverständlich ist sehe ich aber nicht,
je nach aktueller Anwendung könntest du auch für einen String, eine Liste, gar ein Array, für ein Image, für einen Comparator, usw. für beliebige Dinge eine Id benötigten, soll das überall separat eingebaut werden oder gar in Object als Grundfunktion?
ziemlich drastische Änderung

 

[Michael...]

Hab den Thread jetzt nur überfolgen...

Wenn Du eine numerische ID für einen JButton oder sonstiges Objekt benötigst, kannst Du Dir zur Not auch eine eigene Button-Klasse schreiben, die von JButton erbt und um die ID als Attribute erweitert - inkl. passender Methoden.

 

[Andi_CH]

So im Sinne von

((MyButton)event.getSource()).getId()

?

Wäre doch schön nur

event.getSourceId()

schreiben zu können ;-)

Mir ist noch nicht ganz klar was eigentlich der Unterschied ist (Unterschied im Sinn von Vor- und Nachteilen)
Also erst die dritte (neue) Variante:

MyButton extends JButton

Die hat einen kleinen Makel: Der Zugriff auf die Id geht über getSource und casten (mit möglichen Exceptions)

Erste (Original) Variante: Die Klasse Calculator implementiert das Interface ActionListener, hat also die Methode actionPerformed(ActionEvent e) worin ich aber erst einmal rausfinden muss, dass die Taste "5" gedrückt wurde. Die Buttons bekommen this als ActionListener.

Zweite Variante: Man kann an jeden Button ein eigenes Action Objekt

class MyAction implements Action

hängen, habe dann z.B. eine numerische Id in der Action und kann im Fall von "5" die Id gleich als Wert an den Rechner übergeben. Die Buttons bekommen eine Action und diese (im Moment noch) this als Referenz auf die Businesslogik. Die Action ist programmiertechnisch wesentlich einfacher. Numerische Id - switch case ... Unterschiedliche Action-Klassen je nach Tastentyp sind möglich ...

Hat denn die Methode mit der Action einen Nachteil, ausser dass pro Taste ein Objekt generiert wid? (Was mich nicht beunruhigt - soooo viele Tasten gibt es ja auch wieder nicht )