Skip to content

[사전과제] 동시성 제어#21

Open
daseulkim128 wants to merge 2 commits intohanghae-skillup:mainfrom
daseulkim128:main
Open

[사전과제] 동시성 제어#21
daseulkim128 wants to merge 2 commits intohanghae-skillup:mainfrom
daseulkim128:main

Conversation

@daseulkim128
Copy link

[사전과제 작성]

작업 내용

  1. synchronized를 활용한 동시성 코드 작성
  2. latestOrderDatabase는 concurrentMap를 이용

발생했던 문제와 해결 과정을 남겨 주세요.

synchronized를 사용하지 않았을때, 내부적으로 동시성이 보장 (?) 되어서 테스트 할때 패스가 된점

이번 주차에서 고민되었던 지점이나, 어려웠던 점을 알려 주세요.

요구사항에 java만을 활용하여 사전과제를 제출해야 했어서, 스프링 전파 옵션이나 db가 제공하는 락 사용이 아닌 synchronized의 단점을 알고 있지만 일단 사용.
한번도 대용량 관련 동시성에 관련한 코드를 작성해본적이 없고, 클래스 기반으로 하는 로직을 생성해보지 않아서 클래스 분리부터 어떻게 할지 고민이 되었음

리뷰 포인트

전반적으로 클래스 부터 잘 짜여졌는지 , 테스트 코드 작성이 잘되었는지에 대한 의견이 궁금합니다.

기타 질문

테스트를 위해 synchronized를 의도적으로 사용하지 않았을 때도 테스트가 통과된 점이 궁금합니다

@daseulkim128 daseulkim128 changed the title [사전과제] [사전과제] 동시성 제어 Jan 8, 2025
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