MVC: System.out.prinln in View umleiten geht nicht richtig

[robochris]

Hallo Zusammen,

vielleicht kann mir jemand bei meinem etwas verzwickten Problem helfen:

1. eine Java Klasse gibt im Sekundentakt Stausmeldungen mit System.out.println aus
2. Ich möchte die Statusmeldungen nicht in der Kommandozeile sehen, sondern auf einer GUI mit verschiedenen Bedienknöpfen

Ich habe das ganze mit einem MVC Design Pattern ( Model View Controller ) versucht zu lösen.
Jetzt gibt es das folgende Problem: Im Controller rufe ich die Methode einer Java-Klasse auf, die normalerweise ihre Meldungen auf die Statuszeile mit System.out.println ausgibt. Diese schreibt die Meldungen jetzt aber in die MVC_View Klasse. Leider werden aber die Statusmeldungen im Bedienfenster erst angezeigt, wenn die Methode beendet ist. Oder anders ausgedrückt: Wenn die Methode der Klasse 10 Sekunden lang Meldungen an den View geschickt hat, werden diese erst sichtbar, wenn die Methode schon beendet ist. Ich habe versucht, an verschiedenen Stellen eine frame.repaint() einzufügen, aber es hilft nix: die Meldungen werden erst am Schluss angezeigt.

Hat jemand eine Idee?

chris

 

[Marco13]

Wenn das, was die Meldungen verursacht, im Event-Dispatch-Thread läuft, kann eben nicht neu gezeichnet werden. Aber mangels Code... kann man nur sagen, dass man nichts genaues sagen kann...

 

[robochris]

Hallo Marco,

der Code mittlerweile etwas umfangreich, deshalb poste ich mal den wichtigsten Ausschnitt:

Im Controller werden die Event-Handler ziemlich am Anfang gestartet. Also der Controller startet den view und der view nutzt die Funktion "initialize" des Controllers um die "view" Referenz mitzuteilen.
Danach wird beim Drücken einer Taste der Actionlistener im Controller aufgerufen.
Das Problem ist jetzt, dass die Ausgaben des Objektes

hv.loadFile( model.getHexFileName());

erst angezeigt werden, wenn die "LoadFile" Methode schon durchgelaufen ist.

Hier der Code-Ausschnitt aus dem Controler:

         /**
	 * Initialisiert Model und View.
	 * @param m Mathematisches Modell
	 * @param v zum Controller gehoeriger View
	 */
	public void initialize( MVCdataModell m, MVCmainView v)
	{
	    model = m;
	    view = v;
		hv=new HexFileViewer();
		hv.initialize(m,v);
	}
         /**
	 * Verarbeiten der Bildschimreingaben.
	 */
	  public void start()
	  {
....
		  view.loadButton.addActionListener(new ActionListener()
		  {
			  public void actionPerformed(ActionEvent e)
		      {
				  System.out.println(model.getHexFileName());
				  hv.loadFile( model.getHexFileName());
				  
				  //hv.anzeigen();
		      }
		  });
...

 

[Marco13]

Jup, da lag meine Kristallkugel wohl wieder mal richtig. Der einfachste Workaround wäre ein eigener Thread dafür

         /**
	 * Verarbeiten der Bildschimreingaben.
	 */
	  public void start()
	  {
....
		  view.loadButton.addActionListener(new ActionListener()
		  {
			  public void actionPerformed(ActionEvent e)
		      {
				  System.out.println(model.getHexFileName());
                                  Thread t = new Thread(new Runnable()
                                  {
                                      public void run()
                                      {
				          hv.loadFile( model.getHexFileName());
                                      }
                                  });
                                  t.start();
				  
				  //hv.anzeigen();
		      }
		  });
...

[/quote]

Allerdings musst du dir natürlich drüber im Klaren sein, was passieren soll, wenn ein Benutzer einen epilleptischen Anfall hat, und 2000mal hintereinander den Button klickt usw....

 

[robochris]

Hallo Marco,

danke für die Info. Es lag tatsächlich daran. Mit Deiner Methode funktioniert's. Der Text wird jetzt immer während des Methodenlaufs im View angezeigt.
Allerdings habe ich noch ein weiteres seltsames Problem: Mein Anzeigefenster ist mit einer Scrollbar versehen. Eigentlich soll die Scrollbar automatisch scrollen ( also der Text nach oben geschoben werden ) damit immer der neues Text sichtbar wird. Man sieht zwar, wie sich die JTextArea füllt, aber das Scrollen geht einfach nicht automatisch.

		frame= new JFrame(); // Hautpfenster erzeugen
		frame.setTitle(titel);
		
		messageText=new JTextArea(5,20); // Lauftext Fenster
		messageTextScroller=new JScrollPane(messageText);
		messageTextScroller.setVerticalScrollBarPolicy(ScrollPaneConstants.VERTICAL_SCROLLBAR_ALWAYS);

Und das, obwohl ich in der draw-Methode der View das Autoscrolling eingebaut habe.

  public void draw()
	  {
		    //System.out.print( "Daten des Modells visualisieren ");
		    String[] S=model.getMessages();	    
		    messageText.append("\n"+model.getMessages()[S.length-1]);
	
		    String s=model.getHexFileName().getAbsolutePath();
		    hexFileName.setText(s);
		    s=model.getComPort();
		    comPort.setText(s);

		    JScrollBar bar=messageTextScroller.getVerticalScrollBar();  
		    int x=bar.getMaximum();
		    bar.setValue(x); 

		    System.out.println("draw");
		    frame.repaint();
	  }

Das Autoscrolling passiert erst, wenn ich einen neuen Button gedrückt habe.

Hast Du da vielleicht auch noch einen Tipp?

 

[Marco13]

Naja, dieses Autscrolling ist auch wieder fragwürdig. Nach dem "append"
textArea.setCaretPosition(textArea.getText().length()-1);
sollte es schon tun.

 

[Marco13]

Aber noch so als nachtrag: Swing-Components dürfen im allgemeinen nur im Event-Dispatch-Thread verändert werden - es KANN sein, dass du das jetzt (wo Teile deines Programmes in einem anderen Thread laufen) stärker berücksichtigen mußt. Das "append" ist laut Doku aber OK.

 

[robochris]

Hallo Marco,

das hat auch funktioniert. Man muss aber die -1 weglassen:

textArea.setCaretPosition(textArea.getText().length());

Die andere Method habe ich aus dem Buch "Java 6 ist auch eine Insel". Wenn ich es richtig gesehen habe, posted der Author hier auch manchmal im Forum. Vielleicht kann man das im Buch ja ändern, bzw. anmerken.

Aber noch so als nachtrag: Swing-Components dürfen im allgemeinen nur im Event-Dispatch-Thread verändert werden - es KANN sein, dass du das jetzt (wo Teile deines Programmes in einem anderen Thread laufen) stärker berücksichtigen mußt. Das "append" ist laut Doku aber OK.

Das verstehe ich leider nicht. Aber da das Programm erst mal funktioniert, werde ich es so lange ignorieren, bis ein Problem auftaucht.

Vielen Dank für Deine kompetente Hilfe,

chris