2

Нужно найти максимальное и минимальное значение в массиве. Наваял такой "быдлокод" =)

{
    for (i = 0; i < myArray.length; ++i)
    if (myArray[i] > myArray[i + 1]) max = myArray[i];
    else min = myArray[i];
}
document.write(max);
document.write('</br>');
document.write(min);

Неверно работает. Подскажите почему. Не судить строго, я новичок в JS.

1
  • 2
    потому что Вы сравниваете два соседних элемента, а значит в min и max получите два последних элемента массива. чтобы заставить этот код работать как надо, сравнивайте каждый элемент с min и с max но, мнится мне, существует более элегантное решение.
    – ikoolik
    3 апр 2012 в 14:52

8 ответов 8

44

Во имя Джавахарлала Неру, зачем такие сложности?!

>>> Math.min.apply(null, [1,3,5,-1,8,0])
-1
>>> Math.max.apply(null, [1,3,5,-1,8,0])
8

Справочник

13
  • 1
    ахаха, дико плюсую! недавно занялся js, про прототипы знаю, а про такие банальности нет!
    – Spectre
    3 апр 2012 в 15:25
  • а подкинете ссылку? чтоб я разобрался в этих строках и возможно тоже посмеялся ;) или хоть скажите что почитать)
    – Zein
    3 апр 2012 в 15:58
  • @Zein, "строки" это консоль Firebug, остальное здесь developer.mozilla.org/en/JavaScript/Reference/Global_Objects/…
    – karmadro4
    4 апр 2012 в 5:12
  • 2
    более короткий в ES2015: Math.max(...arr)
    – zemil
    4 дек 2017 в 17:07
  • 1
    Да, но только есть один очень интересный нюансик. Всё бы хорошо, когда массив небольшой, но как только массив огромен, например о миллионе элементов, тогда Chrome, например, Вам гневно скажет, что Uncaught RangeError: Maximum call stack size exceeded! Потому что apply создаёт стэк непомерных размеров. И тогда в этом случае используем стандартные средства)) 20 мар 2019 в 11:09
9

Для поиска минимума вы должны делать отдельную проверку, сравнивая текущее myArray[i] с min, и если меньше -- менять min. Да и вообще, сравнивать надо не соседние значения, а текущие min и max с исследуемым myArray[i]

  min = myArray[0];
  max = min;
  for (i = 1; i < myArray.length; ++i) {
      if (myArray[i] > max) max = myArray[i];
      if (myArray[i] < min) min = myArray[i];
  }
5
  • Спасибо,работает,вот только когда в массиве появляется 3х значное число,то оно почему-то не считается большим за 2х значное((
    – Zein
    3 апр 2012 в 15:02
  • Это как? Массив для проверки можно увидеть?
    – alexlz
    3 апр 2012 в 15:22
  • как в коммент скрин сбросить? =))
    – Zein
    3 апр 2012 в 15:46
  • 1
    Bведенo 5 числa 23 234 4 1 78 MAX 78 № 5 MIN 1 № 4 Різниця 77
    – Zein
    3 апр 2012 в 15:46
  • Поддерживаю предыдущего комментатора. Решение ошибочно. 23 мар 2020 в 14:46
6
let arr = [22,4,7];
Math.min(...arr);
4
Array.prototype.min = function(){
    var min = parseInt(this[this.length-1]), el;
    for(var i=this.length-2; i>=0; i--){
        el = parseInt(this[i]);
        if(el<min){
            min = el;
        }
    }
    return min;
};

Array.prototype.max = function(){
    var max = parseInt(this[this.length-1]), el;
    for(var i=this.length-2; i>=0; i--){
        el = parseInt(this[i]);
        if(el>max){
            max = el;
        }
    }
    return max;
};

// использование:
var array = [1,3,5,-1,8,0];
document.write(array.min());// -1
document.write('</br>');
document.write(array.max());// 8

Если в myArray не числа, а строки с цифрами.

UPD: добавил parseInt

6
  • тут та же проблема, с трехзначными числами..
    – Zein
    3 апр 2012 в 15:18
  • не может такого быть: пруф
    – Spectre
    3 апр 2012 в 15:22
  • Может, может. Если в myArray не числа, а строки с цифрами.
    – alexlz
    3 апр 2012 в 15:49
  • я реализую в Dreamweaver,причина может быть в next = prompt( "Введите число массива : "); хотя я не мудрец в яваскриптинге,если интересно могу выложить полностью код..
    – Zein
    3 апр 2012 в 15:50
  • 2
    По prompt вы получаете строку, пусть и содержащую цифры. если сравнивать "13" и "124", то "124" будет меньше. Лексически, по порядку символов. Сравнивается слева направо. Если же их преобразовать к целым (parseInt), то результат сравнения 13 и 124 будет ожидаемым. 13 будет меньше.
    – alexlz
    3 апр 2012 в 16:12
1

//несколько строк кода и максимальное число известно надо минимальное в if поменяйте > на <

var maxNum = [5, -1, 40, 355, 17, 3];

for (var i = 0; i < maxNum.length; i++) {
    if(maxNum[i] > maxNum[0]){
        maxNum[0] = maxNum[i];
    }

}
alert(maxNum[0]);

P.S.: И даже если в трехзначном числе будет первая цифра < первой цифры двухзначного все равно выведет трехзначное число

0
let arr = [20, 3, 4, 5, -50, 100, 19, 200, 5000, 599];
let max = arr[0];

arr.forEach(function (elem, index) {

    if(index > 0 ){
        if(max < elem){
            max = elem;
        }

    }
});

console.log(max);

И Дело в шляпе, максимальное значение найдено!

0

Предлагаю через reduce:

let arrOfNumbers = [
    1, -4653, -3, 4, 5, 54, 684, 7, 8, 9, 10,
];


let maxNumber = (a, b) => {
    return a > b ? a : b;
},
    minNumber = (a, b) => {
        return a < b ? a : b
    };

console.log(arrOfNumbers.reduce(maxNumber))
console.log(arrOfNumbers.reduce(minNumber))
0

Например для того, чтобы найти минимальное и максимальное значение в массиве. В функции используем метод sort():

`var min = function(list){
    list.sort((a, b) => a - b);
    return list[0];
 }

console.log('min', min([-52, 56, 30, 29, -54, 0, -110]))`

Вернёт // -110

`var min = function(list){
    list.sort((a, b) => b - a);
    return list[0];
 }

console.log('min', min([-52, 56, 30, 29, -54, 0, -110]))`

Вернёт // 56

Ещё один пример сортировки массива c помощью методов Math.min и Math.max:

const min = (list) => Math.min(...list);
const max = (list) => Math.max(...list);

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.