Paint und JColorChooser

[muemmel_0811]

Hi zusammen,

entweder ich bin zu blöd oder zu blöd zum Lesen oder sonst wie, aber ich hab leider noch keine Lösung für mein Problem gefunden
Per Button möchte in die Farbauswahl aufrufen, wo dann eine Farbe gewählt werden kann und sich dann der Strich, der jetzt in blau gezeichnet ist, sich in bspw. grün färbt.
Zunächst mal der Code, den ich bisher hab:

public class Akk_q extends JPanel{
    private static final long serialVersionUID = 1L;
    PictureBack pb = new PictureBack();
    Akk_q() {
        double[][] cr = {{800},{175,TableLayout.FILL,rand}};
        setLayout(new TableLayout(cr));
        add(pb, "0,0");
    }
    public class PictureBack extends JPanel{
        private static final long serialVersionUID = 1L;
        PicturePanel pp = new PicturePanel();
        JLabel lbl1 = new JLabel("Rahmenfarbe ändern");
        JButton btnRaFa = new JButton("Rahmenfarbe ändern");
        FarbDialog fb = new FarbDialog();        
        PictureBack() {
            setBackground(new Color(234,234,234));
            setBorder(BorderFactory.createLoweredBevelBorder());
            setLayout(null);
            pp.setBounds(10,10,255,155);
            add(pp);
            btnRaFa.setBounds(290,10,180,26);
            btnRaFa.addActionListener(fb);
            add(btnRaFa);
        }
    }
    public class PicturePanel extends JPanel{
        private static final long serialVersionUID = 1L;
        PicturePanel(){
            setBackground(new Color(255,255,255));
            setLayout(null);
        }
        public void paintComponent(Graphics g) {
            super.paintComponent(g);
            Graphics2D g2 = (Graphics2D) g;
            g2.setColor(Color.BLUE);    g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING,RenderingHints.VALUE_ANTIALIAS_ON);
            BasicStroke stil0 = new BasicStroke(1.0f, BasicStroke.CAP_BUTT, BasicStroke.JOIN_MITER);
            g2.setStroke(stil0);
            Line2D.Float stab0 = new Line2D.Float(30.0f, 40.0f, 30.0f, 130.0f);
            g2.draw(stab0);
        }
    }
    private class FarbDialog implements ActionListener {
        public void actionPerformed(ActionEvent ae) {
            Color col = JColorChooser.showDialog(pb.pp,"Farbe wählen", Color.black);
            if(col != null) {
                pb.pp.repaint();
            }
        }
    }
}

Der Aufruf des Farbdialogs funktioniert schon, aber dann setzt es bei mir aus, denn das ganze passiert ja in der public void actionPerformed, wo ich ja keinen Rückgabewert irgendwie erhalten und weiter verwenden kann. Stellt sich also bei mir die Frage, wie bekomme ich den Wert von col in die paintComponent-Methode bekomme, so dass der Strich dann in der neuen Farbe gezeichnet wird - ich komm nicht drauf, bitte helft mir!

Danke und Grüße,
muemmel_0811

 

[MiDniGG]

Mit einer get-Methode kannst dir theoretisch die Farbe holen... Nur versteh ich nicht wieso Du den ColorChooser in ne extra Klasse auslagerst... -.- Lass das doch einfach da wo Du auch zeichnest und dann brauchst auch nicht lange mit getter und setter rumhantieren...

 

[Schandro]

Bei so nem einfachen Listener innerhalb der eigentlichen Klasse reicht es, das du ne Variable vom Typ Color als Membervariable hast, auf die sowohl der Listener als auch die paint-Methode zugreift...

 

[muemmel_0811]

Hallo Ihr zwei,

Danke für die schnellen Antworten, aber ich poste ja nicht grundlos im Anfängerforum...

Wärd Ihr vielleicht so nett und würdet Ihr mir Eure Anrgegungen so in Code verpacken, damit ich ungefähr weiß, was ich nun tun soll...
Mal als Beispiel:

Nur versteh ich nicht wieso Du den ColorChooser in ne extra Klasse auslagerst... -.- Lass das doch einfach da wo Du auch zeichnest

Zeig mir doch, wie das dann aussehen soll - die Extraklasse hab ich ja wg. dem Button - ich hab das bisher immer so gemacht und ehrlich gesagt auch noch nicht darum gekümmert, ob es da noch andere sinnvollere Methoden gibt, damit ich die actionPerformed aufrufen kann ohne eine Extra-Klasse zu erstellen... ???:L

Danke und Grüße,
muemmel_0811

 

[Schandro]

Ich denke mal, er meint damit das du auch direkt am Ort von addActionListener(..) eine anonyme Implementation des ActionListener-Interfaces hättest machen können anstatt dafür extra ne innere Klasse zu erstellen.
Hat mit deinem Problem aber nichts zu tun.

Weißt du was eine Membervariable ist?

 

[MiDniGG]

Ich denke mal, er meint damit das du auch direkt am Ort von addActionListener(..) eine anonyme Implementation des ActionListener-Interfaces hättest machen können anstatt dafür extra ne innere Klasse zu erstellen.
Hat mit deinem Problem aber nichts zu tun.

Weißt du was eine Membervariable ist?

Ja meinte ich. Was wirklich blödsinn is im Zusammenhang

 

[muemmel_0811]

Weißt du was eine Membervariable ist?

Lass mich kurz googeln und dann meld ich mich wieder...

 

[muemmel_0811]

Oh man :autsch: - jetzt hab ich mir 'ne gefühlte halbe Ewigkeit den Wolf gegoogelt, nur um herauszufinden, dass Membervariablen == Instanzvariablen und ja, dass weiß ich sogar, was es ist

Also, was nun wie weiter?

 

[MiDniGG]

Oh man :autsch: - jetzt hab ich mir 'ne gefühlte halbe Ewigkeit den Wolf gegoogelt, nur um herauszufinden, dass Membervariablen == Instanzvariablen und ja, dass weiß ich sogar, was es ist

Also, was nun wie weiter?

Naja. Eben ne Variable definieren, welche Du dann in der Action-Methode befüllen kannst

 

[muemmel_0811]

Naja. Eben ne Variable definieren, welche Du dann in der Action-Methode befüllen kannst

Super - Danke Dir - jetzt hab's sogar ich kapiert :toll:
Und für die Nachwelt:

public class Akk_q extends JPanel{
    private static final long serialVersionUID = 1L;
    Color farbe = Color.black;
    PictureBack pb = new PictureBack();
    
...

      public void paintComponent(Graphics g) {
            super.paintComponent(g);
            Graphics2D g2 = (Graphics2D) g;
            g2.setColor(farbe);

...

private class FarbDialog implements ActionListener {
        public void actionPerformed(ActionEvent ae) {
            Color col = JColorChooser.showDialog(pb.pp,"Farbe wählen", Color.black);
            if(col != null) {
                farbe = col;
                pb.pp.repaint();
            }
        }
    }

Vielen Dank nochmal für die geduldige Hilfe!
Grüße vom muemmel_0811

 

[MiDniGG]

Super - Danke Dir - jetzt hab's sogar ich kapiert :toll:
Und für die Nachwelt:

Ich hätte Lehrer werden sollen ^^

 

[muemmel_0811]

Ich hätte Lehrer werden sollen ^^

Dein Bild sieht nicht so aus, als ob Du schon kurz vor'm Renteneintritt stehst und von daher bleibt die Möglichkeit ja noch

Danke nochmal!

 

[MiDniGG]

Dein Bild sieht nicht so aus, als ob Du schon kurz vor'm Renteneintritt stehst und von daher bleibt die Möglichkeit ja noch

Danke nochmal!

LOL. Na wer weiß. Vllt is dieses Bild ja auch aus den guten alten 80ern oder das bin gar nicht ich... :-D

Aba nein. Mich mit irgendwelchen Kindern rumärgern is nich mein Ding und vor n paar Leuten reden... Dafür bin ich auch nicht geschaffen -.- Ich bleib bei meiner FI-AE-Ausbildung