1. C++ / Говнокод #1547

    +13

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    __inline BOOL IsNumeric(LPTSTR lpsBuf)
{
 int i, nSize = _tcslen(lpsBuf);
 for (i = 0; i < nSize; i++)
  if ((lpsBuf[i] < _T('0')) || (lpsBuf[i] > _T('9')))
   return (FALSE);
 return (TRUE);
}

    Серъёзный алгоритм.

    Запостил: Exhumer, 12 Августа 2009

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

    • ava guest 12.08.2009 16:41 # +1

      Да, вместо предварительного вычисления длины строки надо было просто проходить по ней, пока не встретится '\0'. Глупая ошибка.

      for (; *lpsbuf >= _T('0') && *lpsbuf <= _T('9'); ++lpsbuf) ;
      return *lpsbuf == 0;

      Как-то так.
      Ответить

    • ava guest 14.08.2009 20:09 # 0

      еще тот велосипед))
      надо пользоваться boost::lexical_cast :)
      Ответить

    • ava Dimarius 14.08.2009 20:32 # 0

      Не знаю точно, что имел ввиду автор, но, если он хотел проверить, не является ли строка представлением числа, то он забыл проверить в начале знак минуса :)
      Ответить

    • ava ElWray 17.08.2009 16:49 # 0

      Не кошерный алгоритм...
      Ответить

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