ActionListener problem

[Chiqsaw]

Hallo,
Ich hoffe ihr könnt mir Helfen.Wir haben gestern in der Schule den ActionListener kennengelert.ich habe mich als gleich drangesetzt ein programm zu schreiben.Ich weis es ist billig da ich es noch nicht wirklich kann aber ich lerne ja
Mein problem ist das ich bei disem Programm zwar den wert eingeben kann udn den button drücken kann doch es weder reagiert noch eine fehlermeldung ausgibt.(ich benutze Eclipse)
kann mir vllt jemand von euch verraten was ich falsch gemahct habe?

package gui;

import java.awt.*;
import java.awt.event.*;

import javax.swing.*;


public class Umrechner extends JFrame implements ActionListener
{
	private static final long serialVersionUID = 1L;
	private double euro, sf, dm, dol;
	private String s;
	
	private JTextField eingabe;
	private JLabel ausgabe,ausgabe2,ausgabe3;
	private JLabel eu;
	
	public Umrechner()
	{
		JButton a,b,c;
		setTitle("Umrechner");
		getContentPane().setBackground(Color.LIGHT_GRAY);
		this.setLayout(null);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		eingabe=new JTextField(" ",10);
		eingabe.setBounds(35,60,60,20);
		add(eingabe);
		eu = new JLabel ("Euro");
		eu.setBounds(100,45,50,50);
		add (eu);
		
		a=new JButton ("Schweizer Franken");
		a.setBounds(150,20,150,30);
		add (a);
		a.addActionListener(this);
		ausgabe =new JLabel("                           ");
		ausgabe.setBounds(310,10,100,50);
		add (ausgabe);
		
		b=new JButton ("Deutsche Mark");
		b.setBounds(150,55,150,30);
		add (b);
		b.addActionListener(this);
		ausgabe2 =new JLabel("                           ");
		ausgabe2.setBounds(310,55,100,50);
		add (ausgabe2);
		
		c=new JButton ("US-Dollar");
		c.setBounds(150,90,150,30);
		add (c);
		c.addActionListener(this);
		ausgabe3 =new JLabel("                           ");
		ausgabe3.setBounds(310,90,100,50);
		add (ausgabe3);
		
		
	}
	



	public void actionPerformed(ActionEvent e)
	{
		
		if (e.getSource().equals("Schweizer Franken"))
		{
			s=eingabe.getText();
			euro = Double.valueOf(s).doubleValue();
			sf=euro*1.52;
			sf=(double)Math.round(sf*100)/100;
			ausgabe.setText("SF: " + sf);
		}
		
		else if (e.getSource().equals("Deutsche Mark"))
        {
                s=eingabe.getText();
            	euro = Double.valueOf(s).doubleValue();
            	dm=euro*0.51;
            	dm=(double)Math.round(dm*100)/100;
            	ausgabe2.setText("DM: " + dm);
		}

		else if (e.getSource().equals("US-Dollar"))
		{
			s=eingabe.getText();
			euro = Double.valueOf(s).doubleValue();
			dol=euro*1.45;
			dol=(double)Math.round(dol*100)/100;
			ausgabe3.setText("Dollar: " + dol);
		}
	}
	
	public static void main(String[] args) 
	{
		Umrechner hauptfenster = new Umrechner();
		
		hauptfenster.setSize(600,200);
		hauptfenster.setLocation(500,200);
		hauptfenster.setVisible(true);
		
	}
}

 

[eRaaaa]

Hi, an sich funktioniert das schon, nur deine if-Abfragen im Listener sind falsch!

  if (e.getSource().equals("Schweizer Franken"))

Du kannst dir ja mal mit System.out.println ausgeben, was dir e.getSource() liefert

Schreibe beispielsweise lieber:

public void actionPerformed(ActionEvent e) {
	JButton source = (JButton) e.getSource();

	if (source.getText().equals("Schweizer Franken")) {
//.......

Oder aber man setzt bei den Buttons einen actionCommand:

a.setActionCommand("schweiz");

dann kannst du im Listener auch sagen

if(e.getActionCommand().equals("schweiz"){

Gruß

 

[Chiqsaw]

Danke für die schnellen Antworten

habe das jetzt mal so porbiert.

package gui;

import java.awt.*;
import java.awt.event.*;

import javax.swing.*;


public class Umrechner extends JFrame implements ActionListener
{
	private static final long serialVersionUID = 1L;
	private double euro, sf, dm, dol;
	private String s;
	
	private JTextField eingabe;
	private JLabel ausgabe,ausgabe2,ausgabe3;
	private JLabel eu;
	
	public Umrechner()
	{
		JButton a,b,c;
		setTitle("Umrechner");
		getContentPane().setBackground(Color.LIGHT_GRAY);
		this.setLayout(null);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		eingabe=new JTextField(" ",10);
		eingabe.setBounds(35,60,60,20);
		add(eingabe);
		eu = new JLabel ("Euro");
		eu.setBounds(100,45,50,50);
		add (eu);
		
		a=new JButton ("Schweizer Franken");
		a.setBounds(150,20,150,30);
		add (a);
		a.setActionCommand("schweiz");
		ausgabe =new JLabel("                           ");
		ausgabe.setBounds(310,10,100,50);
		add (ausgabe);
		
		b=new JButton ("Deutsche Mark");
		b.setBounds(150,55,150,30);
		add (b);
		a.setActionCommand("deutsch");
		ausgabe2 =new JLabel("                           ");
		ausgabe2.setBounds(310,55,100,50);
		add (ausgabe2);
		
		c=new JButton ("US-Dollar");
		c.setBounds(150,90,150,30);
		add (c);
		a.setActionCommand("US");
		ausgabe3 =new JLabel("                           ");
		ausgabe3.setBounds(310,90,100,50);
		add (ausgabe3);
		
		
	}
	



	public void actionPerformed(ActionEvent e)
	{
		
		if(e.getActionCommand().equals("schweiz"))
		{
			s=eingabe.getText();
			euro = Double.valueOf(s).doubleValue();
			sf=euro*1.52;
			sf=(double)Math.round(sf*100)/100;
			ausgabe.setText("SF: " + sf);
		}
		
		if(e.getActionCommand().equals("deutsch"))
        {
                s=eingabe.getText();
            	euro = Double.valueOf(s).doubleValue();
            	dm=euro*0.51;
            	dm=(double)Math.round(dm*100)/100;
            	ausgabe2.setText("DM: " + dm);
		}

		if(e.getActionCommand().equals("US"))
		{
			s=eingabe.getText();
			euro = Double.valueOf(s).doubleValue();
			dol=euro*1.45;
			dol=(double)Math.round(dol*100)/100;
			ausgabe3.setText("Dollar: " + dol);
		}
	}
	
	public static void main(String[] args) 
	{
		Umrechner hauptfenster = new Umrechner();
		
		hauptfenster.setSize(600,200);
		hauptfenster.setLocation(500,200);
		hauptfenster.setVisible(true);
		
	}
}

doch es hat nicht geklappt.
es funtioniert genau das selbe nicht.

 

[eRaaaa]

Ne ne so nicht

Das [c]a.addActionListener(this);[/c] musst du natürlich belassen Das darfst du nicht wegnehmen

Desweiterne hast du beim Copy&Paste vergessen die Variable zu ändern
[c]a.setActionCommand("US");[/c] <-- a??

 

[Chiqsaw]

danke is mir danna uch aufgefallen mit dem copy and paste fehler...ich habe ind er zwischenzeit mal die andere variante

public void actionPerformed(ActionEvent e) {
    JButton source = (JButton) e.getSource();
 
    if (source.getText().equals("Schweizer Franken")) {
//.......

probiert udn diese klappt perfekt! danke nochmal(ich werde die andere methode(die du jetzt gerade angegeben hast aber auchnochmal probieren)