Probleme bei Rückgabewert

[ulr!ch]

Hi JavaGemeinde,

ich habe eine externe Klasse namens Trial, die ungefähr (reduziert) so aus sieht:

import javax.swing.*;

public class Trial
    extends Thread {
  int sound_time, trial_case;
// [...] 

  public Trial(/*[... Importe ...]*/) {
// Übergabe der Importe
  }

  public void run() {
    Thread thread_trial = new Thread(this);

// verschiedene Berechnungen, Bestimmen des trial_case über if-Anweisungen aus den Importen
switch (trial_case) {
      case 1:
	sound_time = 3500;
	break;
      case 2:
	sound_time = 4180;
	break;
      case 3:
	sound_time = 3656;
	break;
      case 4:
	sound_time = 4164;
	break;
    }

// Thread beendet sich selbst
    thread_trial.interrupted();
  }

// Die Methode trialDauer gibt die Dauer der akustischen Rückmeldung wieder
  public int trialDauer() {
    return(sound_time);
  }
}

Wieso bekomme ich mit dem Methodenaufruf:

int abc = new Trial(/* ...Importe... */).trialDauer();

keinen Rückgabewert bzw. ist dieser Null?
Womit hat das was zu tun:
a) mit den if-Anweisungen?
b) mit dem Thread?

Steh' auf dem Schlauch und bin für Lösungsvorschläge wirklich dankbar!

By<e Ulrich

 

[bambi]

Das sollte aber was zurueckgeben... Was passiert denn bei Dir? Exception...

 

[bambi]

Hab' gerade gesehen, dass Du in Deinem switch-case gar kein default hast. Das wuerde ich an Deiner Stelle aber schon
einbauen - gerade eben um Fehler zu finden ist das ganz nuetzlich... :wink:

 

[Wildcard]

Das sind KEINE Importe sonder Parameter :shock:

  public Trial(/*[... Importe ...]*/) { 
// Übergabe der Importe 
  }

Wenn dein Konstruktor wirklich so aussieht, dann wird der Thread nie gestartet, insofern ist trialDauer eben noch auf 0.
Das bringt sowieso nichts, da ein Thread eben nebenlaufig ist, diese direkte Wertabfrage also nur zufällig funktionieren würde.
Vermeide besser '_' das ist schlechter Stil

 

[ulr!ch]

Hab' gerade gesehen, dass Du in Deinem switch-case gar kein default hast. Das wuerde ich an Deiner Stelle aber schon einbauen - gerade eben um Fehler zu finden ist das ganz nuetzlich... :wink:

Steht auch drin, habe ich bloß nicht rein geschrieben.
Nachtrag:

default:
System.err.println( "Was soll hier passieren?" );

Zur anderen Frage: Er gibt halt '0' aus.
So als wenn er die Anweisungen sound_time = 123 nicht liest.

By<e Ulrich

 

[ulr!ch]

Wenn dein Konstruktor wirklich so aussieht, dann wird der Thread nie gestartet, insofern ist trialDauer eben noch auf 0. Das bringt sowieso nichts, da ein Thread eben nebenlaufig ist, diese direkte Wertabfrage also nur zufällig funktionieren würde. Vermeide besser '_' das ist schlechter Stil

a) Wieso funktioniert folgendes nicht?

Trial trial_abc = new Trial(/* Parameter */);
trial_abc.start();
int abc = trial_abc.trialDauer();
System.out.println(abc);

b) Den zweiten Einwand verstehe ich nicht. Meinst du, zu dem Zeitpunkt, an dem ich die Abfrage starte, befindet sich der Thread noch in Bearbeitung und ist zu den Anweisungen sound_time = 123 noch gar nicht vorgedrungen?
c) Wieso_ist_das_schlechter_Stil? Kann das zu Problem führen?

By<e Ulrich

 

[Wildcard]

Den zweiten Einwand verstehe ich nicht. Meinst du, zu dem Zeitpunkt, an dem ich die Abfrage starte, befindet sich der Thread noch in Bearbeitung und ist zu den Anweisungen sound_time = 123 noch gar nicht vorgedrungen?

Zunächst mal steht hier nirgends sound_time = 123; was soll ich dazu also sagen?

c) Wieso_ist_das_schlechter_Stil? Kann das zu Problem führen?

Ja! Mit Menschen die was gegen schlechten Stil haben :wink:

 

[ulr!ch]

Zunächst mal steht hier nirgends sound_time = 123; was soll ich dazu also sagen?
Ja! Mit Menschen die was gegen schlechten Stil haben :wink:

a) sound_time wird in der switch/ case-Anweisung festgelegt
b) Das ist ein schlagendes Argument, gegen das ich mich natürlich mit Vergnügen auflehne. 8)

By<e Ulrich

 

[Wildcard]

Dann weißt du ja jetzt warum's nicht funktioniert!?

 

[ulr!ch]

Ja, gut,
das es zufällig - je nach dem wo im Code aufgerufen - einen Rückgaebwert liefert, habe ich mittlerweile eingesehen.
Gibt es eine andere Möglichkeit? Ohne eine neue Klasse oder das aufsplitten in verschiedene? Oder ist der Tenor - kurzum - Threadklassen liefern keine Rückgaebwerte?

By<e Ulrich

 

[ulr!ch]

Dann weißt du ja jetzt warum's nicht funktioniert!?

:bahnhof: Warum würde ich hier schreiben, wenn ich's wüsste? Um euch auf die Probe zu stellen?...
Lieber Wildcard: Meinst du vielleicht, dass es an der switch-Anweisung liegt? :?

By<e Ulrich

 

[bambi]

c) Wieso_ist_das_schlechter_Stil? Kann das zu Problem führen?

Ja! Mit Menschen die was gegen schlechten Stil haben :wink:

...und mit Menschen, die die Java-Konvetion liebgewonnen haben :wink:

 

[Wildcard]

int abc = new Trial(/* ...Importe... */).trialDauer();

Zu diesem Zeitpunkt ist nur sichergestellt das der Konstruktor beendet wurde.
Wenn du hier einen Wert zuweist müsste das auch funktionieren.
Falls du den Thread erst in der anderen Klasse startest ist es also nicht zufällig, sondern absolut sicher das du keinen Wert erhälst.

Gibt es eine andere Möglichkeit? Ohne eine neue Klasse oder das aufsplitten in verschiedene? Oder ist der Tenor - kurzum - Threadklassen liefern keine Rückgaebwerte?

Kommt immer drauf an was man haben möchte.

 

[ulr!ch]

Falls du den Thread erst in der anderen Klasse startest ist es also nicht zufällig, sondern absolut sicher das du keinen Wert erhälst.

So? Dann erkläre mir mal bitte das:

Trial trial_abc = new Trial(/* Parameter */);
trial_abc.start();
System.out.println(trial_abc.trialDauer());
  try {
    thread_xyz.sleep(trial_abc.trialDauer());
  }
  catch (InterruptedException ie) {
    System.out.println("IE im thread_abc");
  }

liefert 0, wohingegen:

Trial trial_abc = new Trial(/* Parameter */);
trial_abc.start();
  try {
    thread_xyz.sleep(trial_abc.trialDauer());
    System.out.println(trial_abc.trialDauer());
  }
  catch (InterruptedException ie) {
    System.out.println("IE im thread_abc");
  }

den korrekten Wert von 3656 zurückliefert.

By<e Ulrich

 

[Wildcard]

So? Dann erkläre mir mal bitte das:

Das ist ja auch etwas völlig anderes!
Wenn du hier einen Wert verlangst, und den Thread erst später startest ist kja wohl klar das da noch kein Wert ist

int abc = new Trial(/* ...Importe... */).trialDauer();

In deinem 2.Beispiel kann es funktionieren, muss aber nicht. So einfach ist das :wink:

 

[Wildcard]

[edit]SRY! Einmal hätte gereicht[/edit]

 

[ulr!ch]

Wenn du hier einen Wert verlangst, und den Thread erst später startest ist kja wohl klar das da noch kein Wert ist

mhm... ? Also in beiden Beispielen wird zuerst der Thread gestartet. Und zweitens - falls das deine Vermutung sein sollte - ist dieser Thread auch im zweiten Bsp. noch nicht durchgelaufen.
Also, wieso? Ich verstehe dich einfach nicht. Wirklich, ist ja nicht böse gemeint.

By<e Ulrich

 

[Wildcard]

Wenn du hier einen Wert verlangst, und den Thread erst später startest ist kja wohl klar das da noch kein Wert ist

mhm... ? Also in beiden Beispielen wird zuerst der Thread gestartet. Und zweitens - falls das deine Vermutung sein sollte - ist dieser Thread auch im zweiten Bsp. noch nicht durchgelaufen.
Also, wieso? Ich verstehe dich einfach nicht. Wirklich, ist ja nicht böse gemeint.

Das bezieht sich auf den Anfang dieses Topics (wollte eigentlich Threads schreiben, aber das führt jetzt wohl ins Chaos )!
Wenn du diesen Aufruf machst (und der Thread sich nicht selbst im Konstruktor startet sondern von deiner Klasse gestartet wird)

int abc = new Trial(/* ...Importe... */).trialDauer();

Hat der Thread die run Methode noch gar nicht betreten wenn trialDauer() (sollte übrigens getTrialDauer() heißen :wink: ) aufgerufen wird. Folglich hast du deiner Variablen noch keinen Wert zugewiesen, und folglich bekommst du auch 0 zurück. Es wird lediglich der Konstruktor ausgeführt!
Bei deinen beiden späteren Beispielen rufst du trialDauer() erst nach dem Thread-start auf. Folglich kann man nicht vorhersagen ob der Thread oder deine Hauptklasse schneller war, also bekommst du entweder ein Ergebnis, oder keins, oder mit viel Pech einen Osterhasen zurück. Kann man eben nicht wissen

 

[ulr!ch]

Ok, danke, dass habe ich nun verstanden. Yeah!

Aber... rein praktisch gesehen... bin ich genau so schlau wie vorher... oder doch fast.

Wenn du diesen Aufruf machst (und der Thread sich nicht selbst im Konstruktor startet sondern von deiner Klasse gestartet wird)

:? Ist das nun die Lösung, dass der Thread im Konstruktor gestartet wird?
Wenn ja, wie sieht das aus, davon habe ich noch nichts gehört.

Dank vorab,
Ulrich

 

[Wildcard]

Ist das nun die Lösung, dass der Thread im Konstruktor gestartet wird?

Auch dann lässt sich das Ergebniss nicht genau vorherbestimmen.
Kannst du mir mal beschreiben was du gerne machen möchtest, dann wird sich schon eine Lösung finden.

 

[ulr!ch]

Kannst du mir mal beschreiben was du gerne machen möchtest, dann wird sich schon eine Lösung finden.

Also diese Trial-Klasse spielt einen Sound-File der Länge sound_time ab.
In dieser Zeit soll die aufrufende Klasse schlafen.
Ich möchte (aus anderen wichtigen Gründen) nicht die beiden Klassen zusammenlegen.
Das ist eigentlich schon alles, nur das ich nicht bestimmen kann wie lange sound_time/ sleepy-time anhält.
Eine if-Anweisungvorzuschalten sieht ziemlich bescheuert aus und ich dachte ich könnte mir die Länge ausgeben lassen... Hast du vielleicht noch eine andere Ideee?

By<e Ulrich

 

[Wildcard]

Dann musst du mit wait() und notify() arbeiten.
Am besten du liest dir das mal durch :wink:

 

[ulr!ch]

Dann musst du mit wait() und notify() arbeiten.
Am besten du liest dir das mal durch :wink:

Super, vielen Dank, das war es, was ich wissen wollte.
Werde ich mir mal zu Gemüte führen...

By<e Ulrich

 

[ulr!ch]

Also JavaGemeinde,

ich gehöre gesteinigt und aus der Welt der Hobby-Programmierer ausgeschlossen, aber ich bekomme das mit dem wait(); und notify(); nicht hin.
Schaut euch doch mal bitte das an und sagt mir, was ich verkehrt mache:

public class Trial
    extends Thread {
  int sound_time, trial_case;
  public Trial(/* Parameter */) {
// Übergabe der Parameter
  }
  
  public void run() {
    Thread thread_trial = new Thread(this);
// [...]

    try {
      thread_trial.sleep(sound_time);
    }
    catch (InterruptedException ie) {
      System.out.println("IE in thread_trial");
    }

    try {
      fertig();
    }
    catch (InterruptedException ie) {}

// Thread beendet sich selbst
    thread_trial.interrupted();
  }

  public synchronized void fertig()
      throws InterruptedException
  {
    notifyAll();    
  }
}

Das sieht in IMHO schon ziemlich umständlich aus... :?

Die aufrufende Klasse besitzt folgenden Code

Trial trial_abc = new Trial(/* Parameter */);
trial_abc.start();
  try {
    thread_xyz.wait();
       }
  catch (InterruptedException e) {}

Muss hier nicht stehen, dass er solange warten soll bis... ja was eigentlich... bis halt notify... oder ist das automatisch?
Ich befürchte, ich habe da irgendetwas grunsätzliches nicht verstanden.

Hilfe, bitte!

By<e Ulrich
PS: Fliegt immer mit "IllegalMonitorStateException: current thread not owner" raus.

 

[Beni]

Ich hab nicht alles gelesen, aber zu wait und notifiy:

// Irgendein Objekt, welches in beiden Threads sichtbar ist.
Object object = ...

// ein erster Thread, der warten soll
public void run(){
  synchronized( object ){
    object.wait();
  }
}

// ein zweiter Thread, der den ersten starten soll
public void run(){
  synchronized( object ){
    object.notify();
  }
}

Das synchronized wird benötigt, um den Monitor zu bekommen (andernfalls gibt es diese Exception).

 

[ulr!ch]

// Irgendein Objekt, welches in beiden Threads sichtbar ist.

Was heißt sichtbar? Muss ich es dann übergeben?
Danke für die Hilfe!

By<e Ulrich

 

[Beni]

Ich meinte mit sichtbar, dass beide Thread dasselbe Objekt (oder dieselbe Instanz, wenn dir das mehr sagt) haben.

Also ja, müsstest du irgendwo irgendwie übergeben.

 

[ulr!ch]

Aber das kann doch alles gar nicht sein? :?
Ich habe eine Klasse A, die eine externe Klasse B aufruft, solange B läuft soll A schlafen.
Das kann doch gar nicht so schwer sein. Ich bin doch bestimmt nicht der erste, der so ein Problem hat.
Ich kann doch nicht den Thread übergeben, dann kann ich direkt eine Klasse draus machen.
Kann ich nicht durch isInterrupted(); den Zustand des Threads in Klasse B abfragen oder kriege den Status wegen der Nebenläufigkeit der Threads nicht?

Danke, Ulrich

 

[ulr!ch]

Habe das Thema geändert und unter "IsInterrupted()" neu gepostet.
Irgendwie bin ich vo notify();/ wait(); und den Rückgabewerten zu genervt.
Sorry!

By<e Ulrich