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

Attributes의 Map을 계속 탐색하는 대신에 Key를 타입으로 분리 #80

Open
sanhee opened this issue Dec 20, 2021 · 0 comments

Comments

@sanhee
Copy link
Owner

sanhee commented Dec 20, 2021

추가적으로 Attributes의 Map을 계속 탐색하는 대신에 Key를 타입으로 분리해볼 수 있을 것 같습니다.
Map<Key, String> 과 같이 만들 수 있을 것 같아요. Key 내부에서 올바른지 판단을 하니 책임소재도 명확해지고 불필요한 전체 순회도 없어질거라 생각합니다.
물론 Attributes가 성능에 영향을 줄 정도로 많은 일은 드물겠지만, 개선해볼 수 있을 것 같아요

이런식으로 분리하다보면 나중에 Value 파트도 스프링의 MultiValueMap처럼 리스트 타입으로 해야 하거나 하면 쉽게 분리할 수 있을 것 같아요

Originally posted by @Dae-Hwa in #75 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant