Javax.swing - Timer(2 Button)

[mr bond345]

Hallo zusammen!

Aufgabe ist: Ein Fenster immer von rot auf grün zu wechseln. Es gibt einen roten button und einen grünen. Wenn man richtig drückt, dann soll gezählt werden, wenn man falsch drückt und wenn man das drücken verpasst.

Schaut mal in Zeile 60, eigentlich müsste das ja funktionierte? Aber wenn ich das Programm im eclipse ausführe, dann passiert nix, es kommt nicht mal das Fenster.

Jedoch, wenn ich die if-anweisung(ohne der while) in den timer_actionlistener reinmachen, dann funktioniert alles.

Warum bitte funktioniert die while hier nicht? Hab ich zu viel in C gedacht?

mfg bond

package Reaktionstester;

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;

public class MainFrame extends JFrame {
  private JPanel contentPane;
  private JButton button_red;
  private JButton button_green;
  private Timer timer1;
  private JPanel panel1;
  private int zaehler_false = 0;
  private int zaehler_verpasst = 0;
  private int zaehler_richtig = 0;
  private int zaehler_widh = 0;
  
  public MainFrame() {
    try {
      setDefaultCloseOperation(EXIT_ON_CLOSE);
      FrameInit();
    }
    catch (Exception exception) {
      exception.printStackTrace();
    }
  }

  /**
   * Component initialization.
   *
   * @throws java.lang.Exception
   */
  private void FrameInit() throws Exception {
    contentPane = (JPanel) getContentPane();
    contentPane.setLayout(new BorderLayout());
    setSize(new Dimension(400, 300));
    setTitle("Frame Title"); 

    
    panel1 = new JPanel();
    panel1.setLayout(new FlowLayout());
    panel1.setBackground(Color.RED);
    
    button_red = new JButton("red");
    button_red.setBackground(Color.RED);
    button_red.addActionListener(new ButtonRed_ActionListener());
    button_green = new JButton("green");
    button_green.setBackground(Color.GREEN);
    button_green.addActionListener(new ButtonGreen_ActionListener());
    
    panel1.add(button_red, BorderLayout.CENTER);
    panel1.add(button_green, BorderLayout.CENTER);
  
    contentPane.add(panel1, BorderLayout.CENTER);
    
    timer1 = new Timer(500, new timer1_ActionListener());
   
    timer1.start();
    
	while(timer1.isRunning() == true)
	{
		if(zaehler_widh == 20)
		{
			timer1.stop();
			System.out.println("Falsch:" + zaehler_false);
			System.out.println("Richtig:" + zaehler_richtig);
			System.out.println("Verpasst:" + zaehler_verpasst);
		}  
	}
  }
  
  private class ButtonRed_ActionListener implements ActionListener {
	 
	  public void actionPerformed(ActionEvent ae) {
	
		  if(panel1.getBackground() == Color.green)
		  {
			  zaehler_false++;
		  }
		  else
		  {
			  zaehler_richtig++;
		  }
		 
	  }
  }
  
  private class ButtonGreen_ActionListener implements ActionListener {
	  public void actionPerformed(ActionEvent ae) {
		  
		  if(panel1.getBackground() == Color.red)
		  {
			  zaehler_false++;
		  }
		  else
		  {
			  zaehler_richtig++;
		  }
		 
	  }
  }
  
  private class timer1_ActionListener implements ActionListener {
	  public void actionPerformed(ActionEvent ae) {
			
		
		  if(panel1.getBackground() == Color.GREEN)
		  {
			  panel1.setBackground(Color.RED);
		  }
		  else
		  {
			  panel1.setBackground(Color.GREEN);
		  }
		  zaehler_widh++;  
		} 
  }		
}

 

[Marcinek]

Das funktioniert nicht, weil diese While Anweisung den EDT blockiert und die anderen Ausführungen nicht zum zuge kommen.

Starte das in einem weiteren Thread.

 

[mr bond007]

Ok, danke!

Was ist hier mit Thread gemeint? Was hältst du von der lösung, die if in den timer_actionlistener reinzu machen?

Ich will ja auch die verpassten Drücke zählen. Aber wie kann ich eine variable hochzählen, wenn kein button gedrückt wird?

Soll ich das im timer_listener machen? Ich mein ich brauch irgendwas, das mir sagt, dass der button nicht gedrückt wurde.

 

[mr bond1342]

Ich hab gesucht nach einen Befehl, der eine boolean variable zurück gibt, falls der Button gedrückt, oder nicht gedrückt ist, aber habe leider nichts gefunden.

Gibt es denn so was ähnliches?

 

[Marcinek]

Hätte das auch in einen actionlistener getan.

So eine Methode gibt es nicht.

 

[mr bond007]

Aso ok danke.

Es gibt ja den Befehl event.getSource(), diesen wollte ich im timer listener einbauen, wenn der rückgabewert nicht den 2 Buttons enstpricht hochzählen, aber leider geht das nicht Listener übergreifend :/.

Ich stehe jetzt endgültig an, ich finde einfach nichts in den docs von oracle-java. Gibt es den irgendeinen ähnlichen Befehl, oder wie setzte ich die Aufgabe am besten um?

 

[Marcinek]

Ich verstehe die Frage nicht.

Du sollst doch eine Variable hochzählen ... Und du hast zwei Listener.. Wieso brauchst du da e.getSource()?

 

[mr bond007]

Brauche ich eh nicht, war nur eine schlechte Idee.

Ja ich will eine Variable hochzählen, wenn diese buttons nicht gedrückt werden. Das muss ich ja wohl im timer_actionlistener machen oder? Weil im button_listener wirds schlecht gehn.

Und ja ich wie könnte ich es denn machen?

 

[Marcinek]

Verstehe ich immernoch nicht.

Du hast drei Actionlistner, die auf verschiedene Sachen reagieren.

Wo ist das Problem hier richtig zu zählen?

 

[mr bond007]

Ich muss varialbe++ machen, wenn KEIN Button gedrückt wird.

Wenn KEIN Button gedrückt wird, dann werden die 2 Button_listener nicht aufgerufen. Der timer_actionlistener wird nur jede 500ms aufgerufen.

Und wie mach ich das jetzt? Wie kann ich überprüfen, ob ein bzw. kein Button gedrückt wurde?

 

[Marcinek]

Na wenn der Timer Listener zuschlägt ist die Zeit vorbei und man hat keinen knopf gedrückt.

 

[mr bond007]

Mh, das verstehe ich nciht ganz.

Wenn die 500ms vorbei sind, da könnte vorher, aber doch ein knopf gedrückt worden sein.

Darum wie weiß ich denn, wenn die 500ms vorbei sind, ob ich jetzt vorher einen Knopf gedrückt habe, oder nicht?

 

[Marcinek]

Dann brauchst du noch eine Variable.

 

[mr bond007]

Danke, aber welche denn?

Ich finde einfach nichts in der doc von oracle.

Ich habs mit:

if(button_red.getAction() == null && button_red.getAction() == null)
		  {
			 zaehler_verpasst++;  
		  }

versucht, aber klappt auch nicht. Ich dachte wenn man keine Knopf drückt wird null zurückgegeben und dann halt zählen.

 

[mr bond007]

Oder meinst du das so?:

ich zähle einfach die knopfdrücke und dann ziehe ich diese von 20 hab = zaehler_verpasst.

 

[mrbond007]

Fertig, was sagt ihr dazu. Zum Code selber etc.?

package Reaktionstester;

import java.awt.*;
import javax.swing.*;

import java.awt.event.*;

public class MainFrame extends JFrame {
  private JPanel contentPane;
  private JButton button_red;
  private JButton button_green;
  private Timer timer_color_change;
  private JPanel panel1;
  private int counter_false = 0;
  private int counter_missed = 0;
  private int counter_right = 0;
  private int zaehler_widh = 0;
  
  public MainFrame() {
    try {
      setDefaultCloseOperation(EXIT_ON_CLOSE);
      FrameInit();
    }
    catch (Exception exception) {
      exception.printStackTrace();
    }
  }

  /**
   * Component initialization.
   *
   * @throws java.lang.Exception
   */
  private void FrameInit() throws Exception {
    contentPane = (JPanel) getContentPane();
    contentPane.setLayout(new BorderLayout());
    setSize(new Dimension(400, 300));
    setTitle("Frame Title"); 

    
    panel1 = new JPanel();
    panel1.setLayout(new FlowLayout());
    panel1.setBackground(Color.RED);
    
    Button_ActionListener bal = new Button_ActionListener();
    
    button_red = new JButton("red");
    button_red.setBackground(Color.RED);
    button_red.addActionListener(bal);
    button_green = new JButton("green");
    button_green.setBackground(Color.GREEN);
    button_green.addActionListener(bal);
    
    panel1.add(button_red, BorderLayout.CENTER);
    panel1.add(button_green, BorderLayout.CENTER);
  
    contentPane.add(panel1, BorderLayout.CENTER);
    
    timer_color_change = new Timer(500, new timer_color_change_ActionListener());
   
    timer_color_change.start();
  }
  
  private class Button_ActionListener implements ActionListener {
	 public void actionPerformed(ActionEvent ae) {
		 if(ae.getSource() == button_red)
		 {
			 if(panel1.getBackground() == Color.green)
		  	 {
				 counter_false++;
		  	 }
		  	 else
		  	 {
		  		 counter_right++;
		  	 }
			 button_green.setEnabled(false);
			 button_red.setEnabled(false);
		 }
		  
		 if(ae.getSource() == button_green)
		 {
			 if(panel1.getBackground() == Color.red)
		  	 {
				 counter_false++;
		  	 }
		  	 else
		  	 {
		  		 counter_right++;
		  	 }
			 button_red.setEnabled(false);
			 button_green.setEnabled(false);
		 }
	 }
  }
 
  
  private class timer_color_change_ActionListener implements ActionListener {
	public void actionPerformed(ActionEvent ae) {
		if(panel1.getBackground() == Color.GREEN)
		{
			panel1.setBackground(Color.RED);
		}
		else
		{
			panel1.setBackground(Color.GREEN);
		}
		zaehler_widh++;  
		  
		button_green.setEnabled(true);
		button_red.setEnabled(true);
		  
		if(button_red.getAction() == null && button_red.getAction() == null)
		{
			counter_missed++;  
		}
		  
		 
		if(zaehler_widh == 20)
		{
			timer_color_change.stop();
			counter_missed = 20 - (counter_false + counter_right);
			System.out.println("Falsch:" + counter_false);
			System.out.println("Richtig:" + counter_right);
			System.out.println("nicht gedrückt:" + counter_missed);
		} 
	} 
  }		
}

 

[mr bond007]

Bitte leute schaut doch mein Programm an. Ich möchte ja nur wissen wie es besser geht bzw. obs so passt.

 

[mr bond007]

bitte leute kommt schon^^