threads

[Bastian]

folgende situation:

Sudoku.java

public class Sudoku extends Thread {

	
	public void init() {

		// Vorbelegung aller Felder mit Nullen

		// Markiere für alle Felder alle ZufallsZahlen als "nicht versucht"

	}
	
	public void fill() {
	
			// Suche das erste Feld mit Wert Null
	

			// Fertig, falls kein freies Feld mehr gefunden wurde

	
			// Suche für dieses Feld eine Zufallszahl, die noch nicht versucht
			// wurde
			// und die keine doppelten Einträge erzeugt
	


			// Falls passende ZufallsZahl gefunden wurde, schreibe diese in das
			// Feld
			// Falls nicht, backtrack
	


	}

	public boolean doppelt(int row, int col, int ZufallsZahl) {

		// Doppelte Einträge in Reihe?
	 
		// Doppelte Einträge in Spalte?
 
		// Bestimme linke obere Ecke der Region
 
		// Doppelte Einträge in Region?
 
	}

	public boolean zahlenUebrig(int row, int col) {

	}

	public void backtrack(int row, int col) {

		// Markiere für das aktuelle Feld alle Zufallszahlen als "nicht
		// versucht"


		// Setze das vorige Feld auf Null zurück

	}

	public int[][] getSudoku() {
		return Sudoku;
	}
}

Main.java:

[...]
	public void fuelleSpielfeld(int level) {
	
		sudoku.init();
		sudoku.fill();
               
               //fülle alle Zahlen in alle Felder
	}

so, wie programmiere ich das ganze in threads??
ich habe gedacht, ich lasse Sudoku.java von Thread erben, schreibe alles was in fill() steht (der code in der fill()-methode kann unter umständen sehr lange zum ausführen brauchen) in die run()-methode und rufe dann in Main.java

Sudoku sudoku = new Sudoku();
sudoku.init()
sudoku.start();

auf. geht aber nicht. die ganze gui (swing) bleibt hängen, also funktioniert irgendwas mit dem thread nicht richtig...

 

[Redfrettchen]

Hi,
versuch mal in deiner Schleife (ich vermute mal, dass in run() sowas wie ne schleife steht) ab und an Thread.sleep(long) bzw. Thread.yield() aufzurufen.
Aber an sich sollte Sudoku-Lösen (oder ist es Erstellen?) nicht so lange dauern und extensiv viel Prozessorleistung/RAM beanspruchen.
Wenns nicht hilft, kannst du ja mal den kompletten Code (oder die relevanten Stellen: run(), init()) posten.

 

[SlaterB]

selbst bei Dauerbelastung sollte die GUI nicht weiter geschädigt sein,
wenn die GUI nicht reagiert, dann ist das meist ein Zeichen dafür dass im Event-Thread der GUI irgendwas berechnet wird,
also z.B. von einem Button aus die fill()-Operation gerufen wird statt sie korrekt in einem Thread zu starten,

hier mal ein Beispiel für einen durchgängig beschäftigten Thread:

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

public class TestGUI extends JFrame {

	JTextArea a = new JTextArea(10, 10);

	public TestGUI() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setSize(400, 400);

		getContentPane().add(a, BorderLayout.CENTER);
		new Sudoku().start();
	}

	class Sudoku extends Thread {

		public void run() {
			while (true) {
				int j = 0;
				for (int i = 0; i <= 200000000; i++) {
					j++;
				}
				a.append("\neine Runde fertig");
			}
		}
	}
	
	public static void main(String[] args) {
		new TestGUI().setVisible(true);
	}
}

 

[Bastian]

hm, ich glaube ihc habs, allerdings gibt es ein neues prob.

wie kann ich in Main.java die getSudoku()-methode erst aufrufen, wenn der thread bereits abgelaufen ist?

 

[Leroy42]

Indem du dich mit join() an ihn bindest. In einer GUI-Anwendung würde ich
das aber nicht unbedingt so machen, weil du damit alles lahmlegst.

Wäre es nicht besser, daß der Sudoko-Erzeugungsthread am Ende selbst eine Methode
aufruft, die dein Spiel dann startet?

 

[Bastian]

ja, ich habe es jetzt im prinzip so gemacht, wie du gesagt hast:

import javax.swing.JTextField;

public class LoesungThread extends Thread {
	Loesung l1 = new Loesung();
	int[][] problem;
	JTextField[][] fields = new JTextField[9][9];
	
	public void setFields(JTextField[][] fields){
		this.fields = fields;
	}
	
	public void setProblem(int[][] problem){
		this.problem = problem;
	}
	
	public void run(){
		l1.init(problem);
		l1.fill();
		for (int x = 0; x <= 8; x++) {
			for (int y = 0; y <= 8; y++) {
				fields[x][y].setText(Integer.toString(l1.getSudoku()[x][y]));
			}

		}
	}
}

allerdings lässt dieser thread sich immer nur einmal aufrufen, beim zweiten mal gibts ne IllegalThreadStateException...

 

[SlaterB]

warum bloß immer so kompliziert und voller Halbwahrheiten?..

wenn du nicht weißt wie man mit Threads umgeht, dann versuche dieses separate Thema doch erstmal in einfachen Testprogrammen zu lösen?
wozu da 81 Textfelder, reicht denn nicht ein einzelnes?
glaubst du IllegalThreadStateException hat irgendwas mit der Anzahl der Textfelder oder Soduku zu tun?
das ganze drumherum macht es nur schwerer,

und ist wahrscheinlich auch ein Grund warum du immer nur Ausschnitte postest,
IllegalThreadStateException, wo tritt die denn auf?
wieso den Thread mehrmals starten? wer startet den, auf welches Ereignis hin? mit welchen 1-3 Codezeilen?
das ist alles wichtig


von mir wiederum ein Beispiel in dem alles problemlos läuft..,
(am Ende der run-Operation wird ein neuer Thread gestartet)

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

public class TestGUI extends JFrame {

	JTextArea a = new JTextArea(10, 10);

	public TestGUI() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setSize(400, 400);

		getContentPane().add(a, BorderLayout.CENTER);
		LoesungThread l = new LoesungThread();
		l.setArea(a);
		l.start();

	}

	public class LoesungThread extends Thread {

		JTextArea area;

		public void setArea(JTextArea newArea) {
			this.area = newArea;
		}

		public void run() {
			area.append("\nNew Problem");
			try {
				// 3 Sekunden warten
				for (int i = 0; i < 10; i++) {
					Thread.sleep(300);
					area.append(".");
				}
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
			area.append("\nProblem solved");




			LoesungThread l = new LoesungThread();
			l.setArea(a);
			l.start();
		}
	}

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

 

[Bastian]

*teil gelöscht*
antworten, bitte immer noch auf die fragen oben beziehen

 

[SlaterB]

also ich habe nicht behauptet dass du deinen ganzen Code posten sollst ,
und auf 'in mein Programm bitte folgendes einbauen' mag ich dann auch nicht mehr reagieren

aber vielleicht findet sich jemand anders, viel Glück