TASK 135

Этот сайт лучше всего просматривать в современном браузере с включённым JavaScript.

TASK 135

qwerttrewq

Здравтсвуйте я не могу пройти задачу пробывал много раз

export const maxElement = (arr) => {
for(let i=-0;i<=arr.length;i++){
if(arr===5 ){
return arr;} else if (arr===-1 )
return arr}
}

Coderslang_Master

@qwerttrewq в массиве могут быть числа намного больше чем 5.

Попробуй предположить, что максимальный элемент массива находится на нулевой позиции.
Потом сравни этот максимальный элемент со вторым элементом массива. Если он окажется больше - замени максимальный элемент и перейди к третьему элементу.

Продолжай пока не пройдешь все элементы массива.

qwerttrewq

Здравствуйте я нашел решение этой задачи но мне нужно подробное объяснение того что происходит в ходе этого цикла потому что как я не пытался понять не выходит кажись я чтото упуская из виду надеюсь вы поможете мне это найти
Просто не поняв суть я не могу идти дальше а так просто пользоваться циклами я могу

export const maxElement=(arr)=>{   
    let max=arr[0];

    for (let i=0;i<arr.length;i++)
        if (max<arr[i]) 
         
         {max=arr[i]}

    return max};

Max

qwerttrewq
У вас есть функция maxElement, она принимает один аргумент и сохраняет его в переменную с именем arr.
Первым делом, функция сохраняет первое значение из массива arr в переменную max.
Значит, на данный момент у нас есть:

max=arr[0] - что бы там ни значилось, просто какое-то значение из переданного массива.
arr=[...] - массив каких-то значений (их может быть и 100, и 1000, и 1, и вообще ни одного [] ).

Цикл - объявляется с переменной i, ей мы даём значение 0, а сам цикл ограничиваем длинной переданного в функцию массива i < arr.length
Итак, цикл будет повторяться столько раз, сколько значений содержит переданный массив arr
Каждый раз будет выполняться проверка if (max < arr[i]) то есть, мы сравниваем, каждое значение массива с тем. которое хранится у нас в переменной max. И если значение в массиве оказывается больше, то мы перезаписываем эту переменную уже на это большее значение.
*здесь у вас сдвинуты как-то фигурные скобки, стандарт выглядит немного иначе и, как мне кажется, понятнее:
if (max < arr[i]) { max = arr[i]; }
после окончания цикла мы возвращаем переменную max которая содержит самое большое значение, или единственное, если в переданном массиве оно было одно, или никакого значения, елси массив был пустой.

Maxim

Чувствую себя тупицей, в теме все подробно расписано, из всей этой информации у меня получился код:

export const maxElement = (arr) => {
  let max = arr[0];
  for (i = 0; i < arr.length; i++) {
    if (max < arr[i]){
      max = arr[i];}
  return max;
  }
}

Но при проверке выдает ошибку " 3. Функция maxElement должна возвращать максимальный элемент в массиве"
Не могу понять что я сделал не так.

Coderslang_Master

Maxim все нормально, просто return внутри цикла написан, поэтому он выполняется не до конца.

salton

Что происходит вот тут - max < arr?
Или давайте по другому:

export const maxElement = (arr) => {
  let max = arr[0];
  for (i = 0; i < arr.length; i++) {
    if (max < arr[i]){
      max = arr[i];}
console.log(max )
  return max;
  }
}

скопируйте этот код себе вместо вашего и запустите на выполнение- посмотрите что у вас выводиться в консолью

AlanVendy

Я пошёл таким способом:

export const maxElement = (arr) => {
return Math.max.apply(null, arr);
}

Результат вроде как верный, но проверить задачу в целом не могу))

Igor2332

export const maxElement = (arr) => {
let max = arr[0] ;
for (let i=0; i <arr.length; i++) {
if (max < arr ); {
max =arr ;
}
}
return max;
}

Не могу понять,почему самый большой элемент массива 2 выводится -5 , а не -1
в чем ошибка?

Coderslang_Master

Igor2332 убирай точку с запятой после ифа и добавь индекс, чтобы присваивать максимуму текущий элемент массива: max = arr[i]

hardcoder

export const maxElement = (arr) => {
arr.sort((a,b) => b-a);
return arr[0];
}
решил с помощью метода sort, методы массивов здорово упрощают жизнь.