RowSorter beim JTable bringt Programm durcheinander

[Guest]

Hi,
ich habe ein kleines Problem:
Ich habe einen Drag and Drop fähigen Jtable und habe bei diesem aktiviert daswenn man oben auf den spaltenname klickt die spalten sortieren kann.
Desweiteren habe ich in meinem Programm einen Button mit dem man die grade markierte zeile aus dem Jtable löschen kann...nun habe ich jedoch das Problem das wenn ich die Spalten sortiere das löschen durcheinandere kommt...wenn ich eine Zeile markiere wird eine andere Zeile gelöscht(die an dem das element vorher war)
Kann man das Problem irgendwie umgehen?

Hier mal ein paar quelltext ausschnitte

//Funktion des Delete Buttons
if(DnD.table.getRowCount()==0)
            	{
            		JOptionPane.showMessageDialog(null, "Keine Elemente zum Löschen vorhanden!");
            	}

            	else if(DnD.table.getSelectedRow()==-1)
            	{
            		JOptionPane.showMessageDialog(null, "Wählen Sie Bitte ein Objekt zum Löschen!");
            	}
            	else{
            		
            	DnD.model.removeRow(DnD.table.getSelectedRow());
            	
            	}

    TableRowSorter<TableModel> sorter = new TableRowSorter<TableModel>();
     
     table.setRowSorter( sorter );
    
     sorter.setModel( model );

MFG

 

[hdi]

Ahhh so einen Fehler hatte ich auch mal, moment... *workspace durchsuch*

...HAHAHAHA! :lol: Habs gefunden, 1:1 diese Zeile in einem meiner Projekte, mitsamt Kommentar:

table.setAutoCreateRowSorter(true); // <------- HUSO !!!

So, mal schauen wo war jetz das Problem...

Ah genau, also bei mir war es so, dass ich nach jedem Verändern des Table-Models
(Hinzufügen/Löschen/Zelleninhalte ändern), wieder machen musste:

table.setAutoCreateRowSorter(true);

Da du scheinbar einen eigenen hast, musst du halt speziell diesen immer neu setzen.

Hoffe das hilft auch bei dir, viel Glück :toll:

 

[HoaX]

ich denke ehr dass er TableSorter#convertRowToModelIndex oder so aufrufen will, da ja der angezeigte index nichtmehr mit dem index im model übereinstimmt

 

[Guest]

@ hdi: deine methode funktioniert leider nicht

@Hoax: genau sowas brauche ich...werd mir die funktion mal anschauen

 

[Guest]

irgendwie komm ich mit der FKt nicht so richtig klar...die Fkt gibt einen Wert zurück,nur was soll ich damit?kann mir wer helfen?

int test=DnD.table.convertRowIndexToModel(DnD.model.getRowCount()-1);

 

[Ebenius]

irgendwie komm ich mit der FKt nicht so richtig klar...die Fkt gibt einen Wert zurück,nur was soll ich damit?kann mir wer helfen?

int test=DnD.table.convertRowIndexToModel(DnD.model.getRowCount()-1);

Die ergibt auch keinen Sinn. In convertRowIndexToModel wirft man einen View-Index rein. Hier wird aber einen Modell-Index übergeben.

In Zeile 14 (oben) brauchst Du die Methode!

DnD.model.removeRow(convertRowIndexToModel(DnD.table.getSelectedRow()));

Ebenius

 

[Guest]

danke für die hilfe dasfunktioniert ohne probleme

 

[Guest]

hab noch was gefunden was der sorter durcheinandere gebracht hat

und zwar beeinträchtigt er auch das DragandDrop in meinem Jtable

static void enableRowDragging() {
        
	table.addMouseMotionListener(new MouseMotionAdapter() {

    @Override
    public void mouseDragged(MouseEvent evt) {
    	rowTo = table.convertRowIndexToModel(table.rowAtPoint(evt.getPoint()));
        if (rowTo != rowFrom && rowTo > -1 && rowTo < model.getRowCount()) {
        	model.moveRow(rowFrom, rowFrom, rowTo);
            rowFrom = rowTo;
            }
        }
    });
    table.addMouseListener(new MouseAdapter() {

    @Override
    public void mousePressed(MouseEvent evt) {
    	rowFrom = table.convertRowIndexToModel(table.rowAtPoint(evt.getPoint()));
        }
    });
}

hier ist jedoch schon ein Indexconverter drinn...aber es geht tritzdem nicht mehr richtig wenn ich ein element verschieben will dann ändert sich mit jedem element was ich nach unten komme die komplette liste(alle elemente springen umher)

kann man das auch noch irgendwie beheben?

 

[Ebenius]

Das verstehe ich nicht. Du willst die Zeilen vertauschen bei mouseDragged? Willst Du nicht lieber bei mouseReleased tauschen? Sonst kann das ja nicht funktionieren.

Ebenius

 

[Guest]

wenn ich das dragged in released ändere geht garnix mehr

mfg

 

[Guest]

keiner ne idee?

 

[Guest]

hat denn wirklich keiner ne idee?

 

[Ebenius]

Ich sag doch, es liegt am mouseDragged... Lass Dir doch mal bei mouseDragged per "System.out.println(...)" was ausgeben. Und dann stell Dir for, dass Du bei jedem dieser Events eine Zeile mit der Original-Zeile aus mousePressed vertauschst. Du wirst sehen, dass das keinen Sinn ergeben kann!

 

[Guest]

ok ich schaus mir mal an...aber wie gesagt ich habe schon das dragged in released geändert gehabt und da ging das drag and drop nicht mehr

 

[Guest]

also ich hab das teil jetzt mal mit System.out beobachtet

Der Ganze index haut nach dem Sortieren nicht mehr hin
in der Obenren hälfte sind nur ungerade zaheln(1,3,5,7,9 usw) und in der unteren Hälfte sind nur grade zahlen(0,2,4,6,8 usw)
ich müsste also nach dem ordnen den index irgendwie wieder neu schreiben oder irgendsowas
hat da wer ne idee?

 

[Ebenius]

Das Problem ist nicht, dass der Index falsch ist, sondern dass Du falsch einfügst. Da bin ich ziemlich sicher. Kannst Du ein kleines kompilierbares Beispiel posten, mit dem ich spielen kann?

 

[Guest]

ich glaub ein beispiel aus dem programm zu machen dauert zu lange...hab einfach mal das komplette prog hochgeladen

http://rapidshare.com/files/189640747/Audioplayer_beta.rar

ich hoffe das geht auch so
eventuell findest du ja noch ein paar bugs

MFG

 

[Ebenius]

[off topic] Nur so zur Info: Wenn Du Dich bei java-forum.org als Nutzer anmeldest, hast Du eigene Dateien in die Du was hochladen kannst, ohne dass Werbung beim Download kommt.

 

[Ebenius]

Bis zu Deinem DnD-Problem bin ich bisher noch nicht gekommen...

Hinweise

1. Keine Umlaute in Java benutzen. Das geht zwar, ist aber Mist, weil es nicht so gut auszutauschen geht. Die Menü.java konnte ich in meinem UTF-8 Linux mit Ark nicht extrahieren... Generell auch im Source-Code möglichst auf ASCII-7bit beschränken. Das erspart Ärger! Danke L-etron-X für die Hilfe beim Entpacken.
2. Methodennamen und Variablennamen camelCase (erstes Wort klein) schreiben: "dateiEinlesen()", "saveOrCancel"
3. Code Formatter verwenden; am besten 2 SPC zur Einrückung und Zeilenlänge begrenzen auf 78.
4. Solche Ausdrücke andersherum, das spart ggf. den Null-Check: cmd.equals("Beenden") → "Beenden".equals(cmd)
5. Unnötige Casts vermeiden. Die Klasse Menü ist ein ActionListener (Bsp. Zeile 19), da muss kein cast hin.
6. MAIN nicht großschreiben. Da fühle ich mich ja ANGESCHRIEN! Main tut's völlig.
7. Immer die Software in Packages tun, nie im default package. Wie das geht? Sun Java™ Tutorial: Learning the Java Language: Packages
8. Entweder Du arbeitest mit Dateien (java.io.File) oder mit URLs (java.net.URL). Beides komisch zu vermischen ist nicht gut: "java.net.URISyntaxException: Illegal character in path at index 8: file:/C:\Dokumente und Einstellungen\Seidemann\Desktop\15-Crusher.wav"
9. Exceptions nicht nur abfangen sondern auch richtig handhaben (MessageBox), zum Beispiel beim Datei einlesen.

 

[HoaX]

Ist es nicht unsinn eine sortierte liste von hand neu sortieren zu wollen? einzige möglichkeit die mir dafür einfällt: daten schon sortiert im model halten und auf TableRowSorter verzichten. Dann is aber auch nixmehr mit auf spaltenname klicken zum sortieren lassen ...

 

[Ebenius]

Dein Sortierproblem bekommst Du so gelöst:

table.addMouseMotionListener(new MouseMotionAdapter() {

  public void mouseDragged(MouseEvent evt) {
    final int viewRowTo = table.rowAtPoint(evt.getPoint());
    if (viewRowTo > -1) {
      final int modelRowFrom = table.convertRowIndexToModel(viewRowFrom);
      final int modelRowTo = table.convertRowIndexToModel(viewRowTo);
      if (modelRowTo != modelRowFrom) {
        model.moveRow(viewRowFrom, viewRowFrom, modelRowTo);
        viewRowFrom = table.convertRowIndexToView(modelRowTo);
        System.out.println(viewRowFrom);
        System.out.println(modelRowTo);
      }
    }
  }
});
table.addMouseListener(new MouseAdapter() {

  @Override
  public void mousePressed(MouseEvent evt) {
    viewRowFrom = table.rowAtPoint(evt.getPoint());
  }
});

Oben musst Du statt rowFrom und rowTo nur das da deklarieren:

static int viewRowFrom;

Was es Dir aber bring bleibt mir ein Rätsel. Es ergibt nicht wirklich Sinn, die Tabelle händisch umzusortieren, während die Tabelle auch sortiert...

[ edit ] Der Hoax war schneller und hat auch recht.

Ebenius

 

[Guest]

danke für den quellcode,aber damit gets auch nicht immer noch das gleiche prob
mir ist aber grade noch was eingefallen wie man es eventuell lösen könnte...ist es möglich den Rowsorter auch wieder auszuschalten?Weil dann könnte man das teil ja einfach sobald man ein element in der liste verschiebt wieder ausschalten
Aso und das das Prog das kann ist von unserem Prof so gewollt also das man sortieren kann und dannach wieder per hand ordnen

 

[Ebenius]

danke für den quellcode,aber damit gets auch nicht immer noch das gleiche prob

Dann habe ich Dein Problem wohl falsch verstanden.

Du solltest eventuell auf das Auto-Sorting verzichten und gleich Buttons zum Sortieren des Datenmodells anbieten. Dann bist Du das Problem los. Ausschalten des RowSorters beim Klick hilft nicht. Dann würde ja der Nutzer auf eine Zeile klicken und eigentlich wäre's eine ganz andere, weil die Sortierung wegfällt.

 

[Guest]

ok dann werd ich wohl auf buttons umsteigen

danke für deine hilfe