Discussion :

[FotoXe33]

Bonjour,

Je suis en TP de JS (niveau BTS donc pas énorme non plus), et on nous demande de classer 10 mots par ordre alphabétique...

J'ai beau cherché sur google, je ne trouve pas de fonction pour classer alphabétiquement une liste...

Comment faire SVP ?

Je ne demande pas une solution toute machée, juste un p'tit coup de pouce, genre le nom de la fonction ou un indice...

Merci d'avance...

[SpaceFrog]

mets tes mots dans un tableau (array) au moyen d'un split sur une chaine par exemple et regarde du coté de sort()

[FotoXe33]

Oui j'y avais pensé a split avec un array merci

Bon merci pour le sort(), voila mon script :

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
 
		var total = "";
		for(var i=1; i<11; i++){
			var mot = prompt("Mot "+i);
			total = total+";"+mot;
		}
		var solo = total.split(';');
		while(solo[i] && i<10){
			var Reg=new RegExp(solo[i],'gi');
		    solo.sort();
		    i++;
		}
		document.write("<h1>Tri croissant :</h1><br><br>" + solo.join(",<br>") + "<br><br>");
		var solo2 = solo.reverse();
		document.write("<h1>Tri décroissant :</h1><br><br>" + solo2.join(",<br>"));

Mais il y a juste un tout petit "bug" visuel... Comment le régler ?

Merci en tout cas

[SpaceFrog]

evite le document.write et passe plutot par document.createElement...

quel est le bug visuel ?

[FotoXe33]

Ce bug :

Tri croissant :


,
a,
b,
c,
d,
e,
f,
g,
h,
i,
j

Tri décroissant :


j,
i,
h,
g,
f,
e,
d,
c,
b,
a,

[E.Bzz]

Bonjour,
il te manque juste un test : au premier passage dans la boucle, si total est encore vide, il ne faut pas le ";"

Code :

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

total = total+";"+mot;

(ça génère un élément vide dans ton array ....)

A+

[SpaceFrog]

ha ok...
ben ça vient de ta concaténation vu que tu ajoutes ; avant le mot

je te conseillerais plutot de mettre le ; après
ensuite avant de splitter tu enlèves le dernier ;
ici je le fais avec une reg

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
var total = "";
		for(var i=1; i<11; i++){
			var mot = prompt("Mot "+i);
			total += mot+";";
		}
		var reg=/;$/;
		var solo = total.replace(reg,'').split(';');
		alert(solo)
		while(solo[i]){
			 solo.sort();
		    i++;
			}
		document.write("<h1>Tri croissant :</h1><br><br>" + solo.join(",<br>") + "<br><br>");
		var solo2 = solo.reverse();
		document.write("<h1>Tri décroissant :</h1><br><br>" + solo2.join(",<br>"));

Autres remarques:
+= pour la concaténation
ta boucle while, pas besoin de seconde condition
la regexp est inutile dans la boucle
et le document.write mais ça je l'ai déja dit ^^

[FotoXe33]

Ok ok !

Le += c'est comme en php .= Merci pour l'astuce...
D'accord pour le while, ça m'arrive de me compliquer pour rien
Pourquoi document.write n'est pas bon ? Je sais que tu m'as dis de mettre autre chose, mais quelle est la raison ?

Juste un truc que je n'ai pas compris dans ton code :

Code :

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

var reg=/;$/;

A quoi ça sert ?

Merci en tout cas

[SpaceFrog]

lorsque tu concatènes ta chaine de mots:

tu obtiens ceci

au premier mot:
mot1;
au second
mot1;mot2;

etc ...
tu auras donc toujours un ; à la fin de la chaine
comme l'a dit Ebzz lors du split sur le ; le dernier element du tableau est vide.
il faut donc supprimer le ; final avant de splitter
tu pourrais le faire avec un subtring, masi je préfère utiliser une expression régulière

Code :

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

re=/;$/

qui matche le ; final de la chaine de caractères
que je remplaces ensuite par ... rien

Pour ce qui est de document.write je le déconseille simplement parce que c'est une methode qui insère au pied de biche du string dans une page, mais si ce string est en fait du code html, le code est interprété pour l'affichage mais les objets insérés ne sont pas forcément pris en compte pas le DOM ce qui peut poser des problèmes pour ensuite manipuler ces objets.
Dans ton cas de figure ça ne pose pas de problème majeur , masi autatn prendre de bonne habitudes dès le début