Nach Druck eines Buttons reagieren andere Button nicht

[andi11]

Hallo zusammen,

im folgenden Programm läuft nach Druck auf einen Button ein Ball vom Rand des Fensters zur Mitte (2D Grafik). Es gibt 3 Button, jeder Button gibt eine andere Startposition vor.
Wird nun ein Durchlauf durch den Druck auf einen Button gestartet und danach ein anderer Button gedrückt, soll der aktuelle Lauf abgebrochen werden und der neue gestartet werden.
Das funktioniert allerdings nicht. Wurde ein Button gedrückt, reagieren die anderen Button nicht mehr, nur der Button "Schließen".

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.*;
import java.io.ByteArrayOutputStream;
import java.io.PrintStream;

import javax.swing.Timer;
import javax.swing.JApplet;
import javax.swing.JButton;
import javax.swing.JOptionPane;
import javax.swing.JPanel;


public class ProgStart extends JApplet implements ActionListener {
   
   int [] StartPos= {0,0};
   private final Timer timer = new Timer(100, null);
   
   private JButton VonLinks1;
   private JButton VonLinks2;
   private JButton VonRechts;
   private JButton Schliessen;
   
   private Thread t;

   private boolean Hoehenkontrolle = false;
   
   JPanel MainPanel = new JPanel();
   Feld Feld = new Feld();
   
   public void init()
   {
     
       setSize(800,800);
       
       MainPanel.setLayout(null);
       getContentPane().add(MainPanel);
       MainPanel.setBounds(0, 0, 800, 800);
       
       Feld.setLayout(null);
       MainPanel.add(Feld);
       Feld.setBounds(50, 20, 700, 700);
       setVisible(true);
       
       VonRechts = new JButton("Von rechts");
       VonLinks1 = new JButton("Von links1");
       VonLinks2 = new JButton("Von links2");
       Schliessen = new JButton("Schließen");
       
       MainPanel.add(VonRechts);   
       MainPanel.add(VonLinks1);
       MainPanel.add(VonLinks2);
       MainPanel.add(Schliessen);     

       int PosxStart = 50;
       int PosyStart = 750;
       int BreiteButton = 160;
       int HoeheButton = 40;
       int delta = 20;
       
       VonRechts.setBounds(PosxStart, PosyStart, BreiteButton, HoeheButton);
       VonLinks1.setBounds(PosxStart+(BreiteButton+delta)*1, PosyStart, BreiteButton, HoeheButton);
       VonLinks2.setBounds(PosxStart+(BreiteButton+delta)*2, PosyStart, BreiteButton, HoeheButton);
       Schliessen.setBounds(PosxStart+(BreiteButton+delta)*3, PosyStart, BreiteButton, HoeheButton);

       VonRechts.addActionListener(this);
       VonLinks1.addActionListener(this);
       VonLinks2.addActionListener(this);
       Schliessen.addActionListener(this);
       
       VonRechts.setEnabled(true);
       VonLinks1.setEnabled(true);
       VonLinks2.setEnabled(true);
       
       t = new Thread(new Runnable()
       {     
       
         @Override
         public void run()
         {
           Weg Weg = new Weg(StartPos);
           timer.start();
   
          timer.addActionListener(event ->
          {
         
               if (Pack1.GlobalVar.AktPos == Pack1.GlobalVar.SollPos)
               {
                 timer.stop();
                 
               }
               
               if (Hoehenkontrolle == true && Pack1.GlobalVar.HoeheUeberschritten == true)
               {
                 timer.stop();
                 
               }
               
               Weg.NaechstePos();
               repaint();   
             
        });   
         };
       });
   }
   
  public void Start()
   {   
   }

   public void actionPerformed(ActionEvent e)
   {

       if (e.getSource() == VonRechts)
       {     

         StartPos[0] = 700;   
         StartPos[1] = 350;
         t.start();
     
       }
       
       if (e.getSource() == VonLinks1)
       {         
         StartPos[0] = 0;   
         StartPos[1] = 350;
         Hoehenkontrolle = true;
         t.start();     
       }
       
       
       if (e.getSource() == VonLinks2)
       {
         StartPos[0] = 0;   
         StartPos[1] = 350;
         t.start();   
       }
       
       if (e.getSource() == Schliessen)
       {
        System.exit(0);
       }   
   }
   
  public void destroy()
  {     
  }
   
  public void stop()
  {     
  }

}

 

[X5-599]

Ich bin mir fast sicher, dass es daran liegt, dass du deinen Thread mehrmals starten willst. Laut Oracle darf man das nicht tun:
"It is never legal to start a thread more than once. In particular, a thread may not be restarted once it has completed execution."
Auszug aus der API Doku für Thread.https://docs.oracle.com/javase/7/docs/api/java/lang/Thread.html

 

[andi11]

Ich hatte vorher schon ein paar Sachen ausprobiert. Zuerst hatte ich gar keinen Thread. Dabei wurde der Ball gar nicht angezeigt. Dann habe ich unter jedem Button unter "public void actionPerformed" einen Thread eingebaut. Dabei trat das selbe Problem auf wie jetzt.

 

[andi11]

Nun habe ich den Fehler gefunden. Ich habe zwar den Timer nochmal geändert und die Threads rausgeworfen, aber letzt endlich denke lag es nicht daran, sondern die Startposition wurde in der Klasse „Weg“ nicht übernommen. Hier hatte ich nämlich noch eine if-Abfrage Wenn die Anzahl der Schritte = 0 sind dann übernehme Startposition sonst nicht.

package Pack1;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.*;
import java.io.ByteArrayOutputStream;
import java.io.PrintStream;

import javax.swing.Timer;
import javax.swing.JApplet;
import javax.swing.JButton;
import javax.swing.JOptionPane;
import javax.swing.JPanel;


public class ProgrammStart extends JApplet implements ActionListener {
   
   int [] StartPos= {0,0};
   private final Timer timer = new Timer(100, null);
   
   private JButton VonLinks1;
   private JButton VonLinks2;
   private JButton VonRechts;
   private JButton Stop;

   private boolean Hoehenkontrolle = false;
   
   JPanel MainPanel = new JPanel();
   Feld Feld = new Feld();
   Weg Weg = new Weg();
   
   
   public void init()
   {
     
       setSize(800,800);
       
       MainPanel.setLayout(null);
       getContentPane().add(MainPanel);
       MainPanel.setBounds(0, 0, 800, 800);
       
       Feld.setLayout(null);
       MainPanel.add(Feld);
       Feld.setBounds(50, 20, 700, 700);
       setVisible(true);
       
       VonRechts = new JButton("Von rechts");
       VonLinks1 = new JButton("Von links1");
       VonLinks2 = new JButton("Von links2");
       Stop = new JButton("Stop");
       
       MainPanel.add(VonRechts);   
       MainPanel.add(VonLinks1);
       MainPanel.add(VonLinks2);
       MainPanel.add(Stop);     

       int PosxStart = 50;
       int PosyStart = 750;
       int BreiteButton = 160;
       int HoeheButton = 40;
       int delta = 20;
       
       VonRechts.setBounds(PosxStart, PosyStart, BreiteButton, HoeheButton);
       VonLinks1.setBounds(PosxStart+(BreiteButton+delta)*1, PosyStart, BreiteButton, HoeheButton);
       VonLinks2.setBounds(PosxStart+(BreiteButton+delta)*2, PosyStart, BreiteButton, HoeheButton);
       Stop.setBounds(PosxStart+(BreiteButton+delta)*3, PosyStart, BreiteButton, HoeheButton);

       VonRechts.addActionListener(this);
       VonLinks1.addActionListener(this);
       VonLinks2.addActionListener(this);
       Stop.addActionListener(this);
       
       timer.addActionListener(this);   

   }
   
  public void Start()
   {         
   }

   public void actionPerformed(ActionEvent e)
   {
     
     try
     {

       if (e.getSource() == VonRechts)
       {     

         StartPos[0] = 700;   
         StartPos[1] = 350;
         Pack1.GlobalVar.AnzSchritte=0;

         timer.start();
     
       }
       
       if (e.getSource() == VonLinks1)
       {         
         StartPos[0] = 0;   
         StartPos[1] = 350;
         Pack1.GlobalVar.AnzSchritte=0;
         Hoehenkontrolle = true;
         timer.start();     
       }
       
       
       if (e.getSource() == VonLinks2)
       {
         StartPos[0] = 0;   
         StartPos[1] = 350;
         Pack1.GlobalVar.AnzSchritte=0;
         timer.start();   
       }
       
       if (e.getSource() == Stop)
       {
         timer.wait();
       }   
   
       if (e.getSource() == timer)
       {
           Weg.NaechstePos(StartPos);
           repaint();           
         
           if (Pack1.GlobalVar.AktPos == Pack1.GlobalVar.SollPos)
           {
             timer.wait();
             return;
           }
           
           if (Hoehenkontrolle == true && Pack1.GlobalVar.HoeheUeberschritten == true)
           {
             Hoehenkontrolle = false;
             timer.wait();
             return;
           }

       }
       
     } catch(Exception ex2)
     {   
       timer.stop();
     }
     
   }
   
   public void stop()
  {     
  }
   
  public void destroy()
  {     
  }
}