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

Feat(#40): 필터 소개 API 추가 #42

Merged
merged 2 commits into from
Aug 11, 2024
Merged

Conversation

tmddus2
Copy link
Collaborator

@tmddus2 tmddus2 commented Aug 11, 2024

작업 사항

  • 필터 소개 API

고민한 점들

  • DB에 긴 문자열을 저장할 때 varchar로 길이를 조절하는 게 좋을까, text 라는 자료형을 쓰는 게 좋을까
    VARCHAR vs TEXT

  • 한 테이블에 너무 많은 정보를 우겨넣는 게 아닌가?

    • 필터 소개 관련 상세 정보(필터 소개 제목, 필터 소개 설명, 태그)들을 따로 테이블을 추가해서 관리할까 했는데, 필터 사진이 필터 상세에도 쓰이고 필터 소개에도 쓰여서 애매하다고 생각했다.
      • 필터 테이블과 필터 소개 테이블에 두 번 사진 저장 ➡️ 중복 저장으로 저장공간 낭비
      • 외래키 연결 ➡️ 필터 테이블에 저장된 사진 list와 필터 소개 테이블에 저장된 소개 정보 llist(제목, 설명) 을 [사진, 제목, 설명] 식으로 합쳐줘야 하는데 어떻게 합쳐줄지 애매. index 기준으로 하기엔 데이터 관리가 너무 힘들겠다고 생각
    • 그래서 한 테이블에 객체 list 형태로 column을 관리하기로 했다

@tmddus2 tmddus2 linked an issue Aug 11, 2024 that may be closed by this pull request
2 tasks
@tmddus2 tmddus2 self-assigned this Aug 11, 2024
@tmddus2 tmddus2 merged commit 0a23242 into main Aug 11, 2024
1 check passed
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.

필터를 소개한다
1 participant