Discussion :

[Ceubex]

Bonjour,

J'ai un JTable très grand (plusieurs milliers de lignes) et je voudrais le trier par rapport à une colonne contenant des nombres. Ma procédure est la suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
TableRowSorter sorter = new TableRowSorter<DefaultTableModel>(monModel);
		  NumberComparator comparateurNombre = new NumberComparator();
	      sorter.setComparator(3, comparateurNombre);
             monTableau.setRowSorter(sorter);

la classe NumberComparator est la suivante (je marque "-" quand c'est un NaN ou Infinity) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class NumberComparator implements Comparator<String> {
 
	public int compare(String o1, String o2) {
		Double d1,d2;
		try {
			if(o1.equals("-")) d1=10000000.;
			if(o2.equals("-")) d2=10000000.;
			d1 = Double.parseDouble(o1);
			d2 = Double.parseDouble(o2);
			return (int) (d1-d2);
		} catch (Exception e) {
			//e.printStackTrace();
		}
		return 0;
	}
}

Mon soucis est qu'on dirait qu'il traite le tri par paquet (d'environ 500) et la colonne est trié par exemple comme cela :

1
2
3
40
56
60
-
-
-
2
6
7
40
80
-
-
-

[ninow]

Dans les cas où l'une des valeurs est égale à "-", ton code passe quand même sur le "parseDouble" et fini par retourner 0. Ton problème vient probablement de là.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
class NumberComparator implements Comparator<String> {
 
	public int compare(String o1, String o2) {
		final double d1,d2;
		try {
			if(o1.equals("-")){
                             d1 = Double.MAX_VALUE;
                        }else{
                             d1 = Double.parseDouble(o1);
                        }
			if(o2.equals("-")){
                             d2 = Double.MAX_VALUE;
			}else{
                             d2 = Double.parseDouble(o2); 
                        }
			return (int) (d1-d2);
		} catch (Exception e) {
			//e.printStackTrace();
		}
		return 0;
	}
}

Ce qui équivaut aussi à (je préfère car cest plus synthétique)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
        class NumberComparator implements Comparator<String> {
            public int compare(String o1, String o2) {
                final double d1, d2;
                try {
                    d1 = o1.equals("-") ? Double.MAX_VALUE : Double.parseDouble(o1);
                    d2 = o2.equals("-") ? Double.MAX_VALUE : Double.parseDouble(o2);
                    return (int) (d1 - d2);
                } catch (Exception e) {
                    //e.printStackTrace();
                }
                return 0;
            }
        }

Note : j'ai remplacé 10000000 par Double.MAX_VALUE qui me semble plus approprié dans ton cas