Thread von außen beenden

[Yamato]

Ich möchte einen Button mit folgender Funktionalität:
Auf einen Klick wird eine Berechnung in einem eigenen Thread gestartet, auf einen weiteren Klick (noch während der Berechnung) wird der Thread beendet.
Ich hätte es ja gerne so:

class ButtonClick implements ActionListener {

	  public void actionPerformed(ActionEvent e) {

		  if (threadAktiv == false) {
			  threadAktiv = true;
			  button.setLabel("Berechnung abbrechen");
			  CalcThread calc = new CalcThread();
			  calc.start();
		  }
		  else {
			  threadAktiv = false;
			  button.setLabel("Berechnung starten");
			  calc.stop();
		  }
	  }
}

d. h. in threadAktiv merke ich mir, ob die Berechnung schon einmal gestartet wurde.
Nur kann ich calc.stop() an der entsprechenden Stelle nicht aufrufen.
Wie kann man denn aber auf das Thread-Objekt zugreifen?

Ich kann zwar den Thread vorher global deklarieren & erzeugen, aber nach dem ersten stop() ist er dann weg und ein start() geht nicht mehr.

 

[SlaterB]

irgendwo merken musst du ihn ja, da führt kein Weg dran vorbei,
aber das Erzeugen muss lokal passieren:

class ButtonClick implements ActionListener {

     CalcThread calc;

     public void actionPerformed(ActionEvent e) {

        if (threadAktiv == false) {
           threadAktiv = true;
           button.setLabel("Berechnung abbrechen");
           if (calc != null) { // edit: ok, ist wohl nicht nötig durch die threadAktiv-Variable
               calc.stop();
           }
           calc = new CalcThread();
           calc.start();
        }
        else {
           threadAktiv = false;
           button.setLabel("Berechnung starten");
           if (calc != null) {
               calc.stop();
           }
        }
     }
}

 

[Yamato]

Mit deiner Methode kann ich auf ihn zugreifen und ihn beenden - aber nur einmal. Nach einem weiteren Klick wird er nicht wieder neu erzeugt.
Vermutlich, weil die Deklaration außerhalb von actionPerformed passiert?

 

[SlaterB]

tja, was soll man dazu sagen? ich sehe absolut keine Fehlermöglichkeit, was sollte auch der Ort der Deklaration damit zu tun haben..,

bei mir zu Hause funktionierts in einem Beispiel, liegt vielleicht Code deines CalcThreads?

------------

Beispiel:
ein Thread der jede Sekunde eine Meldung in eine JTextArea schreibt:

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



public class TestGUI extends JFrame implements ActionListener {

	private static int count;
	private JTextArea ta = new JTextArea();
	private JButton button = new JButton("Berechnung starten");
	private CalcThread calc;
	private boolean threadAktiv = false;

	public TestGUI() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		button.addActionListener(this);
		JScrollPane sp = new JScrollPane(ta);

		getContentPane().add(button,BorderLayout.NORTH);
		getContentPane().add(sp,BorderLayout.CENTER);
		setSize(600, 400);
		setVisible(true);
	}

	public void actionPerformed(ActionEvent e) {
		if (threadAktiv == false) {
			threadAktiv = true;
			button.setLabel("Berechnung abbrechen");
			calc = new CalcThread();
			calc.start();
		} else {
			threadAktiv = false;
			button.setLabel("Berechnung starten");
			calc.pleaseStop();
		}
	}






	class CalcThread extends Thread {

		private int number;
		
		public CalcThread() {
			number = TestGUI.count;
			TestGUI.count++;

		}

		public void pleaseStop() {
			schreibe("beendet seine Arbeit nach PleaseStop");
			stop();
		}


		public void run() {
			schreibe("beginnt seine Arbeit");
			try {
				while (true) {
					Thread.sleep(1000);
					schreibe("arbeitet noch");
				}
			} catch (InterruptedException e) {
				e.printStackTrace();
				schreibe("beendet seine Arbeit nach InterruptedException");
				
			}
		}

		private void schreibe(String st) {
			st = "CalcThread "+number+": "+st;
			ta.append(st+"\n");
//			System.out.println(st);  // Alternative zur JTextArea
		}

	}









	public static void main(String[] args) {
		new TestGUI();
	}

}

 

[Yamato]

Ich habe mittlerweile herausgefunden, daß es am Browser liegt (ist ein Applet):

import java.applet.*;
import java.awt.*;
import java.awt.event.*;
import java.math.*;
import java.lang.Thread.*;

public class Faktorisierung extends Applet {

	Label label1, label2;
	Button button;
	TextField edit;
	boolean threadActive;

	public void init() {

		super.init();
		setSize(500, 180);
		setLayout(null);

		label1 = new Label("N =");
		label1.setBounds(25,43,25,25);
		label2 = new Label("Gefundener Faktor:");
		label2.setBounds(25,75,250,25);
		button = new Button("Start");
		button.setBounds(200,120,100,25);
		button.addActionListener(new ButtonClick());
		edit = new TextField(100);
		edit.setBounds(60,45,400,21);

		add(label1);
		add(label2);
		add(button);
		add(edit);
		show();
		threadActive = false;
	}

  class CalcThread extends Thread {

	  public void run() {

		  BigInteger N = new BigInteger(edit.getText());
		  BigInteger P = new BigInteger("1");
		  BigInteger x = new BigInteger("1");
		  BigInteger y = new BigInteger("1");
		  BigInteger a = new BigInteger("1");
		  BigInteger cmp = new BigInteger("1");
		  int s = 1;

		  label2.setText("Gefundener Faktor:");

		  while (true) {
			  P = cmp;
			  for (int i = 1; i <= s; i++) {
				  x = x.multiply(x);
				  x = x.add(a);
				  x = x.remainder(N);
				  y = y.multiply(y);
				  y = y.add(a);
				  y = y.multiply(y);
				  y = y.add(a);
				  y = y.remainder(N);
				  P = P.multiply(x.subtract(y));
			  }
			  P = P.gcd(N);
			  if (P.compareTo(cmp) > 0) {
				  break;
			  }
		  }

		  label2.setText("Gefundener Faktor: " + P.toString());
		  threadActive = false;
		  button.setLabel("Start");
	  }
  }

  class ButtonClick implements ActionListener {

		CalcThread calc;

	  	public void actionPerformed(ActionEvent e) {

			if (threadActive == false) {
			  	threadActive = true;
			  	button.setLabel("Abbrechen");
			  	calc = new CalcThread();
			  	calc.start();
		  	}
		  	else {
			  	threadActive = false;
			  	button.setLabel("Start");
			  	calc.stop();
		  	}
	  	}
  	}
}

Die Seite sieht so aus:

<html>
<head>
<title>Faktorisierung</title>
</head>
<body>
<h3><u>Faktorzerlegung einer natürlichen Zahl</u></h3>


Geben Sie eine Zahl ein (sie sollte nicht zu klein sein).</p>
<applet code="Faktorisierung.class" width=500 height=180>
</applet>
</body>
</html>

Gibt man z.B. 1000000000000000000000000007 als Zahl ein, hat der Rechner erst mal zu tun. Mit dem Internet Explorer funktioniert alles wunderbar, aber Opera kommt offensichtlich nicht an den Zeilen 72 und 73 vorbei, nachdem die Berechnung abgeschlossen wurde (wohl aber bei 71!)... eigenartig.

 

[The_S]

Ich habe mittlerweile herausgefunden, daß es am Browser liegt (ist ein Applet):

Gibt man z.B. 1000000000000000000000000007 als Zahl ein, hat der Rechner erst mal zu tun. Mit dem Internet Explorer funktioniert alles wunderbar, aber Opera kommt offensichtlich nicht an den Zeilen 72 und 73 vorbei, nachdem die Berechnung abgeschlossen wurde (wohl aber bei 71!)... eigenartig.

Kann eigentlich net sein. Sicher dass du die Seite net aus dem cache liest?

 

[Yamato]

Ich hab nach jeder Neukompilation auf "Neu laden" geklickt, was ja eigentlich ausreicht.

 

[Mag1c]

Hi,

hast du mal geguckt, ob dort evtl. eine Exception fliegt ? Das wäre jedenfalls für mich im Augenblick die einzige Erklärung, wieso er 71 ausführt aber nicht 72 und 73 (wenn es denn wirklich so ist).

EDIT: ob der neueste Code des Applets benutzt wird, kannst du im "Java Plug-in Control Panel" unter Cache -> View nachsehen.

Gruß
Mag1c

 

[SlaterB]

oder irgendwo ne Textausgabe/ Anzeige "Version xy"