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

    • Ну, по две скобки в обертке, конечно, излишек.
      Но вот в логике скобки стоит ставить всегда, даже если приоритет не требует.
      Читать потом легче, да и в голове лично у меня все приоритеты не держутся.
      Ответить
    • Это может быть просто свидетельством ускоренного рефакторинга или правки. Бывает такое - && или || снесли, а лишние скобки не убрали. Потому что некогда :)
      Ответить
    • лисперы снисходительно улыбаются
      Ответить
    • Вот у нас так: если в условии есть присваивание, то наш гнусный компилятор требует двойные скобки. У этих ребят тот же компилятор наверно, вот они и решили двойные скобки везде ставить и не париться. На говнокод не тянет
      Ответить
      • mucho gusto
        Ответить
      • Кто гнусный? Присваивание в ифе можно писать только так:
        if ((var = getValue()) != 0) // NULL, false
          ... var ... ;

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

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