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

    +134

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    for (var i = 0; i < numberPhone.Length; i++)
                {
                    if (numberPhone[i] == ',')
                        return resultPhone;
                    if (Char.IsNumber(numberPhone[i]))
                    {
                        resultPhone += numberPhone[i];
                    }
                }
                return resultPhone;

    Запостил: vladb9582, 14 Августа 2014

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

    • ava kegdan 14.08.2014 23:26 # 0

      >>numberPhone

      намбырфоны атакуют!

      Почему бы не юзать кошерное PhoneNumber ?

      >> if (numberPhone[i] == ',') return resultPhone;

      Запятая - терминальный символ нового поколения.

      Ну и из контекста не понятно, че нужно было и правильно ли автор складывает циферки или нет (Хотя я бы '0' вычел бы, да)
      Ответить

      • ava Fike 15.08.2014 03:45 # +4

        > Почему бы не юзать кошерное PhoneNumber ?

        мгимо финишд
        Ответить

      • ava JuiceRU 15.08.2014 09:12 # +1

        А по-моему все очевидно.
        Есть строка телефонов через запятую. А автор возвращает первый телефон в цифровом формате.
        А где говно? и как надо тогда?
        Ответить

        • ava 1024-- 15.08.2014 09:36 # +1

          Думаю, kegdan говорит ещё про split или регулярки.
          http://ideone.com/nEyRIe
          Ответить

          • ava JuiceRU 15.08.2014 10:32 # +3

            Ну согласен, что split бы предпочтительнее. Да и потом, если вдруг понадобиться не первый, а все телефоны.
            Понимаю, что регулярку написали просто для примера, но например, если телефон с кодом города (000) 000-00-00, и усе. А как там манагеры записывали телефоны, вдруг еще какая-нибудь кракодябля попадется, так что лучше долой все ненамберы. Хотя в таком случаи, пример "000-00-00 доб. 000" плохо преобразуется, а регулярка наоборот спасет.
            Короче, все зависит от базы этих телефонов.
            Ответить

            • ava eth0 15.08.2014 18:35 # 0

              Меня один раз попросили помочь, в справочнике одноэски были данные в наименовании прям, всё вперемешку, имена, фамилии, организации, местные малосимвольные телефоны, прямые одиннадцатисимвольные сотовые, время, когда звонить можно и куча всякого другого мусора. Попросили выделить хотя бы все телефоны, которые получится. После фейспальма в сторону разработчика и идиота-набивателя пришлось выгружать всё и писать злую регулярку.
              Ответить

        • ava kegdan 15.08.2014 10:30 # 0

          Я таки подумал. что resultPhone - инт. Но теперь виду, он может быть строкой.
          Ответить

    • ava guest6 04.01.2023 20:04 # 0

      какой пиздец
      Ответить

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