1

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

Cannot read property 'push' of undefined

при добавлении в массив. Из-за чего она появляется, и как её исправить?

console.log(flatList([
  [
    [2]
  ],
  [4, [5, 6, [6], 6, 6, 6], 7]
]));

function flatList(array) {
  let res;
  array.forEach(el => {
    res = repeat(el);
  });
  return res;
}

var tmp = tmp || [];

function repeat(itms) {
  if (typeof itms === 'number') {
    tmp.push(itms);
  } else {
    itms.forEach(el => repeat(el));
  }
}

7
  • при выполнении сниппета нет указанной ошибки.
    – Grundy
    21 окт 2018 в 21:32
  • @Grundy попробуйте выполнить код из консоли, ответ мне уже предложили, но почему мой код выдает эту странную ошибку до сих пор не понятен
    – user275900
    22 окт 2018 в 6:22
  • что в консоли, что в сниппете - указанной ошибки нет.
    – Grundy
    22 окт 2018 в 6:30
  • 1
    редактор не отображает ошибку - потому что ее нет. Если она есть у тебя - значит у тебя другой код.
    – Grundy
    22 окт 2018 в 6:33
  • 1
    А вот так и надо было вставлять в вопрос. Проблема в том, что ты flatList вызываешь до того, как в переменную массив запишешь.
    – Grundy
    22 окт 2018 в 6:47

3 ответа 3

3

Проблема заключается в том, что вызов функции flatList происходит до того, как присвоилось значение в переменную tmp.

Поэтому внутри функции, эта переменная будет undefined, и будет возникать указанная ошибка.

Для решения, нужно присваивать значение tmp до вызова функции, в которой используется эта переменная:

var tmp = tmp || [];
...
console.log(flatList([
  [
    [2]
  ],
  [4, [5, 6, [6], 6, 6, 6], 7]
]));
2

Ошибки, про которую Вы пишите, - нет.

Функция repeat ничего не возвращает.

1
  • запустите из консоли, местный редактор этого не показывает, в предложенном ответе к этой задаче ф-я repeat тоже ничего не возвращает
    – user275900
    22 окт 2018 в 6:26
1

Вот рабочая версия, хотя вторую проверку на число можно и убрать.

function flatList(arr){
    let res = [];
    arr.forEach(el => {
        repeat(el,res);
    });
    return res;
}
function repeat(itms,arr){
    if(typeof itms === 'number'){
        arr.push(itms);
    }else{
        itms.forEach(el => {
            if(typeof el === 'number')
                arr.push(el);
            else
                repeat(el,arr);
        });
    }
}
console.log(flatList([[[2]], [4, [5, 6, [6], 6, 6, 6], 7]]));

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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