[wook] step 13-1 미션 제출합니다! #74
Merged
Conversation
This file contains 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
constructor 에서 cb를 실행하도록 하였다. cb가 MyPromise.resolve와 MyPromise.reject를 인자로 받도록하였다.
콜백함수의 인자를 외부에서 받아서 처리하는것이 아니므로 내부의 this.value를 인자로 넣어주어 Promise객체가 인자로 들어가도록 처리하였다.
then은 내부에서 cb가 상태가 변했을때 실행되도록한다. 따라서 resolve에서 상태를 바꾸고, then을 한번 더 호출해야한다. 이에따라 then의 콜백함수를 변수로 기억하도록 하였다. (변수가 하나 더 생겨서 살짝 지저분한것같다)
fast-return 을 사용해 2뎁스를 예방
1. this.callback를 this.callbackQueue로 변경 2. executeThensCallback 함수를 생성 이 함수는 등록된 콜백함수를 실행하고 promise 객체의 value를 변경하고 콜백큐를 수정하며 resolve에서 실행된다. 3. 하나의 promise에서 실행되므로 resolve에서 while문을 통해 콜백큐에 등록된 콜백들을 실행한다. 4. 그러나 이런 방식은 Promise생성시 pending이 resolve로 바뀌면 나머지 Promise는 자동으로 resolve상태로 담아있게된다.
step12-1 작업 머지 받아오기
crongro
approved these changes
Jul 4, 2019
|
|
||
| resolve(value) { | ||
| this.value = value; | ||
| this.state = 'fulfilled'; |
There was a problem hiding this comment.
promise가 fullfilled / rejected 이런 상태를 가지고 있죠.
이번기회에 의미를 잘 기억해두세요.
| }).then((data2) => { | ||
| console.log(`data2 인 ${data2}를 잘 받았습니다.`); | ||
| return data2 | ||
| }).then((data3) => { |
There was a problem hiding this comment.
(계속 같은 data네요 ㅎㅎ)
아무튼 잘 했어요.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
요구사항만 동작하는 myPromise 입니다!
resolve에서 while문을 돌면서 myPromise 의 콜백큐에 등록된 콜백들을 실행합니다.
then함수는 새로운 myPromise를 리턴하게 만드려고 했는데.. 그러지 못한게 아쉽네요 ㅠㅠ