9조 BE 코드리뷰 4회차

[yooonwodyd]

PR

✨ 작업 내용

• 유저 도메인 관련 Swagger 추가

관련 사항

유저 도메인 API가 어느정도 완성되어 리팩토링을 하는 과정을 거치려고 합니다. 고민인 점은 User Entity에서 개인정보와 관련된 공통적인 부분을 UserInfo로 분리했는데, 지금 와서 다시 보니 불필요하지 않았나 싶어 여쭈어봅니다.

유저 이름이나 사진에 접근하려고 해도 userInfo를 통해 접근해야 하는 작업이 추가되었고, UserInfo를 별도의 테이블로 분리해서 관리하게 될 경우를 생각하기도 했는데 현 시점에서 의미가 없는 코드 처럼 보이기 때문입니다.
처음 목적은 도메인에 더 많은 책임을 주자는 취지였는데, 이를 제대로 반영하지 못하고 있다는 점 또한 그렇습니다.

행복한 주말 보내십쇼! 감사합니다

[leaf-upper]

실제 현업에서는 개인정보와 관련된 데이터는 별도 보안 네트워크망에서 분리보관 저장하고 있습니다. 개인정보는 유출되면 안되는 민감한정보이기 때문입니다.
특정 데이터, 클래스를 분리 했을때는 그 이유가 분명하면 좋을것 같긴합니다. 객체지향에서 주로 말하는 5대 원칙에서도 객체를 책임에 따라 분리하는것을 추천합니다. 즉 책임에 따라 분리하는것이지 데이터의 용도에 따라 분리하는것은 좋지못할수도 있을것 같아요.
개인정보와 관련된 부분을 왜 UserInfo로 분리해서 저장하려고 하셨는지 궁금하고, 그에 대해 적절한 책임과 기능이 없다면 다시 합쳐도 좋지 않을까 싶어요~

[leaf-upper]

👍 👍 고생많으셨습니다 벌써 4주차네요~ 앞으로 몇주간은 중간고사 기간이실텐데 중간고사도 화이팅입니다.