Observer funktioniert nicht

[Nemesys88]

Hallo zusammen!

Habe eine Klasse A mit einem Label, das ich in Klasse D updaten möchte.
Würde das wenns geht am liebsten mit einem Observer machen, der will aber noch nicht so wie ich...

Die Klassen mit dem Observer-relevanten Code:

public class MainFrame extends JFrame implements Observer
... 
	@Override
	public void update(Observable o, Object text) {
		label1.setText((String) text);
	}

public class Data extends Observable
...
				String text = "Summe ECTS: " + berechne_summe() + "               " + "Notendurchschnitt: " +  Math.round(berechne_durchschnitt() * 100. ) / 100.;
				
				setChanged();
				notifyObservers(text);

Was fehlt hier jetzt noch ?
addObserver in der ersten Klasse?
Wie muss der aussehen ?

Danke schonmal.

 

[Flown]

Du musst schon noch deinem Observable den Observer hinzufügen.

 observable.addObserver(...);

 

[Nemesys88]

Sorry wenn ich nochmal so doof fragen muss, aber wie muss der aussehen ?

Ich hab im Konstruktor von MainFrame jetzt stehen:

Data daten = new Data();
daten.addObserver(this);

Da sich aber immernoch nichts tut, geh ich davon aus, dass das so nicht richtig sein kann... :/

 

[Flown]

So ich hab dir mal ein kleines KSKB zusammengestellt, wie was wann abläuft:

import java.util.Observable;
import java.util.Observer;

public class MyObserver implements Observer {
  
  public static void main(String[] args) {
    MyObserver observer = new MyObserver();
    MyObservable observable = new MyObservable(10);
    observable.addObserver(observer);
    
    observable.setX(12);
  }
  
  @Override
  public void update(Observable o, Object arg) {
    if (o instanceof MyObservable) {
      System.out.println("x has changed from " + arg + " to " + ((MyObservable) o).getX() + "!");
    }
  }
  
  static class MyObservable extends Observable {
    private int x;
    
    public MyObservable(int x) {
      this.x = x;
    }
    
    public void setX(int x) {
      int oldX = this.x;
      this.x = x;
      setChanged();
      notifyObservers(oldX);
    }
    
    public int getX() {
      return x;
    }
  }
}

 

[Nemesys88]

Also meiner Meinung nach hab ich jetzt alles genau so gemacht und trotzdem funktioniert es nicht...

public class Data extends Observable
...
String text = "blabla";

setChanged();
notifyObservers(text);

public class MainFrame extends JFrame 
...
Data daten = new Data();
MyObserver watcher = new MyObserver(label1);
daten.addObserver(watcher);

public class MyObserver implements Observer {
	
	private JLabel label1;

	public MyObserver(JLabel _label) {
		label1 = new JLabel();
		label1 = _label;
	}

	@Override
	public void update(Observable o, Object arg) {
		label1.setText("hallo");
	}

}

 

[Flown]

Was passiert wenn du dein Label änderst und dann die Größe des Frames veränderst? Aktualisiert sich dein Label? Wenn ja dann fehlt dir nur noch ein refresh auf dein label/frame.

 

[Nemesys88]

Nein leider nicht... :/

Mir fällt auch auf, dass er garnicht in die Update-Funktion springt!

Wenn ich da in die erste Zeile ein System.out.println("test") mache wird das niemals ausgegeben.

 

[Flown]

Kannst du mal den ganzen Code posten?

 

[Nemesys88]

import java.util.Observable;
import java.util.Observer;

import javax.swing.JLabel;

public class MyObserver implements Observer {
	
	private JLabel label1;

	public MyObserver(JLabel _label) {
		label1 = new JLabel();
		label1 = _label;
	}

	@Override
	public void update(Observable o, Object arg) {
		System.out.println("test");
		label1.setText("hallo");
	}

}

public class Data extends Observable
...
public void read() {

				String text = "Summe ECTS: " + berechne_summe() + "               " + "Notendurchschnitt: " +  Math.round(berechne_durchschnitt() * 100. ) / 100.;

				setChanged();
				notifyObservers(text);
			   }

public class MainFrame extends JFrame
MainFrame() {
...
		Data daten = new Data();
		daten.read();

		// adds Label
		label1 = new JLabel();
		label1.setText(("Summe ECTS: " + daten.berechne_summe() + "               " + "Notendurchschnitt: " +  Math.round( daten.berechne_durchschnitt() * 100. ) / 100.));
		label1.setHorizontalAlignment(SwingConstants.CENTER);
		label1.setBorder(new LineBorder(Color.BLACK));
		add(label1,BorderLayout.SOUTH);
		
		MyObserver watcher = new MyObserver(label1);
		daten.addObserver(watcher);
...

Die Observer-Klasse ist so komplett, die andern sind leider viel zu groß um sie komplett zu posten, die Data.java hat 400 Zeilen Code...
Die Data.java hat halt ne Methode read() die an vielen Stellen im Programm aufgerufen wird, halt zb. im Konstruktor von MainFrame (= Programmstart).
Im selbigen Konstruktor konstruiere ich auch das Label das ich der neuen "MyObserver"-Klasse mitgegeben habe.

 

[Flown]

Du solltest vorher schon deinen Observer setzen bevor du read aufrufst.

Und du brauchst in dem Konstruktor deines Observers kein neues JLabel erstellen, sondern nur übergeben.

 

[Nemesys88]

Ok

		MyObserver watcher = new MyObserver(label1);
		daten.addObserver(watcher);
		
		daten.read();

funktioniert dann, aber leider nur für den Programmstart...

Wenn ich aus Klasse C jetzt daten.read() aufrufe passiert aber immer noch nichts,
und ich kann auch keinen watcher definieren,
weil Klasse C das Label ja garnicht kennt...

Im Endeffekt hab ich also rein garnichts gewonnen...

Ich hab nach wie vor das Problem, das Klasse C das Label aus Klasse A (= MainFrame) nicht kennt...

edit: Ich hatte eigentlich gehofft mit dem Observer das Problem lösen zu können, das das Label in jeder Klasse bekannt sein muss, in der ich es ändern will...

 

[Flown]

Ich kann dir sagen, dass es, wenn es sauber implementiert ist auch so funktioniert. Aber dein Fehler liegt nicht in dem was du geschrieben hast, sondern in einem anderen Teil. Wenn du willst kannst du mir eine PN senden mit deinem ganzen Code und ich schau mal nach was es da haben kann.

 

[Joose]

[ot]@Nemesys88: Bitte verwende in Zukunft die

-Tags für Java-Code, danke! :)[/ot]

 

[Nemesys88]

Klar mach ich.

Problem ist übrigens gelöst!

Danke an Flown.