Информация об изменениях

Сообщение Re[3]: Уменьшить Cognitive Complexity метода. рефакторинг. от 14.09.2018 13:25

Изменено 14.09.2018 13:26 мамут ушёл, и я пойду

Re[3]: Уменьшить Cognitive Complexity метода. рефакторинг.
Здравствуйте, HAXT, Вы писали:

HAX>Здравствуйте, Artem Korneev, Вы писали:


HAX>Спасибо Артём,


HAX>Product то нормально в саму мапу встраивается а вот zoneId, dateFormat передаются с верхнего вызова и если dateFormat можно заинжектить в этом классе, то zoneId передаётся из UI через контроллер, и тут проблемы с объявлением мапы.


Если цель отформатировать даты под юзера, то это должно быть настройкой отчета (csvBuilder) , который перехватывает даты через перегрузку метода addNextValue и форматирует.

Если цель форматировать конкретное поле в конкретной зоне, то это должно быть часть экстрактора этого полы (захардкодено)

В самом крайнем случае можно передавать Map c постпроцессорами для отдельных полей и проверять
Re[3]: Уменьшить Cognitive Complexity метода. рефакторинг.
Здравствуйте, HAXT, Вы писали:

HAX>Здравствуйте, Artem Korneev, Вы писали:


HAX>Спасибо Артём,


HAX>Product то нормально в саму мапу встраивается а вот zoneId, dateFormat передаются с верхнего вызова и если dateFormat можно заинжектить в этом классе, то zoneId передаётся из UI через контроллер, и тут проблемы с объявлением мапы.


Если цель отформатировать даты под юзера, то это должно быть настройкой отчета (csvBuilder) , который перехватывает даты через перегрузку метода addNextValue и форматирует.

Если цель форматировать конкретное поле в конкретной зоне, то это должно быть частью экстрактора этого поля (захардкодено)

В самом крайнем случае можно передавать Map c постпроцессорами для отдельных полей и проверять