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

[BE] feat: 소셜미디어 CRUD 기능 추가 (#802) #848

Merged
merged 11 commits into from
Apr 11, 2024
Merged

Conversation

seokjin8678
Copy link
Collaborator

@seokjin8678 seokjin8678 commented Apr 8, 2024

📌 관련 이슈

✨ PR 세부 내용

관련 이슈처럼 소셜미디어 CRUD 기능을 추가했습니다.

이슈에서는 CUD 기능만 추가하기로 되어 있지만, 관리자 페이지에 빠르게 기능 구현이 필요하고, 구현의 어려움이 없으므로 해당 PR에서 작업하였습니다.

소셜미디어의 추가와 수정에 문자열의 길이에 대한 검증 로직이 없는데, 관리자 기능이라 추가하지는 않았습니다.
(추후 500 에러가 발생하는 것을 보고 추가하면 될 것 같네요)

소셜미디어 조회 API는 2개인데, 그중 식별자를 통한 단건 조회 기능은 수정 기능에서 사용할 목적으로 만들었습니다. (상세 조회)

목록 조회는 ownerId, ownerType에 해당하는 소셜미디어를 모두 조회해 오는데, 따로 limit을 두지는 않았습니다.

이유는 ownerIdownerType의 조합에서 생성할 수 있는 소셜미디어의 최대 개수는 socialMediaType 개수와 동일하기에, 너무 많은 row로 인해 발생하는 성능 이슈가 없습니다.

SocialMediaType.values().length를 limit 절에 사용한다면 나름 성능 최적화가 가능할 것 같네요. 다만 모든 socialMediaType을 만드는 경우가 없을 것 같아 현실성은 떨어져 보입니다. 😂

또한 SocialMediaCommandService의 TODO 주석에서 남겨두었지만, 추상화된 예외가 필요할지 얘기하고 싶습니다.

더 객체 지향적인 방법으로 풀어나간다면 SocialMediaCreateValidator 인터페이스를 만들어서 어떻게 할 것 같은데...

현재 소셜미디어의 owner는 School과 Artist 2개뿐이고, 여기서 더 추가될 것 같지는 않아서 그저 오버 엔지니어링이 될 것 같네요. 😂

그 외, 메서드 이름이 적절한지 봐주시면 감사하겠습니다!

@seokjin8678 seokjin8678 added BE 백엔드에 관련된 작업 ADMIN 📬 API API가 변경되거나 추가되는 작업 🏗️ 기능 기능 추가에 관한 작업 labels Apr 8, 2024
@seokjin8678 seokjin8678 self-assigned this Apr 8, 2024
@seokjin8678 seokjin8678 linked an issue Apr 8, 2024 that may be closed by this pull request
@github-actions github-actions bot requested review from BGuga, carsago and xxeol2 April 8, 2024 14:16
Copy link

github-actions bot commented Apr 8, 2024

Test Results

197 files  197 suites   32s ⏱️
669 tests 669 ✅ 0 💤 0 ❌
682 runs  682 ✅ 0 💤 0 ❌

Results for commit 8064e16.

♻️ This comment has been updated with latest results.

@seokjin8678
Copy link
Collaborator Author

관리자 프론트 페이지에 기능 완성 되었고, 추가 리뷰 없다고 판단하여 머지하도록 하겠습니다!
의견 있으시면 이슈로 남겨주세요!!

@seokjin8678 seokjin8678 merged commit e4b753a into dev Apr 11, 2024
3 checks passed
@seokjin8678 seokjin8678 deleted the feat/#802 branch April 11, 2024 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ADMIN BE 백엔드에 관련된 작업 🏗️ 기능 기능 추가에 관한 작업 📬 API API가 변경되거나 추가되는 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] 소셜 미디어 CUD 기능을 추가한다.
1 participant