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

    +133

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    panel1.Visible = checkBoxCCF.Checked;
panel2.Visible = checkBoxReliabilty.Checked;
panel3.Visible = checkBoxRisk.Checked;
panel4.Visible = checkBoxSaftey.Checked;
panel5.Visible = checkBoxSensitivity.Checked;
panel6.Visible = checkBoxThroughput.Checked;
panel7.Visible = checkBoxUncertainity.Checked;

    Почему половина переменных нормальные, половина нет? Логика некоторых погромистов зашкаливает...

    Запостил: kostoprav, 28 Мая 2014

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

    • ava kipar 28.05.2014 14:40 # −4

      Тоже часто так делаю. Не будешь же придумывать название для каждой бесполезной панельки и лейбла, единственная функция которых - чтоб их положили на форму и они там лежали. А если это чекбокс который мы проверяем в коде - лучше сделать нормальное название.
      А потом раз, и понадобилось управлять видимостью этих панелек - можно конечно переименовать, но если это единственное место где мы к ним обращаемся, можно и забить.
      Ответить

      • ava Vasiliy 28.05.2014 14:46 # 0

        В коде управлять видимостью элементов. Сколько помню в Дельфи был TAction как раз для этих целей там все настраивается визуально и все.
        Ответить

      • ava bormand 28.05.2014 15:18 # +3

        > Не будешь же придумывать название для каждой бесполезной панельки и лейбла
        Это верно, поэтому некоторые системы позволяют... не давать им имени вообще. И автодополнение не засирается никому не нужным мусором, и когда все-таки захочется к ним обратиться - не будет искушения оставить panel1 как есть, а придется дать нормальное имя.
        Ответить

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