Discussion :

[jacksparo]

Bonjour,
j'ai probleme dans la fonction suivante, apparament elle fait pas le tri correctement :elle compare les jours , aprés les mois et enfin les années.
voila focntion:

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
function ts_sort_date(a,b) {
 
    aa = ts_getInnerText(a.cells[SORT_COLUMN_INDEX]);
    bb = ts_getInnerText(b.cells[SORT_COLUMN_INDEX]);
    if (aa.length == 10) {
        dt1 = aa.substr(6,4)+aa.substr(3,2)+aa.substr(0,2);
    } else {
        yr = aa.substr(6,2);
        if (parseInt(yr) < 50) { yr = '20'+yr; } else { yr = '19'+yr; }
        dt1 = yr+aa.substr(3,2)+aa.substr(0,2);
    }
    if (bb.length == 10) {
        dt2 = bb.substr(6,4)+bb.substr(3,2)+bb.substr(0,2);
    } else {
        yr = bb.substr(6,2);
        if (parseInt(yr) < 50) { yr = '20'+yr; } else { yr = '19'+yr; }
        dt2 = yr+bb.substr(3,2)+bb.substr(0,2);
    }
    if (dt1==dt2) return 0;
    if (dt1<dt2) return -1;
    return 1;
}

j'utilise comme expression reguliére /^\d\d[\/-]\d\d[\/-]\d\d\d\d$/)

Merci d'avance

[Astartee]

apparament elle fait pas le tri correctement :elle compare les jours , aprés les mois et enfin les années.

Je ne me suis pas trop plongée dans le code, parce qu'il est un peu illisible (d'ailleurs, pense à la balise [code], merci), mais:
- si tu cherches à trier des dates, il faut bien évidemment comparer d'abord les années, puis les mois, puis les jours
- la comparaison de chaînes de caractères se fait par les méthodes equals et compareTo (et non "==" ou "<")


Ceci dit, plutôt que de réinventer la roue, je te propose une autre approche à ce problème:
- transformer tes chaînes de caractère en Date avec la méthode parse de la classe DateFormat
- comparer simplement les dates avec la méthode compareTo de la classe Date

EDIT : c'est pas malin de poster dans le forum Java pour du Javascript, hein

[le_chomeur]

trouvé sur le net : http://javascript.internet.com/forms/date-sorter.html

[RomainVALERI]

Un petit détail à connaître :
Tu peux simplifier ta regexp en mettant en facteur les caractères répétés.
Exemple : une expression comportant uniquement 4 chiffres pourrait s'écrire

Code javascript :

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

/^\d\d\d\d$/

, mais plus simplement :

Code javascript :

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

/^\d{4}$/

De plus, si tu veux autoriser le caractère "-" comme séparateur, il faut l'échapper quand tu le mentionnes entre les crochets, comme tu as fait pour le caractère "/".

Enfin, attention : ta regexp va accepter des dates au format "01/09-2008", c'est-à-dire des séparateurs non-identiques... ce n'est peut-être pas un problème, à toi de voir... ? (au cas où ça te gêne, utilise "\1" pour désigner le même contenu que celui décrit dans la première paranthèse de la regexp, voir exemple final)

Pour ton expression complète, en prenant en compte ces 3 remarques, ça donnerait quelque chose comme :

Code javascript :

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

1
2
3
 
// expression contenant uniquement : 2 chiffres, puis un séparateur "/" ou "-", 2 autres chiffres, le même séparateur que le premier employé, et enfin 4 chiffres
/^\d{2}([\/\-])\d{2}\1\d{4}$/

[Bovino]

Un petit tour sur la FAQ ???

[le_chomeur]

vi , derrière il ne reste plus qu'a créer la méthode de trie

[SpaceFrog]

un split revers join ... suivi d'un sort et c'est fini ...