Jawa / Говнокод #16363 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
private void setLock(boolean lock) {
        if (lock) {
            view1.setEnabled(false);
            view2.setEnabled(false);
            view3.setEnabled(false);
            view4.setEnabled(false);
            view5.setEnabled(false);
            view6.setEnabled(false);
        } else {
            view1.setEnabled(true);
            view2.setEnabled(true);
            view3.setEnabled(true);
            view4.setEnabled(true);
            view5.setEnabled(true);
            view6.setEnabled(true);
        }
    }

Copy & Paste наше всё.
P.S. Названия переменных изменены, не пугайтесь.

Запостил: ilsy ilsy, (Updated )

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

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

    > Названия переменных изменены, не пугайтесь.
    Тех, кто видел делфи, view6 не напугаешь...
    Ответить

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

    а как правильно? вот нужно у всех переменных setEnabled сделать
    Ответить

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

      Самое простое и очевидное:
      view1.setEnabled(!lock);
            view2.setEnabled(!lock);
            view3.setEnabled(!lock);
            view4.setEnabled(!lock);
            view5.setEnabled(!lock);
            view6.setEnabled(!lock);

      Заметьте, кода стало в два раза меньше
      Можно ещё сделать массив и в цикле пробежаться

      Ещё предложения?
      Ответить

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

        Массив не так уж и логично использовать, как я понял код с андроида, там так или иначе надо каждую вьюху по отдельности привязывать, соответсвенно под каждую будет своя ссылка
        Ответить

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

        lock = ! lock;
        А дальше по списку. Не нужно повторять отрицание.
        Ответить

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

      Ну хотя бы так:
      private void setLock(boolean lock) {
    view1.setEnabled(!lock);
    view2.setEnabled(!lock);
    view3.setEnabled(!lock);
    view4.setEnabled(!lock);
    view5.setEnabled(!lock);
    view6.setEnabled(!lock);
}
      Ответить

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

        Ну можно и так... Всех детей перебрать то точно можно. И фильтрануть по классу тоже.
        Ответить

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

          Не факт, что блокируются все view - их может быть и больше. Я преположил, что views - массив нужных view, созданный при создании этих обьектов
          А так - да
          шарпик 
          foreach(var chd in Controls)
    if (chd is SomeControlType) 
        chd.Enabled = !lock;
          Ответить

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

Переведи на "PHP", guest!

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

    ↑
    8
    ↓