Это будет нормально работать и является простой реализацией, но с архитектурной точки зрения это не очень хорошая практика, потому что это тесно связывает вашу остальную реализацию с дизайном вашей базы данных и предоставляет ее потребителю API.
Вы должнычтобы объекты запроса / ответа входили в ваш API и выходили из него, отделив его от базы данных и позволяя вам выполнять проверки и другие бизнес-правила на уровне выше доступа к данным.Если вы обнаружите, что вам приходится писать много кода присваивания между сущностью и dtos, вы можете использовать такой инструмент, как http://modelmapper.org/ или http://mapstruct.org/