feat(async): エラーファーストコールバック #509
Conversation
|
Deploy preview for js-primer ready! Built with commit 0445cff |
source/basic/async/README.md
Outdated
| } | ||
| }); | ||
| // failtureTaskは失敗するため、`error`にはErrorオブジェクトが入る | ||
| callTaskAsync(successTask, (error, result) => { |
There was a problem hiding this comment.
このサンプルやっぱり長いなー。
2/3ぐらいにしたい…
There was a problem hiding this comment.
s/successTask/failtureTask/では・・・？
| setTimeout(() => { | ||
| try { | ||
| const result = task(); | ||
| callback(null, result); |
There was a problem hiding this comment.
これ一回result変数で受けているのはなんででしょう・・・？
There was a problem hiding this comment.
特別な理由はないですが、resultという変数名で意味を説明したいのとインラインで書くと評価順が異なるため結果は同じですがやりたいことが異なるためですね。
callback(null, task());このケースでは結果が同じになりますが、もし左辺に副作用があるものが存在していた場合に結果が異なる場合がありますね。(awaitを使うと実際そういうミスをしやすい。副作用があるかどうで書き方を変えるよりは、同じ書き方を取れる方を優先したいですね)
やりたいことはtask()の返り値をcallback に渡したいのであって、callbackを呼ぶためにtask() を実行するではないからですね。(いいたいこととしては、「まずtask()を評価する」ことが先にある点ですね)
短くかけますが余計なミスが発生する可能性もあるので、一時変数resultを作ったほうが誰が読んでも同じに読めると思えるからですね。
source/basic/async/README.md
Outdated
| } | ||
| }); | ||
| // failtureTaskは失敗するため、`error`にはErrorオブジェクトが入る | ||
| callTaskAsync(successTask, (error, result) => { |
There was a problem hiding this comment.
s/successTask/failtureTask/では・・・？
#94