Checkboxen erstellt mit MouseClicked event

[Xnor]

Hallo
Vorweg, ich bin ein blutiger Anfänger.

import java.awt.event.*;

public class NewJFrame extends javax.swing.JFrame {

    public javax.swing.JCheckBox[] radioButton = new javax.swing.JCheckBox[8];
    public int[] zahlen = new int[8];

    public NewJFrame() {
        initComponents();
        setRadioButton();
    }

    public void setRadioButton() {
      int loc = 0;
      for(int i = 0; i <= 7; i++){
        radioButton[i] = new javax.swing.JCheckBox();
        radioButton[i].setBounds(16, 16+loc, 50, 17);
        radioButton[i].setText("" + (int)Math.pow(2,i));
        jPanel1.add(radioButton[i]);
        loc = loc + 15;
        radioButton[i].addMouseListener(new MyML(){
            @Override
            public void mouseClicked(MouseEvent e){
                for(int i = 0; i<=7;i++){
                    int zahl = 0;
                    if(radioButton[i].isSelected() == true){
                    zahlen[i] = (int)Math.pow(2,i);
                } else {zahlen[i] = 0;}
                    for(int v = 0; v <= 7; v++){
                        zahl = zahl + zahlen[v];
                        jLabel1.setText("" + zahl);
                    }
                }
            }
        });
      }
    }
[...]

Sieht dann in etwa so aus:


bei jedem klick auf eins der Checkboxen wird die Summe berechnet und im Label unten aktualisiert.

Mein Problem ist jetzt, dass bei einem zu schnellen anklicken der Checkboxen, die Summe nicht neu berechnet und angezeigt wird.
Ist es möglich dieses Problem zu lösen?

Vielen dank schonmal im Vorraus,
MfG Xnor

 

[Developer_X]

kein Problem, ging uns allen mal so.
Das kannst du eig. ganz einfach regeln:

	JCheckBox[] radioButton;
	JPanel jPanel1;
	JLabel jLabel1;
    public void setRadioButton()
    {
        int loc = 0;
        int i = 0;
        
        for(i = 0; i <= 7; i++)
        {
        	radioButton[i] = new javax.swing.JCheckBox();
        	radioButton[i].setBounds(16, 16+loc, 50, 17);
        	radioButton[i].setText("" + (int)Math.pow(2,i));
        	jPanel1.add(radioButton[i]);
        	loc = loc + 15;
         	radioButton[i].addActionListener(new ActionListener()
         	{
				public void actionPerformed(ActionEvent arg0) 
				{
					if(radioButton[i].isSelected()==true)
					{
						int i = Integer.parseInt(radioButton[i].getText());
						i = Integer.parseInt(jLabel1.getText()) + i;
						jLabel1.setText(""+i);
					}
					else
					{
						int i = Integer.parseInt(radioButton[i].getText());
						i = Integer.parseInt(jLabel1.getText()) - i;
						jLabel1.setText(""+i);					
					}
					jLabel1.repaint();
				}
        	});
        }
    }

So in etwa müsste das aussehen.
Verstehst du?
Wenn du noch weitere fragen hast, stell sie ruhig.
Ich hoffe ich konnte dir helfen,

Developer_X

 

[Xnor]

Vielen dank für die schnelle und hilfreiche Antwort

kleine Anmerkung zum Quellcode von dir:

if(radioButton[i].isSelected()==true)

funktioniert so nicht, genauso wie

radioButton[i].getText()

da er das "i" nicht kennt, bzw. sagt er mir:
"local variable i accessed from within inner class; needs to be declared final" und zum zweiten "i"
"variable i might not have been initialized"

da aber mein Fehler wohl in dem event lag, hab einzig mein Quellcode diesbezüglich geändert.
Die Texte bei den RadionButtons waren nur provisorisch/vorübergehend die Zahlen die zusammen gezählt werden sollten, damit mans bisschen besser nachvollziehen kann.
Die Rechnung soll weiterhin über

(int)Math.pow(2,i)

laufen

Sieht dann nach kleiner aber feiner Veränderung so aus.

    public void setRadioButton() {
      int loc = 0;
      for(int i = 0; i <= 7; i++){
        radioButton[i] = new javax.swing.JCheckBox();
        radioButton[i].setBounds(16, 16+loc, 50, 17);
        radioButton[i].setText("D" + i);
        jPanel1.add(radioButton[i]);
        loc = loc + 15;
        radioButton[i].addActionListener(new ActionListener(){
            public void actionPerformed(ActionEvent arg0){
                for(int i = 0; i<=7;i++){
                    int zahl = 0;
                    if(radioButton[i].isSelected() == true){
                    zahlen[i] = (int)Math.pow(2,i);
                } else {zahlen[i] = 0;}
                    for(int v = 0; v <= 7; v++){
                        zahl = zahl + zahlen[v];
                        jLabel1.setText("" + zahl);
                    }
                }
            }
        });
      }
    }

Es gibt bestimmt immernoch eine Menge dran aus zu setzen und ich bin immer für weitere Verbesserungsvorschläge offen.
Vom prinzip her sollte es aber der Funktionsweise entsprechen, da die Checkboxs Namen eben anders sein werden und ich den Wert in der variable "zahl" später weiter verwenden möchte.
Das Anzeigen des Ergebnisses im Label fällt später weg und diese Methode wird nur ein kleinen Teil des Quellcodes einnehmen.

Vielen dank nochmal
MfG Xnor

 

[eRaaaa]

Es gibt bestimmt immernoch eine Menge dran aus zu setzen und ich bin immer für weitere Verbesserungsvorschläge offen.

dann fang ich mal an
ihr erstellt 8 neue actionlistener! erstellt einen actionlistener und hängt diesen einen an alle checkboxen. die aktion ist ja an sich immer die gleiche. dann ist imo diese for-schleife innerhalb des listeners auch nicht mehr nötig, ehrlich gesagt versteh ich eh nicht was die da sucht =)

 

[Xnor]

wenn du diese For-Schleife meinst:

for(int i = 0; i<=7;i++){
                    int zahl = 0;
                    if(radioButton[i].isSelected() == true){
                    zahlen[i] = (int)Math.pow(2,i);
                } else {zahlen[i] = 0;}

ist es nach meinem geringen Wissensstand über Java die einzige Lösung, da er sonst, wie in Post #3 (von mir) beschrieben, das "i" nicht kennt, bzw. ein Fehler anzeigt.

Wie ich alle CheckBoxen an eine ActionListener anhänge weiß ich aus diesem Grund genauso wenig.

MfG Xnor

 

[javimka]

Umgekehrt, du sollst 1 ActionListener an alle CheckBoxen ranhängen. Dazu bastelst du eine, die sogar eine for Schleife haben darf, die dann durch alle CheckBoxen durchgeht und den Wert berechnet. Mit einer weiteren for-Schleife (ausserhalb der anderen Schleife) hängst du dann diesen einen ActionListener an die CheckBoxen an.

 

[Xnor]

Umgekehrt, du sollst 1 ActionListener an alle CheckBoxen ranhängen. Dazu bastelst du eine, die sogar eine for Schleife haben darf, die dann durch alle CheckBoxen durchgeht und den Wert berechnet. Mit einer weiteren for-Schleife (ausserhalb der anderen Schleife) hängst du dann diesen einen ActionListener an die CheckBoxen an.

Vielen dank für den Ansatz,

Ich werde mich morgen ran machen und versuchen dies dann bestmöglich um zu setzen.

MfG Xnor

 

[eRaaaa]

Umgekehrt, du sollst 1 ActionListener an alle CheckBoxen ranhängen. Dazu bastelst du eine, die sogar eine for Schleife haben darf, die dann durch alle CheckBoxen durchgeht und den Wert berechnet. Mit einer weiteren for-Schleife (ausserhalb der anderen Schleife) hängst du dann diesen einen ActionListener an die CheckBoxen an.

also ich meinte das eig. ungefähr so:

//ItemListener
     ItemListener listener = new ItemListener() {

			@Override
			public void itemStateChanged(ItemEvent e) {
				JCheckBox box = (JCheckBox) e.getSource();
				if (e.getStateChange() == ItemEvent.SELECTED) {
					summe += Integer.parseInt(box.getActionCommand());
				} else {
					summe -= Integer.parseInt(box.getActionCommand());
				}
				label.setText(String.valueOf(summe));
			}
		};

//checkboxen erstellen
		JCheckBox[] boxes = new JCheckBox[8];
		JCheckBox box;
		for (int i = 0; i < boxes.length; i++) {
			box = new JCheckBox();
			box.setText((int) Math.pow(2, i) + "");
			box.setActionCommand(String.valueOf((int) Math.pow(2, i)));
//checkbox dem listener zuteilen
			box.addItemListener(listener);
			panel.add(box);
		}

 

[Xnor]

Hallo, vielen dank für das Beispiel

So ein Quellcode ist mir dann doch schon viel zu Kompiziert

---------------------------------------------------------

box.setActionCommand(String.valueOf((int) Math.pow(2, i)));

wenn ich das richtig verstanden hab, wird hier der Checkbox der Wert zugeteil, doch frag ich mich was es genau bedeutet, sodass ichs auch nachvollziehen kann.
"setActionCommand" und "String.valueOf" sind mir fremd.
---------------------------------------------------------
welche Funktion genau übernimmt

JCheckBox box

werden diese CheckBoxen die dann in der Schleife über "box" erstellt werden in das Array "gepackt", oder welches vorteil hat es gegenüber dieser Zeile z.B.

boxes[i] = new JCheckBox();

---------------------------------------------------------
Die ganze ItemEvent Geschichte...
Die IF-Abfrage kann ich mir ja durch die Übersetzung der Begriffe herleiten, doch versteh ich, wie schon oben erwähnt, die ganze Geschichte mit dem "jCheckBox box" nicht, genauso hier:

JCheckBox box = (JCheckBox) e.getSource();

Ich hoffe, dass es nicht zuviel Unwissenheit in einem Post ist und bitte um Verständnis

MfG Xnor

 

[eRaaaa]

also bei mir wird garnichts in das checkboxen array gelegt (zeile 18 kannst du also ignorieren, hat sich beim copy paste eingeschlichen) bei mir werdne die checkboxen einfach aufs panel geaddet, das wars --> das ganze sollte eig. auch nur ein beispiel dazu sein, wie ich das mit dem listener gemeint habe.

JCheckBox box; übernimmt sogesehen noch garkeine funktion
das ist einfach nur eine deklaration.

zu dme ItemEvent. find ich hier schöner als mit actionlistener, denn so sagt mir auch gleich das event, ob nun die checkbox angeklickt, oder wegklickt wurde.
das frag ich hier ab: [c]e.getStateChange() == ItemEvent.SELECTED[/c]

[c]
JCheckBox box = (JCheckBox) e.getSource();
[/c]
e.getSource() liefert dir das objekt welches das event ausgelöst hat. in diesem fall halt eine deiner checkboxen. damit du aber auch die methoden der checkbox nutzen kannst musst du es halt auf checkbox casten(hier kannst du dir ja sicher sein dass es sich um eine checkbox handeln muss).
so kann ich mir jetzt mittels box.getActionCommand() den wert holen, den ich vorher mit setActionCommand gesetzt habe(hier halt der wert (int) Math.pow(2, i)). diesen wert addiere ich nun, oder subtrahiere, je nachdem was getStateChanged mir geliefert hat.

aber eig. sollte dich das alles garnicht irritieren, sorry! ich wollte eig. eher auf die sache mit dem listener hinaus ;(

 

[Xnor]

Hallo

Kleine Anpassung des Quellcodes von "eRaaaa":

    public int zahl;
    public JCheckBox[] radioButton = new JCheckBox[8];    

    public void setRadioButton(){
        JCheckBox box;
        int loc = 0;

        ItemListener listener = new ItemListener() {
            @Override
            public void itemStateChanged(ItemEvent e) {
                JCheckBox box = (JCheckBox) e.getSource();
                if (e.getStateChange() == ItemEvent.SELECTED) {
                    zahl += Integer.parseInt(box.getActionCommand());
                } else {
                    zahl -= Integer.parseInt(box.getActionCommand());
                }
                spin1.setValue(zahl);
            }
        };

        for (int i = 0; i < 8; i++) {
            box = new JCheckBox();
            box.setText("D" + i);
            box.setBounds(16, 54+loc, 40, 17);
            box.setActionCommand(String.valueOf((int) Math.pow(2, i)));
            loc = loc + 25;

            box.addItemListener(listener);
            radioButton[i] = box;
            jPanel3.add(box);
        }
    }

dazu ist jetzt noch ein Spinner dazugekommen, der bei Veränderung die CheckBoxen anders Selektiert:
(dank eurem Quellcode hergeleitet )

    public JSpinner spin1;

    public void setSpinner(){
         //JSpinner erstellen
         spin1 = new JSpinner();
         spin1.setBounds(17, 251, 53, 20);
         jPanel3.add(spin1);

         //Zahlenbereich festlegen
         SpinnerModel model = new SpinnerNumberModel(0,0,255,1);
         spin1.setModel(model);

         //Event
         ChangeListener listener = new ChangeListener(){
             public void stateChanged(ChangeEvent e){
                 int wert = Integer.parseInt(spin1.getValue().toString());
                    for(int i = 0; i <= 7; i++){
                        if(wert%2 == 1){
                            radioButton[i].setSelected(true);
                            wert = wert / 2;
                        } else {
                            radioButton[i].setSelected(false);
                            wert = wert / 2;
                        }
                    }
             }
         };
         spin1.addChangeListener(listener);
    }

Funktioniert einbahnfrei alles und so ist dieses Thema auch im grunde gelöst.
Für Verbesserungsvorschläge bin ich denoch jederzeit offen.

Vielen dank nochmal für alle Antworten.

MfG Xnor