Swing JScrollPane nimmt das JPanel zum Zeichnen nicht an

[Jay1980]

Servus,

ich kann mir gerade nicht erklären warum es die JScrollPane nicht aktualisiert.

Ich habe eine JTabbedPane die JScrollPanes trägt, die als Reiter funktionieren.
Nun sind in den ScrollPanes meist JPanels, ein Reiter etwa soll nach einer ausgelösten Action ein JPanel bekommen, das als Zeichenfläche fungieren soll. Naja das Action Handling funktioniert, die Zeichenklasse leitet sich von JPanel ab und zeichnet auch nur eine Testlinie, da sollte auch nichts anbrennen. Ich denke es liegt an einem Layoutmanager oder einem Fehler beim 'Einsetzen' des neuen JPanels. Ich habe aus der JScrollPane des Tabs alle Komponenten removed und dann das neue JPanel geaddet. Als Ergebnis wird das alte JPanel entfernt aber das neue nicht angezeigt.
Danke vorab.

Hier der Code:

... Klasse JobaDrawPanel extends JPanel ...
/**
	  * paintComponent()
	  * 
	  * 
	  */
	 protected void paintComponent( Graphics g )
	 { 
		super.paintComponent(g); // Elternklassen ihre Sachen zeichnen lassen
		
		// Testlinie
		g.drawLine( 20, 40, 400, 40 );
		
...

.... Klasse JobaGui ....

public void actionPerformed(ActionEvent evt)
	{
		if ( evt.getActionCommand() == "Testbutton" )
		{
			JOptionPane.showMessageDialog(null, "ActionListener aus Testbutton ist da...");	
			// stoße Berechnung an
			
			// Objekt der Huellenberechnung starten
			JobaConvexHull jch = new JobaConvexHull(); // Konstruktor mit Dummyarray
			
			// Variablen für den JobaDrawPanel-Konstruktor beschaffen
			int cosysx = jch.getMaxXSoFar()[0] + 50;
			int cosysy = jch.getMaxYSoFar()[1] + 50;
			int[][] ptd = jch.getPoints();
			int[][] cptd = jch.getConvexHullPoints();
			int[][] lcptd = jch.getLeftHullSortedByY();
			int[][] rcptd = jch.getRightHullSortedByY();
			
			// Erwartung JobaDrawPanel-Konstruktor: int cosysx, int cosysy, int[][] ptd, int[][] cptd, int[][] lptd, int[][] rcptd
			JobaDrawPanel jdp = new JobaDrawPanel(cosysx, cosysy, ptd, cptd, lcptd, rcptd);
			jdp.setBackground(Color.blue);
			drawTabScrollPane.removeAll();
			drawTabScrollPane.add(jdp);
			drawTabScrollPane.revalidate();
			drawTabScrollPane.repaint();
		}

 

[SlaterB]

> drawTabScrollPane.add(jdp);

drawTabScrollPane.setViewPortView(jdp);
oder so

 

[Jay1980]

Okay danke, leider geht es immer noch nicht.

Ich habe versucht, die umfassende Berechnung rauszunehmen und das Problem einzukreisen...
Also das JTabbedPane Tab 'DrawPanel' ist weiss, das Optionen-Tab hat einen button mit der Action. Nun klick ich auf den Button und will ein neues JPanel in die ScrollPane laden. Das 'alte' weisse JPanel ist auch weg, man sieht aber nur einen grauen Hintergrund ohne meine JPanel+JLabel-Kombo.

Das ist der aktuelle Code, ich dachte das muesste so problemlos laufen oder was habe ich vergessen:

JPanel jdp = new JPanel();
			JLabel jdpLabel = new JLabel("Teststring");
			jdp.add(jdpLabel);
			
			drawTabScrollPane.removeAll();
			drawTabScrollPane.setViewportView(jdp);
			drawTabScrollPane.revalidate();
			drawTabScrollPane.repaint();
			tabbedPane.revalidate();
			tabbedPane.repaint();

 

[SlaterB]

vielleicht ist das neue JPanel von der Größe 1x1 Pixel, dann wird man auch nicht viel erkennen,
setPreferredSize könnte helfen, auch einen Border und/ oder eine Hintergrundfarbe zu setzen sorgt für minimale Sichtbarbkeit

falls noch nicht geklärt:
vollständiges Testprogramm bitte, möglichst kurz

edit:
du benutzt ein Standard-JPanel, kein eigenes mit paintComponent, das sollte dann wohl sichbar sein,

was ist mit
> tabbedPane.revalidate()
ist das ein wichtiges Objekt oder vielleicht eher unbeteiligt?
im Zweifel direkt auf das JFrame validate() aufrufen,
oder Testprogramm, läßt alle Fehlerquellen sofort aufdecken

edit 2:
hmm,
> drawTabScrollPane.revalidate();
steht da ja auch..

 

[Jay1980]

Okay, ich habe mal eine Beispielklasse getippt, die man dann kompilieren kann und die das Problem nachstellt.

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTabbedPane;
import javax.swing.JTextArea;


public class Main implements ActionListener
{
	String dbs;
	JFrame mainFrame;
	JTabbedPane tabbedPane;
	JPanel drawTabPanel;
	JPanel optionTabPanel;
	JScrollPane drawTabScrollPane;
	JScrollPane optionTabScrollPane;
	JPanel debugTabPanel;
	
	

	public static void main(String[] args)
	{
		Main main = new Main();
		main.initialize();
	}
	
	public void initialize()
	{
		// mainFrame
		mainFrame = new JFrame("Konvexe Huelle");
		
		this.mainFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.mainFrame.setSize(1200, 900);
		
		Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
		this.mainFrame.setLocation( 
				(d.width - this.mainFrame.getSize().width)/2 , 
				(d.height - this.mainFrame.getSize().height)/2 );
		
		// JTabbedPane mit ScrollPanes
		tabbedPane = new JTabbedPane();
		
		// Reiter erzeugen samt Inhaltsbereichen
		drawTabPanel = new JPanel();
		optionTabPanel = new JPanel();
		debugTabPanel = new JPanel();
		drawTabScrollPane = new JScrollPane( drawTabPanel );
		optionTabScrollPane = new JScrollPane( optionTabPanel );
		
		// DrawTabPanel
		drawTabPanel.setBorder(BorderFactory.createLineBorder( Color.green ));
		drawTabPanel.setBackground(Color.white);
		
		// OptionTabPanel
		optionTabPanel.setBorder(BorderFactory.createLineBorder( Color.red ));
		JButton testBut = new JButton("Testbutton");
		testBut.addActionListener( this );
		optionTabPanel.add(testBut);
		optionTabPanel.revalidate();
		
		// DebuggingTab	
		dbs = "+++++ DebugString +++++\n";
		
		JTextArea debugTabPanelTextArea = new JTextArea(dbs, 15, 20);
		JScrollPane debugTabPanelScrollPane = new JScrollPane(debugTabPanelTextArea);
		debugTabPanel.add(debugTabPanelScrollPane);
		debugTabPanel.revalidate();
		debugTabPanel.repaint();
		
		// Reiter einhaengen
		tabbedPane.addTab( "Zeichenfläche", drawTabScrollPane );
		tabbedPane.addTab( "Optionen", optionTabScrollPane );
		tabbedPane.addTab( "Debugging", debugTabPanel );
		
		// Frame zusammenfuegen
		mainFrame.add(tabbedPane);
		mainFrame.setVisible(true);
	}
	
	public void actionPerformed(ActionEvent evt)
	{
		if( evt.getActionCommand() == "Testbutton")
		{
			JPanel jdp = new JPanel();
			jdp.setPreferredSize(new Dimension( 300, 300) );
			jdp.setBorder(BorderFactory.createLineBorder(Color.green));
			JLabel jdpLabel = new JLabel("Teststring");
			jdp.add(jdpLabel);
			
			drawTabScrollPane.removeAll();
			drawTabScrollPane.setViewportView(jdp);
			drawTabScrollPane.revalidate();
			drawTabScrollPane.repaint();
			tabbedPane.revalidate();
			tabbedPane.repaint();
		}
	}

}

 

[SlaterB]

ok, sieht so aus, als wäre
> drawTabScrollPane.removeAll();
schuld, damit entfernst du den inneren Aufbau, also z.B. den ViewportView, aus der GUI-Anzeige-Struktur,
dass mir das noch nie aufgefallen ist.., werde ich mir merken für zukünftige ähnliche Fragen

removeAll() weg, setViewportView reicht