JPanel und ScrollPanel funktionieren nicht

[lokly]

Der JPanel und ScrollPanel funktionieren nicht richtig:

Ich will eine unterschiedliche Menge an JPanels erstellen, die dann in ein weiteres Panel geadded werden. Das klappt soweit auch, allerdings nur 7 oder 8 Stück, danach werden mir die Zahlen, um welches Panel es sich handelt nicht mehr angezeigt, obwohl sich da eigentlich die Scrollbar aktivieren sollte.
Der weitere Fehler ist: Wenn man die Fenstergröße so ändert, dass die Scrollbars angezeigt werden und diese dann bewegt, dann wird der Hintergrund der Panels undeutlich bzw. schwarz, da stimmt wohl was mit der paint Methode nicht, aber finde meinen Fehler nicht.

Wäre echt super, wenn mir jemand weiter helfen könnte..

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.GridLayout;

import javax.swing.DefaultComboBoxModel;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JSplitPane;
import javax.swing.ListModel;


public class TestFenster {

	public static void main(String[]args){
		new Fenster();
	}
	
}

class Fenster extends JFrame {
	private JSplitPane mainWindowSplitPane;
	private JPanel mainWindowPanel;
	private JSplitPane optionSplitPane;
	private JScrollPane buttonScrollPane;
	private JPanel buttonPanel;
	private JList jList;
	private JScrollPane jListScrollPane;
	private JScrollPane mainWindowScrollPane;
	private int num;
	

	public Fenster(){
		this.setSize(1050, 600);
		initGUI();
		
		//Hier ist der Fehler!
		num = 7;
		this.mainWindowPanel.setLayout(new GridLayout(num,1));

		for(int i=0;i<num;i++){
			mainWindowPanel.add(new ModuleTable(i+1));
		}
		
		
		
		this.setVisible(true);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);	
	}
	
	private void initGUI(){
			//Set Look & Feel
			try {
				javax.swing.UIManager.setLookAndFeel(javax.swing.UIManager.getSystemLookAndFeelClassName());
			} catch(Exception e) {
				e.printStackTrace();
			}
		
			mainWindowSplitPane = new JSplitPane();
			getContentPane().add(mainWindowSplitPane, BorderLayout.CENTER);
			mainWindowSplitPane.setPreferredSize(new java.awt.Dimension(1059, 578));
			{
				mainWindowScrollPane = new JScrollPane();
				mainWindowSplitPane.add(mainWindowScrollPane, JSplitPane.LEFT);
				mainWindowScrollPane.setPreferredSize(new java.awt.Dimension(741, 574));
				{
					mainWindowPanel = new JPanel();
					mainWindowScrollPane.setViewportView(mainWindowPanel);
					mainWindowPanel.setPreferredSize(new java.awt.Dimension(691, 570));
				}
			}
			{
				optionSplitPane = new JSplitPane(JSplitPane.VERTICAL_SPLIT);
				mainWindowSplitPane.add(optionSplitPane, JSplitPane.RIGHT);
				optionSplitPane.setPreferredSize(new java.awt.Dimension(294, 574));
				{
					jListScrollPane = new JScrollPane();
					optionSplitPane.add(jListScrollPane, JSplitPane.TOP);
					jListScrollPane.setPreferredSize(new java.awt.Dimension(292, 254));
					{
						ListModel jListModel = 
							new DefaultComboBoxModel(
									new String[] { "Item One", "Item Two" });
						jList = new JList();
						jListScrollPane.setViewportView(jList);
						jList.setModel(jListModel);
						jList.setPreferredSize(new java.awt.Dimension(288, 244));
					}
				}
				{
					buttonScrollPane = new JScrollPane();
					optionSplitPane.add(buttonScrollPane, JSplitPane.BOTTOM);
					buttonScrollPane.setPreferredSize(new java.awt.Dimension(366, 177));
					{
						buttonPanel = new JPanel();
						buttonScrollPane.setViewportView(buttonPanel);
						buttonPanel.setPreferredSize(new java.awt.Dimension(288, 241));
					}
				}
			}
		
	}
	

}



class ModuleTable extends JPanel {

	private int tableNumber;
	
	public ModuleTable(int tableNumber){
		
		this.tableNumber=tableNumber;
		
	}
	
	
	
	public void paint(Graphics g){
		
		//Draw BackGround
		g.setColor(Color.GRAY);
		
		for(int i=0;i<this.getWidth();i+=10){
			for(int j=0;j<this.getHeight();j+=10){
				g.drawLine(0,j, getWidth(), j);
				g.drawLine(i, 0, i, getHeight());
			}
		}
		//End Background
		
		
		g.setColor(Color.BLACK);
		
		g.drawString(String.valueOf(this.tableNumber), 10, 70);
		g.drawLine(0, 80, getWidth(), 80);
		g.drawLine(30, 0, 30, this.getHeight());
		
		
	}
	
	
	
	
}

 

[Duepsi]

Dein Problem ist folgendes, soweit ich das nachvollziehen kann:
Du fügst die einzelnen ModuleTable's nicht direkt mainWindowScrollPane hinzu, sondern mainWindowPanel. Das allerdings hat eine feste, preferierte Größe, die du in Zeile 72 festlegst (691 * 570) und verwendet ein GridLayout (Zeile 42). Das heißt, dass jede neue ModuleTable so gestaucht wird, dass alle Komponenten in mainWindowPanel Platz finden.
Damit

g.drawString(String.valueOf(this.tableNumber), 10, 70);

(Zeile 140) funktioniert, muss die Höhe der Komponente mindestens 70 betragen. Dies gilt, wenn maximal 8 Komponenten zu mainWindowPanel hinzugefügt wurden, da 570/8 > 70, 570/9 < 70. Deswegen werde deine Zahlen nicht mehr gezeichnet, wenn du num > 8 setzt.
Du kannst dir das klarmachen, indem du die oben genannte Zeile durch

g.drawString(String.valueOf(this.tableNumber), 10, getHeight()/2);

ersetzt.


Eine mögliche Lösung für dein Problem wäre also die dynamische Anpassung der Größe von mainWindowPanel. Leider könnte dies immer noch zu Problemen führen, da JPanel das Interface Scrollable nicht implementiert und daher grundsätzlich nicht für JScrollPanes geeignet ist. Um dies zu vermeiden kannst du stattdessen eine JList verwenden. Das sind allerdings nur zwei spontane Ideen ohne Garantie, da ich selbst noch nie mit mehreren Komponenten in einer JScrollPane gearbeitet habe.


Noch ein paar Anmerkungen am Rande:
Überschreibe paintComponent() anstatt von paint(). Melde dich noch mal, wenn dein zweiter Fehler nach Lösen des oben beschriebenen Problems weiterhin besteht.

Außerdem hättest du deinen Beispielcode einfacher gestalten können, indem du sämtlichen, die rechte Seite von mainWindowSplitPane betreffenden, Code entfernt hättest.


Viele Grüße,
Simon

 

[Marco13]

Ja, so fast: Statt paint zu überschreiben eben

public void paintComponent(Graphics g)
   {
      super.paintComponent(g);
      //Draw BackGround
      g.setColor(Color.GRAY);
...

Und die Sache mit den Größen ... stimmt schon: Das mainWindowPanel hat eine zu kleine PreferredSize, um mehr ModuleTables aufzunehmen. Eine mögliche Abhilfe wäre, die Zeile 72

//mainWindowPanel.setPreferredSize(new java.awt.Dimension(691, 570));

auszukommentieren, und statdessen die PreferredSize der einzelnen ModuleTables zu setzen. Vielleicht gleich in deren Konstruktor - die wissen ja selbst am besten, wie groß sie sein wollen 🙂

   public ModuleTable(int tableNumber){

      this.tableNumber=tableNumber;
      setPreferredSize(new java.awt.Dimension(691, 81));

   }
}

Wenn man das so macht, dann wird die PreferredSize des mainWindowPanel automatisch berechnet, und zwar auf Basis von seinem Layout und dem PreferredSize der enthaltenen Components - d.h. es gibt automatisch die passende PreferredSize zurück, die es erlaubt, die aktuelle Anzahl ModuleTables in ihrer PreferredSize darzustellen.

@Duepsi: Eine mögliche Lösung für dein Problem wäre also die dynamische Anpassung der Größe von mainWindowPanel.
... was ja automatisch gemacht wird, wenn man die PreferredSize nicht explizit setzt 🙂

Leider könnte dies immer noch zu Problemen führen, da JPanel das Interface Scrollable nicht implementiert und daher grundsätzlich nicht für JScrollPanes geeignet ist.

Das stimmt aber nicht! JEDE Component kann in einer ScrollPane liegen (und oft ist es nur ein JPanel). Scrollable muss man nur implementieren, wenn man ein ganz bestimmtes Scroll-Verhalten erreichen will. In diesem Fall könnte so ein "bestimmtes Scroll-Verhalten" z.B. sein, dass bei jedem Klick auf den Pfeil der Scrollbar genau um EIN ModuleTable gescrollt wird (wenn man Scrollable implementiert, kann man also z.B. die "Scroll-Schritt-Größe" angeben)

Und noch @lokly: Du solltest evtl. allgemein etwas "sparsamer" mit den PreferredSizes umgehen. In den meisten Fällen fuktioniert diese automatische Berechnung der PreferedSize auf Basis des Layouts und des Inhalts sehr gut. Nur in wenigen Fällen muss man die PreferredSize von Hand setzen - und einer dieser Fälle ist eben bei solchen Components wie den ModuleTables, die eine bestimmte Größe haben sollten, weil man an eine bestimmte Stelle etwas reinzeichnen will.

 

[lokly]

Super, vielen Dank dafür!
Das setPreferredSize hats echt gebracht! Da habt ihr mich echt nen gutes Stück nach vorne gebracht..