26.05.2016 / 09:10 | |
aNNiMON Супервизор Сейчас: Offline
Имя: Витёк Регистрация: 11.01.2010
| Не знаю даже, в чём бóльшая проблема, в куче if в commandAction или в куче объектов форм. Можно было переиспользовать одну-две формы. Или сложить всё в массив
__________________
let live Изменено aNNiMON (26.05 / 09:11) (всего 1 раз) |
26.05.2016 / 09:49 | |
Витаминыч Супермодератор Сейчас: Offline
Имя: Василиус Откуда: RZN Регистрация: 20.04.2012
| aNNiMON, кучу форм, в общем, можно сократить, есть метод deleteAll для очистки формы и тогда ифы можно сократить
сначала хотел так сделать, но с той реализацией такого не получилось бы, пэтому просто переписал на свои формы
__________________
わからない!! |
26.05.2016 / 09:54 | |
Кальцид Пользователь Сейчас: Offline
Имя: Влад Откуда: Карадарас Регистрация: 05.12.2011
| У нас такой темы небыло на химие |
26.05.2016 / 10:05 | |
aNNiMON Супервизор Сейчас: Offline
Имя: Витёк Регистрация: 11.01.2010
| Витаминыч, commandAction или какой-нибудь общий метод (как getShowMainForm, только чтоб он на все твои формы распространялся) ты так и не вставил. А ведь с ними было бы куда меньше кода. 1 из 2 | Презентация |
if (d == a1) {
if (c == a1.getShowMainForm()) {
mainDisplay.setCurrent(mainForm);
}
}
if (d == a2) {
if (c == a2.getShowMainForm()) {
mainDisplay.setCurrent(mainForm);
}
}
// и ещё есть
if ((AnswerForm) d).isShowMainFormCommandSelected(c)) {
mainDisplay.setCurrent(mainForm);
}
public class AnswerForm extends Form {
//...
public boolean isShowMainFormCommandSelected(Command c) {
return c == showMainForm;
}
}
А если б ещё интерфейс создать или абстрактный класс с этим методом isShowMainFormCommandSelected, тогда можно будет одним сравнением покрыть команду выхода в главное меню на всех формах. if ((AbstractForm) d).isShowMainFormCommandSelected(c)) { ... }
__________________
let live Изменено aNNiMON (26.05 / 10:07) (всего 4 раза) |
26.05.2016 / 14:26 | |
Bogdan-G Пользователь Сейчас: Offline
Имя: Богдан Откуда: Москва Регистрация: 13.02.2011
| проект можно прогнать PMD и FindBugs.
|
26.05.2016 / 14:41 | |
Витаминыч Супермодератор Сейчас: Offline
Имя: Василиус Откуда: RZN Регистрация: 20.04.2012
| aNNiMON, я такие конструкции впервые вижу ((Класс) объект).метод()Надо подумать Bogdan-G, прогони __________________
わからない!! Изменено Витаминыч (26.05 / 14:42) (всего 1 раз) |
26.05.2016 / 16:31 | |
DominaN Пользователь Сейчас: Offline
Имя: Кирилл Откуда: Смоленск Регистрация: 28.09.2011
| Витаминыч, не слишком быстрые конструкции, так-то. Но иногда приходится использовать
|