This repository was archived by the owner on Dec 26, 2024. It is now read-only.
Conversation
minsu11
reviewed
Nov 6, 2023
minsu11
reviewed
Nov 6, 2023
Contributor
There was a problem hiding this comment.
Main에서 Consumer를 Interrupt를 해줄 때 결과 값에 중지합니다가 뜬 후에 물건 판매완료와 퇴장하는 결과값이 나오는 의도로 하신 이유가 있을까여?? deadlock이 걸릴 가능성을 생각하셔서 하신건가용?
minsu11
reviewed
Nov 6, 2023
NANNDA3463
reviewed
Nov 6, 2023
hyeonjaez
reviewed
Nov 7, 2023
NANNDA3463
reviewed
Nov 7, 2023
Contributor
There was a problem hiding this comment.
이 자료형의 필요성에 대해 생각하게 됩니다
add를 하지만 pop을 하지 않으니 queue여야 하는 이유가 있을까요
Thread를 순차적으로 깨워주기 위해서라기엔 이 자료형이 비어있을 때 모든 쓰레드를 notifyAll하는 구조이고... 만약 이 의도대로라면 pop을 하고, 그 자료형을 깨워주는 식으로 하면 어떨까싶습니다
실제로 구현될 지는 미지수지만요
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
과제 - 문제 2: semaphore 은 아직 풀지 못한 관계로 제외하고 코드리뷰 부탁드립니다!