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

[베이] Chapter 1. 서버란 무엇인가(소켓&멀티 프로세스) #10

Merged
merged 3 commits into from
Apr 10, 2024

Conversation

h-ye-ryoung
Copy link

@h-ye-ryoung h-ye-ryoung commented Apr 8, 2024

#️⃣ 연관된 이슈

#1

✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?
  • 적절한 라벨을 설정하였나요?
  • 스터디원들에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?

📌 주안점

@h-ye-ryoung h-ye-ryoung added the ✨ Chapter01 챕터 1 label Apr 8, 2024
@h-ye-ryoung h-ye-ryoung self-assigned this Apr 8, 2024
@h-ye-ryoung h-ye-ryoung changed the title Bey/#1 [베이] Chapter 1. 서버란 무엇인가(소켓&멀티 프로세스) Apr 8, 2024
public int number = 1; //초기화

//수행할 메소드
public void clean(String name) {
Copy link

@suuu0719 suuu0719 Apr 9, 2024

Choose a reason for hiding this comment

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

Synchronize 키워드가 필요한 것 같습니다!

@suuu0719 suuu0719 self-requested a review April 9, 2024 09:43
| --- | --- |
| 2번 방식
Shared Memory 모델: | 프로세스 사이에 공유 공간을 만들어 메시지를 저장한다. |
- 독립된 메모리 공간을 가지므로, 동기화 문제가 발생하지 않는다.

Choose a reason for hiding this comment

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

Shared memory를 사용하면 메모리를 공유해서 동기화 문제가 발생할 것 같습니다..! Lock과 같은 기법으로 해결할 수 있을 것 같아요 :)

Copy link

@bflykky bflykky left a comment

Choose a reason for hiding this comment

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

LGTM

@h-ye-ryoung h-ye-ryoung merged commit cc31478 into bey/main Apr 10, 2024
@h-ye-ryoung h-ye-ryoung deleted the bey/#1 branch April 10, 2024 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants