@Buckstabue

Здравствуйте. Возникло недопонимание с руководителем в следующем вопросе: я реализовывал локигу синхронизации с сервером, для этого создал специальный интерфейс ISyncUpCommand с единственным методом execute(), был список из этих команд, который синхронизируют разные сущности по очереди. Но руководитель удалил этот интерфейс и переделал список из ISyncUpCommand в список из Callable<Void>. Лично мне не понравилось это решение. Оно лишает контекста, в очередь команд можно засунуть что угодно, в том числе не связанное с синхронизацией. Плюс масштабируемость страдает. Что, если мы захотим как можно скорее остановить текущую команду? Надо будет добавить метод stop() и снова возвращать старый интерфейс обратно. Кто как думает?
Я еще понимаю делать общий интерфейс для коллбеков - там все примитивно, четко и ясно. Но здесь это попахивает экономией на спичке и загрязнению кода

0

@KEKCoGEN

Buckstabue, то что вы написали, надо было сказать вашему руководителю а не сюда. Если сказали, каковы были его аргументы? По вашему описанию нельзя сказать что лучше т.к никто тут не знает что у вас за проект и что там происходит.

0