Thread stoppen.

[Dit_]

Hallo zusammen!

Wie stoppe ich Thread von goButton? Bin Anfänger und habe hier im Forum nichts passendes zum Thema gefunden.
Bin für jede Hilfe dankbar
Gruss
Dit

// ///////////
			// GO BUTTON//
			// ///////////
			{
				goButton = new JButton();
				getContentPane().add(
						goButton,
						new AnchorConstraint(244, 926, 317, 813,
								AnchorConstraint.ANCHOR_REL,
								AnchorConstraint.ANCHOR_REL,
								AnchorConstraint.ANCHOR_REL,
								AnchorConstraint.ANCHOR_REL));
				goButton.setText("GO");
				goButton.setPreferredSize(new java.awt.Dimension(74, 25));
				goButton.setFocusTraversalKeysEnabled(false);
				goButton.setEnabled(false);
				goButton.setForeground(new java.awt.Color(0, 0, 0));
				goButton.setBackground(new java.awt.Color(212, 208, 200));

				goButton.addActionListener(new ActionListener() {

					@Override
					public void actionPerformed(ActionEvent e) {
						goButton.setEnabled(false);
						stopButton.setEnabled(true);
						editButton.setEnabled(false);
						resetButton.setEnabled(false);
						analyseButton.setEnabled(false);

						new Thread() {

							public void run() {
								erstelleObj();
								sucheDasWort();
							}

							/**
							 * Beginnt die suche
							 * 
							 * @param pass
							 */
							private void sucheDasWort()() {
								if (modell.sucheDasWort()) {
									wort.setText(ausgabe.getDasWort());
									editButton.setEnabled(true);
									resetButton.setEnabled(true);
									kopierenButton.setEnabled(true);
									stopButton.setEnabled(false);
									gefundenInText.setText(pass.gebeSuchZeit()
											+ "\n" + pass.gebeSuchRate());
								}
							}
						}.start();
					}
				});
			}
			// /////////////
			// STOP BUTTON//
			// /////////////
			{
				stopButton = new JButton();
				getContentPane().add(
						stopButton,
						new AnchorConstraint(336, 927, 408, 814,
								AnchorConstraint.ANCHOR_REL,
								AnchorConstraint.ANCHOR_REL,
								AnchorConstraint.ANCHOR_REL,
								AnchorConstraint.ANCHOR_REL));
				stopButton.setText("Stop");
				stopButton.setPreferredSize(new java.awt.Dimension(74, 25));
				stopButton.setEnabled(false);

				stopButton.addActionListener(new ActionListener() {

					@Override
					public void actionPerformed(ActionEvent e) {
							//Wie stoppe ich Thread von goButton?
					}
				});
			}

 

[SlaterB]

unmöglich,

du kannst aber im Thread-Objekt einen boolean einführen und den auf false setzen,
der Thread müsste von selbst ab und zu nachschauen, welchen Wert die Variable hat und dann selber aufhören

 

[lichtemo]

Erstelle den Threadnicht in dem ActionListener Objekt sondern in der gleichen Methode wie den Button. Achtung: du musst ihn als final deklarieren.
Dann kannst du im ActionListener den Thread mit .start() starten und an einer anderen Position in dieser Methode wieder beenden.

 

[Schandro]

Erstelle den Threadnicht in dem ActionListener Objekt sondern in der gleichen Methode wie den Button. Achtung: du musst ihn als final deklarieren.
Dann kannst du im ActionListener den Thread mit .start() starten und an einer anderen Position in dieser Methode wieder beenden.

Nein, der Thread muss sich selber beenden (wie SlaterB bereits geschrieben hat). Auf KEINEN Fall auf den Thread stop() aufrufen!

 

[lichtemo]

Dann bleibt wohl keine andere Lösung als die breits genannte. Ich würde diesen Thread trotdem nicht anonym im ActionListener deklarieren, sondern an einer anderen Stelle in der Klasse, so dass man von überall darauf zugreifen kann. dann auf in dem ActionListener ein Objekt davon erstellen!

 

[SlaterB]

das muss natürlich sein, stimmt,
ohne Zugriff und bekannte Klasse kann man praktisch nichts machen,

außer in der umgebenden Klasse ein Klassenattribut zu setzen, darauf könnte eine anonyme innere Klasse auch zugreifen

 

[lichtemo]

Das andere ist aber vom Code her gesehen "schöner".

 

[Dit_]

ok aber wenn ich in dem Thread

boolean abbruch;

einführe, dann kann ich ich doch diese immer noch nicht durch stopButton ändern. oder ? :shock:

 

[SlaterB]

im Moment nicht, wenn du aber eine ordentliche Klasse mit getter/setter definierst und den Thread speicherst wie von lichtemo vorgeschlagen, dann schon

 

[Dit_]

ok nicht anonym heisst

Thread t1 = new Thread {....?

wo erstelle ich die klasse vor dem Listener oder danach ?

 

[lichtemo]

Entweder als eigenständige Klasse (was am schönsten wäre) oder interne Klasse.

class Sucher extends Thread {
 
   public void run () {
        //was auch immer
   }
}
//....
new Sucher().start();

 

[Dit_]

ok folgendes problem.
Habe Extra Klasse erstellt so wie lichtemo geschrieben hat.
Dann

...
@Override
					public void actionPerformed(ActionEvent e) {
						erstelleObj();
						
						anfang = System.currentTimeMillis();
						suche = new Suche(pass);
						
						goButton.setEnabled(false);
						stopButton.setEnabled(true);
						editButton.setEnabled(false);
						resetButton.setEnabled(false);
						analyseButton.setEnabled(false);
						
						suche.start();
						
						
							ende = System.currentTimeMillis();
							wort.setText(pass.getDasWort());
							gefundenInText.setText(gebeSuchZeit() + "\n"
									+ gebeSuchRate(pass.getVersuche()));
							editButton.setEnabled(true);
							resetButton.setEnabled(true);
							kopierenButton.setEnabled(true);
							stopButton.setEnabled(false);
						...

jetzt macht das programm weiter nach

suche.start();

obwohl suche noch gar nicht abgeschlossen ist Was soll ich machen ?

P.S. Abbruch geht jetzt

 

[SlaterB]

es ist ja gerade der Sinn eines Thread, dass der Aufrufer danach weitermachen kann, beide Dinge parallel bearbeitet werden,

denkbares Vorgehen:
den Code danach in eine andere Methode, die dann der Thread am Ende seiner run-Methode aufruft,
war das nicht vorher schon so?..

 

[Dit_]

ja ok, aber die Klasse Suche kann doch nicht auf die Buttons von GUIMain klasse nicht zugreifen.

 

[SlaterB]

geht nicht gibts nicht, übergib das GUI-Objekt an die Thread-Klasse, im Konstruktor oder per setter
und am Ende der run-Methode rufe
gui.sucheBeendetStelleFolgendeErgebnisseDar(..);
auf,

in der GUI-Klasse muss entsprechende Methode vorhanden sein

 

[Dit_]

ok läuft jetzt danke euch allen