Skip to content

Refactor/music#32

Merged
mjkhub merged 8 commits intomainfrom
refactor/music
Jun 30, 2025
Merged

Refactor/music#32
mjkhub merged 8 commits intomainfrom
refactor/music

Conversation

@mjkhub
Copy link
Copy Markdown
Contributor

@mjkhub mjkhub commented Jun 30, 2025

결과물 발표때 RestTemplate을 왜 사용하냐는 질문을 받았었고," HTTP 커넥션 풀을 사용할 수 있어서 아직은 사용해도 될 것 같다" 라는 답변을 드렸었습니다.

HTTP/1.1은 Keep Alive라는 필드가 있어서 커넥션 재사용이 가능한데, 이를 적용하는 구체적인 코드를 공유드리고 싶어서 추가해보았습니다 :)

추가적으로 API 응답에 사용되는 클래스를 엔티티 이름으로 폴더로 구분하여 관리하는 방식을 제안드리고 싶습니다.

api 라는 하나의 폴더에 모든 클래스를 두면 시간이 지났을 때, 구분하기 어려울 것 같더라구요.

@sudo-Terry
Copy link
Copy Markdown
Member

추가적으로 API 응답에 사용되는 클래스를 엔티티 이름으로 폴더로 구분하여 관리하는 방식을 제안드리고 싶습니다.
api 라는 하나의 폴더에 모든 클래스를 두면 시간이 지났을 때, 구분하기 어려울 것 같더라구요.

단순한 하나의 자원에 대한 DTO들은 말씀해주신 대로 묶어도 괜찮을 것 같습니다. 근데 서비스가 조금더 복잡해지거나, 당장에도 화면 렌더링에 필요한 엔드포인트와 같이 도메인 단위로 나누기에 애매한 경우도 있어서 화면 단위로 나누는건 어떨까 싶네요. 여러 엔드포인트에서 활용되는 것들은 common으로 빼도 되고 말씀하신 대로 그 안에서 도메인별로 나눠도 될 것 같구요

@mjkhub
Copy link
Copy Markdown
Contributor Author

mjkhub commented Jun 30, 2025

좋습니다~ 그럼 우선 화면 단위로 API를 관리하는 것으로 하시죠!

@mjkhub mjkhub merged commit 2420c90 into main Jun 30, 2025
@mjkhub mjkhub deleted the refactor/music branch June 30, 2025 12:55
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