Performanz beim Bilderzeichnen, >1000 bilder

[cmmi]

Hallo,

ich lese jetzt schon seit ein paar Tagen hier mit, und habe auch schon die Tutorials durch. Die Suche hat mir auch keine brauchbaren Ergebnisse gebracht.
Also, was ist mein Problem? Ich habe eine Liste von Sequenzen, jede Sequenz ist wiederum eine Liste an Symbolen. Für jedes Symbol gibt es eine Grafikdatei.

Ich möchte nun diese Sequenzliste anzeigen, mit der Möglichkeit rein und rauszuzoomen sowie natürlich scrollen, da die liste nicht komplett auf den Bildschirm passt. Ach ja, die Bilder müssen farblich markierbar sein, da das ganze zum Anzeigen von Mustern dienen soll. Daher handelt es sich bei den Bildern um transparente pngs.

Was habe ich schon probiert: eine JList mit eigenem CellRenderer, das war aber performance technisch nicht wirklich toll, da war kein "flüssiges" betrachten und nutzen der Liste möglich.
Momentan spiele ich mit einer eigenen Component welche alles selber zeichnet. Das ist schon ganz in Ordnung, aber auch noch nicht so wahnsinnig schnelle. Habe eben mal versucht das Zeichnen einer einzelnen Sequenz in einen eigenen Thread auszulagern, aber das scheint auch nicht wirklich zu klappen.

Daher meine Frage, wie bekommt man eine solche Datenstruktur sinnvoll und vor allem flott auf den Schirm? Jemand eine Idee? Wenn nötig poste ich gerne den Code, aber vielleicht hat ja schon jemand eine generelle Idee?

Danke & greet,
Sebastian

 

[Wildcard]

Fang bloß nicht mit Threads oder ähnlichem an. Zeig doch mal bitte den Renderer den du geschrieben hast.
Ein Umstieg von PNG auf GIF, oder das explizite verwenden eines 'compatible' Images (Bitmaskentransparenz), kann die Sache übrigens stark beschleunigen.

 

[Guest]

So, mein ListRenderer sieht so aus:

public class SequenceListCellRenderer2 implements ListCellRenderer {

	public static final int	SequenceListRowRenderer		= 1;
	public static final int	SequenceListHeaderRenderer	= 2;
	public static final boolean	STRINGVIEW	= true;
	public static final boolean	ICONVIEW		= false;
	private boolean					simpleView;
	private int	renderType;
	private Dimension	headerPanelDim;
	private Dimension	headerLabelDim;
	private Font	font;
	private final int			fontSize							= 12;
	private final int			fontStyle						= Font.PLAIN;
	private final String		fontName							= "Arial";
	private int rowHeight;
	private Hashtable<Symbol, Icon> iconCache;
	private int length = -1;
	private Dimension labelDim;

	public SequenceListCellRenderer2(int renderType, boolean simpleView) {

		assert renderType >= SequenceListRowRenderer;
		assert renderType <= SequenceListHeaderRenderer;
		this.renderType = renderType;
		this.simpleView = simpleView;
		this.iconCache = new Hashtable<Symbol, Icon>();
		if (simpleView) {
			this.rowHeight = 20;
			this.headerPanelDim = new Dimension(60,20);
			this.headerLabelDim = new Dimension(60,20);
		} else {
			this.rowHeight = 60;
			this.headerPanelDim = new Dimension(60,60);
			this.headerLabelDim = new Dimension(60,60);
		}
		this.font = new Font(this.fontName, this.fontStyle, this.fontSize);
	}

	public Component getListCellRendererComponent(JList list, Object value,
			int index, boolean isSelected, boolean cellHasFocus) {

		switch (this.renderType) {
			case SequenceListRowRenderer:
				return this.getListRowRendererComponent(list, value, index,
						isSelected, cellHasFocus);
			case SequenceListHeaderRenderer:
				return this.getListHeaderRendererComponent(list, value, index,
						isSelected, cellHasFocus);
			default:
				return null;
		}

	}

	private Component getListHeaderRendererComponent(JList list, Object value,
			int index, boolean isSelected, boolean cellHasFocus) {

		Sequence sequence = (Sequence) value;

		JPanel panel = new JPanel(new FlowLayout(FlowLayout.LEFT, 0, 2));
		panel.setMinimumSize(this.headerPanelDim);
		panel.setPreferredSize(this.headerPanelDim);
		panel.setMaximumSize(this.headerPanelDim);

		JLabel header = new JLabel(sequence.getIdentifier());
		int sizeNeeded = sequence.getIdentifier().length() * 12;
		if (sizeNeeded > this.headerLabelDim.getWidth()
				|| sizeNeeded > this.headerPanelDim.getWidth()) {
			this.headerLabelDim.setSize(sizeNeeded, this.headerLabelDim
					.getHeight());
			this.headerPanelDim.setSize(sizeNeeded, this.headerPanelDim
					.getHeight());
		}
		header.setMinimumSize(this.headerLabelDim);
		header.setPreferredSize(this.headerLabelDim);
		header.setMaximumSize(this.headerLabelDim);
		header.setFont(this.font);

		panel.add(header);
		return panel;
	}

	private Component getListRowRendererComponent(JList list, Object value,
			int index, boolean isSelected, boolean cellHasFocus) {

		Sequence sequence = (Sequence) value;
		SequenceMarker marker;

		JPanel panel = new JPanel(new FlowLayout(FlowLayout.LEFT, 0, 2));
		
		if (this.length == -1 || this.labelDim == null) {
			if (this.simpleView) {
				length = sequence.getLength() * 20;
				labelDim = new Dimension(20, this.rowHeight - 4);
			} else {
				length = sequence.getLength() * (sequence.getSymbol(0).getIcon().getIconWidth());
				labelDim = new Dimension(sequence.getSymbol(0).getIcon().getIconWidth(), this.rowHeight - 4);
			}
		}
		
		JLabel symbol;
		Symbol sym;
		Icon icon;
		for (int i = 0; i < sequence.getLength(); i++) {
			symbol = new JLabel();
			if (this.simpleView) {
				symbol.setText(sequence.getSymbol(i).getIdentifier());
				symbol.setFont(this.font);
			} else {
				//symbol.setIcon(sequence.getSymbol(i).getIcon());
				sym = sequence.getSymbol(i);
				icon = this.iconCache.get(sym);
				if (icon == null) {
					icon = sym.getIcon();
					this.iconCache.put(sym, icon);
				}
				symbol.setIcon(icon);
			}
			symbol.setMinimumSize(labelDim);
			symbol.setPreferredSize(labelDim);
			symbol.setMaximumSize(labelDim);
			symbol.setHorizontalAlignment(SwingConstants.CENTER);
			marker = sequence.getMarkerAtPosition(i);
		
			if (marker != null) {
				symbol.setBackground(marker.getPattern().getColor());
				symbol.setBorder(BorderFactory.createLineBorder(marker.getPattern().getColor(), 2));
			} else {
				symbol.setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5));
			}
			symbol.setOpaque(true);
			panel.add(symbol);
		}

		panel.setMinimumSize(new Dimension(length, this.rowHeight));
		panel.setPreferredSize(new Dimension(length, this.rowHeight));
		panel.setMaximumSize(new Dimension(length, this.rowHeight));

		return panel;
	}

[/code][/quote]

 

[Wildcard]

Ein Renderer legt keine neuen Components an! Erstell dir genau die 2-3 Labels die du brauchst, und setz deren Text/Icon entsprechend immer wieder neu.
Die JComponents werden nur zum Zeichnen verwendet, können als problemlos mehrfach verwendet werden.

 

[cmmi]

Ein Renderer legt keine neuen Components an! Erstell dir genau die 2-3 Labels die du brauchst, und setz deren Text/Icon entsprechend immer wieder neu.
Die JComponents werden nur zum Zeichnen verwendet, können als problemlos mehrfach verwendet werden.

Danke, da hast Du wohl recht, aber es wird trotzdem nicht wirklich schneller. Habe gerade alles soweit entfernt.

Es sind eben immer noch über 4000 Bilder, welche er da rendert, auch wenn es nur ca 20 verschiedene sind. Gibt es doch noch irgendetwas um das zu beschleunigen?

 

[Wildcard]

Im Prinzip dürften ja nur wenige Icons gleichzeitig zu sehen sein und nur die müssen gerendert werden.
Wie gesagt, compatible images sind wesentlich schneller als Bilder mit komplettem Alphakanal. Versuch mal das, oder pack den Profiler aus um zu sehen ob du wirklich die Zeit beim Rendern der Bilder verlierst.

 

[cmmi]

Im Prinzip dürften ja nur wenige Icons gleichzeitig zu sehen sein und nur die müssen gerendert werden.
Wie gesagt, compatible images sind wesentlich schneller als Bilder mit komplettem Alphakanal. Versuch mal das, oder pack den Profiler aus um zu sehen ob du wirklich die Zeit beim Rendern der Bilder verlierst.

@Wildcard Danke für den Tipp mit den compatible images, damit funktioniert es jetzt ohne Probleme. Zumindest mit meiner selbsgestrickten Component. Aber die gefällt mir optisch eh besser als die Liste.
Auch Zoomen funktioniert ohne zeitliche Verzögerungen, genial.

Danke nochmal. =)

Hier noch der Code den ich zum laden der Bilder verwende, falls es jemanden interessiert:

public class LoadImagesThread extends Thread {

	private HashMap<Symbol, BufferedImage> cache;
	private Symbol	symbol;
	
	public LoadImagesThread(ExtendedThreadGroup threads, HashMap<Symbol, BufferedImage> images, Symbol symbol) {
		super(threads, "loading image from " + symbol);
		this.cache = images;
		this.symbol = symbol;
	}
	
	public void run() {
		try {
			/**
			 * Idea from:
			 * @see [url]http://www.curious-creature.org/2005/10/04/drag-and-drop-effects-part-1-swing-demo/[/url]
			 */
			BufferedImage source = ImageIO.read(new File(this.symbol.getImagePath()));
			GraphicsConfiguration config = GraphicsEnvironment.getLocalGraphicsEnvironment().getDefaultScreenDevice().getDefaultConfiguration();
			BufferedImage converted = config.createCompatibleImage(source.getWidth(), source.getHeight(), source.getTransparency());
			Graphics g = converted.createGraphics();
			g.drawImage(source, 0, 0, null);
			g.dispose();
			this.cache.put(this.symbol, converted);
		} catch (IOException e) {
			e.printStackTrace();
		}
	}

}