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

    +121

    1. 1
    2. 2
    3. 3
    4. 4
    if (Convert.ToBoolean(ViewState["IsProgram"]) != null && Convert.ToBoolean(ViewState["IsProgram"]))
{
    // do work
}

    Это взорвало мой мозг...

    Запостил: rstrg, 13 Ноября 2010

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

    • ava Govnocoder#0xFF 13.11.2010 14:03 # +1

      Никогда не пишите с бодуна!
      Ответить

    • ava Lure Of Chaos 13.11.2010 14:32 # 0

      урок конвертации пройден )
      Ответить

    • ava Oleg_quadro 13.11.2010 14:52 # +1

      это не true, это — труЪ.
      Ответить

    • ava GLeBaTi 14.11.2010 08:23 # 0

      В C#, у bool, значение null бывает?
      Ответить

      • ava Мистер Хэнки 14.11.2010 09:03 # 0

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

      • ava rstrg 14.11.2010 16:57 # +1

        ну смысл в том что Convert.ToBoolean(ViewState["IsProgram"]) != null всегда будет true, видимо это написали на всякий случай, не понимая как это на самом деле работает.
        Чтобы это не было говонокодом нужно было написать так:
        if (Convert.ToBoolean(ViewState["IsProgram"]))
        {
        // do work
        }
        Ответить

        • ava abatishchev 16.11.2010 15:16 # +1

          Очевидно, что стоило написать:
          if (this.ViewState["IsProgram"] as bool? ?? false) { }
          Ответить

          • ava Back 20.11.2010 13:05 # 0

            не обязательно as
            if ((bool?)this.ViewState["IsProgram"] ?? false) { }
            тоже вполне подходит
            Ответить

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