Комментарии пользователя Орхан Гасанлы

Может ли Service иметь репозитории других классов?

Орхан Гасанлы @azerphoenix Куратор тега Java

Valeriy_Morozov,
Добрый день.
Насчет маппера - можно подключить библиотеку ModelMapper

<dependency>
            <groupId>org.modelmapper.extensions</groupId>
            <artifactId>modelmapper-spring</artifactId>
            <version>3.0.0</version>
        </dependency>

Затем создать интерфейс и его реализацию:

public interface MapperService {

    /**
     * Note: outClass object must have default constructor with no arguments
     *
     * @param <D> type of result object.
     * @param <T> type of source object to map from.
     * @param entity entity that needs to be mapped.
     * @param outClass class of result object.
     * @return new object of <code>outClass</code> type.
     */
    <D, T> D map(T entity, Class<D> outClass);

    /**
     * Note: outClass object must have default constructor with no arguments
     *
     * @param entityList list of entities that needs to be mapped
     * @param outCLass class of result list element
     * @param <D> type of objects in result list
     * @param <T> type of entity in <code>entityList</code>
     * @return list of mapped object with <code><D></code> type.
     */
    <D, T> List<D> mapAll(Collection<T> entityList, Class<D> outCLass);

    /**
     * Maps {@code source} to {@code destination}.
     *
     * @param source object to map from
     * @param destination object to map to
     */
    <S, D> D map(S source, D destination);

}

Имплементация:

@Service
@RequiredArgsConstructor
public class MapperServiceImpl implements MapperService {

    private final ModelMapper modelMapper;

    @Override
    public <D, T> D map(final T entity, Class<D> outClass) {
        return modelMapper.map(entity, outClass);
    }

    @Override
    public <D, T> List<D> mapAll(final Collection<T> entityList, Class<D> outCLass) {
        return entityList.stream().map(entity -> map(entity, outCLass)).collect(Collectors.toList());
    }

    @Override
    public <S, D> D map(final S source, D destination) {
        modelMapper.map(source, destination);
        return destination;
    }
}

Теперь, если понадобится замаппить любой объект, то достаточно:

Order order = mapperService.map(orderCreationRequest, Order.class);

или наоборот:

OrderCreationResponse orderCreationResponse = mapperService.map(order, OrderCreationResponse.class);

Теперь, дальше...
У вас есть и UserRepository и UserService. Можно удалить UserRepository и оставить только UserService. Вместо OrderMapper автовайрить MapperService. DishRepository также заменить на DishService. В общем, идея такова, что все репозитории, кроме репозитория соответствующей сервису сущности заменить на {Foo}Service. Оставить из репозитория только OrderRepository.

Но в принципе, это не ошибка. Т.е. можно и так оставить. Вопрос в том, как принято в команде работать.

Войдите на сайт