Observer Pattern - hasChanged() immer false

[Helgon]

Hey, ich hab folgendes Problem. Mein Subjekt gibt immer zurück, dass das flag das von setChanged() gesetzt wird false sei.

public class Main implements Observer {

    public static void main(String[] args) {
    	
    	final MainFrame frame = new MainFrame();
    	frame.main(null);
    	
    	Thread t = new Thread(){
    		
    		@Override public void run(){
    			
    			while(true){
    				if(frame.teller.hasChanged()){
    					System.out.println("x");
    
            // [...........]
    
}

	public void addProgress(Object object){
		setChanged();
		notifyObservers(object);
	}

Das Subjekt und die Observer werden in der Instanz von MainFrame erzeugt. (Das funktioniert auch alles richtig)

Ich möchte aber, dass wen die Observer benachrichtigt werden (in MainFrame wurde ein Button Event ausgelöst) ich das auch in der Klasse Main weis, um von dort aus mit dem Wert einer TextBox eine neue Klasse zu erzeugen.

Ich muss ja aber wissen, wann der Button gedrückt wurde und das müsste doch der Moment sein wo setChaged()==true sein müsste, oder etwa nicht? Nur bei mir ist das nie der Fall? Ist der Moment vllt zu kurz und der Thread bekommt es nicht mit?

Wenn ja, wie kann ich das Problem sonst lösen?

Grüße

 

[AmunRa]

Ich galub um dir helfen zu koennen, fehlt wohl etwas Code.

Mach mal ein KSKB wo das fehlerhafte verhalten auftritt

 

[Helgon]

Alles klar, ich wollts übersichtlich halten - Eigentlich gibts ja kein "fehlerhaftes Verhalten". Alles läuft wie es soll, nur die Flag scheint nie gesetzt zu sein (was sie aber natürlich wird)

 public static void main(String[] args) {
    	
    	final MainFrame frame = new MainFrame();
    	frame.main(null);
    	Thread t = new Thread(){
    		@Override public void run(){   			
    			while(true){
    				if(frame.teller.hasChanged()){
    					System.out.print("x");
    				}
    			}  			
    		}   		
    	};    	
    	t.start();   	
    }  
}

package isbnreader;

import java.util.Observable;

import javax.swing.JProgressBar;

public class ProgressTeller extends Observable {
	
	public void addProgress(Object object){
		setChanged();                          /// Wenn das Erzähler Subjekt die Methode addProgres(Obj o) aufruft, dann müsste die Flag durch kurzeitig gesetzt werden, sodass hasChanged() == true ist
		notifyObservers(object);            /// hier wird die flag dann ja nach dem benachrichtigen auf false wieder gesetzt
	}

}

import java.util.Observable;

public class ProgressListener implements Observer {

	final private String name;
	ProgressListener(String name){this.name = name;}
	
	@Override
	public void update(Observable o, Object arg) {
		try {((JProgressBar) arg).setValue(((JProgressBar) arg).getValue()+20);} 
                catch(ClassCastException e) {e.printStackTrace();}
	}
}

package isbnreader;
import java.awt.Label;
import java.util.Observer;

import javax.swing.*;

public class MainFrame extends JFrame {
	
     private JProgressBar jProgressBar1;
     
     ProgressTeller teller = new ProgressTeller();
	 ProgressListener listener1  = new ProgressListener("lis1");
     
    public MainFrame() {
    	initComponents();
    	teller.addObserver(listener1);
    }
    
    public  void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {	
		teller.addProgress(jProgressBar1);
	}
	
    public static void main(String args[]) {		     	    
	        java.awt.EventQueue.invokeLater(new Runnable() {
	            public void run() {
	                new MainFrame().setVisible(true);
	            }
	        });        
	  }
    
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">//GEN-BEGIN:initComponents
    public  void initComponents() {
    	
         jProgressBar1 = new JProgressBar();

        jButton1.setText("Suchen");
        jButton1.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                jButton1ActionPerformed(evt);
            }
        });

 
        pack();
    }// </editor-fold>//GEN-END:initComponents

    public JProgressBar getBar(){
    	return jProgressBar1;
    }
    
}

Ka, wie ich ein KSKB kleiner verpacken kann, sry

Aber hoffe es hilft damit mir wer helfen kann. Wie gesagt funktionieren tut alles, nur ich würd gern wissen wie ich auch Klassenübergreifend benachrichtigt werden kann (um dann weitere AKtionen auszuführen)

Grüße

P.S.: Siehe Kommentare 2ter Code Block

 

[AmunRa]

Das hab ich leider total falsch verstanden und kann dir leider nicht helfen

 

[Michael...]

     	Thread t = new Thread(){
    		@Override public void run(){   			
    			while(true){
    				if(frame.teller.hasChanged()){
    					System.out.print("x");
    				}
    			}  			
    		}   		
    	};    	
    	t.start();

Das ist schonmal nichts. Der Thread läuft die ganze Zeit und belastet Deine CPU zu 100%. Ausserdem ist das Oberver Pattern ja nicht dafür da, um ständig ein Objekt zu fragen, ob es was neues gibt. Sondern, damit das Objekt aktiv mitteilen kann, dass sich etwas geändert hat.
Ich weiß zwar nicht, was das werden soll, aber momentan reicht für die Lösung Deiner Anforderung ein simpler Listener.

import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JProgressBar;

public class ProgressBarDemo extends JPanel {
	private JProgressBar progressBar;
	
	public ProgressBarDemo() {
		JButton button = new JButton("Suchen");
		this.setLayout(new GridLayout(2,1, 10, 10));
		this.add(progressBar = new JProgressBar());
		this.add(button);
		
		button.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent evt) {
				progressBar.setValue(progressBar.getValue() +20);
			}
		});
	}
	
	public static void main(String[] s) {
		JFrame frame = new JFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.add(new ProgressBarDemo());
		frame.pack();
		frame.setVisible(true);
	}
}

 

[Helgon]

Hey, danke für die Antwort und das du dir die Mühe gemacht hast den Code zu schreiben, aber ich glaub du verstehst nicht ganz worauf ich hinaus will

Hast natürlich recht, dumm den Thread da ganze Zeit laufen zu lassen, aber ich wusst mir leider nicht anders zu helfen (und war ja nurn Test).

Wie du es geschrieben hast, ist es mir natürlich klar, aber wenn ich in

KlasseA

ein Objekt von

KlasseB

erzeuge und der Button/der EventHandler sich in

KlasseB

befindet. Wie bekommt das

KlasseA

mit? Sagen wir Beispielsweise soll dann in

KlasseA

ein Objekt von

KlasseC

erzeugt werden.

Da hab ichs ja mit dem Observer Pattern probiert.

Grüße

Hoffe du weißt was ich meine, ansonsten bitte frag und ich versuchs nochmal zu erklären

 

[bygones]

brute-force:

public class KlasseA {
 public void foo() {
     klasseB b = new KlasseB(this);
 }

 public void createC() {
    // was auch immer
 }
}

public class KlasseB {
  private final KlasseA a;
  
  public KlasseB(KlasseA a){
    this.a = a;
  }

  public void bar() {
    // irgendwas
    a.createC();
 }
}

also eine Art Callback Mechanismus. Brute-force deswegen, da KlasseB KlasseA direkt kennt. Schoener ist es, wenn KlasseA ein Interface Implementiert (zb einen Listener) und KlasseB nur das Interface kennt.

Das sollte dann aber auch mit dem Observer Pattern gehen - oder jeglichen Listener Interface das man selbst baut

 

[Helgon]

Danke für die Antwort. Diesen "Brute-Force" Weg hatte ich auch schon ausprobiert, aber der erschien mir zu unsauber

And Interfaces hab ich gar nicht gedacht. Werd mal gucken ob ich da was hinkrieg. Danke für die Idee!

 

[bygones]

Danke für die Antwort. Diesen "Brute-Force" Weg hatte ich auch schon ausprobiert, aber der erschien mir zu unsauber

ist es eigentlich auch, da sehr unflexibel

Aehnlich wie bei MouseListener, ActionListener etc sollte man ein Interface definieren, dass auf gewisse Aktionen reagiert. Bei dir dann ist die KlasseB der Halter aller Listener und kann neue aufnehmen bzw wenn noetig, welche entfernen.

Wenn dann das Ereignis eintritt, benachrichtigt KlasseB alle seine Listener.

 

[Michael...]

Bin ja nicht gerade Fan des Observer Patterns und von den in der Java API implementierten Klassen.

Ich würde sowas mit einem Listener implementieren. Der Einfachheit halber hier ein Bsp. mit ActionListener. Es könnte natürlich auch ein beliebiger selbst definierter Listener sein:

import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class DemoListener {
	
	class KlasseA extends JPanel implements ActionListener {
		private JLabel label;
		private KlasseB klasseB;
		
		public KlasseA() {
			this.setLayout(new GridLayout(2,1));
			klasseB = new KlasseB();
			klasseB.registerListener(this);
			label = new JLabel("Push the button!!");
			this.add(klasseB);
			this.add(label);
		}
		
		public void actionPerformed(ActionEvent e) {
			KlasseC klasseC = new KlasseC();
			System.out.println("new object created: " + klasseC);
			label.setText(klasseC.toString());
		}
	}
	
	class KlasseB extends JPanel {
		private JButton button;
		
		public KlasseB() {
			this.setLayout(new GridLayout(1, 1));
			this.add(button = new JButton("Suchen"));
		}
		
		public void registerListener(ActionListener listener) {
			button.addActionListener(listener);
		}
	}
	
	class KlasseC {
		public String toString() {
			return "Ich bin ein Objekt vom Typ " + this.getClass().getSimpleName();
		}
	}
	
	public void starteDemo() {
		JFrame frame = new JFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.add(new KlasseA());
		frame.setBounds(50, 50, 300, 80);
		frame.setVisible(true);
	}
	
	public static void main(String[] s) {
		new DemoListener().starteDemo();
	}
}

 

[Helgon]

Ich wollt grad wieder schon ne Frage Posten, aber da hab ich den Post von dir gesehn (Michael).

Das ist glaub ich ca. was ich umsetzen wollte, bin ich aber nicht drauf gekommen.

Werds mir dann gleich mal ganz genau zu Gemüte führen und schauen das ichs dann bei mir auch so umsetzen kann.

Vielen Dank euch beiden auf jeden Fall.