Trier par date décroissante de comparaison ne fonctionne pas comme prévu

En essayant de donner un sens à la sortie suivante:

public class CommunicationComparator implements Comparator<Communication> {
    @Override
    public int compare(Communication comm1, Communication comm2) {
        long t1 = comm1.getDate().getTime();
        long t2 = comm2.getDate().getTime();
        return (int) (t2 - t1);
    }
}

La méthode getDate() renvoie un java.sql.Timestamp.

Voici la sortie avant le tri:

for (Communication n : retVal) {
    System.out.println(n.getDate().toString());
}

2012-10-03 10:02:02.0

2012-10-07 03:02:01.0

2012-10-08 13:02:02.0

2012-10-09 03:02:00.0

2012-11-26 10:02:05.0

2012-11-28 11:28:11.0

2012-12-03 12:03:01.0

2012-12-06 15:03:01.0

2012-12-13 14:03:00.0

2012-12-28 11:03:00.0

2012-12-28 13:49:21.0

Et après:

Collections.sort(retVal, new CommunicationsComparator());

2012-12-13 14:03:00.0

2012-12-06 15:03:01.0

2012-12-03 12:03:01.0

2012-11-28 11:28:11.0

2012-10-09 03:02:00.0

2012-10-08 13:02:02.0

2012-11-26 10:02:05.0

2012-10-07 03:02:01.0

2012-10-03 10:02:02.0

2012-12-28 13:49:21.0

2012-12-28 11:03:00.0

Des idées pourquoi les deux objets ne soient pas classés correctement? Je suis en utilisant le JDBC MySQL mise en œuvre de ce Timestamp.

Pourquoi êtes-vous de tri sur l'horodatage de la soustraction? Plus précisément, quelle est la spec où elle est mise en œuvre?
java.sql.Timestamp implémente Comparable et a donc un compareTo() méthode. Pourquoi ne pas déléguer directement comme return comm1.getDate().compareTo(comm2.getDate());?
C'est la meilleure approche, et honnêtement je me sens stupide de ne pas y penser. C'est ce que je vais faire.
Vous êtes les bienvenus. Il ressemble que Pierre a copié cette réponse par la suite, donc je ne vais pas le rediffuser.

InformationsquelleAutor maple_shaft | 2013-01-02

18

La différence entre les 2 dernières dates et les dates antérieures débordement d'entier.

Peut-être une meilleure solution serait de comparer les valeurs, plutôt que de soustraire eux.
```
    long t1 = comm1.getDate().getTime();
    long t2 = comm2.getDate().getTime();
    if(t2 > t1)
            return 1;
    else if(t1 > t2)
            return -1;
    else
            return 0;
```
- Un look parfait et sûr de moi.
- Ce qu'un débutant complet d'erreur que j'ai faite! Merci pour le rappeler à moi. 🙂
- Une courte façon de faire cela à l'aide de l'opérateur ternaire: return t1 == t2 ? 0 : t1 < t2 ? -1 : 1;
- merci homme vous sauver ma journée.
- Je ne dirais pas une erreur de débutant, mais un très bel aperçu par l'op répondre chevalier.
InformationsquelleAutor James McMinn
6

Si l'écart est supérieur à 25 jours, un débordement se produit. (Un int ne peut pas représenter une plus grande différence de temps en millisecondes que d'environ 25 jours). Cela permettra de faire la comparaison incorrecte.

Cela peut être résolu en modifiant l'instruction return dans:
```
return Long.signum(t2 - t1);
```
- Vous avez encore une (certes irréaliste pour ce cas d'utilisation comme les deux valeurs sont positives) de débordement de la possibilité. Il est généralement préférable de le comparer au lieu de soustraire...
InformationsquelleAutor gogognome
6

Vous pouvez utiliser
```
return Long.compare(t2, t1);
```
mais vous êtes mieux comparant les dates.
```
return comm2.getDate().compareTo(comm1.getDate());
```
InformationsquelleAutor Peter Lawrey
4

Ma première idée est que le problème est un dépassement de capacité.
t1 et t2 sont longs. Les différentes ne pourront pas entrer dans un int.

Je voudrais vérifier que.

Si on compare sur le deuxième niveau est assez bon pour vous, vous devriez essayer:
```
return (int) ((t2 - t1)/1000);
```
Cela ne garantit pas qu'il n'y aura pas de débordements.
J'aurais au moins ajouter un test.

Je pense que la meilleure réponse n'est pas la mienne.
Mon préféré est:
```
    if(t2 > t1)
        return 1;
    else if(t1 > t2)
        return -1;
    else
        return 0;
```
- La Question est, est la différence aussi nt toujours un int. Il semble donc.
InformationsquelleAutor MrSmith42

Vous devez vous connecter pour publier un commentaire.