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

    +947.7

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    foreach (string id in sourceIDs.Split(new char[] { ',' }, StringSplitOptions.RemoveEmptyEntries))
                {
                    sourceId = Convert.ToInt32(id);
                    break;
                }

    Как взять первую цифру в строке - конечно без цикла тут никак

    Запостил: eval_2009, 03 Ноября 2009

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

    • да и new char[] { ','} отжигает....
      Ответить
      • а как надо? ;)
        Ответить
        • Можно просто
          Split(',');
          Ответить
          • Не прокатит, так как Split требуе char[], а не просто char
            Ответить
            • практика показывает что прокатит и Split(',')
              Ответить
              • Split(',') прокатит, но вот Split(',', StringSplitOptions.RemoveEmptyEntries) не прокатит
                Ответить
                • зато прокатит Split(",", StringSplitOptions.RemoveEmptyEntries); - вот так и надо было.
                  Ответить
              • насколько я помню, внутренне там всё равно создаётся new char[',']

                поэтому на такие случае я всегда использую static'и
                Ответить
    • Не цифру, а число.
      и без TryParse тут тоже всё плохо.
      new char[] { ',' } - без этого тут не обойтись, но можно было бы вынести во внешнюю константу, раз уж так хочется.
      Ответить
      • Постер же акцентировал внимание на цикле. Почему вы все так привязались к split?
        Ответить
    • Здесь присутствует еще и нечто более глубокое - просто распарсить строку и взять первый элемент из массива не получится - в случае если в строке нет ни одного ID получим ошибку, нужна проверка строки на "".
      А так - типа ошибки не возникнет )))
      Ответить

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