В чем ошибка использования arguments функции?

Question

[Gagatyn]

В чем ошибка? Как исправить её?

Answer 1

[Алексей Шашенков]

arguments - это псевдомассив

var args = Array.prototype.slice.call(arguments);
var result= args.reduce((s, cur) =>{ 
      return s+cur; 
   });

Comments

[Gagatyn]

Алексей Шашенков ПСЕВДОМАССИВ, как я мог забыть.

Answer 2

[Сергей Соколов]

Calc = function(){
  this.sum = function(){
    return Array
      .prototype
      .slice
      .call(arguments)
      .reduce(function(p,c){return +p+c}, 0)
  }
}

let c = new Calc;
c.sum(2,4,6) // 12

Comments

[Negwereth]

function foo(...args) {
  console.log(args);
}

[Сергей Соколов]

Negwereth: эт да, только не поддерживается в IE и некоторых других.

[Negwereth]

Сергей Соколов: как и arrow function, вами же использованная

[Сергей Соколов]

Negwereth: вы правы. Поменять её на function(p,c){return p+c} легко (сделал). А попробуйте заменить spread на полифилл(?) : )

[Negwereth]

Сергей Соколов: ну так или

return Array
      .prototype
      .slice
      .call(arguments)
      .reduce( function(p,c){return p+c})

или

function (...args) {
  return args.reduce((res, next) => res + next, 0);
}

[Сергей Соколов]

Negwereth: так ли нужен второй аргумент 0 ?

[Negwereth]

Сергей Соколов: Почитайте, это начальное значение, передастся в первую итерацию первым аргументом

[Сергей Соколов]

Negwereth: я в курсе. Но работает и без него. В чём подвох? ; )

[Kovalsky]

Сергей Соколов: "это начальное значение, передастся в первую итерацию первым аргументом" - в этом и подвох. Если передано начальное значение, выполнение начинается с него. Если нет, то с первого элемента массива.

[1,2].reduce((a, b, i) => {
	console.log(`${i}) ${a} + ${b}`);
	return a + b;
});
// 1) 1 + 2

[1,2].reduce((a, b, i) => {
	console.log(`${i}) ${a} + ${b}`);
	return a + b;
}, 5);
// 0) 5 + 1
// 1) 6 + 2

Обратите внимание на индексы итераций.

Answer 3

[Dmitriy Mozgovoy]

Хоть и мелочь, но для кроссплатформенности не рекомендуется передача arguments в другие функции (типа Array.prototype.slice.call(arguments) ), ибо в V8 это убивает оптимизацию функции, и это, как я знаю, все еще актуально.
Кроме того, нет никакой причины использовать slice перед вызовом reduce- все методы массива умеют переваривать массивоподобные объекты:

let sum= Array.prototype.reduce.call(arguments, (acc, value)=>acc+value, 0))

Но для оптимизации кода под V8 нужно руками формировать массив с arguments через обход циклом for, либо написать helper, вроде

function toArray() {
    let len = arguments.length;
    if (len === 0)return [];
    return len > 1 ? Array.apply(null, arguments) : [arguments[0]];
}

И вызывать его исключительно через .apply

let sum= toArray.apply(null, arguments).reduce( (acc, value)=>acc+value, 0))