Fehler im ActionListener?

[IvaNiK]

Hallo!
Ich habe ein Programm geschrieben, das in einem JLabel mit hintergrundfarbe ein Bild anzeigt. Es gibt 3 Buttons in einer Toolbar die die Hintergrundfarbe des JLabels ändern können. Es gibt auch ein Menü um zwischen 3 verschiedenen Bildern wählen zu können die im JLabel angezeigt werden sollen.
Leider kommt bei mir wenn ich auf einen Button Klicke in der Toolbar oder auf ein MenuItem in dem Menü immer ein Fehler....irgendeine Exception (Bei den Buttons in der ToolBar diese: Exception in thread "AWT-EventQueue-0" java.lang.ClassCastException: javax.swing.JButton und bei den MenuItems diese hier: Exception in thread "AWT-EventQueue-0" java.lang.ClassCastException: javax.swing.JMenuItem).

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
public class Bildbetrachter extends JFrame{
    
    JLabel hintergrund;
    private JButton red, blue, yellow;
    /** Creates a new instance of Bildbetrachter */
    public Bildbetrachter(){
        super("PixShow");
        setSize(400,400);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        getContentPane().setLayout(new BorderLayout());
        
        Listener bild = new Listener();
        //Menue
        JMenuBar mbar = new JMenuBar();
        JMenu bildM = new JMenu( "Bild" );
        mbar.add( bildM );
        setJMenuBar( mbar );
        //Menue-Items
        JMenuItem wolf = new JMenuItem("Wolf");
        wolf.addActionListener(bild);
        JMenuItem katze = new JMenuItem("Katze");
        katze.addActionListener(bild);
        JMenuItem hund = new JMenuItem("Hund");
        hund.addActionListener(bild);
        bildM.add(wolf);
        bildM.add(katze);
        bildM.add(hund);
        
        
        //Toolbar
        JToolBar buttons = new JToolBar();
        red = new JButton(new ImageIcon("rot.jpg"));
        red.addActionListener(bild);
        blue = new JButton(new ImageIcon("blau.jpg"));
        blue.addActionListener(bild);
        yellow = new JButton(new ImageIcon("gelb.jpg"));
        yellow.addActionListener(bild);
        buttons.add(red);
        buttons.add(blue);
        buttons.add(yellow);
        getContentPane().add(buttons, BorderLayout.NORTH);

        hintergrund = new JLabel(new ImageIcon("test.jpg"));
        hintergrund.setBackground(Color.red);
        hintergrund.setOpaque(true);
        getContentPane().add(hintergrund);
        
        setVisible(true);
    }
    public static void main(String [] args){
        Bildbetrachter pixViewer = new Bildbetrachter();
    }
    
}

class Listener extends JFrame implements ActionListener{
    public void actionPerformed(ActionEvent evt){
        JLabel source = (JLabel)evt.getSource();
        Object cmd = evt.getActionCommand();
        
        if(cmd == "Wolf"){
            source.setIcon(new ImageIcon("test.jpg"));
        }
        if(cmd == "Katze"){
            source.setIcon(new ImageIcon("katze.jpg"));
        }
        if(cmd == "Hund"){
            source.setIcon(new ImageIcon("hund.jpg"));
        }
        if(cmd == new ImageIcon("rot.jpg")){
            source.setBackground(Color.red);
        }
        if(cmd == new ImageIcon("gelb.jpg")){
            source.setBackground(Color.yellow);
        }
        if(cmd == new ImageIcon("blau.jpg")){
            source.setBackground(Color.blue);
        }
    }
}

Thx im Vorhinein
mfg IvaNiK

 

[Bleiglanz]

du fügst zu JMenuItem den Listener hinzu

und erwartest bei evt.getSource() auf einmal ein JLabel??

 

[Guest]

du fügst zu JMenuItem den Listener hinzu

und erwartest bei evt.getSource() auf einmal ein JLabel??

Ja ok wenn ich jetzt zu source ein MenuItem erwarte....wie kann ich dann den hintergrund des labels ändern? bzw. das Bild im Label? Der ActionListener ist ja in einer eigenen Klasse....wie mach ich denn das? Muss ich im ActionListener ein eigenes neues Label erzeugen?

pls HELP
mfg IvaNiK

 

[Guest]

Habs jetzt geändert....auf JMenuItem und eines auf JLabel....wieder der selbe Fehler.

 

[mic_checker]

dann poste deinen neuen code und was anderes:
warum leitest du den actionlistener von jframe ab? kann es sein das du dich noch nicht wirklich mit der materie befasst hast?

noch was: getActionCommand() liefert String, hol doch auch einfach das u. vergleich dann entsprechend (mit equals!).

 

[Guest]

dann poste deinen neuen code und was anderes:
warum leitest du den actionlistener von jframe ab? kann es sein das du dich noch nicht wirklich mit der materie befasst hast?

noch was: getActionCommand() liefert String, hol doch auch einfach das u. vergleich dann entsprechend (mit equals!).

Weil ich (nach meiner Programmierlehrerin) die GUI vom restlichen Programmcode trennen soll. Und sie meint ich soll keine anonymen Klassen machen und den ActionListener in eigenen Klassen machen...

 

[IvaNiK]

Hier der aktuelle Code, aber es kommt wieder der Fehler..

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
public class Bildbetrachter extends JFrame{
    
    JLabel hintergrund;
    private JButton red, blue, yellow;
    /** Creates a new instance of Bildbetrachter */
    public Bildbetrachter(){
        super("PixShow");
        setSize(400,400);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        getContentPane().setLayout(new BorderLayout());
        
        Listener bild = new Listener();
        //Menue
        JMenuBar mbar = new JMenuBar();
        JMenu bildM = new JMenu( "Bild" );
        mbar.add( bildM );
        setJMenuBar( mbar );
        //Menue-Items
        JMenuItem wolf = new JMenuItem("Wolf");
        wolf.addActionListener(bild);
        JMenuItem katze = new JMenuItem("Katze");
        katze.addActionListener(bild);
        JMenuItem hund = new JMenuItem("Hund");
        hund.addActionListener(bild);
        bildM.add(wolf);
        bildM.add(katze);
        bildM.add(hund);
        
        
        //Toolbar
        JToolBar buttons = new JToolBar();
        red = new JButton(new ImageIcon("rot.jpg"));
        red.addActionListener(bild);
        blue = new JButton(new ImageIcon("blau.jpg"));
        blue.addActionListener(bild);
        yellow = new JButton(new ImageIcon("gelb.jpg"));
        yellow.addActionListener(bild);
        buttons.add(red);
        buttons.add(blue);
        buttons.add(yellow);
        getContentPane().add(buttons, BorderLayout.NORTH);

        hintergrund = new JLabel(new ImageIcon("test.jpg"));
        hintergrund.setBackground(Color.red);
        hintergrund.setOpaque(true);
        getContentPane().add(hintergrund);
        
        setVisible(true);
    }
    public static void main(String [] args){
        Bildbetrachter pixViewer = new Bildbetrachter();
    }
    
}

class Listener extends JFrame implements ActionListener{
    public void actionPerformed(ActionEvent evt){
        JMenuItem source = (JMenuItem)evt.getSource();
        JButton source2 = (JButton)evt.getSource();
        
        JLabel label = new JLabel();
        if(source.equals(new JMenuItem("Wolf"))){
            label.setIcon(new ImageIcon("test.jpg"));
        }
        if(source.equals(new JMenuItem("Katze"))){
            label.setIcon(new ImageIcon("katze.jpg"));
        }
        if(source.equals(new JMenuItem("Hund"))){
            label.setIcon(new ImageIcon("hund.jpg"));
        }
        if(source2.equals(new ImageIcon("rot.jpg"))){
            label.setBackground(Color.red);
        }
        if(source2.equals(new ImageIcon("gelb.jpg"))){
            label.setBackground(Color.yellow);
        }
        if(source2.equals(new ImageIcon("blau.jpg"))){
            label.setBackground(Color.blue);
        }
    }
}

 

[mic_checker]

Weil ich (nach meiner Programmierlehrerin) die GUI vom restlichen Programmcode trennen soll. Und sie meint ich soll keine anonymen Klassen machen und den ActionListener in eigenen Klassen machen...

sorry, aber was hat das damit zu tun? Tut mir leid wenn ich das so sagen muss, aber ich bezweifle das du MVC verstehen wirst wenn ich deinen restlichen Code seh.

Du kannst ja nicht einmal auf JButton parsen mit getSource() und in der andern Zeile auf JMenuItem...klar das da ne Exception geschmissen wird.
Außerdem wirst du mit deinem Konzept nicht weiterkommen, du legst dort ein neues Label an, dessen Hintergrundfarbe du änderst, das hat aber keine auswirkungen auf dein anderes label das du verändern willst.

 

[Guest]

Außerdem wirst du mit deinem Konzept nicht weiterkommen, du legst dort ein neues Label an, dessen Hintergrundfarbe du änderst, das hat aber keine auswirkungen auf dein anderes label das du verändern willst.

Ok..und wie kann ich jetzt auswirkungen auf mein anderes Label nehmen? Frage ich mich sowieso schon die ganze Zeit.

 

[mic_checker]

kommt drauf an.

musst du eine extra klasse verwenden ? also du darfst nicht in deiner Haupt - GUI ActionListener implementieren?

Dann übergeb deiner Handler KLasse ne Referenz auf das Label oder du schreibst in deiner Haupt Klasse ne Methode die den Hintergrund verändert und du übergibst ne Referenz auf die Hauptklasse....

 

[edgrar_u]

hab das mit dem GUI nur kurz durchprobiert -> alles ohne gewähr ( ausprobieren kannstes natürlich trotzdem)

1. deine action listener klasse musst du nich von JFRame ableiten , das is überflüssig

2. der compiler wird nicht unterscheiden können ob das jetzt en Button is oder JMenu Item bei evt.getSource();mach doch einfacher:
frag die labels der Teile ab mit evt.getActionCommand() wie zuvor beschrieben, dann vergleichst du das ganze mit equals
String actionCommand = evt.getActionCommand();
if (actionCommand.equals "....was auch immer"){
was er dann machen soll }


3. du erschaffst bei deinen If (obj.equals (... anonyme Klassen - warum weis ich nich , auf jeden fall is das nich notwendig und birgt wahrscheinlich fehler
-> Mal an die anderen was passiert da eigentlich wenn er macht:
if(source.equals(new JMenuItem("Wolf"))){ ...wird dann ein neuer Button erzeugt der gleich wieder verwindet ??

 

[Guest]

OK, das Problem mit den Fehlermeldungen hätt ich jetzt gelöst! Nur noch eins habe ich jetzt:
Wie kann ich auf das JLabel aus der GUI Klasse in der ActionListener Klasse zugreifen?

 

[lin]

machst das JLabel static

oder schreibst ne Methode in der Gui-Klassse, mit der du drauf zugreifen kannst...

 

[Guest]

machst das JLabel static

oder schreibst ne Methode in der Gui-Klassse, mit der du drauf zugreifen kannst...

Jo Habs static gemacht....wie kann ich jetzt eigentlich im ActionListener die Buttons mit den Bildern vergleichen mit dem getActionCommand?
das funktioniert ja nicht mit equals oder?

 

[Guest]

Lass dir das hier mal durch den Kopf gehen.

public class Bildbetrachter extends JFrame {

  public Bildbetrachter(String title) {
    super(title);
    getContentPane().setLayout(new BorderLayout());

    // Hintergrund
    JLabel hintergrund = new JLabel(new ImageIcon("test.jpg"));
    hintergrund.setBackground(Color.red);
    hintergrund.setOpaque(true);
    getContentPane().add(hintergrund, BorderLayout.CENTER);

    // Menue
    JMenu bildM = new JMenu("Bild");
    bildM.add(new JMenuItem(new ImageAction("Wolf", hintergrund, "wolf.jpg")));
    bildM.add(new JMenuItem(new ImageAction("Katze", hintergrund, "katze.jpg")));
    bildM.add(new JMenuItem(new ImageAction("Hund", hintergrund, "hund.jpg")));
    JMenuBar mbar = new JMenuBar();
    mbar.add(bildM);
    setJMenuBar(mbar);

    // Toolbar
    JToolBar buttons = new JToolBar();
    buttons.add(new JButton(new ColorAction(new ImageIcon("rot.jpg"), hintergrund, Color.RED)));
    buttons.add(new JButton(new ColorAction(new ImageIcon("blau.jpg"), hintergrund, Color.BLUE)));
    buttons.add(new JButton(new ColorAction(new ImageIcon("gelb.jpg"), hintergrund, Color.YELLOW)));
    getContentPane().add(buttons, BorderLayout.NORTH);
  }

  public static void main(String [] args) {
    Bildbetrachter pixViewer = new Bildbetrachter("PixShow");
    pixViewer.addWindowListener(
      new WindowAdapter() {
        public void windowClosing(WindowEvent event) {
          System.exit(0);
        }
      }
    );
    pixViewer.setSize(400,400);
    pixViewer.setLocationRelativeTo(null);
    pixViewer.setVisible(true);
  }

}

class ImageAction extends AbstractAction {
  private JLabel label    = null;
  private ImageIcon image = null;
  private String imageSrc = null;

  public ImageAction(String name, JLabel label, String imageSrc) {
    super(name);
    this.label = label;
    this.imageSrc = imageSrc;
  }

  public void actionPerformed(ActionEvent event) {
    label.setIcon(getImage());
  }

  private ImageIcon getImage() {
    if(image==null)
      image = new ImageIcon(imageSrc);
    return image;
  }
}

class ColorAction extends AbstractAction {
  private JLabel label = null;
  private Color color  = null;

  public ColorAction(Icon icon, JLabel label, Color color) {
    super(null, icon);
    this.label = label;
    this.color = color;
  }

  public void actionPerformed(ActionEvent event) {
    label.setBackground(color);
  }
}