Вниз  Chemicals
- 26.05.2016 / 09:10copy  #462666
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
Не знаю даже, в чём бóльшая проблема, в куче if в commandAction или в куче объектов форм. Можно было переиспользовать одну-две формы. Или сложить всё в массив
__________________
 let live

Изменено aNNiMON (26.05 / 09:11) (всего 1 раз)
- 26.05.2016 / 09:49copy  #462667
Витаминыч
  Супермодератор

Витаминыч 
Сейчас: Offline
aNNiMON, кучу форм, в общем, можно сократить, есть метод deleteAll для очистки формы
и тогда ифы можно сократить

сначала хотел так сделать, но с той реализацией такого не получилось бы, пэтому просто переписал на свои формы
__________________
 わからない!!
- 26.05.2016 / 09:54copy  #462668
Кальцид
  Пользователь

Кальцид 
Сейчас: Offline
У нас такой темы небыло на химие :gg:
- 26.05.2016 / 10:05copy  #462669
aNNiMON
  Супервизор

aNNiMON 
Сейчас: Offline
Витаминыч, commandAction или какой-нибудь общий метод (как getShowMainForm, только чтоб он на все твои формы распространялся) ты так и не вставил. А ведь с ними было бы куда меньше кода.
1 из 2Презентация
  1. if (d == a1) {
  2.     if (c == a1.getShowMainForm()) {
  3.         mainDisplay.setCurrent(mainForm);
  4.     }
  5. }
  6. if (d == a2) {
  7.     if (c == a2.getShowMainForm()) {
  8.         mainDisplay.setCurrent(mainForm);
  9.     }
  10. }
  11. // и ещё есть
А если б ещё интерфейс создать или абстрактный класс с этим методом isShowMainFormCommandSelected, тогда можно будет одним сравнением покрыть команду выхода в главное меню на всех формах.
  1. if ((AbstractForm) d).isShowMainFormCommandSelected(c)) { ... }

__________________
 let live

Изменено aNNiMON (26.05 / 10:07) (всего 4 раза)
- 26.05.2016 / 14:26copy  #462677
Bogdan-G
  Пользователь

Bogdan-G 
Сейчас: Offline
проект можно прогнать PMD и FindBugs.
- 26.05.2016 / 14:41copy  #462679
Витаминыч
  Супермодератор

Витаминыч 
Сейчас: Offline
aNNiMON, я такие конструкции впервые вижу :gg: ((Класс) объект).метод()
Надо подумать

Bogdan-G, прогони :gg:
__________________
 わからない!!

Изменено Витаминыч (26.05 / 14:42) (всего 1 раз)
- 26.05.2016 / 16:31copy  #462680
DominaN
  Пользователь

DominaN 
Сейчас: Offline
Витаминыч, не слишком быстрые конструкции, так-то. Но иногда приходится использовать
Наверх  Всего сообщений: 27
Фильтровать сообщения
Поиск по теме
Файлы топика (4)
Поиск по форуму