trouver le plus grand nombre entier dans le tableau javascript

Question

Possible en double:
How might I find the largest number contained in a JavaScript array?

Je ne parviens pas à obtenir ce code au travail, je l'ai été à elle pendant un certain temps à essayer de comprendre. Quand je regarde la console, il affiche simplement 0 quelqu'un peut-il m'aider à comprendre ce que j'ai fait de mal. Voici mon code:

var array = [3 , 6, 2, 56, 32, 5, 89, 32]; 
var largest= 0; 

for (i=0; i<=largest;i++){ 
    if (array>largest) { 
     var largest=array[i]; 
    } 
} 




console.log(largest); 

Answer 1

Le code ci-dessous est fixe et devrait fonctionner. Le problème était que dans cette ligne if (array>largest) { Vous ne fournissiez pas l'index du tableau. En changeant le code à ceci if (array[i]>largest) { cela fonctionne. Notez que j'ai ajouté le [i] à la fin de array dans l'instruction if.

var array = [3 , 6, 2, 56, 32, 5, 89, 32]; 
var largest= 0; 

for (i=0; i<=largest;i++){ 
    if (array[i]>largest) { 
     var largest=array[i]; 
    } 
} 




console.log(largest); 

Answer 2

Vous avez quelques petites erreurs. Tout d'abord:

if (array>largest) { 

Il devrait plutôt être:

if (array[i]>largest) { 

Deuxième:

for (i = 0; i <= largest; i++) { 

devrait être

for (i = 0; i <= array.length; i++) { 

Answer 3

Vous avez dans votre code deux questions. Tout d'abord, array>largest devrait être array[i]>largest. Deuxièmement, vous déclarez une nouvelle variable la plus grande à l'intérieur du if qui n'est pas la même que celle à l'extérieur. Supprimez var de l'affectation de la nouvelle valeur la plus grande.

Answer 4

var arr = [3, 6, 2, 56, 32, 5, 89, 32]; 
var largest = arr[0]; 

for (var i = 0; i < arr.length; i++) { 
    if (largest < arr[i]) { 
     largest = arr[i]; 
    } 
} 
console.log(largest); 

• Vous devez définir i ou bien elle devient une variable globale.
• Ne redéfinissez pas le plus grand dans la boucle.
• Puisque vous parcourez le tableau, utilisez i < array.length au lieu de i <= largest.
• Puisque vous comparez chacun des éléments du tableau à largest, utilisez if(largest < array[i]) au lieu de if(array > largest)
• Vous devez définir plus égal au premier élément du tableau, car si tous les chiffres sont négatifs?
• array est un nom de variable incorrect car il est trop similaire à Array (le constructeur de tableau). Essayez plutôt arr.

Une doublure:

var largest = Math.max.apply(0, array);

Plus d'infos ici: Javascript max() function for 3 numbers

Answer 5

var array = [3 , 6, 2, 56, 32, 5, 89, 32]; 
var largest= array[0]; 

for (i=0; i<=largest;i++){ 
    if (array[i]>largest) { 
     var largest=array[i]; 
    } 
} 

Answer 6

Juste une ligne :)

var array = [3 , 6, 2, 56, 32, 5, 89, 32], 
    largest = array.sort().reverse()[0]; 

ou mieux encore

... 
    largest = array.sort()[array.length - 1]; 

UPD, tout le code ci-dessus est suce quand vous ajoutez par exemple 9 dans le tableau je pense que par le nombre par défaut traités comme des chaînes de tri, il y a meilleure version

var array = [3 , 6, 2, 56, 32, 5, 89, 32, 9], largest; 
array.sort(function(a, b) { 
    largest = a > b ? a: b; 
}); 

bien que dans la performance boucle forEach sage suggéré dans les commentaires sont meilleurs http://jsperf.com/array-sorting-javascript-stack

UPD2, d'accord, le code ci-dessus a quelques mauvaises parties, donc ne fonctionnera pas comme prévu. Un autre essai:

array.sort(function(a, b) { 
    return a - b; 
}); 
largest = array[array.length - 1];