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

fix [#76] ArtistResolver의 load 함수에서 target이 리스트 타입인 경우 탐색 가능하도록 수정 #77

Merged
merged 2 commits into from
Jan 17, 2025

Conversation

ch1hyun
Copy link
Collaborator

@ch1hyun ch1hyun commented Jan 17, 2025

✨ Issue Number ✨

closes #76

✨ To-do ✨

  • ArtistResolver에 리스트 타입 타겟도 탐색 가능하도록 수정

✨ Description ✨  

image

load 함수의 target 변수가 리스트 타입인 경우 target.getClass()의 결과가 ArrayList 클래스입니다.
반대로 load 함수에서 리플렉션을 사용해 필드 값을 순회하다 Field 변수가 리스트 타입인 경우 field.getType()의 결과가 List.class 입니다.

부모 인터페이스를 찾아보는 방법으로 해결할 수 있을 것 같지만, 성능 최적화, 구현 난이도가 높다고 생각해 스프린트에 구현하기로 결정했습니다.
현재는 ArrayList.class, List.class 모두 mapper에 등록했습니다.

image

load 함수에 현재 타입이 리스트인 경우 처리하도록 구현했습니다.

@ch1hyun ch1hyun added the 🐞 BugFix 버그 수정 label Jan 17, 2025
@ch1hyun ch1hyun self-assigned this Jan 17, 2025
Copy link
Collaborator

@Ivoryeee Ivoryeee left a comment

Choose a reason for hiding this comment

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

부모 인터페이스 분리는 말씀하신대로 앱잼 이후 스프린트에서 리팩토링하더라도 충분할 것 같네요! 굿굿 수고 많으셨습니다!

@ch1hyun ch1hyun merged commit dc27e72 into develop Jan 17, 2025
1 check passed
@ch1hyun ch1hyun deleted the fix#76 branch January 17, 2025 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BugFix 버그 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] ArtistResolver의 load 함수에 전달되는 오브젝트 타입에 리스트 추가
2 participants