Где должны находиться методы конвертации DTO -> Entity и наоборот?

Question

[Biaci_Anj]

Я расположил методы в сервисах. Удобно, но возникла проблема при тестировании. Метод конвертации в ДТО вызывается почти в каждом методе сервиса. И теперь при тестировании я должен каждый раз проверять, совпадает ли результат с нужным, вместо того, чтобы просто verify вызывался ли метод и все ( verify нельзя вызвать не на моке ).
Возникла идея перенести метод в отдельный класс, но я слаб в хороших практиках.
Подскажите, пожалуйста, куда было бы правильно перенести эти методы конвертации?
Мне создать отдельный пакет mapper, а в нем классы? Допустим, класс EmployeeMapper и в нем два метода toEmployee, toEmployeeDTO? Правильно ли я мыслю?

P.S. мапстракт не могу использовать

Answer 1

[BorLaze]

Мне создать отдельный пакет mapper, а в нем классы? Допустим, класс EmployeeMapper и в нем два метода toEmployee, toEmployeeDTO? Правильно ли я мыслю?

Да.

Можно долго расписывать, почему именно так, но особо смысла не вижу.

Comments

[acwartz]

А как же modelMapper и подобные?

[BorLaze]

acwartz, а что маппер?

Это библиотека; а где ее вызывать, это уже другой вопрос.

Для каких-то простых преобразований, где достаточно вызвать библиотечные методы и все - тут да, городить отдельный layer смысла нет.

Но, по опыту, такое прокатывает редко - обычно приходится писать какие-то кастомные правила DTO <-> Entity <-> DTO, а это уже как раз стоит прятать в отдельные компоненты и оформлять дополнительным слоем рядом с контроллерами.

Если уж совсем по феншую делать, так сказать.

[Biaci_Anj]

BorLaze, подскажите, пожалуйста, стоит ли сверху над маперами делать интерфейс, где будут перечислены все методы классов мапперов?
Или это лишнее

[BorLaze]

Параметризованный интерфейс не помешает. По крайней мере, все будет выглядеть "пусть безобразно, зато однообразно" :-)

Что-то типа

public interface MyMapper<D, E> {
    D toDto(E entity);
    E toEntity(D dto);
}

тогда конкретную имплементацию можно оформить как

public class MyBookMapper implements MyMapper<BookDto, Book> {

    @Override
    public BookDto toDto(Book entity) {
        ...
    }

    @Override
    public Book toEntity(BookDto dto) {
        ...
    }
}

Answer 2

[acwartz]

В сервисе MagicDTO инкапсулирующем ModelMapper. За примерами сюда. И забыть все эти toEntity, toDTO как страшный сон.

Comments

[BorLaze]

Ничего не понял. Сходил за примерами туда. Вижу:

@Component
public class UnicornMapper {

    @Autowired
    private ModelMapper mapper;

    @Override
    public Unicorn toEntity(UnicornDto dto) {
        return Objects.isNull(dto) ? null : mapper.map(dto, Unicorn.class);
    }

    @Override
    public UnicornDto toDto(Unicorn entity) {
        return Objects.isNull(entity) ? null : mapper.map(entity, UnicornDto.class);
    }
}

И как это стыкуется с "забыть все эти toEntity, toDTO как страшный сон"?

[acwartz]

BorLaze, если знать про дженерики то вот так:

public <D, T> D map(final T entity, Class<D> outClass) {
    return modelMapper.map(entity, outClass);
  }
  public <S, D> D map(final S source, D destination) {
    modelMapper.map(source, destination);
    return destination;
  }

[Jacen11]

acwartz, а если поля имеют разные названия? а списки? а если внутри еще объекты? В проекте нормального размера не помогут твои дженерики