回答編集履歴

最終更新

2018/04/11 08:59

投稿

miyabi-sun

スコア21482

answer CHANGED Viewed

@@ -32,8 +32,12 @@
 やっぱり使い方分かってないじゃないですか。
 まずコールバック引数というのは1つ以上の引数を必ず参照する作りにしてください。
-中身でエラーが出てもtry~catchで取る事が出来ないので、第一引数はかならずerrになるからです。
+中身でエラーが出てもtry~catchで取る事が出来ないので、
+非同期処理を司るライブラリなんかは全て第一引数はかならずerrにするというルールがあるからです。
+もし後続の処理を行われたらまずいという場合は、
+`new Error('エラー理由')`でエラーを作成して第一引数に設定してください。
 そして第二引数以降に、コールバック関数が使いたいと考えている変数を全て引数としてぶち込んで下さい。
 これはnewObjをasyncCallbackで使えるようにした例です。
@@ -47,13 +51,15 @@
     }, 0);
 }
-// コールバック関数は必ず第一引数をerrにしてください。
+// コールバック関数は必ず第一引数をerrにする
-// objとnewObjが使いたいと思ってそうでしたので両方要求する関数にしました。
+// objとnewObjが使いたさそうでしたので両方要求する関数に
 function asyncCallback (err, obj, newObj) {
     if (err) console.error(err);
+    // 本命の処理を行う
+    console.log(obj);
-    console.log(err, newObj);
+    console.log(newObj);
 }
-const obj = {};
+// objもnewObjもここで宣言しない方が良いので削除
-const newObj = makeNewObj(obj, asyncCallback);
+makeNewObj({}, asyncCallback);
 ```

再調整

2018/04/11 08:59

投稿

miyabi-sun

スコア21482

answer CHANGED Viewed

@@ -31,22 +31,27 @@
 ああ、なるほど…
 やっぱり使い方分かってないじゃないですか。
-コールバック引数というのは1つ以上の引数を必ず参照する作りにしてください。
+まずコールバック引数というのは1つ以上の引数を必ず参照する作りにしてください。
+中身でエラーが出てもtry~catchで取る事が出来ないので、第一引数はかならずerrになるからです。
+そして第二引数以降に、コールバック関数が使いたいと考えている変数を全て引数としてぶち込んで下さい。
+これはnewObjをasyncCallbackで使えるようにした例です。
 ```JavaScript
 function makeNewObj(obj, callback) {
     // いろいろな処理がこの辺に
     // 非同期コールバックを表現するためにsetTimeoutを利用
     setTimeout(function() {
         // コールバックに値を引き継ぎたいので関数適用時の引数として渡す
-        callback(null, {new: true});
+        callback(null, obj, {new: true});
     }, 0);
 }
 // コールバック関数は必ず第一引数をerrにしてください。
-// if: もしobjも使いたければ引数を3つ取る関数にする
+// objとnewObjが使いたいと思ってそうでしたので両方要求する関数にしました。
-function asyncCallback (err, newObj) {
+function asyncCallback (err, obj, newObj) {
+    if (err) console.error(err);
-    console.log(err, newObj)
+    console.log(err, newObj);
 }
 const obj = {};

returnを削除出来て居なかったので再修正

2018/04/11 08:51

投稿

miyabi-sun

スコア21482

answer CHANGED Viewed

@@ -41,13 +41,12 @@
         // コールバックに値を引き継ぎたいので関数適用時の引数として渡す
         callback(null, {new: true});
     }, 0);
-    // 新しいオブジェクトを返します。
-    return {new: true};
 }
 // コールバック関数は必ず第一引数をerrにしてください。
+// if: もしobjも使いたければ引数を3つ取る関数にする
-function asyncCallback (err, value) {
+function asyncCallback (err, newObj) {
-    console.log(err, value)
+    console.log(err, newObj)
 }
 const obj = {};

全面的に書き直し

2018/04/11 08:41

投稿

miyabi-sun

スコア21482

answer CHANGED Viewed

@@ -1,83 +1,55 @@
-そもそも、質問文のそれはコールバック関数を適用出来ていません。
+> 以下はそのコードを質問用に簡略化したコードなのですが、関数式の代入先オブジェクトを関数に渡すコールバック関数の中で参照出来るのは、どういう仕組みからなのでしょうか？(質問1)
-```JavaScript
-function otherLibrary (callback) {
-    // 非同期コールバック
+コールバック関数適用と戻り値は完全に異なるものです。
-    // ネットワークやI/O通信が終わり次第、イベントループに設定されて下記のコードが実行される
-    // まぁ今回は横着してコンマ0秒で通信完了したことにしてprocess.nextTickを利用
+非同期処理でコールバック関数を渡したからといって、戻り値を返してはならないというルールは存在しません。
-    process.nextTick(() => callback(err, value));
-}
-function makeNewObj(obj, callback) {
+setTimeoutなんかはこの両者の違いを上手に使っている典型例の一つで、
-    // いろいろな処理がこの辺に
+一度コールバック関数で遅延させると同時に、IDが戻り値として返ってきます。
-    otherLibrary(callback);
+やっぱり実行を辞めたい場合はキャンセル関数に渡して上げる事で実行を取り下げる事が可能です。
+```JavaScript
+var id = setTimeout(() => console.log('test'), 10000);
-    // こいつただのオブジェクト
+// 異常を感じたのでやっぱりキャンセルだ！
-    return {new: true};
+clearTimeout(id);
-}
-function asyncCallback (err, value) {
-    // err, valueはこの関数内でアクセスすることになる
-    console.log(err, value);
-}
-const newObj = makeNewObj(obj, asyncCallback);
-// これはあくまでmakeNewObjectの戻り値
-// errやvalueにはどう足掻いてもたどり着けない。
+// test <- キャンセルされたので文字は表示されない
-// { new: true }
 ```
-こんな関係になっています。
-従って、ライブラリの先では「はいはい実行しておくよ、そのまま次に行ってね」と通信を開始するだけです。
+> とりあえず僕的にはこういうことが可能らしいということはわかったのですが、直感的ではないと感じていて、これは良い作法なのかどうかも気になります。(質問2)
-なので、上記のerrやvalueを同期的なコードで受け取る事は絶対不可能です。
+setTimeoutのように意味があれば良い作法ですし、
-(ES2017のasync / awaitを活用すれば見てくれだけは同期的に出来ますけどね)
+意味が無ければ辞めたほうが良いでしょう。
+[UNIX哲学](https://ja.wikipedia.org/wiki/UNIX%E5%93%B2%E5%AD%A6)に「一つのことを行い、またそれをうまくやるプログラムを書け。」とあります。
----
+あまり色々な機能をゴテゴテと付け足した関数は汎用性が失われ使い勝手が悪くなります。
+これらの事から、setTimeoutのようにキャンセルすることを見越してキー情報を返す。
-コールバック関数というのは、子供のお使いみたいなものです。
+…くらいの使い方以外はすることないんじゃないですかね？
-では、朝子供が学校にいったママの行動を例としながら説明していきましょう。
-まず朝子供に朝食を食べさせ学校に向かわせます。
-ここまで子供は自宅に居るのでコントロール可能です（同期処理）
-ママは子供を学校に送る関数を叩き、子供は自分の手元を離れて学校へ移動しはじめました。
+> もし、これが良い作法ではない場合、このようにコールバックの中で代入先を参照したいような場合に、どのような方法が取れるのかを教えて頂けると嬉しいです。(質問3)
+？？？
+ああ、なるほど…
-子供が学校に行っている間に、ママは書き置きを作ります。
+やっぱり使い方分かってないじゃないですか。
-「学校から帰ってきたら、プリントを机に出す、手を洗う、冷蔵庫のプリンを食べる」
-子供が学校から返ってきたら、その書き置きを読み、
-自発的にプリント机に出し、手を洗って、冷蔵庫のプリンを取り出し始めます。
+コールバック引数というのは1つ以上の引数を必ず参照する作りにしてください。
-この書き置きがJavaScriptにおけるコールバック関数なのです。
-子供が学校に行っている間、ママは学校で何があったかを把握することが出来ません。
-なので、学校へ送り出す関数の戻り値で、先生が渡したプリントを取得することは出来ません。
-このイメージをコードにあらわしてみました。
 ```JavaScript
-function toSchool (child, kakioki) {
+function makeNewObj(obj, callback) {
+    // いろいろな処理がこの辺に
+    // 非同期コールバックを表現するためにsetTimeoutを利用
-  setTimeout(function () {
+    setTimeout(function() {
-    var print = '今日の学級通信';
+        // コールバックに値を引き継ぎたいので関数適用時の引数として渡す
-    kakioki(null, print);
+        callback(null, {new: true});
+    }, 0);
+    // 新しいオブジェクトを返します。
-  }, 8 * 60 * 60 * 1000);
+    return {new: true};
-  return child + 'が登校しました';
 }
-var child = '太郎';
-var result = toSchool(child, function (err, print) {
-  // 慣習としてコールバック関数の第一引数はエラー
+// コールバック関数は必ず第一引数をerrにしてください。
-  if (err) {
-    // 異常あり！子供が返ってこない！？電話する等の対応
+function asyncCallback (err, value) {
-    throw err;
+    console.log(err, value)
-  }
+}
-  // 以降は書き置きの内容
-  console.log('プリントを机の上に出す');
-  console.log(print); // 今日の学級通信
-  console.log('手洗い');
-  console.log('冷蔵庫のプリンを食べる');
-});
-console.log(result);
+const obj = {};
-// 太郎が登校しました
+const newObj = makeNewObj(obj, asyncCallback);
 ```

コードをもうちょっと洗練

2018/04/11 08:38

投稿

miyabi-sun

スコア21482

answer CHANGED Viewed

@@ -63,15 +63,21 @@
   return child + 'が登校しました';
 }
+var child = '太郎';
+var result = toSchool(child, function (err, print) {
-// 約8時間経った後に実行
+  // 慣習としてコールバック関数の第一引数はエラー
-function kakioki (err, print) {
+  if (err) {
-  if (err) throw err; // 異常あり！子供が返ってこない！？電話する等の対応
+    // 異常あり！子供が返ってこない！？電話する等の対応
-  console.log(print);
+    throw err;
-  // 今日の学級通信
-}
+  }
+  // 以降は書き置きの内容
+  console.log('プリントを机の上に出す');
+  console.log(print); // 今日の学級通信
-var child = '太郎';
+  console.log('手洗い');
-var result = toSchool(child, kakioki);
+  console.log('冷蔵庫のプリンを食べる');
+});
 console.log(result);
 // 太郎が登校しました
 ```

関係をもう少し詳しく

2018/04/11 08:24

投稿

miyabi-sun

スコア21482

answer CHANGED Viewed

@@ -50,4 +50,28 @@
 この書き置きがJavaScriptにおけるコールバック関数なのです。
 子供が学校に行っている間、ママは学校で何があったかを把握することが出来ません。
-なので、学校へ送り出す関数の戻り値で、先生が渡したプリントを取得することは出来ません。
+なので、学校へ送り出す関数の戻り値で、先生が渡したプリントを取得することは出来ません。
+このイメージをコードにあらわしてみました。
+```JavaScript
+function toSchool (child, kakioki) {
+  setTimeout(function () {
+    var print = '今日の学級通信';
+    kakioki(null, print);
+  }, 8 * 60 * 60 * 1000);
+  return child + 'が登校しました';
+}
+// 約8時間経った後に実行
+function kakioki (err, print) {
+  if (err) throw err; // 異常あり！子供が返ってこない！？電話する等の対応
+  console.log(print);
+  // 今日の学級通信
+}
+var child = '太郎';
+var result = toSchool(child, kakioki);
+console.log(result);
+// 太郎が登校しました
+```

process.nextTickを追加

2018/04/11 08:20

投稿

miyabi-sun

スコア21482

answer CHANGED Viewed

@@ -4,7 +4,8 @@
 function otherLibrary (callback) {
     // 非同期コールバック
     // ネットワークやI/O通信が終わり次第、イベントループに設定されて下記のコードが実行される
+    // まぁ今回は横着してコンマ0秒で通信完了したことにしてprocess.nextTickを利用
-    callback(err, value);
+    process.nextTick(() => callback(err, value));
 }
 function makeNewObj(obj, callback) {