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

[3w/유용상] 비동기 면접 질문 추가 #13

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Miral3
Copy link
Member

@Miral3 Miral3 commented Apr 28, 2022

수정 사항

  • 참고 링크 list로 구분
  • 콜백헬 발생 원인 추가
  • 프로미스 메소드 추가
  • 콜백 헬 해결 방법 수정
  • async/await promise 차이 삭제
  • async/await 추가
  • async/await 꼬리질문 추가

@Miral3 Miral3 self-assigned this Apr 28, 2022
@Miral3 Miral3 changed the title [3w/유용용상] 비동기 면접 질문 추가 [3w/유용상] 비동기 면접 질문 추가 Apr 28, 2022
Comment on lines 36 to +39
### 4. 콜백 헬을 해결할 수 있는 방법은 무엇이 있을까요?

프로미스를 사용하여 해결할 수 있습니다. 프로미스는 후속 처리 메소드(then이나 catch 메소드)를 체이닝하여 여러 개의 프로미스를 연결해서 사용할 수 있습니다.
프로미스를 사용하여 어느정도 해결할 수 있습니다. 프로미스는 후속 처리 메소드(then이나 catch 메소드)를 체이닝하여 여러 개의 프로미스를 연결해서 사용할 수 있습니다.
하지만 이는 중첩된 Promise, then메소드로 인해 또다른 지옥을 발생시킬 수 있습니다. 이를 위해 async/await를 사용할 수 있습니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

2-1 콜백헬은 왜 발생할까요? 에 대한 해결 방안이 될 수 있으므로 같이 묶으면 가독성이 더 좋을 것 같아요. 그 뒤에 프로미스 체인 관련하여 내용이 나오니까 더 자연스러울 것 같습니다. ㅎㅎ

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.

과감한 수정 보여주세요 ㅎㅎ 과감한 수정 후 -> pr에서 의견을 나눌 수 있는 장치가 있으니, 이전 작성 내용도 대폭 수정할 수 있다고 생각합니다 ㅎ.ㅎ

Copy link
Member Author

Choose a reason for hiding this comment

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

넵! 그러는 편이 좀더 효과적인 방법일것 같네요

@Miral3 Miral3 requested review from loopy-dev and aurora32s April 28, 2022 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants