Говнокод

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

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

Запостил:

rstrg, 13.11.2010 (Updated 24.03.2018)

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

Govnocoder#0xFF 16 лет ago # 0

Никогда не пишите с бодуна!
Ответить
Lure Of Chaos 16 лет ago # 0

урок конвертации пройден )
Ответить
Oleg_quadro 16 лет ago # 0

это не true, это — труЪ.
Ответить
GLeBaTi 16 лет ago # 0

В C#, у bool, значение null бывает?
Ответить
- Мистер Хэнки 16 лет ago # 0 ⇈
  
  у примитива не бывает конечно, а вот у враппера запросто
  Ответить
- rstrg 16 лет ago # 0 ⇈
  
  ну смысл в том что Convert.ToBoolean(ViewState["IsProgram"]) != null всегда будет true, видимо это написали на всякий случай, не понимая как это на самом деле работает.
  Чтобы это не было говонокодом нужно было написать так:
  if (Convert.ToBoolean(ViewState["IsProgram"]))
  {
  // do work
  }
  Ответить
  - abatishchev 16 лет ago # 0 ⇈
    Очевидно, что стоило написать:
    
    if (this.ViewState["IsProgram"] as bool? ?? false) { }
    Ответить
    - Back 16 лет ago # 0 ⇈
      
      не обязательно as
      if ((bool?)this.ViewState["IsProgram"] ?? false) { }
      тоже вполне подходит
      Ответить
      - abatishchev 16 лет ago # 0 ⇈
        
        ага, но as чуть быстрее каста, afaik
        Ответить
        
        Back 16 лет ago # 0 ⇈
        
        да, с этим согласен...
        Ответить