JTable nur sichtbaren Bereich laden

[rbrose]

Hallo,

ich habe eine Tabelle mit 1000 Einträgen. Die Tabelle hat z.B. 2 Spalten: "Vorschau" und "Name".

Ich will jetzt das meine Tabelle mit allen Namen aufgebaut wird, aber die "Vorschauen" nur geladen werden wenn man nach unten scrollt. Heißt, die Vorschauen (JPEG Icons) werden nur geladen wenn man die Zeile sieht.


Wie macht man sowas?

Vielen Dank im voraus.

 

[André Uhres]

die Vorschauen (JPEG Icons) werden nur geladen wenn man die Zeile sieht

Wir können über einen ComponentListener die Bewegungen abfangen (componentMoved, componentResized) und dann den sichtbaren Bereich mittels table.getVisibleRect() und table.rowAtPoint(..) abtasten. Falls dort ein Bild noch nicht geladen ist, laden wir es.

 

[ARadauer]

Das AbstractTableModel fordert die Zeilen an die sie darstellen will... nach dem kann man sich richten

 

[rbrose]

Vielen Dank! Das AbstractTableModel ruft "public Object getValueAt(int rowIndex, int columnIndex)" wirklich nur für die Zeilen die sichtbar sind. Das ist schon mal gut.

Aber ich habe noch ein anderes Problem. Meine Tabelle ist in einer JInternalFrame. Wenn ich das JInternalFrame verschiebe oder irgendwo in der Tabelle klicke wird immer die Methode "public Object getValueAt(int rowIndex, int columnIndex)" ausgerufen.

Wieso wird es soo oft aufgerufen? Ist das nicht ein Performance verlust wenn immer wieder alles neu aufgebaut wird obwohl ich nicht scrolle oder neue Daten reinkommen? Kann ich das irgendwie abstellen?

Das Problem ist nämlich dieses, dass ich die Bilder per RMI vom Server laden muss. Wo mache ich das am besten? wenn ich das in der "public Object getValueAt(int rowIndex, int columnIndex)" Methode mache ... dann wird viel zu viel Traffic im Netzwerk stattfinden das nicht sein muss.

Ich hoffe ich habe mein Problem gut erklärt.

 

[André Uhres]

wenn ich das in der "public Object getValueAt(int rowIndex, int columnIndex)" Methode mache ... dann wird viel zu viel Traffic im Netzwerk stattfinden das nicht sein muss.

Wieso? Du brauchst doch die Bilder, die schon geladen sind, nicht nochmal zu laden. Es ist egal wie oft er da durch geht, jedes Bild wird immer nur einmal über RMI geladen.

 

[rbrose]

Das heißt im AbstractTableModel wäre es der Ideale Platz um eine RMI anfrage zu starten? Kann man das irgendwie per Thread machen? Und somit viele Parallel zu starten, oder ist eine nach einander Methode besser?

Du siehst ich habe viele Fragen ;-) Bin mir bei der Tabelle noch nicht ganz sicher wie man das Elegant man.

Wie kann ich den in der "public Object getValueAt(int rowIndex, int columnIndex)" Methode überhaupt überprüfen ob das Bild schon geladen ist oder nicht?

 

[André Uhres]

Das heißt im AbstractTableModel wäre es der Ideale Platz um eine RMI anfrage zu starten? Kann man das irgendwie per Thread machen? Und somit viele Parallel zu starten, oder ist eine nach einander Methode besser?

Du siehst ich habe viele Fragen ;-) Bin mir bei der Tabelle noch nicht ganz sicher wie man das Elegant man.

Wie kann ich den in der "public Object getValueAt(int rowIndex, int columnIndex)" Methode überhaupt überprüfen ob das Bild schon geladen ist oder nicht?

Ob wir den Platz direkt im Model wählen, oder außerhalb in einem Listener, ist imho relativ unerheblich. Mehrere Threads würde ich erst in Betracht ziehen, wenn ein wahrnehmbares Performance Problem besteht. Um zu überprüfen, ob das Bild schon geladen ist, können wir den Typ des Wertes im Model abfragen. Wir setzen als Wert erstmal einen String, der das Bild identifiziert. Nur wenn der Wert ein String ist (instanceof String), laden wir das Bild und ersetzen den String dann durch ein Icon.

 

[rbrose]

Mhh ich glaube ich verstehe es falsch oder ich mache das Falsch:

So ist es bei mir implementiert:

Im TableModel:
[JAVA=42]
@Override
public Object getValueAt(int rowIndex, int columnIndex) {
StorageElementTransfer ser = folderFiles.get(rowIndex);
switch (columnIndex) {
case VORSCHAU_COLUMN:
return ser.getPreviewPath;
case NAME_COLUMN:
return ser.getName();
default:
return "";
}
}
[/code]


Im TableCellRenderer wird dann folgendes gemacht:

[JAVA=42]
@Override
public Component getTableCellRendererComponent(JTable table,
Object value, boolean isSelected, boolean hasFocus, int row,
int column) {
super.getTableCellRendererComponent(table, value, isSelected, hasFocus,
row, column);

if(value instanceof String) {
System.out.println("Image Setzen!!!");
setIcon(preview);
}

return this;
}
[/code]

value ist aber immer vom Typ String obwohl ich ein Icon setzt.

Denkfehler?

 

[André Uhres]

Versuch's mal so:

case VORSCHAU_COLUMN:
    return ser.loadPreview();

public Icon loadPreview(){
    if(preview == null){
        //load from previewPath
    }
    return preview;
}

Da wir in dem Fall in der Klasse "StorageElementTransfer" getrennte Felder für "previewPath" und "preview" haben, entfällt natürlich die Abfrage auf den Datentyp.
Einen eigenen IconRenderer brauchst du übrigens nicht, da in JTable schon einer eingebaut ist 😉

 

[rbrose]

So irgendwie stimmt etwas nicht.

Mein code vom AbstractTableModel:

[JAVA=42]
public class FileTableModel extends AbstractTableModel {

public static final int VORSCHAU_COLUMN = 0;
public static final int NAME_COLUMN = 1;
public static final int SIZE_COLUMN = 2;
public static final int FORMAT_COLUMN = 3;
public static final int COLORSPACE_COLUMN = 4;
public static final int DPI_COLUMN = 5;
public static final int BOUNCE_COLUMN = 6;

private String[] columnNames = {"Vorschau", "Name / Pfad", "Größe", "Format", "Farbraum", "DPI", "Abmessugen (mm)"};

private List<StorageElementTransfer> folderFiles = new ArrayList<StorageElementTransfer>();

private Icon preview;

public List<StorageElementTransfer> getFolderFiles() {
return folderFiles;
}

public void setFolderFiles(List<StorageElementTransfer> folderFiles) {
this.folderFiles = folderFiles;
}

@Override
public String getColumnName(int column) {
return columnNames[column];
}


@Override
public int getColumnCount() {
return 7;
}


@Override
public int getRowCount() {
return folderFiles.size();
}

public Icon loadPreview(String type){
if(preview == null){
if("DIR".equals(type)) {
preview = new ImageIcon(getClass().getResource("/de/develop/ui/resources/icons/FolderForTable.png"));
} else if ("IMG".equals(type)) {
//Hier wird später per Socket das Preview vom Server geholt.
preview = new ImageIcon(getClass().getResource("/de/develop/ui/resources/icons/jpeg.png"));
} else {
preview = new ImageIcon(getClass().getResource("/de/develop/ui/resources/icons/FolderForTable.png"));
}
return preview; //load from previewPath
}
return preview;
}

@Override
public Object getValueAt(int rowIndex, int columnIndex) {
StorageElementTransfer ser = folderFiles.get(rowIndex);
switch (columnIndex) {
case VORSCHAU_COLUMN:
return loadPreview(ser.getType());
case NAME_COLUMN:
return ser.getName();
default:
return "";
}
}

}
[/code]

Das prüfen auf if(preview == null) geht hier nicht, weil die Tabelle mehrere Zeilen hat, bei der zweiten Zeile ist preview aber nicht mehr NULL weil es ja schon mal erstellt worden ist.
Wenn ich das so mache ist ein Icon für alle Zeilen gleich, brauche aber Verschiedene Icons für jede Zeile.

Wie mache ich das? Eine HashMap mit vielen Icons drin die per "rowIndex" und "columnIndex" angesteuert werden?


> Einen eigenen IconRenderer brauchst du übrigens nicht, da in JTable schon einer eingebaut ist

Wenn ich das so mache ohne IconRenderer, kriege ich in der Spalte kein Icon sondern ein "file://bla/bla/bla".

 

[André Uhres]

Du ja auch so geschrieben:

case VORSCHAU_COLUMN:
    return loadPreview(ser.getType());

statt so wie ich vorgeschlagen habe:

case VORSCHAU_COLUMN:
    return ser.loadPreview();

Zum IconRenderer geben wir für die betreffende Spalte im TableModel bei "getColumnClass" einfach "Icon.class" zurück.

 

[rbrose]

Vielen Dank! Jetzt habe ich es verstanden und es funktioniert.

Eine Sache stört mich noch. Wieso wird die komplette Tabelle in einem JInternalFrame beim verschieben des JInternalFrame repainted?

Kann ich das irgendwie abstellen? und die Tabelle NUR repainten wenn ich das will?

 

[hdi]

Component#setIgnoreRepaint(boolean)

aber... die Table muss ja repainted werden, wenn sich die Position auf dem Bildschirm verändert.
Die Pixel im Bildschirm sind ja statische Leuchtioden, die kannst du nicht verschieben. Wenn die Tabelle woanders ist, sind das andere LED's.