Мне нужно сделать усовершенствование существующего приложения RCP в Java 7. Улучшение очень глубоко в классе, в подклассе. Для этого класса не существует тестов.
Подкласс расширяет JobChangeAdapter и в пределах done() вызывает org.eclipse.swt.widgets.Display#syncExec(Runnable). Нам нужно улучшить run() подкласса, который реализует Runnable.
Улучшение - проверка определенных условий, создание и запуск диалогового окна, поэтому я создал «служебный» интерфейс с двумя методами.
boolean isDialogDisplayable();
void displayDialog();
Однако мне сказали, что это плохая схема, которой нужно следовать в обзоре кода, и что я не должен помещать поведение функций в служебные классы.
Мне интересно, почему он считается плохим шаблоном, когда он позволяет легко тестировать?