Swing JRadioButton zeichnen nicht

[Alex2013]

Hey Leute,

wenn ich die folgenden Zeilen kompiliere, dann werden mir keine Fehler mehr angezeigt. Aber die verschiedenen paints, die ich ausführen möchte, werden auch nicht ausgeführt. Ich habe es bereits mit einem ItemListener, der implementierten ActionListener Klasse und der inner Type klasse versucht, bei der ich auch eigtl. bleiben möchte. Jedoch lässt sich durch Aktivieren der Radio Buttons nichts zeichnen. Es wird zwar eine Anfangszeichnung vorgenommen, aber wenn die verschiedenen Buttons ausgewählt werden, dann passiert nichts weiter.

Kann mir zufällig jemand sagen, wo der Fehler liegt? Ich habe schon einige Varianten versucht und komme nicht weiter... :-(

Hier der Code:

import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.Graphics;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.ButtonGroup;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JRadioButton;


public class Funktionsgraph extends JFrame{
   
    private static final long serialVersionUID = 1L;
    JRadioButton funktion1=new JRadioButton("Funktion 1"),funktion2=new JRadioButton("Funktion 2"),funktion3=new JRadioButton("Funktion 3");
    JPanel anhangKnoepfe=new JPanel(),bg=new JPanel();
    Container c;
    Zeichnen zz=new Zeichnen();
    Graphics g;
    ButtonGroup gb;
   
    public Funktionsgraph() {
        c=getContentPane();
        c.setLayout(new BorderLayout());
        setLocationRelativeTo(null);
        setSize(600, 400);
        c.add(zz,BorderLayout.CENTER);
        c.add(anhangKnoepfe,BorderLayout.SOUTH);
        anhangKnoepfe.setLayout(new GridLayout(1, 3));
        bg.add(funktion1);
        bg.add(funktion2);
        bg.add(funktion3);
        anhangKnoepfe.add(funktion1);
        funktion1.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
//                zz.setAuswahl(0);
                zz.setAuswahl(isEnabled());
                funktion2.setSelected(false);
                funktion3.setSelected(false);
                repaint();
            }
        });
        anhangKnoepfe.add(funktion2);
        funktion2.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                zz.setAuswahl(isEnabled());
                funktion1.setSelected(false);
                funktion3.setSelected(false);
                repaint();
            }
        });
        anhangKnoepfe.add(funktion3);
        funktion3.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                zz.setAuswahl(isEnabled());
                funktion1.setSelected(false);
                funktion2.setSelected(false);
                repaint();
            }
        });
        setVisible(true);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }
    public static void main(String[] args) {
        new Funktionsgraph();
    }

}

[/Java]

und hier:

[code=Java]
import java.awt.Graphics;

import javax.swing.JPanel;


public class Zeichnen extends JPanel
{
    private static final long serialVersionUID = 1L;

    boolean auswahl;
    int auswahl1;
    public Zeichnen() {
        auswahl=auswahl;
    }
    public void setAuswahl(boolean b)
    {
        this.auswahl=b;
    }
   
    @Override
    public void paint(Graphics g) {
        super.paint(g);
        g.drawLine(getWidth()-getWidth(), getHeight()/2, getWidth(), getHeight()/2);
        g.drawLine(getWidth()/2, getHeight()-getHeight(), getWidth()/2, getHeight());

        int auswahl = 2;
        switch(auswahl)
        {
        case 0:
            int x=5,y=2*x,x2=-5,y2=-2*x;
            g.drawLine(x, y, x2, y2);
            break;
        case 1:
            int xNeu=5*5,yNeu=2*xNeu,xNeu2=-5*-5,yNeu2=-2*yNeu;
            g.drawLine(xNeu, yNeu, xNeu2, yNeu2);
            break;
        case 2:
            int xNeuNeu=(5*5)*2,yNeuNeu=2*xNeuNeu,xNeuNeu2=-5,yNeuNeu2=-2*xNeuNeu;
            g.drawLine(xNeuNeu, yNeuNeu, xNeuNeu2, yNeuNeu2);
            break;
        }
    }
}

[/Java]

Hat hier jemand eine Idee? :-)

Lg

 

[Blender3D]

int auswahl = 2;
switch(auswahl)
{
case 0:
int x=5,y=2*x,x2=-5,y2=-2*x;
g.drawLine(x, y, x2, y2);
break;
case 1:
int xNeu=5*5,yNeu=2*xNeu,xNeu2=-5*-5,yNeu2=-2*yNeu;
g.drawLine(xNeu, yNeu, xNeu2, yNeu2);
break;
case 2:
int xNeuNeu=(5*5)*2,yNeuNeu=2*xNeuNeu,xNeuNeu2=-5,yNeuNeu2=-2*xNeuNeu;
g.drawLine(xNeuNeu, yNeuNeu, xNeuNeu2, yNeuNeu2);
break;
}

Du setz Auswahl konstant auf 2 --> es wird immer nur der Fall 2 sein.

 

[Blender3D]

Tipp: Du benutzt JFrame ( Swing )
besser als paint() zu überladen wäre paintComponent():

 

[Alex2013]

Ich habe das jetzt mal an ein paar Stellen angepasst, aber an dem Problem hat sich noch nichts verbessert... :-(
Die Variante mit int auswahl=wert habe ich aus einem anderen fremden Programm. Dort hat es in ähnlicher Weise gearbeitet... Zumindest bei dem switch()case

Hier mal der abgeänderte Code:

funktion1.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
//                zz.setAuswahl(0);
//                zz.setAuswahl(isEnabled());
                funktion1.setSelected(true);
                funktion2.setSelected(false);
                funktion3.setSelected(false);
                repaint();
            }
        });

und

int auswahl;
//    int auswahl1;
//    boolean b;
    public Zeichnen() {
        auswahl=2;
    }
    public void setAuswahl(int auswahl)
    {
        this.auswahl=auswahl;
    }
  
    @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.drawLine(getWidth()-getWidth(), getHeight()/2, getWidth(), getHeight()/2);
        g.drawLine(getWidth()/2, getHeight()-getHeight(), getWidth()/2, getHeight());
//        int auswahl = 0;
        switch(auswahl)
        {
        case 0:
            int x=5,y=2*x,x2=-5,y2=-2*x;
            g.drawLine(x, y, x2, y2);
            break;

wie lässt sich dieser Fehler denn beheben?

 

[Blender3D]

Also der erste Code funktioniert nicht weil die Variable auswahl lokal konstant gesetzt wird.
Um das Problem zu lösen musst Du eine Klassen Variabel auswahl definieren. Die mittels der Actionlistener Deiner Buttons gesetzt wird. Der nachfolgende Code demonstriert das. Ausserdem solltest Du nur Klassenvariablen definieren, die du auch als solche verwendest.

JPanel anhangKnoepfe=new JPanel(),bg=new JPanel();
Container c;

z.B. Sollten die lokal defeniert werden.
Ich habe Deinen Code etwas verbessert. Unter Anderem hast Du 3x den selben ActionListener geschrieben.
Schau Dir den Code an und probiere diesen aus. Für Fragen dazu: einfach stellen.

import java.awt.Graphics;
import javax.swing.JPanel;

public class Zeichnen extends JPanel {
    private static final long serialVersionUID = 1L;
    int auswahl = 2;

    @Override
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        g.drawLine(getWidth() - getWidth(), getHeight() / 2, getWidth(), getHeight() / 2);
        g.drawLine(getWidth() / 2, getHeight() - getHeight(), getWidth() / 2, getHeight());

        switch (auswahl) {
        case 0:
            int x = 5, y = 2 * x, x2 = -5, y2 = -2 * x;
            g.drawLine(x, y, x2, y2);
            break;
        case 1:
            int xNeu = 5 * 5, yNeu = 2 * xNeu, xNeu2 = -5 * -5, yNeu2 = -2 * yNeu;
            g.drawLine(xNeu, yNeu, xNeu2, yNeu2);
            break;
        case 2:
            int xNeuNeu = (5 * 5) * 2, yNeuNeu = 2 * xNeuNeu, xNeuNeu2 = -5, yNeuNeu2 = -2 * xNeuNeu;
            g.drawLine(xNeuNeu, yNeuNeu, xNeuNeu2, yNeuNeu2);
            break;
        }
    }

    public void setAuswahl(int burronNumwer) {
        if (burronNumwer < 0 || burronNumwer >= Funktionsgraph.NUM_BUTTONS)
            return;
        this.auswahl = burronNumwer;
    }
}

import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JRadioButton;

public class Funktionsgraph extends JFrame implements ActionListener {
    private static final long serialVersionUID = 1L;
    public static final int NUM_BUTTONS = 3;
    public static final int BUTTON_1 = 0;
    public static final int BUTTON_2 = 1;
    public static final int BUTTON_3 = 2;
    JRadioButton[] funktion = new JRadioButton[NUM_BUTTONS];
    Zeichnen zeichnen = new Zeichnen();

    public Funktionsgraph() {
        JPanel anhangKnoepfe = new JPanel(), bg = new JPanel();
        for (int i = 0; i < NUM_BUTTONS; i++) {
            funktion[i] = new JRadioButton("Funktion " + i);
            bg.add(funktion[i]);
            funktion[i].addActionListener(this);
            anhangKnoepfe.add(funktion[i]);
        }
        Container c = getContentPane();
        c.setLayout(new BorderLayout());
        setLocationRelativeTo(null);
        setSize(600, 400);
        c.add(zeichnen, BorderLayout.CENTER);
        c.add(anhangKnoepfe, BorderLayout.SOUTH);
        anhangKnoepfe.setLayout(new GridLayout(1, 3));

        setVisible(true);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        Object src = e.getSource();
        for (JRadioButton b : funktion)
            b.setSelected(false);
        if (src == funktion[BUTTON_1])
            activateButton(funktion[BUTTON_1], BUTTON_1);
        if (src == funktion[BUTTON_2])
            activateButton(funktion[BUTTON_2], BUTTON_2);
        if (src == funktion[BUTTON_3])
            activateButton(funktion[BUTTON_3], BUTTON_3);
        repaint();
    }

    private void activateButton(JRadioButton button, int auswahl) {
        button.setSelected(true);
        zeichnen.setAuswahl(auswahl);
    }

    public static void main(String[] args) {
        new Funktionsgraph();
    }

}

Stell Fragen.