JScrollPane funktioniert nicht

[TimeIsTheKey]

Hallo

Ich habe mein GUI fast fertig und möchte endlich mit den Funktionen anfangen, aber den letzten Punkt auf meiner Liste krieg ich nicht hin. Ich habe ein JPanel das im JScrollPane ist. In das JPanel werden dynamisch Labels (eigene Klasse) geladen und wenn sobald zu viele drin sind, sollte man scrollen können. Ich weiss allerdings nicht was ich falsch mache. Vielleicht seht ihr ja etwas:

JPanel mainpanel,themenpanel;
JScrollPane themenscroll;

mainpanel = new JPanel(){
	private static final long serialVersionUID = 1L;
	public void paintComponent(Graphics g){
		super.paintComponent(g);
		g.drawLine(imenu.getIconWidth()-1,0,imenu.getIconWidth()-1,getHeight());
		g.drawLine(imenu.getIconWidth()-1,imenu.getIconHeight()-1,getWidth(),imenu.getIconHeight()-1);
	}
};

themenpanel = new JPanel();
themenscroll = new JScrollPane();

mainpanel.setBounds(0,0,getWidth(),getHeight());
mainpanel.setOpaque(false);
mainpanel.setLayout(null);

themenscroll.setBounds(240,60,600,590);
themenscroll.setOpaque(false);
themenscroll.setVerticalScrollBarPolicy(ScrollPaneConstants.VERTICAL_SCROLLBAR_AS_NEEDED);
themenscroll.setLayout(null);

themenpanel.setBounds(0,0,getWidth(),getHeight());
themenpanel.setPreferredSize(new Dimension(themenscroll.getWidth(),themenscroll.getHeight()));
themenpanel.setOpaque(false);
themenpanel.setLayout(null);

themenscroll.add(themenpanel);
mainpanel.add(themenscroll);

public void paintThemas(){
	Point loc = new Point(40,40);
	...
	themenpanel.setSize((int) loc.getX(),themenscroll.getWidth());
}

MfG

 

[sambalmueslie]

Hab dir mal ein kleines Beispiel gebaut wie dein Problem funktionieren könnte.
Aber vorher noch die Frage:
Das was du da machst klingt verdächtig nach einer Liste und die gibt es schon
How to Use Lists (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components)

So hier die Demo

package de.sambalmueslie.simple_scroll_pane_demo;

import java.awt.BorderLayout;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.UIManager;

public class SimpleScrollPaneDemo extends JFrame implements ActionListener {

	enum COMMANDS {
		ADD, REMOVE
	}

	/**  */
	private static final long serialVersionUID = -5511468445314871966L;;

	/**
	 * @param args
	 * @throws Exception
	 */
	public static void main(final String[] args) throws Exception {
		UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
		new SimpleScrollPaneDemo();
	}

	public SimpleScrollPaneDemo() {
		super("Simple scroll pane demo");

		// create the main panel
		setLayout(new BorderLayout());
		add(getControlPanel(), BorderLayout.NORTH);
		add(getMyScrollPane(), BorderLayout.CENTER);

		// do some setup stuff
		setSize(400, 400);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setVisible(true);
	}

	/**
	 * {@inheritDoc}.
	 */
	@Override
	public void actionPerformed(final ActionEvent e) {
		final COMMANDS cmd = COMMANDS.valueOf(e.getActionCommand());
		switch (cmd) {
		case ADD:
			// add a new item to the scroll pane
			getScrollContentPanel().add(new JLabel("Test item"));
			// revaliadte and repaint the content panel of the scroll pane
			// without this lines the changes will not appear
			getScrollContentPanel().revalidate();
			getScrollContentPanel().repaint();
			break;
		case REMOVE:
			// remove the last item from the scroll pane (only if there is one)
			final int index = getScrollContentPanel().getComponentCount();
			if (index > 0) {
				getScrollContentPanel().remove(index - 1);
				// revaliadte and repaint the content panel of the scroll pane
				// without this lines the changes will not appear
				getScrollContentPanel().revalidate();
				getScrollContentPanel().repaint();
			}
			break;
		default:
			break;
		}
	}

	private JPanel getControlPanel() {
		if (controlPanel == null) {
			controlPanel = new JPanel(new FlowLayout());
			// create a control button to add a new label
			final JButton addButton = new JButton("Add new item");
			addButton.setActionCommand(COMMANDS.ADD.name());
			addButton.addActionListener(this);
			controlPanel.add(addButton);
			// create a control button to remove a new label
			final JButton removeButton = new JButton("Remove last item");
			removeButton.setActionCommand(COMMANDS.REMOVE.name());
			removeButton.addActionListener(this);
			controlPanel.add(removeButton);
		}
		return controlPanel;
	}

	private JScrollPane getMyScrollPane() {
		if (myScrollPane == null) {
			myScrollPane = new JScrollPane(
					JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,
					JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
			myScrollPane.getViewport().add(getScrollContentPanel());
		}
		return myScrollPane;
	}

	private JPanel getScrollContentPanel() {
		if (scrollContentPanel == null) {
			scrollContentPanel = new JPanel();
			scrollContentPanel.setLayout(new BoxLayout(scrollContentPanel,
					BoxLayout.PAGE_AXIS));
		}
		return scrollContentPanel;
	}

	private JPanel controlPanel;
	private JScrollPane myScrollPane;
	private JPanel scrollContentPanel;

}

 

[TimeIsTheKey]

Hab dir mal ein kleines Beispiel gebaut wie dein Problem funktionieren könnte.
Aber vorher noch die Frage:
Das was du da machst klingt verdächtig nach einer Liste und die gibt es schon
How to Use Lists (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components)

Es soll auch eine Liste werden, aber ich möchte später in der Applikation ein eigenes GUI machen (sprich spezielle Bilder für Buttons, Listen-Items ect.) und das kriegt man am einfachsten mit JLabel hin.

So hier die Demo

package de.sambalmueslie.simple_scroll_pane_demo;

import java.awt.BorderLayout;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.UIManager;

public class SimpleScrollPaneDemo extends JFrame implements ActionListener {

	enum COMMANDS {
		ADD, REMOVE
	}

	/**  */
	private static final long serialVersionUID = -5511468445314871966L;;

	/**
	 * @param args
	 * @throws Exception
	 */
	public static void main(final String[] args) throws Exception {
		UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
		new SimpleScrollPaneDemo();
	}

	public SimpleScrollPaneDemo() {
		super("Simple scroll pane demo");

		// create the main panel
		setLayout(new BorderLayout());
		add(getControlPanel(), BorderLayout.NORTH);
		add(getMyScrollPane(), BorderLayout.CENTER);

		// do some setup stuff
		setSize(400, 400);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setVisible(true);
	}

	/**
	 * {@inheritDoc}.
	 */
	@Override
	public void actionPerformed(final ActionEvent e) {
		final COMMANDS cmd = COMMANDS.valueOf(e.getActionCommand());
		switch (cmd) {
		case ADD:
			// add a new item to the scroll pane
			getScrollContentPanel().add(new JLabel("Test item"));
			// revaliadte and repaint the content panel of the scroll pane
			// without this lines the changes will not appear
			getScrollContentPanel().revalidate();
			getScrollContentPanel().repaint();
			break;
		case REMOVE:
			// remove the last item from the scroll pane (only if there is one)
			final int index = getScrollContentPanel().getComponentCount();
			if (index > 0) {
				getScrollContentPanel().remove(index - 1);
				// revaliadte and repaint the content panel of the scroll pane
				// without this lines the changes will not appear
				getScrollContentPanel().revalidate();
				getScrollContentPanel().repaint();
			}
			break;
		default:
			break;
		}
	}

	private JPanel getControlPanel() {
		if (controlPanel == null) {
			controlPanel = new JPanel(new FlowLayout());
			// create a control button to add a new label
			final JButton addButton = new JButton("Add new item");
			addButton.setActionCommand(COMMANDS.ADD.name());
			addButton.addActionListener(this);
			controlPanel.add(addButton);
			// create a control button to remove a new label
			final JButton removeButton = new JButton("Remove last item");
			removeButton.setActionCommand(COMMANDS.REMOVE.name());
			removeButton.addActionListener(this);
			controlPanel.add(removeButton);
		}
		return controlPanel;
	}

	private JScrollPane getMyScrollPane() {
		if (myScrollPane == null) {
			myScrollPane = new JScrollPane(
					JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,
					JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
			myScrollPane.getViewport().add(getScrollContentPanel());
		}
		return myScrollPane;
	}

	private JPanel getScrollContentPanel() {
		if (scrollContentPanel == null) {
			scrollContentPanel = new JPanel();
			scrollContentPanel.setLayout(new BoxLayout(scrollContentPanel,
					BoxLayout.PAGE_AXIS));
		}
		return scrollContentPanel;
	}

	private JPanel controlPanel;
	private JScrollPane myScrollPane;
	private JPanel scrollContentPanel;

}

Ich habs doch nach dem gleichen Prinzip gemacht, oder übersehe ich etwas?
Ich hab mal die revalidate()- sowie repaint()-Methode hinzugefügt, aber das machte keinen Unterschied. Auch hab ich das themenpanel in die Viewport des JScrollPane geaddet, aber das half leider auch nicht weiter. Ich hab das Gefühl, dass ich etwas mit meiner PreferredSize-Methode etwas falsch mache.

 

[sambalmueslie]

Hm okay.. ich fürchte aber, das mit "kriegt man am einfachsten hin" stimmt so nicht ganz
den ersten Beweis hast du mit deiner Frage hier.

Was willst du denn konkret machen? Du kannst JList, JTable, JButton usw. total super customizen sei es mit eigenen Renderern oder eigener UI, so dass es Quatsch ist das noch mal neu zu erfinden.
Außer du hast noch einen guten Grund im Ärmel, den du mir bisher nicht genannt hast.

Wenn du mal den DefaultListCellRenderer anschaust. Der macht genau das was du vor hast.
Nämlich ein JLabel als Listenelement zurückzugeben.

Guck mal hier:

package de.sambalmueslie.simple_scroll_pane_demo;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Component;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.DefaultListCellRenderer;
import javax.swing.DefaultListModel;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.UIManager;

public class SimpleScrollPaneDemo2 extends JFrame implements ActionListener {

	enum COMMANDS {
		ADD, REMOVE
	}

	private class MyListCellRenderer extends DefaultListCellRenderer {
		/**  */
		private static final long serialVersionUID = 379634422827767442L;

		@Override
		public Component getListCellRendererComponent(final JList list,
				final Object value, final int index, final boolean isSelected,
				final boolean cellHasFocus) {
			final JLabel entry = (JLabel) super.getListCellRendererComponent(
					list, value, index, isSelected, cellHasFocus);
			// customize the label
			entry.setBackground(Color.GRAY);
			return entry;
		}
	}

	/**  */
	private static final long serialVersionUID = -5357841182272531662L;

	/**
	 * @param args
	 * @throws Exception
	 */
	public static void main(final String[] args) throws Exception {
		UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
		new SimpleScrollPaneDemo2();
	}

	public SimpleScrollPaneDemo2() {
		super("Simple scroll pane demo 2");

		// create the main panel
		setLayout(new BorderLayout());
		add(getControlPanel(), BorderLayout.NORTH);
		add(getMyScrollPane(), BorderLayout.CENTER);

		// do some setup stuff
		setSize(400, 400);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setVisible(true);
	}

	/**
	 * {@inheritDoc}.
	 */
	@Override
	public void actionPerformed(final ActionEvent e) {
		final COMMANDS cmd = COMMANDS.valueOf(e.getActionCommand());
		switch (cmd) {
		case ADD:
			// add a new item to the list
			myListModel.addElement("Test item");
			break;
		case REMOVE:
			// remove the last item from the list (only if there is one)
			final int index = myListModel.getSize();
			if (index > 0) {
				myListModel.remove(index - 1);
			}
			break;
		default:
			break;
		}
	}

	private JPanel getControlPanel() {
		if (controlPanel == null) {
			controlPanel = new JPanel(new FlowLayout());
			// create a control button to add a new label
			final JButton addButton = new JButton("Add new item");
			addButton.setActionCommand(COMMANDS.ADD.name());
			addButton.addActionListener(this);
			controlPanel.add(addButton);
			// create a control button to remove a new label
			final JButton removeButton = new JButton("Remove last item");
			removeButton.setActionCommand(COMMANDS.REMOVE.name());
			removeButton.addActionListener(this);
			controlPanel.add(removeButton);
		}
		return controlPanel;
	}

	private JList getMyList() {
		if (myList == null) {
			myListModel = new DefaultListModel();
			myList = new JList(myListModel);
			myList.setCellRenderer(new MyListCellRenderer());
		}
		return myList;
	}

	private JScrollPane getMyScrollPane() {
		if (myScrollPane == null) {
			myScrollPane = new JScrollPane(
					JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,
					JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
			myScrollPane.getViewport().add(getMyList());
		}
		return myScrollPane;
	}

	private JPanel controlPanel;
	private JList myList;
	private DefaultListModel myListModel;
	private JScrollPane myScrollPane;

}

 

[TimeIsTheKey]

Hm okay.. ich fürchte aber, das mit "kriegt man am einfachsten hin" stimmt so nicht ganz
den ersten Beweis hast du mit deiner Frage hier.

Was willst du denn konkret machen? Du kannst JList, JTable, JButton usw. total super customizen sei es mit eigenen Renderern oder eigener UI, so dass es Quatsch ist das noch mal neu zu erfinden.
Außer du hast noch einen guten Grund im Ärmel, den du mir bisher nicht genannt hast.

Ich glaube nicht, dass es daran liegt, dass ich JLabels benutze. Irgendwo muss ich noch etwas falsch machen. Ich hab dir ein Beispiel in Paint gemacht. Findest es im Anhang. Ich kann dir keinen guten Grund nennen, weil ich nicht weiss, was alles bei JList und seinen Elementen angepasst werden können. Ich möchte eigentlich nur eine Liste mit eigenem GUI. Dabei sollen die Elemente diverse Referenzen halten können. Die Elemente werden dynamisch mit Text befüllt.

Wenn du mal den DefaultListCellRenderer anschaust. Der macht genau das was du vor hast.
Nämlich ein JLabel als Listenelement zurückzugeben.

Guck mal hier:

package de.sambalmueslie.simple_scroll_pane_demo;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Component;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.DefaultListCellRenderer;
import javax.swing.DefaultListModel;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.UIManager;

public class SimpleScrollPaneDemo2 extends JFrame implements ActionListener {

	enum COMMANDS {
		ADD, REMOVE
	}

	private class MyListCellRenderer extends DefaultListCellRenderer {
		/**  */
		private static final long serialVersionUID = 379634422827767442L;

		@Override
		public Component getListCellRendererComponent(final JList list,
				final Object value, final int index, final boolean isSelected,
				final boolean cellHasFocus) {
			final JLabel entry = (JLabel) super.getListCellRendererComponent(
					list, value, index, isSelected, cellHasFocus);
			// customize the label
			entry.setBackground(Color.GRAY);
			return entry;
		}
	}

	/**  */
	private static final long serialVersionUID = -5357841182272531662L;

	/**
	 * @param args
	 * @throws Exception
	 */
	public static void main(final String[] args) throws Exception {
		UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
		new SimpleScrollPaneDemo2();
	}

	public SimpleScrollPaneDemo2() {
		super("Simple scroll pane demo 2");

		// create the main panel
		setLayout(new BorderLayout());
		add(getControlPanel(), BorderLayout.NORTH);
		add(getMyScrollPane(), BorderLayout.CENTER);

		// do some setup stuff
		setSize(400, 400);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setVisible(true);
	}

	/**
	 * {@inheritDoc}.
	 */
	@Override
	public void actionPerformed(final ActionEvent e) {
		final COMMANDS cmd = COMMANDS.valueOf(e.getActionCommand());
		switch (cmd) {
		case ADD:
			// add a new item to the list
			myListModel.addElement("Test item");
			break;
		case REMOVE:
			// remove the last item from the list (only if there is one)
			final int index = myListModel.getSize();
			if (index > 0) {
				myListModel.remove(index - 1);
			}
			break;
		default:
			break;
		}
	}

	private JPanel getControlPanel() {
		if (controlPanel == null) {
			controlPanel = new JPanel(new FlowLayout());
			// create a control button to add a new label
			final JButton addButton = new JButton("Add new item");
			addButton.setActionCommand(COMMANDS.ADD.name());
			addButton.addActionListener(this);
			controlPanel.add(addButton);
			// create a control button to remove a new label
			final JButton removeButton = new JButton("Remove last item");
			removeButton.setActionCommand(COMMANDS.REMOVE.name());
			removeButton.addActionListener(this);
			controlPanel.add(removeButton);
		}
		return controlPanel;
	}

	private JList getMyList() {
		if (myList == null) {
			myListModel = new DefaultListModel();
			myList = new JList(myListModel);
			myList.setCellRenderer(new MyListCellRenderer());
		}
		return myList;
	}

	private JScrollPane getMyScrollPane() {
		if (myScrollPane == null) {
			myScrollPane = new JScrollPane(
					JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,
					JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
			myScrollPane.getViewport().add(getMyList());
		}
		return myScrollPane;
	}

	private JPanel controlPanel;
	private JList myList;
	private DefaultListModel myListModel;
	private JScrollPane myScrollPane;

}

Ich werde mein Programm vielleicht noch umschreiben, muss mir erst noch anschauen was sich alles verändern lässt, aber eigentlich möchte ich gerne herausfinden, wieso es bei mir nicht funtkioniert. Ich mache soweit ich das sehe keine grossen Fehler. Woran merkt das JScrollPane, dass eine Scrollbar nötig ist?

Edit: Anhang vergessen ...

 

[irgendjemand]

hast du dir schon mal die DOC zu JScrollPane durchgelesen ?

um eine JComponent in einem JScrollPane anzuzeigen verwendet man die methode JScrollPane.setViewportView(Component) und nicht Container.add(Component) ...

 

[TimeIsTheKey]

hast du dir schon mal die DOC zu JScrollPane durchgelesen ?

um eine JComponent in einem JScrollPane anzuzeigen verwendet man die methode JScrollPane.setViewportView(Component) und nicht Container.add(Component) ...

Danke für den Hinweis. Ich hab es wie in meinem 2. Posting beschrieben schon versucht, aber es brachte nichts. Ich lass es jetzt aber mal so drin. Hat allerdings wie gesagt noch nicht zur Problemlösung verholfen. Danke dir trotzdem. ^^

Edit:

Ich hab mal eingestellt, dass die Scrollbar immer angezeigt werden soll und sie ist nicht da. Da muss irgendwo ein grober Fehler versteckt sein. :/

Edit 2:

Ein blöder Fehler ist mir in paintThemas unterlaufen. Ich hab dort anstatt in das Panel in das ScrollPane geaddet. Habs abgeändert, aber nun wird nichts im ScrollPane angezeigt.

Edit 3:

Ich hab das Layout des ScrollPane auf null gesetzt. Als ich das weggenommen hab, wurde das Panel angezeigt. Mit setPreferredSize und revalidate und repaint funktioniert es nun doch. :toll:

Danke für eure Hilfe!

 

[irgendjemand]

revalidate() und repaint() brauchst du glaube icht nicht da das bereits im code vom JScrollPane steht