1. JavaScript / Говнокод #8528

    +153

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    function isInteger(s)
{
	var i;
    for (i = 0; i < s.length; i++)
	{   
        // Check that current character is number.
        var c = s.charAt(i);
        if (((c < "0") || (c > "9"))) return false;
    }
    // All characters are numbers.
    return true;
}

    феерично

    Запостил: DrFreez, 15 Ноября 2011

    Комментарии (28) RSS

    • ava demotivator 15.11.2011 20:10 # 0

      Неплохо.
      Ответить

    • ava Lure Of Chaos 15.11.2011 20:16 # 0

      // Check that current character is number.
      // All characters are numbers.

      супер, конечно
      Ответить

    • ava Govnocoder#0xFF 15.11.2011 20:19 # −2

      Автор, что по задумке должна делать функция? Если проверять строку на оцифрованность, то всё нормально, кроме цикла без Break.
      Ответить

    • ava HottDogg 15.11.2011 20:37 # +1 

      function isInteger(s) {
    return !/\D/.test(s);
}
      Ответить

    • ava HottDogg 15.11.2011 20:41 # +5

      А вообще если писать адекватно, то так:
      function isInteger(s) {
    return parseFloat(s) === parseInt(s);
}
      Ответить

      • ava Govnocoder#0xFF 15.11.2011 20:43 # +1

        Адекватно можно вообще без функции обойтись.
        Ответить

        • ava HottDogg 15.11.2011 20:44 # 0

          Зато если она вдруг пригодится, то она есть :)
          Ответить

      • ava Narsul 17.11.2011 17:32 # 0

        при такой реализации будут проблемы, например, с числами вида '015', для parseInt('015') - это 13, так как число в восьмеричном представлении, а вот parseFloat('015') уже считает что это 15.

        Лучше сделать вот так:
        function isInteger(s) {
    return Number(s) === parseInt(s);
}
        Ответить

        • ava HottDogg 17.11.2011 17:35 # 0

          Такие конструкторы лучше вообще не использовать
          parseFloat(s) === parseInt(s, 10);
          Ответить

          • ava Narsul 18.11.2011 11:12 # 0

            Никаких проблем в использовании именно конструктора Number нет, это же не new Function() которая потом вызывает eval.

            Опять же, Ваша реализация снова имеет проблемы, например если s = '012a', функция должна возвратить false, так как это даже не число, однако, Ваша функция вернет true.

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

    • ava Govnocoder#0xFF 15.11.2011 20:53 # +1

      All characters are belong to us!
      Ответить

    • ava DrFreez 15.11.2011 21:09 # 0

      не понятно кто минусует и за что :)
      Ответить

    • ava Narsul 17.11.2011 17:34 # +1

      Кроме всего прочего автор, по всей видимости, забыл, что integer может быть и отрицательным (:
      Ответить

    Добавить комментарий