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

    +132

    1. 1
    2. 2
    3. 3
    long id;
try { id = long.Parse(e.Item.Cells[0].Text); }
catch { id = 0; }

    Эта жемчужина прекрасно описывает весь говнокод производимый различными группами людей в моей компании.

    Запостил: piocsic, 12 Июня 2012

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

    • ava guest 12.06.2012 14:11 # +1

      Что-то подобное написал на 2 курсе. Препод поржал)
      Ответить

    • ava Lure Of Chaos 12.06.2012 16:00 # +1

      длинные стринги

      зато можно вписать текст
      Ответить

    • ava guest 12.06.2012 22:11 # 0

      апофеоз, не спорю (: но все же, как правильно распарсить стринги в число?
      Ответить

      • ava bormand 12.06.2012 22:18 # 0

        http://ideone.com/3M32I
        Как-то так?
        Ответить

        • ava abatishchev 12.06.2012 22:32 # +1 

          int i;
Int32.TryParse(input, out i);
          документация по out гарантирует что переменная будет инициализирована.
          и будет это default(Int32).
          Ответить

          • ava guest 12.06.2012 22:34 # +2

            Да, кстати тогда и инициализировать в ноль не нужно.
            Ответить

          • ava dirtygopher 13.06.2012 13:19 # +1

            Возможно код из .net framework 1.1, насколько я помню, там не было long.TryParse
            Ответить

            • ava guest 22.06.2012 17:53 # +1

              Проверил, так похоже и есть. В msdn по long.TryParse нет инфы для 1.1
              Ответить

              • ava bormand 22.06.2012 18:05 # 0

                Даже в обсираемой всеми Delphi 7 был TryStrToInt ;)
                Как же MS так пролетели.
                Ответить

                • ava guest 22.06.2012 18:21 # 0

                  Потому что судя по всему 1.0 выпускали впопыхах, а 1.1 был во многом просто багфикс 1.0
                  Ответить

                • ava guest 22.06.2012 18:26 # 0

                  А ещё возможно потому что в первых версиях многое тупо без задней мысли копировалось 1-к-1 с Java, и в Java'овском прототипном Long нет метода tryParseLong, есть только parseLong.
                  Ответить

              • ava piocsic 22.06.2012 18:34 # 0

                Судя по перфорсу, код этот был написан во времена дотнет 2.0.

                Да если бы он и был написал во времена 1.0, всё равно нельзя проглатывать все исключения. Надо было хотя бы прописать catch (FormatException) {}
                Ответить

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