JList JScrollPane Problem

[Whitey]

Hallo,

ich habe ein Problem mit der Reaktion des Scrollpane's bei einer JList. Die JList ist in ein JScrollPane eingebettet. Ich habe mir meinen eigenen ListRenderer geschrieben, damit ein Icon und der Name angezeigt werden können.

Ich habe ein Element das einen sehr langen Namen hat und über den rechten Rand der jList hinausreicht. Das Problem ist, dass der Scrollbar darauf nicht reagiert. D.h. ich kann nicht ans Ende des Namens scrollen.

Mir sind die Ideen ausgegangen, vllt. habt ihr ja eine.

ListRenderer:

public class ListRenderer extends DefaultListCellRenderer {

	public Component getListCellRendererComponent(JList list, Object value, int index, boolean isSelected,
						boolean cellHasFocus) {
	JLabel nameLabel=new JLabel();
	JPanel panel=new JPanel();
	JLabel iconLabel=new JLabel();
	
	panel.setBackground(new Color(185,185,185));
	
	iconLabel.setIcon(new ImageIcon(this.getClass().getResource(Resource.getResource("ListRenderer.ImageIcon"))));
	
	Images image = (Images)value;
	
	String imageSizeName = CalculateSize.checkSize(image.getSize());
	int imageSize = CalculateSize.getExtension().intValue();
	
	nameLabel.setText(image.getName()+" Size: "+imageSize+" "+imageSizeName);
	
	if(image.getStatus()==1){
		nameLabel.setForeground(new Color(208,208,208));
	}
	else
		nameLabel.setForeground(new Color(255,255,255));
	
	panel.setLayout(new BorderLayout());
	panel.add(iconLabel,BorderLayout.WEST);
	panel.add(nameLabel,BorderLayout.CENTER);
	panel.setSize((iconLabel.getPreferredSize().width)+(nameLabel.getPreferredSize().width),17);
	
	setComponentOrientation(list.getComponentOrientation());
	return panel;
	}

}

JScrollPane Init:

	private JScrollPane getSpImages() {
		if (spImages == null) {
			spImages = new JScrollPane();
			spImages.setViewportView(getListImages());
			spImages.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_ALWAYS);
			spImages.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_ALWAYS);
		}
		return spImages;
	}

JList Init:

	private JList getListImages() {
		if (listImages == null) {
			listImages = new JList(listModel);
			listImages.setPreferredSize(new Dimension(200, 200));
			listImages.setBackground(new Color(108,108,108));
			listImages.setForeground(Color.white);
			listImages.setFont(new Font("Tahoma", Font.PLAIN, 12));
		}
		
		return listImages;

Dank für jede Hilfe.

Gruß Whitey

 

[Wildcard]

Dein Renderer ist ziemlich daneben. Die Instanzierung von Objekten solltest du unbedingt vermeiden.
Die ScrollPane interessiert sich für die PreferedSize deiner Componente, wenn die nicht zur tatsächlich benötigten Größe passt ist das problematisch.

 

[Whitey]

Kritik ist immer willkommen, da ich noch Anfänger bin. Aber bitte etwas genauer, was meinst du genau? Wenn ich überhaupt keine Objekte erzeugen darf, bringt mir der Renderer nichts.

Danke für deine Hilfe. Werde den Weg mit der PreferredSize weiterverfolgen, klingt logisch.

Hab deine Kritik so interpretiert:

public class ListRenderer extends DefaultListCellRenderer {
	
	JLabel nameLabel=new JLabel();
	JPanel panel=new JPanel();
	JLabel iconLabel=new JLabel();
	Color lightGray = new Color(185,185,185);
	Color white = new Color(255,255,255);
	Color darkGray = new Color(108,108,108);
	ImageIcon icon = new ImageIcon(this.getClass().getResource(Resource.getResource("ListRenderer.ImageIcon")));

	public Component getListCellRendererComponent(JList list, Object value, int index, boolean isSelected,
						boolean cellHasFocus) {
	
	panel.setBackground(darkGray);
	
	iconLabel.setIcon(icon);
	
	Images image = (Images)value;
	
	String imageSizeName = CalculateSize.checkSize(image.getSize());
	int imageSize = CalculateSize.getExtension().intValue();
	
	nameLabel.setText(image.getName()+" Size: "+imageSize+" "+imageSizeName);
	
	if(image.getStatus()==1){
		nameLabel.setForeground(lightGray);
	}
	else
		nameLabel.setForeground(white);
	
	iconLabel.setHorizontalAlignment(JLabel.LEFT);
	nameLabel.setHorizontalAlignment(JLabel.LEFT);
	panel.setLayout(new BorderLayout());
	panel.add(iconLabel,BorderLayout.WEST);
	panel.add(nameLabel,BorderLayout.CENTER);
	//panel.setSize((iconLabel.getPreferredSize().width)+(nameLabel.getPreferredSize().width),17);
	
	setComponentOrientation(list.getComponentOrientation());
	return panel;
	}

}

 

[Wildcard]

Meine Kritik hast du richtig interpretiert.
Die Objekte du du brauchst werden einmalig instanziert und wiederverwendet.
Hintergrund der Sache ist das ein Renderer sehr häufig aufgerufen wird und Objekterzeugung eine sehr teure Operation ist.