1. C# / Говнокод #8310

    +124

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    private bool IsDigit(char c)
        {
            if (digitInIndicatorList.Contains(c))
            {
                return true;
            }
            return false;
        }

readonly static List<char> digitInIndicatorList = new List<char>() { '0', '1', '2', '3', '4', '5', '6', '7', '8', '9' };

    Неустаревающая классика...

    Запостил: fr0mrus, 27 Октября 2011

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

    • ava Lure Of Chaos 27.10.2011 11:32 # 0

      синтетика?
      Ответить

    • ava absolut 27.10.2011 12:32 # +1

      а почему римские не учитывает?
      Ответить

    • ava fr0mrus 27.10.2011 16:58 # 0

      Автор задумался, когда я у него спросил, что будет, если на клавиатуре не будет арабских цифр :) До сих пор молчит...
      Ответить

    • ava byzyan 31.10.2011 16:33 # 0

      вместо
      if (digitInIndicatorList.Contains(c))
      {
      return true;
      }
      return false;
      надо
      писать сразу return digitInIndicatorList.Contains(c)
      Ответить

      • ava SmackMyBitchUp 31.10.2011 18:15 # +2

        ЩИИТООО?!!?!?!?! О__________________________о
        Ответить

        • ava byzyan 01.11.2011 09:15 # 0

          не понятно?))
          Ответить

          • ava Joe_MD 03.11.2011 01:36 # 0

            Насьяльника, а может так?
            try {
            if (digitIndicators.Contains(c)){
            return true;
            } else if(!digitIndicators.Contains(c)) {
            return false;
            } else {
            return false;
            }
            } catch (Exception ex) {
            return false;
            }
            Ответить

            • ava byzyan 03.11.2011 08:36 # 0

              в методе Contains есть одно исключение(ArgumentNullException).. согласен, что нужно проверять, но не городить кучу строк кода...
              чем плохо так?:
              return c!=null?digitIndicators.Contains(c):fals e;
              Ответить

              • ava TauSigma 16.04.2013 13:00 # +1

                В таком случае правильнее будет писать так:
                private bool IsDigit(char c)
{
    if(c==null)
        throw new InvalidProgramException("Неизвестная версия .NET Framework");

    return Char.IsDigit(c);
}
                Ответить

    • ava guest8 09.04.2019 11:49 # −999

      показать все, что скрытоvanished
      Ответить

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