Variable wird nicht ausgelesen?

[zodiac]

Hallo,
ich hab eine Klasse mit einem Interface Runnable, wobei die run() methode schematisch so aussieht:

public void run() {
		while (true){
			if (tableManager.getAction()){
				Mach was!
			}
		}
	}
}

tableManager ist eine Referenz auf ein anderes Objekt und getAction() gibt einen Wahrheitswert zurück. Parallel dazu kann durch Benutzereingaben der Wahrheitswert geändert werden innerhalb eines event threads. Wenn der geändert wurde, soll die run() methode was machen.

Nun liest er aber den Wert gar nicht aus!!! Erst wenn ich eine ausgabe hinzufüge, dann funktioniert es.

public void run() {
		while (true){
System.out.println(tableManager.getAction());
			if (tableManager.getAction()){
				Mach was!
			}
		}
	}
}

Wie kann ich das umgehen? Sodass er die variable automatisch echt lädt.

Grüße, Zodiac

 

[faetzminator]

Das sollte an sich eigentlich schon funktionieren, sofern du natürlich den Thread mit [c]start()[/c] und nicht mit [c]run()[/c] gestartet hast.
Allerdings macht man das normalerweise mit Listeners, so dass bei einer Änderung einer bestimmten Sache (in deinem Fall der boolean) alle Interessenten aufgerufen werden.

 

[andiv]

Es könnte auch daran liegen, dass nicht ausreichend auf Threadsicherheit geachtet wurde. Wenn die Variable auf die zugegriffen wird nicht mindestens volatile ist oder anderweitig happens-before sichergestellt wurde, dann gibt es keine Garantie, dass der Thread die Änderung der Variable aus dem anderen Thread mitbekommt.

 

[bygones]

nimm faetz hinweis...

es ist resourcenmäßig unsinnig ständig nachzufragen "haste schon, haste schon, haste schon".

 

[zodiac]

Den thread habe ich natürlich mit start() gestartet.

Kannst du mir einen CodeBeispiel zu dem Listener geben? Das Problem ist, dass der andere event Thread, der den Boolean ändert, dort auch terminieren muss, damit ich etwas aktualisieren kann.

Vielleicht ist ja auch meine gesamte Herangehensweise falsch, ich schreib nochmal kurz was ich machen will:

Also ich möchte ein pokerspiel programmieren, dazu gibt es die klassen Table, TableManager, TableLayout. Der Tisch besitzt alle notwendigen Variablen, der TableManager steuert das Spielgeschehen und TableLayout die Optik.

Der TableManager reagiert auf alle Eingaben (ActionListener). Nun möchte ich bevor die nächsten Karten gegeben werden, eine kurze Pause machen. Also ist der Ablauf wie folgt:
Eingabe -> TableManager führt was aus -> Jetzt muss die nächste Karte ausgeteilt werden -> Pause -> nächste Karte.

Diese Pause bekomme ich zur Zeit nur hin, indem ich 2 Threads parallel laufen lasse. Einmal dieses run(), was, nach einer Eingabe, 2sekunden pause macht, und dann das weitere Kartengeben einleitet.

public void run() {
		while (true){
			System.out.println(tableManager.getAction());
			if (tableManager.getAction()){
				tableManager.setAction(false);
				try {
					Thread.sleep(2000);
				} catch (InterruptedException e) {
					// TODO Auto-generated catch block
					e.printStackTrace();
				}
				tableManager.run(tableManager.swap);	// Dealt die Karte
			}
		}
	}

 

[Michael...]

Ein Pokerspiel ist ja ersteinmal etwas rein sequenzielles - außer z.B. der Aktionszeitraum der Spieler ist begrenzt und man will einen Timer mitlaufen lassen um dann automatisch zu passen oder zu checken. So was scheint ja in Deinem Fall (noch nicht notwendig), daher benötigst Du auch eigentlich auch keinen zweiten Thread nur um das austeilen der Karten zu verzögern (warum auch immer das verzögert werden soll). Grundsätzlich kann man an jeder Stelle des Programmablaufs den aktuellen Thread per Thread.sleep(...) pausieren lassen.

 

[zodiac]

Ja okay.
Dann muss ich die Frage vielleicht noch etwas umformulieren. Also der Ablauf ist ja eigentlich
--> Einsätze bis irgendwann -> setzrunde vorbei -> nächste community card.

So mein TableLayout aktualisert parallel dazu immer die Einsätze, die Potgröße, etc.

Wenn ich nun in dem event thread eine Pause mittels Thread.sleep() einfüge, dann findet diese Aktualisierung nicht mehr statt. Die Gründe sind mir nicht ganz klar, ich vermute das Neuzeichnen erfolgt immer erst, wenn eine Methode komplett abgearbeitet wurde. Wenn diese also ausgesetzt wird, dann passiert also erst gar nichts und dann liegen auf einmal die nächsten karten da. Sprich: Thread.sleep() kann nicht an eine ganz bestimmte Stelle gesetzt werden, weil das neuzeichnen immer erst am ende erfolgt.

 

[faetzminator]

Kannst du mir einen CodeBeispiel zu dem Listener geben? Das Problem ist, dass der andere event Thread, der den Boolean ändert, dort auch terminieren muss, damit ich etwas aktualisieren kann.

Hier hätten wir ein kleines Beispiel:

import java.util.ArrayList;
import java.util.List;

public class Test {

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

    static interface Listener {
        public void act();
    }

    static class Job {
        List<Listener> listener = new ArrayList<Listener>();

        public boolean addListener(Listener l) {
            return listener.add(l);
        }

        public boolean removeListener(Listener l) {
            return listener.remove(l);
        }

        public void start() {
            // hier der job
            // ...
            for (Listener l : listener) {
                l.act();
            }
        }
    }

    static class JobCaller implements Listener {
        public JobCaller() {
            Job job = new Job();
            job.addListener(this);
            job.start();
        }

        @Override
        public void act() {
            System.out.println("called");
        }
    }
}