Как преобразовать число в javascript?

Question

[booogabooo]

Появилась проблемма...
Суть функции: нужно, чтобы функция, которой передается любое число, возвращала сумму цифр...
Вот такой тест должна проходить

Test.assertEquals(sumDigits(10), 1);
Test.assertEquals(sumDigits(99), 18);
Test.assertEquals(sumDigits(-32), 5);

Вот мой код:

function sumDigits(number){
  var string = new String(number);
  var numberOne = string.charAt(0);
  var numberTwo = string.charAt(1);
  var newNumberOne = new Number(numberOne);
  var newNumberTwo = new Number(numberTwo);
  return newNumberOne + newNumberTwo;
}

Первые 2 теста проходит, а с отрицательным числом - нет...Подскажите...
И еще, мне кажется, что код моя как-то криво написан, через одно место...может, это можно было сделать проще...

Comments

[Lynn «Кофеман»]

А что делать с числом 3.1415? Или 1.2345e40? Да, в конце концов, что ваш код будет делать с числом 3?

[Lynn «Кофеман»]

Just for fun :)

> String(-32).split('').reduce(function(a,b){return a+(b|0)},0)
5

Answer 1

[Александр Таратин]

function sumDigits(number){
  var d = Math.abs(number)
  var r = 0
  while (d!=0)
    {
        r += d%10;
        d = parseInt(d/10);
    }
  return r;
}

Answer 2

[Евгений Петров]

function getSum (num) {
	var sum = 0;

	Math.abs(num).toString().split('').forEach(function (a) {
		sum += +a;
	});

	return sum;
}

Comments

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

С дробными работать не будет, хоть и не указано такое условие :)

[Евгений Петров]

@rock

console.log(Math.abs(7.4567e-6).toString(10).replace(/[0\.]/g, ''));

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

@Petroveg усложняете, ниже проще :)

[Евгений Петров]

@rock проверка на NaN в цикле по всем знакам в числе, включая бестолковые нули и точки, — это проще?) Очень смешно.

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

@Petroveg очень смешно, что раз вам на столько важна именно скорость, вы обходите массив используя forEach вместо обычного цикла. Очень сомневаюсь, что пара лишних вызовов функции критична по скорости по сравнению с компактностью.

[Евгений Петров]

@rock ваша компактность? Я убираю ненужные символы до начала цикла, а не проверяю их внутри. Это гарантированно быстрее и поэтому я не стал писать проверку внутри цикла.

вы обходите массив используя forEach вместо обычного цикла

И опять мимо кассы — forEach проверяет только определённые элементы массива. Как и reduce(), как и map(), как и every().

var x = [];
x[1000] = 1;
x.forEach(function (a) {
	console.log(a);
});

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

@Petroveg jsperf.com/bntytyntny обход простым циклом в 2 раза быстрее вашего, компактный вариант, за счет лишних итераций, на треть медленней. Еще хотите поспорить о производительности js? :)

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

@Petroveg мне вот, например, понравился этот Как увеличить строку на n количество js ? ваш ответ на вопрос. jsperf.com/str-rep какая здесь разница в производительности? :)

[Евгений Петров]

@rock вы сравниваете forEach и for?) Это не ко мне, а к производителям браузеров. Тем не менее, ваш-то по-любому в проигрыше.

[Евгений Петров]

@rock вы хотите обсудить быстродействие встроенных в интерпретатор методов?
Я просто сказал, что ваш «простой» вариант вовсе не простой, а с тернарным оператором вместо предварительной замены в строке. И он медленней. Вы хотите доказать, что он-таки лучше, а я не понимаю — в чём?

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

@Petroveg вы вроде сами за производительность против лаконичности ухватились. Всё, что я сделал - показал, что раз вам она так важна - есть куда больший простор для оптимизации :) Мой вариант в выигрыше за краткостью и приемлемой производительностью, чего о вашем способе повторения строки не скажешь :)

[Евгений Петров]

@rock ваш вариант только и отличается, что reduce вместо forEach, да проверкой вместо предварительной замены. Что мы тут мусолим-то?

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

@Petroveg так я ваш слегка и исправил, вам об этом отписал, тут и говорить не о чем, но вам что-то показалось смешным :)

[Евгений Петров]

@rock про то, что for быстрее всех на свете, а обратный for ещё проворнее, известно давно. Если бы вы написали обратный for, да с предзаменой — без раздумий написал бы, что да, ваше лучше.

Но признать решение, которое короче на 20 символов и априори медленней... В чём смысл беседы?)

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

@Petroveg быстрее далеко не всегда значит лучше, хотя я и не говорил, что моё лучше. Проще. Модуль, регулярка - зачем? Какой-то новичок вообще не поймёт этой регулярки. Очень сомневаюсь, что хоть кому-то из тех, кому потребовалось решить подобную задачу на столько важна производительность. А важна - решение с for, что написал по ссылке, длиннее вашего примерно на столько же, но куда быстрее. Да и если важна производительность, лучше в первую очередь искать в других местах. В тех, что используются куда чаще и можно выгадать не несколько процентов, а ускорить в сотни и тысячи раз, пример я привел. Еще раз повторю - говорить тут больше не о чем, предлагаю закончить.

[Евгений Петров]

@rock о да:) Новичку и reduce как матан для аборигена. Вообще эти сентенции насчёт абстрактных новичков, которые якобы что-то не поймут — кто они, эти неведомые люди, чьё неизвестное мнение обязательно является важнейшим аргументом?)

Пусть разбираются и сразу берут за основу то, что данные желательно подготовить перед работой с ними. А то так можно доупрощаться до того, что prototype станет ругательством — сложно ведь, не поймут.

И, хотя я не вижу, где вы раньше предлагали закончить, соглашусь — всё и так предельно ясно.

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

@Petroveg я скорее не о новичках писал, а о том, что для меня, да и для многих

строка -> берем посимвольно -> сворачиваем во (что-то, так-то)

воспринимается куда легче, чем

что-то; модуль -> строка -> какайто регулярка -> берем посимвольно -> обходим (так-то); что-то

и хорошая читаемость обычно стоит нескольких процентов производительности.

Ушел, удачи.

Answer 3

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

function sumDigits(num){
  return String(num).split('').reduce(function(memo, dig){
    return +dig ? memo + +dig : memo;
  }, 0);
}

Answer 4

[Андрей Э]

function sumDigits(number){
  var r = 0;
      (number+'').replace(/-?[0-9]/g,function(n){
            r+=parseInt(n);
      })
  return r;
}