SeriesService: move catalog numbers related methods from DAO and service

[php-coder]

Move findBy{Michel,Scott,Yvert,Gibbons}Number() to StampsCatalogService.

Tech debt for: 5d148c0 (#120)

[Shagbark]

Я тут подумал, вот первоначальный драфт рефакторинга.
Метод findByCatalogNumber в SeriesServiceImpl. (Пока пишу в этом классе).
List<SeriesInfoDto> findByCatalogNumber(String catalogName, String catalogNumberCode, String lang)

if (catalogName == null || catalogName.isEmpty() {
	return Collections.emptyList();
}

catalogName = Character.toUpperCase(catalogName.charAt(0)) + catalogName.substring(1);

Validate.isTrue(catalogNumberCode != null, catalogName + " number code must be no null");
Validate.isTrue(!catalogNumberCode.trim().isEmpty(), catalogName + "number code must be non empty");

List<Integer> seriesIds = seriesDao.findSeriesIdsByCatalogNumberCode(catalogName, catalogNumberCode);
if (seriesIds.isEmpty) {
	return Collections.emptyList();
}

return seriesDao.findByIdsAsSeriesInfo(seriesIds, lang);

catalogName = Character.toUpperCase(catalogName.charAt(0)) + catalogName.substring(1); можно эту строку убрать, если не требуется обязательного начала сообщения с большой буквы.

В классе Dao в методе seriesDao.findSeriesIdsByCatalogNumberCode(catalogName, catalogNumberCode); можно сделать switch, на примере того, как он реализован в SeriesController.
Можно еще подумать над тем, чтобы как-то унифицировать SQL запрос, но у меня как-то не получается придумать его более простым. Все получается громоздким.

[php-coder]

catalogName = Character.toUpperCase(catalogName.charAt(0)) + catalogName.substring(1); можно эту строку убрать, если не требуется обязательного начала сообщения с большой буквы.

Если бы нам это и понадобилось, то для этого уже есть метод: StringUtils.capitalize()

[Shagbark]

А да,точно. Помню, что где-то был такой метод. :)

[php-coder]

Отлично, что предложил свой вариант!

Ты уже смотрел на класс StampsCatalogServiceImpl? Там есть все ответы ;)

[Shagbark]

Агааа! :)
Я тут еще раз посидел, посмотрел, наваял второй драфт:

Получается необходимо добавить в StampsCatalogDao метод List<Integer> findByCatalogNumber(String catalogCode). В реализацию DAO добавляем еще одно поле private final String findByCatalogCodeSql. В теле метода: return jdbcTemplate.queryForList(findByCatalogCodeSql, Collections.singlentonMap("catalog_code"),String.class)

В класс DaoConfig в методах get{Michel,Scott..}CatalogDao() в вызове new JdbcStampsCatalogDao добавляемenv.getRequiredProperty("здесь property key, в котором лежит определенный SQL запрос"). По сути, реализация метода закончена.

Далее в SeriesService добавляем метод findByCatalogCode(String catalogName, String catalogCode, String lang). В зависимости от catalogName, вызываем метод findByCatalogCode определенного класса StampsCatalogService(поля {scott,gibbons}CatalogService). И в конце метода seriesDao.findByIdsAsSeriesInfo().

Что скажешь?

На самом деле не очень нравится такая реализация. Мне кажется, что нужно потом, как-нибудь, поправить сам класс SeriesService, чтобы не хранить внутри 4 экземпляра интерфейса StampsCatalogService.
Здесь нужно побольше подумать :)

[php-coder]

Отлично! Уже близко! :)

Получается необходимо добавить в StampsCatalogDao метод List findByCatalogNumber(String catalogCode)

Да, метод будет практически таким же как и текущий, изменения будут минимальными.

В класс DaoConfig в методах get{Michel,Scott..}CatalogDao() в вызове new JdbcStampsCatalogDao добавляем

Да, похоже на правду. Не забудь что делать эти изменения нужно не разом для всех, а сначала только для Michel, потом для Scott и т.д.

Далее в SeriesService добавляем метод findByCatalogCode

В заголовке задачи написано "move" поэтому подразумевается, что 4 метода из SeriesService "уйдут". Там где сейчас использовали SeriesService нужно будет использовать StampsCatalogService.

На самом деле не очень нравится такая реализация. Мне кажется, что нужно потом, как-нибудь, поправить сам класс SeriesService, чтобы не хранить внутри 4 экземпляра интерфейса StampsCatalogService.

Ты мыслишь в правильном направлении, но, видимо, я не достаточно точно задачу сформулировал или ты невнимательно ее прочитал :)

Ну что, сделаешь рефакторинг для одного метода, чтобы посмотреть и поревьювить?

[Shagbark]

Да, давай попробуем :)

[Shagbark]

Брр. Что-то совсем запутался.

1. В SeriesController добавляем поле scottStampsCatalogService.
2. В ControllerConfig метод getSeriesCongroller изменяем на:
   return new SeriesController( servicesConfig.getCategoryService(), servicesConfig.getCollectionService(), servicesConfig.getCountryService(), servicesConfig.getSeriesService(), servicesConfig.getSeriesSalesService(), servicesConfig.getTransactionParticipantService() servicesConfig.getScottCatalogService() );
3. А дальше в методе searchSeriesByCatalog вызываем объединенный метод findSeriesIdsByCatalogNumberCode.
   Это для одного метода.

А так у нас получится 4 экземпляра StampsCatalogService в классе SeriesController. Это если просто переносить методы. Я прав?

Или я что-то затупил, и не понял.

[php-coder]

Да, все верно.

[Shagbark]

Все, хорошо. :)