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.

Sign up for GitHub

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

Jump to bottom

[서준환] 5장: 구부러지거나 부러지거나 #48

Merged
merged 1 commit into from
Aug 22, 2024
Merged

[서준환] 5장: 구부러지거나 부러지거나 #48

merged 1 commit into from
Aug 22, 2024

Conversation

Seojunhwan
Copy link
Member

결국 내가 부러져버림

@github-actions github-actions bot added the 5장 label Aug 19, 2024
hyesungoh
hyesungoh approved these changes Aug 20, 2024
View reviewed changes
5장/서준환.md

선언적인 코드를 작성한다고 해도 내부는 명령형 코드인 것을 감안하면 비슷한 맥락이 아닐까 싶습니다. 어렵네요.

> 아! 언어에 포함된 기능들이라면 괜찮다고 하는군요! 라이브러리단의 경우 "." 하나를 작성할 때마다 변경되었을 때 함께 고려할 부분이라는 것을 의미하는 것 같습니다.
Copy link
Member

Choose a reason for hiding this comment

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

👍 👍 👍

metacode22
metacode22 approved these changes Aug 20, 2024
View reviewed changes
Copy link
Collaborator

@metacode22 metacode22 left a comment

Choose a reason for hiding this comment

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

😘

5장/서준환.md

요 상태머신이란 개념을 어디선가 주워듣고 회사에서 사용했는데 흠,, 흠,, 흠..

useReducer를 사용해서 복잡한 상태 변경 로직은 따로 분리하여 처리하는 방법이 더 나았을 것 같다고 승준님께 305번째 말씀 드리는 중,, (뭐가 더 나았을지 아직까지 모르겠네요)
Copy link
Collaborator

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

100Gyeon
100Gyeon approved these changes Aug 20, 2024
View reviewed changes
Copy link
Member

@100Gyeon 100Gyeon left a comment

Choose a reason for hiding this comment

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

대준환

5장/서준환.md

### 유한상태기계

요 상태머신이란 개념을 어디선가 주워듣고 회사에서 사용했는데 흠,, 흠,, 흠..
Copy link
Member

Choose a reason for hiding this comment

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

https://state-machine-viz.vercel.app/dialog

주난님 이거 너무 신기해요 공유 감사합니다👍

Copy link
Member Author

Choose a reason for hiding this comment

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

저는 그 익스텐션이 너무 신기하네요,, 진짜 좋네요 ㄷㄷ

Copy link
Member

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋ 대상조님이 알려주셨습니다 @sjoleee

Copy link
Member Author

Choose a reason for hiding this comment

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

진짜 대상조,, 오지네요

@Seojunhwan Seojunhwan merged commit 77b2918 into main Aug 22, 2024
1 check passed
@Seojunhwan Seojunhwan deleted the 5장/서준환 branch August 22, 2024 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@100Gyeon 100Gyeon 100Gyeon approved these changes

@metacode22 metacode22 metacode22 approved these changes

@hyesungoh hyesungoh hyesungoh approved these changes

@Dongkyuuuu Dongkyuuuu Awaiting requested review from Dongkyuuuu Dongkyuuuu is a code owner

@gutenLEE gutenLEE Awaiting requested review from gutenLEE gutenLEE is a code owner

@Orchemi Orchemi Awaiting requested review from Orchemi Orchemi is a code owner

@sangbooom sangbooom Awaiting requested review from sangbooom sangbooom is a code owner

@sjoleee sjoleee Awaiting requested review from sjoleee sjoleee is a code owner

@sumi-0011 sumi-0011 Awaiting requested review from sumi-0011 sumi-0011 is a code owner

Assignees

@Seojunhwan Seojunhwan

Labels
5장
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Seojunhwan @hyesungoh @100Gyeon @metacode22