javascript - Finding largest element in an array - Stack Overflow

admin管理员组

文章数量:1347215

I am trying to write a JavaScript program that will display the largest number out of 10 numbers inputted by the user. This is what I have so far but it isn't working.

var counter = 1;
var number = new Array();
number.length = 9;
var largest = 0;

while (counter <= 10) {
    number = window.prompt("Enter Numbers 1-10 Number:" + counter + ".");
    number[counter] = parseInt(number);
    counter++;
}

largest = Math.max.apply(Array);
document.writeln("<h1>Largest number is " + largest + "</h1>");

I am trying to write a JavaScript program that will display the largest number out of 10 numbers inputted by the user. This is what I have so far but it isn't working.

var counter = 1;
var number = new Array();
number.length = 9;
var largest = 0;

while (counter <= 10) {
    number = window.prompt("Enter Numbers 1-10 Number:" + counter + ".");
    number[counter] = parseInt(number);
    counter++;
}

largest = Math.max.apply(Array);
document.writeln("<h1>Largest number is " + largest + "</h1>");

• javascript
• arrays

Share Improve this question Follow edited Nov 15, 2023 at 15:43 Ghost 5,08977 gold badges2424 silver badges4848 bronze badges asked Oct 11, 2013 at 16:18 JessicaJessica 18911 gold badge55 silver badges1616 bronze badges 3

• 1 Have you read anything about the javascript arrays? – Akshay Khandelwal Commented Oct 11, 2013 at 16:20
• I am new to Javascript, Just started the chapter this week – Jessica Commented Oct 11, 2013 at 16:34
• 1 For a newer, this question is pretty well formatted as well as it includes information on what you have tried already along with the code. +1 for that. Make sure to read the about page it includes quite useful information :) – Moritz Roessler Commented Oct 11, 2013 at 17:33

Add a ment  | 

5 Answers 5

Sorted by: Reset to default 2

You have several issues:

1. Arrays are 0 indexed. You skipped the 0 index by starting the counter at 0, this will screw up the array calculation. If you go with Math.max.apply(array, number), it will work without the 0 indexing.

2. You overwrote the number variable with every prompt, either use the window.prompt to feed into the parseInt or feed it into a temporary variable.

3. You had incorrect syntax for the apply variable.

var counter = 0;
var number = new Array();
number.length = 9;
var newnumber;
var largest = 0;

while (counter <= 10) {
    newnumber = window.prompt("Enter Numbers 1-10 Number:" + counter + ".");
    number[counter] = parseInt(newnumber);

    counter++;
}

largest = Math.max.apply(Math, number);
document.writeln("<h1>Largest number is " + largest + "</h1>");

You're simply missing the expected thisArg for Function.prototype.apply.

The Syntax, as described by MDN is

fun.apply(thisArg[, argsArray])

As shown in this simple example

Math.max.apply (null,[5,4,3,7,9,]) //9
                ^^^^

where null is used in the example for simplicity, as Math doesn't expect an specific context

What you are trying to do is passing the array number to Math.max, which would then be the thisArg (though you seem to have it mistaken with Array) which would result either way in Math.max being called with zero arguments, which yields according to §15.8.2.11

Given zero or more arguments, calls ToNumber on each of the arguments and returns the largest of the resulting values.

• If no arguments are given, the result is −∞.

1. You're mixing up the array with the temporary variable to hold the input.
2. apply takes two arguments, the context (irrelevant in this case) and the arguments array.

Should be like this:

var counter = 1;
var arr = [];
var count = 3;

while (counter <= count) {
    number = window.prompt("Enter Numbers 1-" + count + " Number #" + counter + ": ");
    if (number === null) break;
    arr[counter++] = parseInt(number);
}

if (arr.length > 0) {
    var largest = Math.max.apply(Array, arr);
    alert("<h1>Largest number is " + largest + "</h1>");
}

var counter = 1;
var number = [];
var largest = 0;

while (counter <= 10) {
    number.push(Number(window.prompt("Enter Numbers 1-10 Number:" + counter + "."), 10));
    counter++;
}

Array.prototype.max = function () {
    return Math.max.apply(Math, this);
};

largest = number.max();
document.writeln("<h1>Largest number is " + largest + "</h1>");

The main problems with your code are:

1. You are missing a this argument from the Math.max.apply function call.

2. You are destroying the number array every time you read a new value from the user. You need a separate variable for the user input.

However, you don't really need to build an array at all for this task. Inside the loop, just pare largest with the number just supplied by the user and update largest if appropriate:

var counter;
var number;
var largest = Number.NEGATIVE_INFINITY;

for (counter = 1; counter <= 10; counter++) {
    number = window.prompt("Enter Numbers 1-10 Number:" + counter + ".");
    number = parseInt(number);
    if (number > largest) {
        largest = number;
    }
}

document.writeln("<h1>Largest number is " + largest + "</h1>");

本文标签： javascriptFinding largest element in an arrayStack Overflow