Variable wird in for - loop nicht richtig hochgezählt

[PaddyFH]

Mit dem code will ich errecihen, dass man je nach dem Wert der Variable "turns" nur einer der beiden buttons ein event auslöst. Versucht habe ich das (hier vereinfacht) mit diesem ausführbaren code:

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class Demo {

   static int myVariable = 1;
  
   public static void main(String[] args) {
          
        JFrame frame = new JFrame();
      
        JPanel mainPanel = new JPanel();
      
        JButton button1 = new JButton();
      
        JButton button2 = new JButton();
      
        mainPanel.add(button1);
        mainPanel.add(button2);
      
        frame.add(mainPanel);
        frame.setVisible(true);
          
        for(; myVariable < 10;) {
              
             if((myVariable % 2) == 0) { 
                  
                 button1.addActionListener(e -> {
                  
                     myVariable++;
                     System.out.println("Button1 "+myVariable);
                });
              
           }  

           else { 
              
                button2.addActionListener(e -> {
                  
                   myVariable++;
                   System.out.println("Button2 "+myVariable);
               });      
           }  
          
       }  
        
   }
  
}

Deshalb passiert auch nur etwas, wenn man "button2" drückt. Aber der zählt hoch wie er lustig ist...

 

[Xyz1]

Aber der zählt hoch wie er lustig ist

Wo ist der Action Listener??

Bearbeitung : Habe ihn gesehen , der AL wird mehrmals hinzugefügt , wenn er n-mal hinzugefügt wird wird um 1- bis n-mal hochgezählt . Je nach dem wie er lustig ist

 

[PaddyFH]

Ich habe das bis jetzt immer mit einer Lambda - funktion gemacht.

button2.addActionListener(e -> {...

Passt das hier dann nicht?

 

[Xyz1]

Passt das hier dann nicht?

Siehe Beitrag # 2.

Eine Frage wäre wie hochgezählt werden soll .

 

[Robat]

dass man je nach dem Wert der Variable "turns" nur einer der beiden buttons ein event auslöst

Wozu die for-Schleife? Damit wird nur der Listener mehrmals hinzugefügt.
Du solltest nicht verhindern ob ein Event ausgelöst wird sondern einfach bestimmen wann etwas in einem Event passiert. Dazu müssen die if-Abfragen in die Listener rein

 

[PaddyFH]

Bearbeitung : Habe ihn gesehen , der AL wird mehrmals hinzugefügt , wenn er n-mal hinzugefügt wird wird um 1- bis n-mal hochgezählt . Je nach dem wie er lustig ist

Ah, okay das macht Sinn. Wie kann ich dann das verhindern?
Edit: hab deine Antwort nicht gesehen...
Danke!

 

[Xyz1]

Du solltest nicht verhindern

Er kann den AL auch 5 und 10-mal hinzufügen, das ist kein Problem . Nur das Adden muss er "einreihen" .

 

[Xyz1]

Schreibe bitte mal ein

volatile

vor

int myVariable

.

 

[mihe7]

dass man je nach dem Wert der Variable "turns" nur einer der beiden buttons ein event auslöst.

Wenn Du nicht willst, dass ein Event ausgelöst wird, musst Du den Button disablen.

Zum Beispiel:

import javax.swing.*;

public class Test {

    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> new Test().show());
    }

    public void show() {
        final JButton button1 = new JButton("Button 1");
        final JButton button2 = new JButton("Button 2");
        button2.setEnabled(false);

        button1.addActionListener(e -> toggleAll(button1, button2));
        button2.addActionListener(e -> toggleAll(button1, button2));

        JPanel panel = new JPanel();
        panel.add(button1);
        panel.add(button2);

        JFrame frame = new JFrame("Test");
        frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
        frame.add(panel);
        frame.pack();
        frame.setVisible(true);
    }

    private void toggleAll(JButton ... buttons) {
        for (JButton button : buttons) {
            toggle(button);
        }
    }

    private void toggle(JButton button) {
        button.setEnabled(!button.isEnabled());
    }
}

 

[Xyz1]

Deshalb passiert auch nur etwas, wenn man "button2" drückt

Das stimmt
Hier nochmal zum Verständnis....

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;

public class Demo {

    static /*volatile*/ int myVariable = 1;

    public static void main(String[] args) {

        JFrame frame = new JFrame();

        JPanel mainPanel = new JPanel();

        JButton button1 = new JButton();

        JButton button2 = new JButton();

        mainPanel.add(button1);
        mainPanel.add(button2);

        frame.add(mainPanel);
        frame.setVisible(true);

        //////
        SwingUtilities.invokeLater(() -> {
            button2.doClick();
            button1.doClick();
            System.out.println("Click ready");
        });
        //////

        for (; myVariable < 10;) {

            if ((myVariable % 2) == 0) {

                button1.addActionListener(e -> {

                    myVariable++;
                    System.out.println("Button1 " + myVariable);
                });

            } else {

                button2.addActionListener(e -> {

                    myVariable++;
                    System.out.println("Button2 " + myVariable);
                });
            }

        }
        System.out.println("AL ready");
    }

}

 

[mihe7]

for (; myVariable < 10;) {

Das kommt erschwerend hinzu

 

[mkdrive2]

1)

for (; myVariable < 10;)

ist eine Endlos-Schleife bis zum ersten Klick des zweiten Buttons. So viele Action-Listener werden entsprechend hinzugefügt.

2) Warum nicht einfach beide Action-Listener einmal hinzufügen und dann eine If-Abfrage innerhalb der Action-Listener, ob sie ausgeführt werden soll?

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class Demo {
    static int myVariable = 1;
 
    public static void main(String[] args) {
        JFrame frame = new JFrame();
        JPanel mainPanel = new JPanel();
        JButton button1 = new JButton();
        JButton button2 = new JButton();
    
        mainPanel.add(button1);
        mainPanel.add(button2);
    
        frame.add(mainPanel);
        frame.setVisible(true);
        frame.pack();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     
        button1.addActionListener(e -> {
            if((myVariable % 2) == 0) {
                myVariable++;
                System.out.println("Button1 "+myVariable);
            }
        });
    
        button2.addActionListener(e -> {
            if((myVariable % 2) == 1) {
                myVariable++;
                System.out.println("Button2 "+myVariable);
            }
        });
    }
}