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] 구매자는 매장 리스트를 조회할 수 있다 #81

Conversation

Hyeon-Uk
Copy link
Contributor

@Hyeon-Uk Hyeon-Uk commented Aug 15, 2024

💡 다음 이슈를 해결했어요.

Issue Link - #1

  • (가능한 한 자세히 작성해 주시면 도움이 됩니다.)

💡 이슈를 처리하면서 추가된 코드가 있어요.

  • (없다면 이 문항을 지워주세요.)

💡 이런 고민을 했어요.

  • (없다면 이 문항을 지워주세요.)

💡 다음 자료를 참고하면 좋아요.

  • (없다면 이 문항을 지워주세요.)

추가 고친 이슈

✅ 셀프 체크리스트

  • 내 코드를 스스로 검토했습니다.
  • 필요한 테스트를 추가했습니다.
  • 모든 테스트를 통과합니다.
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있습니다.
  • 커밋 메세지를 컨벤션에 맞추었습니다.
  • wiki를 수정했습니다.

@Hyeon-Uk Hyeon-Uk linked an issue Aug 15, 2024 that may be closed by this pull request
7 tasks
…ntroller에 구현

- 현재 아무런 파라미터가 없으므로 모든 store의 정보를 parsing해서 보여준다.
- 현재 아무런 파라미터가 없으므로 모든 store의 정보를 store_id 기준으로 오름차순해서 parsing하는지 확인
- 기존 단위테스트를 진행한 StoreApiControllerTest.java 에 MockBean으로 StoreDao를 주입
- StoreDao를 이용한 조회 통합테스트인 StoreApiControllerWithDaoTest.java에 @transactional 어노테이션 추가
- 기존 로직은 메서드에서 매번 팩토리를 생성
- 매번 메서드마다 팩토리를 생성하는 코드 중복이 일어남
- 팩토리를 매번 생성하면 성능적인 오버헤드가 생김
- 팩토리는 Thread-Safe한 객체기 때문에 싱글톤으로 관리되어도 아무런 문제가 없다.
@Hyeon-Uk Hyeon-Uk self-assigned this Aug 16, 2024
@Hyeon-Uk
Copy link
Contributor Author

커밋 정리 후 다시 PR올리겠습니다 ^_^

@Hyeon-Uk Hyeon-Uk closed this Aug 17, 2024
@Hyeon-Uk Hyeon-Uk deleted the feature/72_Hyeon-Uk_구매자는_매장_리스트를_조회할_수_있다 branch August 17, 2024 06:26
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.

[기능] 구매자는 매장 리스트를 조회할 수 있다.
1 participant