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

    +141

    1. 1
    2. 2
    3. 3
    4. 4
    if ((ddlProbType.Text != "" && ddlProbType.Text != "Select") && (txtDesc.Text != ""))
            {
                if ((txtDesc.Text.Trim() == string.Empty))
                {

    Indian style: чем больше скобок, тем лучше!

    Запостил: Nataly, 17 Февраля 2012

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

    • ava 7ion 17.02.2012 13:06 # +3

      Ну, по две скобки в обертке, конечно, излишек.
      Но вот в логике скобки стоит ставить всегда, даже если приоритет не требует.
      Читать потом легче, да и в голове лично у меня все приоритеты не держутся.
      Ответить

    • ava ICELedyanoj 17.02.2012 13:33 # +1

      Это может быть просто свидетельством ускоренного рефакторинга или правки. Бывает такое - && или || снесли, а лишние скобки не убрали. Потому что некогда :)
      Ответить

    • ava Lure Of Chaos 17.02.2012 22:54 # +1

      лисперы снисходительно улыбаются
      Ответить

    • ava kotlomoy 18.02.2012 14:39 # 0

      Вот у нас так: если в условии есть присваивание, то наш гнусный компилятор требует двойные скобки. У этих ребят тот же компилятор наверно, вот они и решили двойные скобки везде ставить и не париться. На говнокод не тянет
      Ответить

      • ava SmackMyBitchUp 18.02.2012 16:40 # 0

        mucho gusto
        Ответить

      • ava Steve_Brown 20.02.2012 09:41 # 0

        Кто гнусный? Присваивание в ифе можно писать только так:
        if ((var = getValue()) != 0) // NULL, false
  ... var ... ;

        И если компилятор на if (var = getValue()) выдает варнинг, то это правильно.
        Ответить

    • ava guest 18.02.2012 17:52 # 0

      причем здесь скобки? говно в хардкоде.
      Ответить

    • ava guest8 09.04.2019 12:43 # −999

      показать все, что скрытоvanished
      Ответить

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