Je ne comprend pourquoi mon code ne fonctionne pas

**Hikaru_03** · 18/04/2020, 22h24

Bonjour à tous, j'apprends le JavaScript et j'ai essayé pour m'entrainer de faire un code qui pourrait remettre dans l'ordre numérique la contenance d'un tableau alors le voici (je le lance tout simplement sur FireFox) :

Code HTML :

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
<!DOCTYPE html>
<html>
    <head>
        <script>
 
            var tableau = [3, 64, 21, 93, 23, 45, 54, 12]
                var resultat = tableau 
                    for (var i = 0; i < resultat.length; i++) {
                        if (resultat[i] > resultat[i + 1]) { 
                                var tmp = resultat[i + 1]    
                                resultat[i + 1] = resultat[i] 
                                resultat[i] = tmp              
                        }
                        console.log(resultat[i] + ",")
                    }
 
        </script>
    </head>
    <body>
    </body>
</html>

Mais malheureusement cela donne :

3, 21, 64, 23, 45, 54, 12, 93,

Alors, j'aimerais bien savoir qu'est ce qui ne va pas avec mon code. Je suis bien aller voir sur internet d'autre façon de trier par ordre numérique un tableau mais ce qui m'intéresse est de savoir quelle est mon erreur.

**javatwister** · 18/04/2020, 22h28

Bonsoir,
As-tu entendu parler de la méthode sort() de l'objet Array?

**Hikaru_03** · 19/04/2020, 09h17

Non je n'en ai pas encore entendu parler, est ce que tu pourrais m'éclaircir sur le sujet ?

**javatwister** · 19/04/2020, 09h54

Tu as cherché un peu? beaucoup?

Code javascript :

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

console.log([3, 64, 21, 93, 23, 45, 54, 12].sort((a,b)=>a-b));

**javatwister** · 19/04/2020, 10h14

Après, ta méthode intuitive est presque opérationnelle; ce que tu tentes s'appelle un tri à bulles. Et tu as juste oublié qu'il fallait décrémenter la variable de 2 à chaque passage...
Et avec une syntaxe moins bavarde:

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
let tableau=[3, 64, 21, 93, 23, 45, 54, 12]
 
for (let i = 0; i < tableau.length; i++) {
	if (tableau[i] > tableau[i+1]) { 
		tableau.splice(i,2,tableau[i+1],tableau[i]);
		i-=2             
	}
}
 
console.log(tableau)

**Hikaru_03** · 19/04/2020, 11h37

Alors déjà merci beaucoup pour tes réponses et si j'ai bien compris le problème c'est que à la fin du premier passage i ne sera plus égal à 0 alors il faut juste lui remettre la valeur de 0 ?

(J'ai essayé et effectivement cela fonctionne bien mais je te pose tout de même la question pour être sur d'avoir bien compris.)

Et je suis aller voir ce qu'était .sort() et effectivement c'est une manière bien plus simple de faire ce que je voulais faire.

**javatwister** · 19/04/2020, 12h00

Tu as compris: on trie les éléments par paire mais dès que a > b, il faut revenir en arrière pour tester la supériorité par rapport au nombre précédent;

Je ne comprend pourquoi mon code ne fonctionne pas

JavaScript

Discussions similaires

Partager

Partager