Observer reagieren beim 2ten mal nicht

[babuschka]

Hallo,

ich hab ein kleines Problem mit dem Observer-Pattern.

Undzwar funktioniert das Observer-Pattern schon, aber nur beim ersten mal :-(
Sobald es ein zweites mal ausgeführt wird, reagieren die Observers (2 Stück) nicht mehr.

1ter Ablauf (richtig):

setChanged->if(hasChanged)->notifyObservers
Observer 1 fürht Code in update aus
Observer 2 macht nix in update
Observer 1 ändert Observable
setChanged->if(hasChanged)->notifyObservers
Observer 1 macht nix in update
Observer 2 fürht Code in update aus

Und das ganze nochmal sobald ein Button geklickt wird.
Jetzt macht er diesen Ablauf nur beim ersten mal. Beim zweiten mal nur soweit:

2ter Ablauf (falsch):

setChanged->if(hasChanged)->notifyObservers

Versteh echt ned warum keine der updateMethoden der Observers "erreicht" wird :-(
Nicht mal ein System.out.println("bla"); am anfang einer update wird ausgegeben...
Fast so als wenn das "Signal" verloren geht...

Wisst ihr weiter?

 

[Kim Stebel]

code posten

 

[babuschka]

Hallo,

hab jetzt hier nur die beteiligten methoden drin:

Im Observable

...
public void setCode(int code) 
{
        setChanged();
        Code = code;
        updateObservers();
}
...
public void updateObservers()
{
    if(Controller.updateDB==true)
    {
	Controller.updateDB=false;
	System.out.println("Gehe nun in update View");
    }
    else if(Controller.updateDB==false)
    {
	Controller.updateDB=true;
	System.out.println("Gehe nun in update DB");
    }
				
    if(hasChanged())
    {
                System.out.println("BLAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA");
                notifyObservers(DTCCode);
    }    
}
....

In Observer 1

...
public void update( Observable ob,Object arg)
{
        System.out.println("UpdateMehtode-DB");
        if(Controller.updateDB==true)
        {	
	xxx
        }
}
...

In Observer 2

...
public void update( Observable ob,Object arg)
{
        System.out.println("UpdateMethode-View");
        if(Controller.updateDB==false)
        {
            xxx
        }
}
...

Die bool wird richtig gesetzt, da bin ich mir sicher. Nur wenn das Pattern zum 2ten mal ausgeführt werden soll, kommt auch noch die anzeige Blaaaaa...., aber keine der updates wird erreicht.
Konsolenausgabe:

Gehe nun in update DB
BLAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA
UpdateMehtode-DB
Gehe nun in update View
UpdateMethode-View
->Repaint
Gehe nun in update DB
BLAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA

 

[Kim Stebel]

guck dir die boolean-variablen mit nem debugger an...
und ==true ist überflüssig...
und statt ==false benutze !

 

[babuschka]

mhhh....aber nach den notifyObservers werden die update-Methoden der Observers aufgerufen....
in denen mach ich erstmal ein System.out.println() und danach überprüf ich auf die bool

da müsst er doch auch was auf der konsole ausgeben und ned einfach nach "BLAAA..." aufhören oder nicht?

 

[Kim Stebel]

anscheinend werden sie eben nicht aufgerufen...was macht notifyObservers? den code davon hast du weggelassen

 

[babuschka]

also meine klasse implementiert observable...
in der java-api steht dazu folgendes

notifyObservers():

notifyObservers

public void notifyObservers()If this object has changed, 
as indicated by the hasChanged method, then notify all 
of its observers and then call the clearChanged method 
to indicate that this object has no longer changed. 
Each observer has its update method called with two arguments: 
this observable object and null. In other words, this method is equivalent to: 

notifyObservers(null)

 

[Kim Stebel]

implementiert? du meinst erweitert...
implementieren kann man nur interfaces...
der fehler wird wohl in einem teil des codes liegen, den du noch nicht gepostet hast. bist du mal mit nem debugger durchgegangen? dieses pseudo-debugging mit system.out.println ist völlig unnötig und umständlich.

 

[babuschka]

sry, mein fehler


Observable wird erweitert...
Observer wird implementiert...das hab ich soweit auch getan...man muss in dieser nur die updateMethode implementieren.


www.galileocomputing.de/openbook/javainsel6/javainsel_08_006.htm#Xxx1000606

http://java.sun.com/j2se/1.5.0/docs/api/java/util/Observer.html


EDIT:
Beim Debugging gibt er auch "UpdateMehtode-DB " beim 2ten Durchgang aus
Ka worans liegen könnt :-(

 

[Kim Stebel]

willst du mich veralbern oder so? anstatt auf meine Frage zu antworten willst _du_ _mir_ was über design patterns beibringen? keine MVC-anwendung kommt ohne observer aus...

 

[babuschka]

ich hab doch nen observer drin ^^

meineKlasse implements Observer

guck dir halt das bsp im link an....so ist mein aufbau des mvc auch

was gibts da ned zu verstehen?

 

[byte]

willst du mich veralbern oder so? anstatt auf meine Frage zu antworten willst _du_ _mir_ was über design patterns beibringen? keine MVC-anwendung kommt ohne observer aus...

Die Frage von ihm bezog sich wohl eher auf die Observer/Observable Implementierung der Java API.

Denn das hier lässt schon schließen, als wenn Du das noch nicht benutzt hast:

...was macht notifyObservers? den code davon hast du weggelassen

Also kein Grund, so überheblich zu reagieren. ???:L

 

[babuschka]

danke byto 🙂

hab nomal den debugger benutzt....hab das problem gefunden, doch ne lösung kenn ich dazu ned

public void updateObservers() 
{ 
    if(Controller.updateDB==true) 
    { 
   Controller.updateDB=false; 
   System.out.println("Gehe nun in update View"); 
    } 
    else if(Controller.updateDB==false) 
    { 
   Controller.updateDB=true; 
   System.out.println("Gehe nun in update DB"); 
    } 
             
    if(hasChanged()) 
    { 
                System.out.println("BLAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA"); 
                notifyObservers(DTCCode); 
    }    
}

beim ersten durchgang läufts normal und beim 2ten geht er in die if(hasChanged()) rein, gibt blaaa auch noch aus ABER dann geht er auf notifyObservers und macht gar nix mehr danach ^^ die updateMethoden werden nicht aufgerufen, deshalb macht er nix mehr

kann ma ihn dazu zwingen die updates aufzurufen?
bin langsam echt am verzweifeln ^^

 

[Kim Stebel]

Denn das hier lässt schon schließen, als wenn Du das noch nicht benutzt hast:

dazu hätte einem ja mal gesagt werden müssen, dass die klasse von observable erbt und er nicht sein eigenes süppchen kocht...

 

[babuschka]

Hallo,
ich hab ein kleines Problem mit dem Observer-Pattern.

ich dachte mit der aussage Pattern is es klar, dass ich ein offizielles design pattern meinte
aber jetzt bitte ned vom thema abkommen ;-)

 

[Kim Stebel]

du sagst nur was über das pattern...nicht über die implementierung

 

[Murray]

beim ersten durchgang läufts normal und beim 2ten geht er in die if(hasChanged()) rein, gibt blaaa auch noch aus ABER dann geht er auf notifyObservers und macht gar nix mehr danach ^^ die updateMethoden werden nicht aufgerufen, deshalb macht er nix mehr

Wenn du beim Debuggen bei notifyObservers stehst - machst du dann "step into"? Kommt er aus dem Methodenaufruf wieder zurück?
Hast du mal Breakpoints auf die beiden Update-Methoden gesetzt?
Kann es sein, dass sich die Observer nach dem ersten Durchgang deregistiert haben? Lass dir doch vor dem Aufruf von notifyObservers() mal das Ergebnis von countObservers() ausgeben.

 

[byte]

Oder wird vielleicht irgendwo zwischen setChanged() und notifyObservers() ein clearChanged() aufgerufen?

 

[maki]

Eine gutes Loggingsystem erspart viel Debugging und Verwunderung.

Meine 0.02 €

 

[babuschka]

Oder wird vielleicht irgendwo zwischen setChanged() und notifyObservers() ein clearChanged() aufgerufen?

nö, das is ned der fall

Kann es sein, dass sich die Observer nach dem ersten Durchgang deregistiert haben? Lass dir doch vor dem Aufruf von notifyObservers() mal das Ergebnis von countObservers() ausgeben.

wow :roll: countObservers ergab vor notifyObservers folgenedes:
1ter durchgang 2
2ter durchgang 0

es sind tatsächlich keine observers mehr angemeldet....trennt er die automatisch, da ich kein deleteObservers ausführe?

 

[babuschka]

hab bisher imma noch ned rausgefunden warum die Observers gelöscht werden :-(

 

[maki]

hab bisher imma noch ned rausgefunden warum die Observers gelöscht werden :-(

Mein Beileid...

Eine gutes Loggingsystem erspart viel Debugging und Verwunderung.

In meinen Anwendungen wird in fast jeder Methode mindestens einmal geloggt, meist benutze ich log4j.

 

[Murray]

Hast du es schon mit Breakpoints in deleteObservers() und deleteObserver( Observer) versucht?

 

[babuschka]

Problem gelöst 🙂

es lag daran, dass ich im code einmal das bestehende observable-objekt neu zuweise ( observable = new observable)
und dann hatte es die observer gelöscht.