Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3주차과제] 이원규 #43

Open
wants to merge 24 commits into
base: OneK-2
Choose a base branch
from
Open

[3주차과제] 이원규 #43

wants to merge 24 commits into from

Conversation

OneK-2
Copy link

@OneK-2 OneK-2 commented Feb 5, 2023

이번 주에 어떤 작업을 했는지 설명해주세요.

  • 피드백 반영하여 카테고리를 enum으로 변경, BaseTimeEntity추가
  • 로그인 기능 리펙토링
  • 회원가입 추가
  • 상품 관련 엔티티, 레포지토리, 서비스 추가

특히 어떤 부분을 리뷰받고 싶나요?

  • User에서 Builder를 사용하는 것이 문제가 없는지 궁금합니다.
  • 지금은 이미지 파일을 상품엔티티의 컬럼으로 설정 했는데, 테이블을 따로 분리하는것이 더 좋은 방법인지 궁금합니다.

이번 주는 어떻게 학습했나요? 아래 질문에 짧게 답변주세요!

이번 주에 학습에 투자한 시간

  • 20시간 이상

학습 하면서 좋았던 점과 아쉬웠던 점

  • 코드를 리펙토링 하면서 작동 원리에 대한 이해도가 상승한것같다.
  • 기존 코드를 수정하는데 시간을 너무 많이 사용해서 인터셉터 관련 공부가 부족했다.
  • 예외처리가 미흡한것 같다.

어려움을 겪는 부분

  • 레디스와 스웨거를 사용해보려 했는데, 생각만큼 잘 되지 않아서 아직 반영은 하지 않은 상태입니다. 관련하여 공부할만한 자료가 있는지 궁금합니다.

스터디 개선되었으면 하는 점

  • 없습니다

Copy link
Collaborator

@won983212 won983212 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

User에서 Builder를 사용하는 것이 문제가 없는지 궁금합니다.

네 문제 없을 것 같습니다. 저도 Entity를 초기화하는 방법으로 Builder를 자주 사용합니다. 다만 @Builder를 붙인 생성자를 private으로 사용하는 걸 권장드립니다. 특별한 이유가 없는 한 Builder이외 다른 방법으로 객체 생성을 하지 않도록 하는 게 좋을 것 같습니다.

지금은 이미지 파일을 상품엔티티의 컬럼으로 설정 했는데, 테이블을 따로 분리하는것이 더 좋은 방법인지 궁금합니다.

이미지 파일이 1개라면 문제가 되지않습니다. 여러 개를 사용하시게 될 경우에는 테이블로 따로 빼셔야합니다. 향후에 여러 개를 등록할 수 있도록 하는 경우에 대비해서 따로 테이블로 빼셔도 좋을 것 같습니다,

@won983212
Copy link
Collaborator

Swagger는 springdoc을 사용하신다면, https://springdoc.org/을 참고해보시면 좋을 것 같습니다. 공식 문서가 꽤 잘 되어있습니다! (OpenAPI 버전3 기준입니다)

Redis는 제가 깊게 공부를 해본 경험이 없어서 추천 드리기가 어렵네요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants