Skip to content

Comments

[BE] dto의 엔티티 생성 책임 mapper로 분리#154

Open
happyAndy0412 wants to merge 2 commits intodevfrom
feature/#113_company_refactoring
Open

[BE] dto의 엔티티 생성 책임 mapper로 분리#154
happyAndy0412 wants to merge 2 commits intodevfrom
feature/#113_company_refactoring

Conversation

@happyAndy0412
Copy link
Collaborator


🚀 어떤 기능을 구현했나요 ?

  • 기존 dto에서 req -> entity 를 생성하는 책임을 mapper클래스로 분리하였습니다
  • 또한, 기존에 생성함수로 빌더를 사용했으나 정적팩토리를 위해 생성자를 이용하여 객체를 생성하도록 하였습니다
  • 영속성 객체의 특성을 근거로, 서비스에서 불필요하게 한번 더 save가되는 메서드를 삭제하고 바로 엔티티의 세터를 이용하여 companyCode를 세팅하도록 했습니다
  • private void makeCompanyCode(Company company) {
    company.assignCompanyCode("C" + company.getId());
    companyRepository.save(company);
    }

🔥 어떤 문제를 마주했나요 ?

✨ 어떻게 해결했나요 ?

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 추후에 리팩토링할때 참고를 위한 코드입니다. mapstruct를 쓰던 이 방식으로 하던 기능과 성능에는 변화가 없으니 머지 요청드립니다

📚 참고 자료 및 회고

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BE] 멀티모듈 분리하기

3 participants