0

Не могу понять почему js сопротивляется и не хочет подставлять переданные аргументы в функцию и выдает ошибку: pipe.js:70 Uncaught TypeError: action is not a function Если один аргумент, то: pipe.js:70 Uncaught TypeError: action is not a function Если несколько(как в примере), то: undefinded

Объясните пожалуйста, буду очень признателен.

const multiply = (a, b) => a * b;
const addOne = x => x + 1;
const square = x => x * x;


function fn(...action) {
  return (...args) => {
    if (action.length > 1) {
      return action.forEach(item => {
        item(...args);
      });
    } else {
      action(...args);
    }
  }
}

console.log(fn(multiply, addOne, square)(3, 4));

4
  • у Array​.prototype​.for​Each() возвращаемое значение undefined 7 мая 2019 в 19:28
  • в приведенном сниппете проблема не воспроизводится
    – Grundy
    7 мая 2019 в 19:37
  • rest parameters - это всегда массив. Поэтому если передали один аргумент, надо его и вызывать action[0](...args)
    – Grundy
    7 мая 2019 в 19:40
  • кстати, а какой результат ты ожидал получить конкретно в данном случае?
    – Grundy
    7 мая 2019 в 19:49

1 ответ 1

0

Если вы хотите получить массив на выходе, то используйте метод Array.prototype.map(), так как Array.prototype.forEach() всегда возвращает undefined.

Будьте бдительны args - массив

const multiply = (a, b) => a * b;
const addOne = x => x + 1;
const square = x => x * x;


function fn(...action) {
  return (args) => {
    if (action.length > 1) {
      return action.map(item => {
        return item(...args);
      });
    } else {
      return action[0](...args); // либо так либо вообще без этой части
    }
  }
}

console.log(fn(multiply, addOne, square)([3, 4]));
console.log(fn(multiply)([3, 4]));

Можно вторую часть опустить.

    const multiply = (a, b) => a * b;
    const addOne = x => x + 1;
    const square = x => x * x;


    function fn(...action) {
      return (args) => {
          return action.map(item => {
            return item(...args);
          });
      }
    }

    console.log(fn(multiply, addOne, square)([3, 4]));
    console.log(fn(multiply)([3, 4]));

4
  • А если так, то тоже не получаетсяfunction fn(...action) { return (...args) => { if (action.length > 1) { for (var i = 0; i < action.length; i++) { action[i](...args); } } else { action(...args); } } } console.log(fn(multiply, addOne, square)(3, 4));
    – kldd0
    7 мая 2019 в 19:39
  • @Proger0912, что не получается-то? :-) ты же ничего не возвращаешь в своем коде, а в консоль выводишь то, что твоя fn возвращает, то есть ничего, поэтому и видишь undefined
    – Grundy
    7 мая 2019 в 19:41
  • Благодарю, разобрался
    – kldd0
    7 мая 2019 в 19:43
  • ... перед args забыл в объявлении. А, ты зачем-то поменял формат аргумента, но зачем?
    – Grundy
    7 мая 2019 в 19:48

Ваш ответ

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

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