Как создать метод для нахождения суммы массива?

Question

[keslo]

Добрый день!

Вопрос такой: необходимо создать метод:
Array.prototype.sum = function() {}
.
Который бы находил сумму массива. Например:
[1,2,3].sum() -> 6

В чем проблема? Не могу понять, как обратиться к данным массива при написании функции sum.
Я новичок в этом. Уверен, что туплю. Но все же прошу помочь в этом вопросе :-)

Answer 1

[Сергей Мельников]

Учите мат часть ))).
Самый простой способ:

Array.prototype.summary = function(){
  return this.reduce(function(x,y){ return x + y }, 0);
}

но, такое добавление в прототип неправильное, вообще, расширение прототипа встроенных объектов считается дурным тоном (простительно только для новичков),
проблемы возникают при переборе массива циклом for in. На практике желательно использовать просто ф-ции:

function summary(array){
  return array.reduce(function(x,y){ return x + y }, 0);
}

Или, если заморочиться, можно безопастно добавить метод в прототип:

Object.defineProperty(Array.prototype,
  "summary", 
  {
    writable: false,
    enumerable: false,
    configurable: false,
    value: function(o) {
      return this.reduce(function(x,y){ return x + y }, 0);
    }
  }
);

Comments

[sergealmazov]

Где такому учат?! Откуда ты, гений?!

[Денис Пушкарев]

[].summary() // => TypeError: Reduce of empty array with no initial value
['1', 2, 3].summary() // => '123'

[Денис Пушкарев]

fix:

return this.reduce(function(x,y){ return x + +y }, 0);

[Сергей Мельников]

@sergealmazov www.ozon.ru/context/detail/id/3881091
"гений" - автор этой книги)

[Сергей Мельников]

@rock спасибо, это вы правильно заметили)

[keslo]

Отлично! Спасибо! Пригодилось :-)

Answer 2

[Mikhail Osher]

Легко и просто.

Array.prototype.sum = function() {
  var total = 0;
  
  for (var i = 0, len = this.length; i < len; i++) {
    total += this[i];
  }
  
  return total;
};

plnkr.co/edit/I7oqrQMgX6pB1QRFw5X8?p=preview

Comments

[sergealmazov]

Выпендрился. У всех jsfiddle. А тут - Plunker )))

[Mikhail Osher]

@sergealmazov , просто я презираю jsfiddle. Ничего личного.)

[sergealmazov]

Туше! )

[Mikhail Osher]

@sergealmazov , а еще plnkr написан на Angular, это лично по мне плюс. :)

Answer 3

[Scorpi]

jsfiddle.net/zpryQ/1 вот так будет лучше

Comments

[Константин]

Ещё можно выпендриться jsfiddle.net/zpryQ/2

[sergealmazov]

jsfiddle.net/96Puc

[Mikhail Osher]

@CheeseMaster длину массива надо тоже в иницализацию вынести, зачем каждый раз пересчитывать - <sarcasm>тратите процессорные ресурсы.</sarcasm>

[Scorpi]

Правильный вариант: jsfiddle.net/79Gt2/1
Короткий вариант jsfiddle.net/zpryQ/3

P.S. извиняюсь если много уведомлений пришло на почту :C

[Денис Пушкарев]

@Scorpi Вы из принципа var не используете? :) За такое пальцы ломать нужно.

[Scorpi]

@rock честно говоря, я на Javascript вообще не пишу, мой язык - Java.
Пытался вспомнить в каких случаях нужно использовать var, и почему-то показалось что при использовании var переменная i станет доступна за пределами цикла. Не угадал значит? :D

[Денис Пушкарев]

@Scorpi да была бы только i - и s, и l без var становятся глобальными переменными. isNaN автоматом приводит значение к числу, parseFloat не нужен, да и isFinite исключает NaN. И вообще зачем тут нужны проверки isNaN и isFinite? Они тупо ломают логику, исключая из суммы NaN и (+|-)Infinity. Достаточно элемент массива привести к числу (s += +this[i]), дабы результатом не была, например, строка. https://codeo.me/4h9

[Scorpi]

@rock ок, я понял, я нуб и опозорился :C

Answer 4

[xaja]

jsfiddle.net/zpryQ

Comments

[Константин]

Жёстко на конкретный массив же завязан, this надо использовать