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

[JT-54] 요일 웹툰/장르 웹툰 테이블 분리 #13

Merged
merged 3 commits into from
Sep 7, 2023

Conversation

kmebin
Copy link
Member

@kmebin kmebin commented Sep 7, 2023

📌 개발 내용

  • 기존의 웹툰 테이블에서 요일-웹툰 테이블과 장르-웹툰 테이블로 분리하였음
  • 웹툰 생성 로직, 웹툰 정보 조회 로직 수정됨

📑 PR 포인트

  • toEntity 가 엔티티 별로 3개 생성되었음 (양방향 참조를 피하기 위함)

👥 협업을 위한 코드리뷰

  1. 세상에 바보같은 질문은 없다.
  2. 실수를 예방하는 팀이 아니라, 실수를 잘 다루는 팀이 되자.
  3. 분업이 아닌 협업을 하자. 우린 모두 같은 문제를 이겨내기 위해 모였다.
  4. 영원한 것은 없으니 대화를 하자.
  5. 서로 존중하자.
  6. 많이 부딪치고 깨닫고 배우자.
  7. 부드럽게 설득하고 열린 마음으로 설득당해보자.

✅ 리마인더

  • 본인의 로컬에서 정상 동작하는지 확인해주세요.
  • 최신 브랜치를 Pull 받고 PR을 요청했는지 확인해주세요.
  • API가 추가되었을 경우 테스트를 하고 PR을 올려주세요.
  • Conflict가 났을 때, UI상에서 해결하지 말고, 본인 local에서 해결해주세요.
  • Commit 메시지 제대로 작성해주세요.

⚙️ 코드리뷰 룰

  • R(Request Change): 해당 블럭은 꼭 변경해주셨으면 좋겠습니다.
  • C(Comment): 웬만하면 고려해주시면 좋겠습니다.
  • Q(Question) : 해당 라인이 궁금합니다.
  • A(Approve): 반영해도 좋고 넘어가도 좋습니다. 혹은 사소한 의견입니다.

@kmebin kmebin self-assigned this Sep 7, 2023
@Getter
@Table(name = "day_of_week_webtoons")
@NoArgsConstructor(access = AccessLevel.PROTECTED)
public class DayOfWeekWebtoon {
Copy link
Member

Choose a reason for hiding this comment

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

Q: 해당 엔티티의 역할이 궁금합니다!

@Getter
@Table(name = "genre_webtoons")
@NoArgsConstructor(access = AccessLevel.PROTECTED)
public class GenreWebtoon {
Copy link
Member

Choose a reason for hiding this comment

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

Q: 해당 엔티티 역할이 궁금합니다.!


@Builder
public record GenreRes(

Copy link
Member

Choose a reason for hiding this comment

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

의미없는 줄바꿈~

Copy link
Member Author

@kmebin kmebin left a comment

Choose a reason for hiding this comment

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

답변은 여기에 적어놓을게염

저희가 기존에는 웹툰 테이블에서 장르 컬럼, 요일 컬럼에서 String으로 관리를 했었습니다.
(ex. "ROMANCE, FANTASY" / "MON, TUE")

그런데 전체 웹툰 리스트 조회 시에 요일별로 나눠서 리스트를 보내줘야 하는데, "MON, TUE"인 경우에는 문자열을 분리해서, 필터링하는 비용이 발생했습니다.

추가로 장르는 사용자가 직접 입력해서 장르를 추가할 수 있다는 점도 고려했습니다. ><><><

@Getter
@Table(name = "genre_webtoons")
@NoArgsConstructor(access = AccessLevel.PROTECTED)
public class GenreWebtoon {
Copy link
Collaborator

Choose a reason for hiding this comment

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

클래스 나누면서 Converter도 필요 없어진건가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

넵 그렇습니다 큐 ㅋ ㅋ

Copy link
Collaborator

@parksey parksey left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@kmebin kmebin merged commit 81b6fad into develop Sep 7, 2023
1 check passed
@kmebin kmebin deleted the feature/JT-54 branch September 14, 2023 07:53
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.

3 participants