Си диез / Говнокод #16067 Ссылка на оригинал

0

  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 kostoprav, (Updated )

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

  • ava kipar # Ссылка на оригинал 0

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

    • ava Vasiliy # Ссылка на оригинал 0 ⇈

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

    • ava bormand # Ссылка на оригинал 0 ⇈

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

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓