Component ändern (repaint)

[public_void_main]

Hi,

ich habe ein Mini-Swing-Programm erstellt, das Charts generieren kann. Diese werden mit JFreeChart hergestellt. Die Grafik wird als Component geholt und dann in ein Panel gesetzt.

Das Panel und auch jede Chart-Component sind als Instanzvariablen vermerkt:

public class PlotView {
    
    JFrame    frame;
    
   Component diskUsageChartComponent;
   Component nrRowsChartComponent;
 ...
}

Wenn der Anwender jetzt einen Button klickt, werden neue Parameter ausgelesen und die Plots neu erstellt. Jetzt sollen natürlich auch die neuen Charts angezeigt werden Daher sieht meine update-Methode ungefähr so aus:

final String title = "Charts";
		
 getChartData();		
 diskUsageChartComponent   = swingWrapper.getChartComponent(swingWrapper.CHART_DISKUSAGE, title);
 // das gleiche fuer weitere Charts

 diskUsageChartComponent.repaint();

So.
Jetzt passiert aber - nichts ???:L

Übersehe ich hier etwas ganz Wichtiges?
Die update() methode wird auch aufgerufen, das JFRame ist visible. Und die neuen Charts sehen definitiv anders aus.

Wenn jemand eine Idee hat, wäre ich sehr interessiert

Gruß
Georg

 

[madboy]

Versuch mal ein

frame.validate();

nach/anstatt deinem repaint().

 

[public_void_main]

Hm nein, das hat leider nicht geholfen. :-(

ICh stell hier nochmal die wichtigsten Code-Teile rein. Vielleicht wird dann ersichtlich, woran es liegt.
In jedem Fall vielen Dank für Eure Hilfe.

package ch.cern.atlas.coolmonitor;

import ...
public class PlotView {

    static CoolChartFactory chartFabric     = CoolChartFactory.getInstance();
    static ICoolChartXYBuilder    chartBuilder;
    static JFreeChartSwingWrapper swingWrapper;
    
    JFrame    frame;
    Container container;
    
    Component diskUsageChartComponent;
    Component nrRowsChartComponent;
    Component nrChannelsChartComponent;
    Component nrTagsChartComponent;
    
    JTextField serverField;
    JTextField dbNameField;
    JTextField folderNameField;
    JTextField coolVersionField;
    JTextField intervalField;
    
    final String implClass = "xyz.ChartXYBuilder";
    
    CoolConfigurationModel model;
    
    public PlotView(CoolConfigurationModel model) throws ... {
    	this.model = model;
    	
    	chartBuilder = chartFabric.create(implClass);
    	
     	frame = new JFrame("Charts for "+model.getFolderName());   
	container = frame.getContentPane();
		
	frame.addWindowFocusListener(new WindowAdapter()  {
		public void windowClosing(WindowEvent event) { System.exit(0);}
	});
    }
    
    
	public void showMainFrame() throws ... {
		this.getChartData();

    	// ============ GUI stuff ============
	    container.setBackground(Color.white);
	    
	    Panel dataPanel = new Panel();
	    dataPanel.setLayout(new FlowLayout());
	    // server field
	    serverField = new JTextField(model.getServer());
	    // db name field
	    dbNameField = new JTextField(model.getDbName());
	    // version field
	    coolVersionField = new JTextField(model.getCoolVersion());
	    // folder name field
	    folderNameField = new JTextField(model.getFolderName());
	    
	    intervalField = new JTextField(model.getInterval());
	    	    
	    JButton setParamsButton = new JButton("Set parameters");
	    
	    setParamsButton.addActionListener(new ActionListener() {

			public void actionPerformed(ActionEvent e) {
				updateModel();
			} 
	    });
	    
	    dataPanel.add(serverField);
	    dataPanel.add(dbNameField);
	    dataPanel.add(coolVersionField);
	    dataPanel.add(folderNameField);
	    dataPanel.add(intervalField);
	    dataPanel.add(setParamsButton);
	    
	    
	    // get the chart components
	    JPanel plotPanel = this.getComponents();
	    
	    // add all sub panels
	    container.add(dataPanel, BorderLayout.NORTH);
	    container.add(plotPanel, BorderLayout.CENTER);    

		frame.setVisible(true);
	}
	
	
	
	protected void updateModel() {
			String server = serverField.getText();
			String dbName = dbNameField.getText();
			String interval = intervalField.getText();
			String folderName = folderNameField.getText();
			String coolVersion = coolVersionField.getText();
			model.setData(server, dbName, coolVersion, folderName, interval);
                    // ^ ^ triggers update method
	}
	
	
	
	
	protected void getChartData() {

		chartBuilder.setServer(model.getServer());
		chartBuilder.setDbName(model.getDbName());
		chartBuilder.setCoolVersion(model.getCoolVersion());
		chartBuilder.setFolderName(model.getFolderName());
		chartBuilder.setInterval(Integer.parseInt( model.getInterval() ));

		swingWrapper = new JFreeChartSwingWrapper(chartBuilder);
	}
	
	
	
	protected JPanel getComponents()  {
	
	    JPanel panel = new JPanel();
	    panel.setLayout(new GridLayout(2,2));
	    panel.add(diskUsageChartComponent);
	    panel.add(nrRowsChartComponent);
	    panel.add(nrChannelsChartComponent);
	    panel.add(nrTagsChartComponent);
	    
	    return panel;
	}
	
	
	
	public void update() {
		
		final String title = "Charts";
		
		getChartData();
		System.out.println(model.getInterval());
		
		try {
			diskUsageChartComponent   = swingWrapper.getChartComponent(...);
...			
		} catch (SQLException e) {
			...		}
		
		System.out.println("View: Received update");
		
		// ================ * UPDATES * ================== //
		diskUsageChartComponent.repaint();
		nrRowsChartComponent.repaint();
		nrChannelsChartComponent.repaint();
		nrTagsChartComponent.repaint();
		
		frame.validate();
//		container.repaint();
//		frame.repaint();
	}
}

 

[Gast2]

du vermischt awt mit swing....das ist nie gut

EDIT: Des weitern versuch mal in der update methode plotPanel.repaint() einzubauen

   public void update() { 
       
      final String title = "Charts"; 
       
      getChartData(); 
      System.out.println(model.getInterval()); 
       
      try { 
         diskUsageChartComponent   = swingWrapper.getChartComponent(...); 
...          
      } catch (SQLException e) { 
         ...      } 
       
      System.out.println("View: Received update"); 
       
      // ================ * UPDATES * ================== // 
plotPanel.repaint() 
   } 
}

 

[public_void_main]

du vermischt awt mit swing....das ist nie gut

Ok, das wusste ich auch noch nicht
An welchen Stellen sollte ich besonders darauf achten?
Ich habe ja folgende Imports:

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Component;
import java.awt.Container;
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.GridLayout;
import java.awt.Panel;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;

Das ist ne Menge AWT ;-). Gibt es fuer all das Ersatz-Pakete in Swing?

Das plotPanel.repaint() werde ich mal ausprobieren, danke für den Hinweis. [/code]

 

[merlin2]

Es gibt für alle AWT-Klassen, die von Component erben, Swing-Entsprechungen.

 

[Gast2]

du solltest du anstatt Component JComponent verwenden.... und anstatt Panel JPanel....

(fast) alle Swing Componenten erkennst du am J...

 

[public_void_main]

danke!

 

[public_void_main]

Hm, ich habe nun alle Components gegen JComponents ausgetauscht, dasselbe für JPanel. Für die übrigen imports habe ich keine Swing-Äquivalente gefunden.
Das plotPanel.repaint() ist ebenfalls eingebaut. Und trotzdem wird die GUI nicht aktualisiert.

Hat noch jemand Tipps? Ich komme da nicht hinter... :-/

 

[Gast2]

schick mal deinen code... am besten, dass man ihn ausführen kann.