Thread funktioniert nicht

[Jack159]

Hintergrund:
Ich habe ein Schere Stein Papier Spiel (mit GUI) gecodet, welches nun auch funktioniert.
Beim klicken des Buttons wird das Spiel gestartet und in der TextArea werden so Sachen wie "Spieler nimmt Stein, PC nimmt Schere, Spieler gewinnt" angezeigt. Da diese anzeige, aber sofort erscheint, würde ich den Text in mehreren Zeitabständen ausgeben lassen, damit es ein wenig spannender wirkt.
Beispiel:
*Schere* (1sek Pause)
*Stein* (1sek Pause)
*Papier* (1sek Pause)
"Spieler nimmt Stein, PC nimmt Schere" (1sek Pause)
"Spieler gewinnt"
Das ganze müsste sich mit einem Thread realisieren lassen, indem man den Thread dann immer für 1 sek einschlafen lässt.

Hier das Spiel ohne Threads, wo der Text sofort erscheint. Die ganze Aktion spielt sich eigentlich in der ActionPerformed Methode des Buttons ab (Funktionierende Version)

import java.awt.BorderLayout;


public class Schere extends JFrame 
{

	private JPanel contentPane;
	private JScrollPane scrollPane;
	private JTextArea textArea;
	private JRadioButton SchereRadioButton;
	private JRadioButton SteinRadioButton;
	private JRadioButton PapierRadioButton;
	public String spielerAuswahl;
	public String computerAuswahl;
	public ArrayList<String> liste = new ArrayList<String>();
    
	
	
	

	/**
	 * Launch the application.
	 */
	public static void main(String[] args) {
		
	
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					Schere frame = new Schere();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame.
	 */
	public Schere() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 547, 429);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		setContentPane(contentPane);
		contentPane.setLayout(null);
		
		scrollPane = new JScrollPane();
		scrollPane.setBounds(42, 28, 444, 221);
		contentPane.add(scrollPane);
		
		textArea = new JTextArea();
		textArea.setWrapStyleWord(true);
		textArea.setLineWrap(true);
		scrollPane.setViewportView(textArea);
		
		 SchereRadioButton = new JRadioButton("Schere");
		SchereRadioButton.setBounds(42, 256, 109, 23);
		contentPane.add(SchereRadioButton);
		
		 SteinRadioButton = new JRadioButton("Stein");
		SteinRadioButton.setBounds(42, 282, 109, 23);
		contentPane.add(SteinRadioButton);
		
		 PapierRadioButton = new JRadioButton("Papier");
		PapierRadioButton.setBounds(42, 308, 109, 23);
		contentPane.add(PapierRadioButton);
		
		ButtonGroup group = new ButtonGroup();
	    group.add(SchereRadioButton);
	    group.add(SteinRadioButton);
	    group.add(PapierRadioButton);
		
		JButton btnNewButton = new JButton("Los");
		btnNewButton.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				
				if(SchereRadioButton.isSelected() | SteinRadioButton.isSelected() | PapierRadioButton.isSelected()) {
					
					textArea.append("Das Spiel beginnt!\n");
					
			
				liste.add("Schere");
				liste.add("Stein");
				liste.add("Papier");
				int zufallsZahl = (int) (Math.random()*3); 
				computerAuswahl = liste.get(zufallsZahl);
					
					
				
				
				if(SchereRadioButton.isSelected()) {
					
					spielerAuswahl = "Schere";
					textArea.append("Spieler nimmt Schere. Der Computer nimmt "+computerAuswahl+".\n");
					
				}
				
                if(SteinRadioButton.isSelected()) {
					
                	spielerAuswahl = "Stein";
					textArea.append("Spieler nimmt Stein. Der Computer nimmt "+computerAuswahl+".\n");
					
				}
                
                if(PapierRadioButton.isSelected()) {
					
                	spielerAuswahl = "Papier";
					textArea.append("Spieler nimmt Papier. Der Computer nimmt "+computerAuswahl+".\n");
					
				}
                
                

                if(spielerAuswahl==computerAuswahl)
                {
                  
                	textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Unentschieden!\n\n");

                }
                else
                {
                  if( (spielerAuswahl == "Schere" && computerAuswahl=="Papier") ||
                      (spielerAuswahl == "Stein" && computerAuswahl=="Schere")   ||
                      (spielerAuswahl == "Papier" && computerAuswahl=="Stein"))
                  {

                	  textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Spieler gewinnt das Spiel!\n\n");
                  }
                  else
                  {
                	  textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Computer gewinnt das Spiel!\n\n");
                  }
                }  
                
                
			}
				
			}
		});
		btnNewButton.setBounds(42, 360, 89, 23);
		contentPane.add(btnNewButton);
		
		
	    
		
	}
}

Hier meine aktuelle Thread-Version, welche nicht funktioniert. Erstmal soll das ganze Zeugs aus der Actionperformed Methode des Buttons, vom Thread aus ausgeführt werden:

import java.awt.BorderLayout;


public class Schere extends JFrame 
{

	public JPanel contentPane;
	public JScrollPane scrollPane;
	public JTextArea textArea;
	public JRadioButton SchereRadioButton;
	public JRadioButton SteinRadioButton;
	public JRadioButton PapierRadioButton;
	public String spielerAuswahl;
	public String computerAuswahl;
	public ArrayList<String> liste = new ArrayList<String>();
	public Runnable r1 = new SchereThread();
    
	
	
	

	/**
	 * Launch the application.
	 */
	public static void main(String[] args) {
		
	
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					Schere frame = new Schere();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame.
	 */
	public Schere() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 547, 429);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		setContentPane(contentPane);
		contentPane.setLayout(null);
		
		scrollPane = new JScrollPane();
		scrollPane.setBounds(42, 28, 444, 221);
		contentPane.add(scrollPane);
		
		textArea = new JTextArea();
		textArea.setWrapStyleWord(true);
		textArea.setLineWrap(true);
		scrollPane.setViewportView(textArea);
		
		 SchereRadioButton = new JRadioButton("Schere");
		SchereRadioButton.setBounds(42, 256, 109, 23);
		contentPane.add(SchereRadioButton);
		
		 SteinRadioButton = new JRadioButton("Stein");
		SteinRadioButton.setBounds(42, 282, 109, 23);
		contentPane.add(SteinRadioButton);
		
		 PapierRadioButton = new JRadioButton("Papier");
		PapierRadioButton.setBounds(42, 308, 109, 23);
		contentPane.add(PapierRadioButton);
		
		ButtonGroup group = new ButtonGroup();
	    group.add(SchereRadioButton);
	    group.add(SteinRadioButton);
	    group.add(PapierRadioButton);
		
		JButton btnNewButton = new JButton("Los");
		btnNewButton.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				
				new Thread(r1).start();
				
			}
		});
		btnNewButton.setBounds(42, 360, 89, 23);
		contentPane.add(btnNewButton);
		
		
	    
		
	}
}

public class SchereThread extends Schere implements Runnable {

	
	public void run() {
		
		if(SchereRadioButton.isSelected() | SteinRadioButton.isSelected() | PapierRadioButton.isSelected()) {
			
			textArea.append("Das Spiel beginnt!\n");
			
	
		liste.add("Schere");
		liste.add("Stein");
		liste.add("Papier");
		int zufallsZahl = (int) (Math.random()*3); 
		computerAuswahl = liste.get(zufallsZahl);
			
			
		
		
		if(SchereRadioButton.isSelected()) {
			
			spielerAuswahl = "Schere";
			textArea.append("Spieler nimmt Schere. Der Computer nimmt "+computerAuswahl+".\n");
			
		}
		
        if(SteinRadioButton.isSelected()) {
			
        	spielerAuswahl = "Stein";
			textArea.append("Spieler nimmt Stein. Der Computer nimmt "+computerAuswahl+".\n");
			
		}
        
        if(PapierRadioButton.isSelected()) {
			
        	spielerAuswahl = "Papier";
			textArea.append("Spieler nimmt Papier. Der Computer nimmt "+computerAuswahl+".\n");
			
		}
        
        

        if(spielerAuswahl==computerAuswahl)
        {
          
        	textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Unentschieden!\n\n");

        }
        else
        {
          if( (spielerAuswahl == "Schere" && computerAuswahl=="Papier") ||
              (spielerAuswahl == "Stein" && computerAuswahl=="Schere")   ||
              (spielerAuswahl == "Papier" && computerAuswahl=="Stein"))
          {

        	  textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Spieler gewinnt das Spiel!\n\n");
          }
          else
          {
        	  textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Computer gewinnt das Spiel!\n\n");
          }
        }  
        
        
	}
		
	}
	
	
}

 

[SlaterB]

SchereThread erbt von Schere, jedes dieser eigenständigen Runnable-Objekte greift auf seine eigenen Schere-Variablen zu,
das was du auf dem Bildschirm siehst ist aber ein einzelnes erstes separates Schere-Objekt, welches von der main-Methode erstellt wurde

-> SchereThread sollte nicht von Schere erben sondern ein (das einzige) Schere-Objekt im Konstruktor übergeben bekommen

 

[Michael...]

Wofür soll der Thread gut sein bzw. warum willst Du das in einen separaten Thread auslagern?
Das ist doch ein rein sequenzielles Spiel.

 

[Jack159]

SchereThread erbt von Schere, jedes dieser eigenständigen Runnable-Objekte greift auf seine eigenen Schere-Variablen zu,
das was du auf dem Bildschirm siehst ist aber ein einzelnes erstes separates Schere-Objekt, welches von der main-Methode erstellt wurde

-> SchereThread sollte nicht von Schere erben sondern ein (das einzige) Schere-Objekt im Konstruktor übergeben bekommen

Welches welches Schere-Objekt sollte SchereThread übergeben bekommen? Kann dir nicht ganz folgen.
Meinst du das Objekt "r1"?

@ Michael:
Ich möchte ja den die Textausgaben immer mit etwa 1sek Abstand ausgeben lassen, anstatt komplett alles sofort. Daher dachte ich, dass man das ganze mit einem Thread lösen könnte, da man einen Thread ja dann ständig für 1sek einfrieren lassen kann, bevor er den nächsten Text ausgibt.

 

[SlaterB]

nein, r1 ist ja der Thread,

sondern
> Schere frame = new Schere();
aus der main-Methode bzw., da der Thread ja aus Schere heraus gestartet wird dort einfach 'this' übergeben, das Schere-Objekt eben

 

[Jack159]

Ich habe jetzt nochmal ins Buch geschaut (Thema Threads). Ich habe es jetzt so gemacht wie es dort erklärt wird, allerdings klappt es immer noch nicht. Fast alles aus der SchereThread-Klasse ist auch rot unterstrichen.
Hier mal die überarbeitete Version, Zeile 16+17+84 wurden überarbeitet:

import java.awt.BorderLayout;


public class Schere extends JFrame 
{

	public JPanel contentPane;
	public JScrollPane scrollPane;
	public JTextArea textArea;
	public JRadioButton SchereRadioButton;
	public JRadioButton SteinRadioButton;
	public JRadioButton PapierRadioButton;
	public String spielerAuswahl;
	public String computerAuswahl;
	public ArrayList<String> liste = new ArrayList<String>();
	public Runnable threadJob = new SchereThread();
	public Thread meinThread = new Thread(threadJob);
    
	
	
	

	/**
	 * Launch the application.
	 */
	public static void main(String[] args) {
		
		
	
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					Schere frame = new Schere();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame.
	 */
	public Schere() {
		
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 547, 429);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		setContentPane(contentPane);
		contentPane.setLayout(null);
		
		scrollPane = new JScrollPane();
		scrollPane.setBounds(42, 28, 444, 221);
		contentPane.add(scrollPane);
		
		textArea = new JTextArea();
		textArea.setWrapStyleWord(true);
		textArea.setLineWrap(true);
		scrollPane.setViewportView(textArea);
		
		 SchereRadioButton = new JRadioButton("Schere");
		SchereRadioButton.setBounds(42, 256, 109, 23);
		contentPane.add(SchereRadioButton);
		
		 SteinRadioButton = new JRadioButton("Stein");
		SteinRadioButton.setBounds(42, 282, 109, 23);
		contentPane.add(SteinRadioButton);
		
		 PapierRadioButton = new JRadioButton("Papier");
		PapierRadioButton.setBounds(42, 308, 109, 23);
		contentPane.add(PapierRadioButton);
		
		ButtonGroup group = new ButtonGroup();
	    group.add(SchereRadioButton);
	    group.add(SteinRadioButton);
	    group.add(PapierRadioButton);
		
		JButton btnNewButton = new JButton("Los");
		btnNewButton.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				
				meinThread.start();
				
			}
		});
		btnNewButton.setBounds(42, 360, 89, 23);
		contentPane.add(btnNewButton);
		
		
	    
		
	}
}

In der Klasse hab ich nur das "extends Schere" rausgenommen.

public class SchereThread implements Runnable {

	
	public void run() {
		
		
		
		if(SchereRadioButton.isSelected() | SteinRadioButton.isSelected() | PapierRadioButton.isSelected()) {
			
			textArea.append("Das Spiel beginnt!\n");
			
	
		liste.add("Schere");
		liste.add("Stein");
		liste.add("Papier");
		int zufallsZahl = (int) (Math.random()*3); 
		computerAuswahl = liste.get(zufallsZahl);
			
			
		
		
		if(SchereRadioButton.isSelected()) {
			
			spielerAuswahl = "Schere";
			textArea.append("Spieler nimmt Schere. Der Computer nimmt "+computerAuswahl+".\n");
			
		}
		
        if(SteinRadioButton.isSelected()) {
			
        	spielerAuswahl = "Stein";
			textArea.append("Spieler nimmt Stein. Der Computer nimmt "+computerAuswahl+".\n");
			
		}
        
        if(PapierRadioButton.isSelected()) {
			
        	spielerAuswahl = "Papier";
			textArea.append("Spieler nimmt Papier. Der Computer nimmt "+computerAuswahl+".\n");
			
		}
        
        

        if(spielerAuswahl==computerAuswahl)
        {
          
        	textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Unentschieden!\n\n");

        }
        else
        {
          if( (spielerAuswahl == "Schere" && computerAuswahl=="Papier") ||
              (spielerAuswahl == "Stein" && computerAuswahl=="Schere")   ||
              (spielerAuswahl == "Papier" && computerAuswahl=="Stein"))
          {

        	  textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Spieler gewinnt das Spiel!\n\n");
          }
          else
          {
        	  textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Computer gewinnt das Spiel!\n\n");
          }
        }  
        
        
	}
		
	}
	
	
}

 

[SlaterB]

du hast den Sinn meiner Worte wohl nicht verstanden und das lässt jetzt auch ein bisschen an allgemeinen Programmierfähigkeiten zweifeln..

natürlich hast du jetzt im SchereThread Schere mit all seinen Attributen und vielleicht Methoden nicht mehr verfügbar,
natürlich ist der ganze GUI-Code nun erstmal kaputt,

aber das extends wegzunehmen war doch nur der kleinere Teil meines Vorschlags, du brauchst im Thread offensichtlich grundsätzlich immer noch den Zugriff auf Schere, deswegen sagte ich ja dass du Schere als Konstruktor-Parameter übergeben sollst
was ungesagt implizierte, dass alle GUI-Zugriffe dann auf dieses Schere-Objekt umzubiegen sind

alle Zugriffe zu ändern ist natürlich nicht so schön, es gibt noch zwei angenehmere Möglichkeiten die ich vorher nicht erwähnt habe:
1.
der Thread ruft einfach nur eine Methode von Schere auf, der ganze Code wandert wieder zurück nach Schere, steht nur nicht direkt im Listener sondern eine Methode die nebenläufig aufgerufen wird,
dann könnte die separate Thread-Klasse auch gleich ganz wegfallen, ein anonym deklarierter Thread reicht (nicht so wichtig wenn du das nicht kennst)

2.
ähnlich, aber doch bisschen anders:
Schere selber implementiert Runnable, und übernimmt die run-Methode exakt wie sie in SchereThread steht,
wird als Thread ausgeführt,
das ist möglich, so hattest du es am Anfang unbewußt quasi auch, von Vererbung über zwei Klassen hinweg abgesehen

 

[twseitex]

Hi,

nimm doch mal den Thread aus seiner Klasse und trage ihn direkt dort ein,
wo er gestartet wird,
per instanz=new Thread
{public void run()
{
... usw.
}

try
{instanz.start();}
catch (IllegalThreadStateException e)
{
.....
}


// +++++ prüfen ob Thread existiert
if(instanz!=null)
{
// +++++ Status holen
if( (instanz.isAlive())
&& (!instanz.isInterrupted())

....
}

instanz.interrupt(); // für stop

Dann hast du das sequentielle Gerüst. Wenn du dann feststellt, es geht ohne
Thread, dann ist das die bessere Wahl.

Ein Thread ist ein paralleler Programmteil, der im Prinzip autark mit eigenen
Ressourcen laufen soll. Daher muss der Thread auch wieder entfernt werden.

Wenn nicht nötig, dann nicht parallel arbeiten.

Cu

 

[Jack159]

2.
ähnlich, aber doch bisschen anders:
Schere selber implementiert Runnable, und übernimmt die run-Methode exakt wie sie in SchereThread steht,
wird als Thread ausgeführt,
das ist möglich, so hattest du es am Anfang unbewußt quasi auch, von Vererbung über zwei Klassen hinweg abgesehen

So hatte ich es auch vor, wusste nur nicht, dass man Threads auch mit nur 1 Klasse insgesamt erstellen kann. Daher die Vererbung mit der 2ten Klasse.
So funktioniert es nun erstmal wie gehabt, nur halt über den Thread bzw. der run() Methode.
Problem: Wie sage ich der run() Methode, dass sie schlafen soll? Ich habe ja kein Thread-Objekt...
Erstelle ich eins mit

Runnable threadJob = new Schere();
    Thread meinThread = new Thread(threadJob);

Gibts wieder ne Fehlermeldung. Sry aber ich bin absoluter Anfänger. Das hier ist mein eigentlich erstes "Projekt" (Andere coden sowas sicherlich in 5Min^^)

Aktueller Code (Funktioniert wie die alte Version. Ziel der verzögerten Textausgabe ist ja noch nicht erreicht)

import java.awt.BorderLayout;
import java.awt.EventQueue;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;
import javax.swing.ButtonGroup;
import javax.swing.JTextArea;
import javax.swing.JScrollBar;
import javax.swing.JScrollPane;
import javax.swing.JButton;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.util.ArrayList;

import javax.swing.ScrollPaneConstants;
import javax.swing.JRadioButton;


public class Schere extends JFrame implements Runnable 
{
 
    public JPanel contentPane;
    public JScrollPane scrollPane;
    public JTextArea textArea;
    public JRadioButton SchereRadioButton;
    public JRadioButton SteinRadioButton;
    public JRadioButton PapierRadioButton;
    public String spielerAuswahl;
    public String computerAuswahl;
    public ArrayList<String> liste = new ArrayList<String>();
    Runnable threadJob = new Schere();
    Thread meinThread = new Thread(threadJob);
    
    
    
    
    
    
 
    /**
     * Launch the application.
     */
    public static void main(String[] args) {
    	
        
    
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    Schere frame = new Schere();
                    frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }
 
    /**
     * Create the frame.
     */
    
public void run() {
        
        
        
        if(SchereRadioButton.isSelected() | SteinRadioButton.isSelected() | PapierRadioButton.isSelected()) {
            
            textArea.append("Das Spiel beginnt!\n");
            
            
    
        liste.add("Schere");
        liste.add("Stein");
        liste.add("Papier");
        int zufallsZahl = (int) (Math.random()*3); 
        computerAuswahl = liste.get(zufallsZahl);
            
            
        
        
        if(SchereRadioButton.isSelected()) {
            
            spielerAuswahl = "Schere";
            textArea.append("Spieler nimmt Schere. Der Computer nimmt "+computerAuswahl+".\n");
            
        }
        
        if(SteinRadioButton.isSelected()) {
            
            spielerAuswahl = "Stein";
            textArea.append("Spieler nimmt Stein. Der Computer nimmt "+computerAuswahl+".\n");
            
        }
        
        if(PapierRadioButton.isSelected()) {
            
            spielerAuswahl = "Papier";
            textArea.append("Spieler nimmt Papier. Der Computer nimmt "+computerAuswahl+".\n");
            
        }
        
        
 
        if(spielerAuswahl==computerAuswahl)
        {
          
            textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Unentschieden!\n\n");
 
        }
        else
        {
          if( (spielerAuswahl == "Schere" && computerAuswahl=="Papier") ||
              (spielerAuswahl == "Stein" && computerAuswahl=="Schere")   ||
              (spielerAuswahl == "Papier" && computerAuswahl=="Stein"))
          {
 
              textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Spieler gewinnt das Spiel!\n\n");
          }
          else
          {
              textArea.append(spielerAuswahl + " gegen " + computerAuswahl+"! Computer gewinnt das Spiel!\n\n");
          }
        }  
        
        
    }
        
    }
    
    public Schere() {
        
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(100, 100, 547, 429);
        contentPane = new JPanel();
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        setContentPane(contentPane);
        contentPane.setLayout(null);
        
        scrollPane = new JScrollPane();
        scrollPane.setBounds(42, 28, 444, 221);
        contentPane.add(scrollPane);
        
        textArea = new JTextArea();
        textArea.setWrapStyleWord(true);
        textArea.setLineWrap(true);
        scrollPane.setViewportView(textArea);
        
         SchereRadioButton = new JRadioButton("Schere");
        SchereRadioButton.setBounds(42, 256, 109, 23);
        contentPane.add(SchereRadioButton);
        
         SteinRadioButton = new JRadioButton("Stein");
        SteinRadioButton.setBounds(42, 282, 109, 23);
        contentPane.add(SteinRadioButton);
        
         PapierRadioButton = new JRadioButton("Papier");
        PapierRadioButton.setBounds(42, 308, 109, 23);
        contentPane.add(PapierRadioButton);
        
        ButtonGroup group = new ButtonGroup();
        group.add(SchereRadioButton);
        group.add(SteinRadioButton);
        group.add(PapierRadioButton);
        
        JButton btnNewButton = new JButton("Los");
        btnNewButton.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
                
            	
                run();
                
                
                
                
            }
        });
        btnNewButton.setBounds(42, 360, 89, 23);
        contentPane.add(btnNewButton);
        
        
        
        
    }
}

 

[Volvagia]

Jetzt wirst du den Event Dispatcher blockieren, dass wird häßlich werden.
Thread (Java 2 Platform SE v1.4.2)
TimeUnit (Java 2 Platform SE 5.0)

Übrigens, schau mal wie man Strings vergleicht. :autsch:

 

[SlaterB]

keine neue Schere, das Grundprinzip sollte doch klar sein, run() direkt aufrufen geht natürlich auch nicht,
wobei ich gar nicht so genau verstehe was du darin machst,

Runnable threadJob =this;
Thread meinThread = new Thread(threadJob);

ist aber grundsäztlich möglich, da Schere nur ein Runnable ist kann es mehrfach verwendet werden, ein Runnable ist ja einfach nur irgendwas mit run() Methode und sonst nix,
wäre Schere ein Thread, dann könnte Schere nur einmal laufen, einen Thread kann man nicht wieder starten,

aber da ich langsam wirklich viel Standardwissen langweilig rezitiere antworte ich jetzt erstmal gar nicht mehr,
aus Buch lernen oder auf andere Antworten hoffen, falls nicht zufällig schon alles geklärt