task 137

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

task 137

Alexey

Реализуй функцию positiveElements в файле helper.js.

Она должна принять массив, сформировать и вернуть новый состоящий только из положительных элементов начального массива.

Оригинальный массив изменять нельзя.

как не пытаюсь ничего не получается, итерация одна он находит первое число и останавливается должен по идее быть массив, но я если и могу его пустой массив создать в начале, он так пустой и остается?
помогите разобраться.


export const positiveElements =(arr)=>{   
  let pos = arr[0];

  for (let i = 0; i <arr.length; i++) 
      if (arr[i] > 0) ;
  return pos;
}

Alexey


export const positiveElements =(arr)=>{   
  let pos = []; 
  for (let i = 0; i < arr.length; i++) {
      if (pos < 0) {
      {pos = arr[i]}
      return arr; 
  }
} else {
if (pos > 0) {
      {pos.push.arr}
      return pos; 
  }}
}

Друзья, кто-нибудь помогите не могу понять как вернуть массив только с позитивными числами, голова уже не работает 3 день сижу над задачей. Пожалуйста!

Coderslang_Master

Alexey к отличному разбору от @salton добавлю несколько советов:

Чтобы получить доступ к текущему элементу массива, нужно написать arr[i]. Похоже что это то что тебе нужно вместо if (pos < 0)
Чтобы добавить новый элемент в массив нужна функция push. Например arr.push(1) добавит элемент 1 в массив arr. Обрати внимание на то как ты пытался использовать push.

Coderslang_Master

Alexey начни с того, что опиши по пунктам алгоритм (своими словами) и пока не думай о коде.

salton

Ну вы на верном пути - собираете массив из положительных чисел... Но многое не понятно:

export const positiveElements = (arr) => {
    let pos = []; // создаем пустой массив (предположительно чтобы собрать туда все позитивные элементы)
    for (let i = 0; i < arr.length; i++)  // определяем условия прохода по входящему массиву
     {
        if (pos < 0) //проверяем меньше ли созданный нами массим чем ноль
        {
            { pos = arr[i] } //если меньше мы пытаемся переприсовить занчение переменной, установив в нее текущее значение входящего элемента
            return arr;      // и на этом решив что с нас хватит возвращаем этот результат
        }
    } else {                // иначе в случае, если наш созданный массив больше чем ноль 
        if (pos > 0) {      // проверим еще раз, что он таки точно больше чем ноль
            { pos.push.arr } // и добавим в него сразу весь входящий массив
            return pos;     // ну и вернем этот результат
        }
    }
}

Alexey


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

я уже ближе? можно ли тут не использовать else?

salton

Alexey я уже ближе?

да)))

Alexey можно ли тут не использовать else?

нужно, тебя вобще не должны интересовать элементы "не положительные".

salton

export const positiveElements =(arr)=>{   
  let pos = [] // пустой массив - он нам по идее  нужен, чтобы положить "положительные" элементы
  for (let i = 0; i < arr.length; i++) {  //условие перебора - тоже нужно
      
    if (arr[i] > 0) // тут мы сравниваем - является ли элемент входящего массива "положительным"
    arr.push(arr[i] > 0); // добавление в новый массив нужного нам элемента. Должно было быть.. но тут вы почему-то добавили внутри добавляемого элемента не нужную проверку
    {arr[i] = pos}; // это очередная загадка.. вы пытаетесь в элемент существующего массива присвоить новосозданный массив
      return arr; // ну и самая главная ошибка мы при первом же проходе входящего массива сразу же возвращаем результат
  }
}

в целом - в разы лучше чем было

Olewa

Не могли бы вы обьяснить мои ошибки тут, не совсем понимаю как реализовывать метод push после проверки

Olewa

Olewa Только что разобрался что не там return, и мы грубо говоря заканчиваем на первом числе

salton

пуш уже как раз хорош (помоему).
вот только return не нужно делать после первого пуша. верните свой массив, который соберет в цикле все нужное, после самого цикла.
попробуйте реорганизовать свой код, выложите сюда. если что - всегда поможем

salton

успехов вам