Swing ActionListener

[AndiE]

Hallo,

ich habe eine "Anfängerfrage". Beim Ausproblieren verschiedener Programme ist mir aufgefallen, dass der "Actionlistener" anders implementiert wird, als in der Hilfe bei Oracle zu SWING.

Oracle:

class MyClass implements Actionlistener{
...
{
...
Jbutton button;
button.addActionListerner(this);
...
}

public void actionPerformed(ActionEvent e) { 
    ...//code that reacts to the action... 
}

}

in Beispielen:

class Test {
...
 actionListener actionHandler;
 JBotton but;
 but.addActionListener(actionHandler);
 actionHandler = new ActionListener() {
 
            @Override
            public void actionPerformed(final ActionEvent e) {
            ...
            }

Meine Frage ist jetzt, aus welchem Grund die zweite Version bei mir nicht funktioniert( JDK 1.7), und wie beide Versionen zueinander stehen.

 

[Marcinek]

Hallo,

was bedeutet "funktioniert nicht"?

Die anonyme inner Class (zweites Beispiel) kann ggf. dazu verführen lokale (In de Methode) deklarierten Variablen zuzugreifen. Das funktioniert in Java aber nur bedingt mit final Variablen.

Ansonsten hat man da nicht so viele unterschiede.

Gruß,

Martin

 

[AndiE]

Stimmt, "funktioniert nicht" ist keine Aussage.

Das ist ein Ausschnitt des einfachen Zeichenprogrammes unter Malen in Swing Teil 2: ein einfaches Malprogramm ? Byte-Welt Wiki.

actionHandler = new ActionListener() {
 
            @Override
            public void actionPerformed(final ActionEvent e) {
         ->       String s = e.getActionCommand();
                if (s.equals("Paint")) {
                    JButton button = (JButton) e.getSource();
                    drawingPanel.setPaintColor(button.getBackground());
                } else if (s.equals("Clear")) {
                    drawingPanel.clearPaint();
                } else if (s.equals("+")) {
                    drawingPanel.increaseBrushSize();
                } else {
                    drawingPanel.decreaseBrushSize();
                }
            }
        };

Ich habe an der mir "->" einen Breakpoint gesetzt. Dieser sollte angesprungen werden, wenn ich einen Button anklicke. Leider passiert das so nicht.

 

[Camino]

Oracle:

Jbutton button;

in Beispielen:

JBotton but;

Der heisst JButton, der arme Knopf...

Ich habe an der mir "->" einen Breakpoint gesetzt. Dieser sollte angesprungen werden, wenn ich einen Button anklicke. Leider passiert das so nicht.

Hast du den ActionListener denn auch deinem Button hinzugefügt? Zeig doch mal noch den Code mit dem Button...

Du musst anstatt einem Beispielcodeausschnitt aus dem Internet schon deinen (relevanten) Code hier zeigen, damit man sehen kann, wo der Fehler liegt.

 

[AndiE]

package malen;

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
 
public class DrawOnImage {
 
    final private DrawingPanel drawingPanel;
    final private JPanel buttonPanel;
    final private JButton clearButton, upSize, downSize;
    private final ActionListener actionHandler;
 
    public DrawOnImage() {
        JFrame frame = new JFrame("DrawOnImage");
        drawingPanel = new DrawingPanel();
        buttonPanel = new JPanel();
        buttonPanel.setLayout(new GridLayout(2, 0, 2, 2));
        addButton(Color.BLACK);
        addButton(Color.BLUE);
        addButton(Color.GREEN);
        upSize = addButton(null);
        upSize.setText("+");
        addButton(Color.RED);
        addButton(Color.ORANGE);
        clearButton = addButton(null);
        clearButton.setText("Clear");
        downSize = addButton(null);
        downSize.setText("-");
        frame.getContentPane().add(new JScrollPane(drawingPanel));
        frame.getContentPane().add(buttonPanel, BorderLayout.SOUTH);
        actionHandler = new ActionListener() {
 
            @Override
            public void actionPerformed(final ActionEvent e) {
                String s = e.getActionCommand();
                if (s.equals("Paint")) {
                    JButton button = (JButton) e.getSource();
                    drawingPanel.setPaintColor(button.getBackground());
                } else if (s.equals("Clear")) {
                    drawingPanel.clearPaint();
                } else if (s.equals("+")) {
                    drawingPanel.increaseBrushSize();
                } else {
                    drawingPanel.decreaseBrushSize();
                }
            }
        };
        frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        frame.pack();
        frame.setLocationRelativeTo(null);
        frame.setVisible(true);
    }
 
    private JButton addButton(final Color color) {
        JButton button = new JButton();
        button.setBackground(new Color(230, 240, 250));
        button.setBorder(BorderFactory.createEtchedBorder());
        if (color != null) {
            button.setForeground(Color.WHITE);
            button.setBackground(color);
        }
        button.setText("Paint");
        buttonPanel.add(button);
        button.addActionListener(actionHandler);
        return (button);
    }
 
    public static void main(final String[] args) {
        Runnable gui = new Runnable() {
 
            @Override
            public void run() {
                DrawOnImage drawOnImage = new DrawOnImage();
            }
        };
        //GUI must start on EventDispatchThread:
        SwingUtilities.invokeLater(gui);
    }
}

Das ist ein Standart-Beispiel. Das müsste doch bei jedem funktionieren. Ich habe einfach den Quellcode kopiert, und dann gestartet. Aber aus irgendeinem Grund reagiert der ActionListener bei mir so nicht.

 

[Camino]

Das ist ein Standart-Beispiel. Das müsste doch bei jedem funktionieren. Ich habe einfach den Quellcode kopiert, und dann gestartet. Aber aus irgendeinem Grund reagiert der ActionListener bei mir so nicht.

Ja, du hast recht. Ich hab das auch gerade mal kopiert und gestartet. Bei mir funktionieren die Buttons und der Listener auch nicht. Hmm, muss man mal schauen, warum...

 

[Marcinek]

zuerst werden die buttons eingefügt und dann der Actionlistener erzeugt.

Vorher wird immer null übergeben.

 

[AndiE]

package malen;

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
 
public class DrawOnImage {
 
    final private DrawingPanel drawingPanel;
    final private JPanel buttonPanel;
    final private JButton clearButton, upSize, downSize;
    private final ActionListener actionHandler;
 
    public DrawOnImage() {
        JFrame frame = new JFrame("DrawOnImage");
        drawingPanel = new DrawingPanel();
        buttonPanel = new JPanel();
        buttonPanel.setLayout(new GridLayout(2, 0, 2, 2));
        actionHandler = new ActionListener() {
        	 
            @Override
            public void actionPerformed(final ActionEvent e) {
                String s = e.getActionCommand();
                if (s.equals("Paint")) {
                    JButton button = (JButton) e.getSource();
                    drawingPanel.setPaintColor(button.getBackground());
                } else if (s.equals("Clear")) {
                    drawingPanel.clearPaint();
                } else if (s.equals("+")) {
                    drawingPanel.increaseBrushSize();
                } else {
                    drawingPanel.decreaseBrushSize();
                }
            }
        };
        addButton(Color.BLACK);
        addButton(Color.BLUE);
        addButton(Color.GREEN);
        upSize = addButton(null);
        upSize.setText("+");
        addButton(Color.RED);
        addButton(Color.ORANGE);
        clearButton = addButton(null);
        clearButton.setText("Clear");
        downSize = addButton(null);
        downSize.setText("-");
        frame.getContentPane().add(new JScrollPane(drawingPanel));
        frame.getContentPane().add(buttonPanel, BorderLayout.SOUTH);
       
        frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        frame.pack();
        frame.setLocationRelativeTo(null);
        frame.setVisible(true);
    }
 
    private JButton addButton(final Color color) {
        JButton button = new JButton();
        button.setBackground(new Color(230, 240, 250));
        button.setBorder(BorderFactory.createEtchedBorder());
        if (color != null) {
            button.setForeground(Color.WHITE);
            button.setBackground(color);
        }
        button.setText("Paint");
        buttonPanel.add(button);
        button.addActionListener(actionHandler);
        return (button);
    }
 
    public static void main(final String[] args) {
        Runnable gui = new Runnable() {
 
            @Override
            public void run() {
                DrawOnImage drawOnImage = new DrawOnImage();
            }
        };
        //GUI must start on EventDispatchThread:
        SwingUtilities.invokeLater(gui);
    }
}

Ich habe den Code geändert, und nun funktioniert es.

Könnte jemand die Macher der Seite kontaktieren, dass der Fehler behoben wird. Die Seite gilt ja als sowas wie ein Standartwerk.

 

[Phash]

na du kannst das doch jetzt selber machen