回答編集履歴

誤記

2019/03/06 12:00

投稿

KSwordOfHaste

スコア18404

answer CHANGED Viewed

@@ -164,22 +164,25 @@
 run(array)
 ```
-ここでArray.prototype.mapではなくfor文を使っていることに注意してください。先に述べたようにmapは「要素の実行が非同期関数の呼び出しかどうか」「非同期関数の呼び出しなら完了までまつ」なんて配慮を一切しない関数ですのでfor文を使ってかかねばならないのです。「for文書かないとだめなのかぁ」とげっそりするかも知れませんが、それをするためには特別な非同期関数の同期をしてくれるようなmapがなければなりません。そういうmap関数は今のところ標準のArrayにはないと思います。
+ここでArray.prototype.mapではなくfor文を使っていることに注意してください。先に述べたようにmapは「要素の実行が非同期関数の呼び出しかどうか」「非同期関数の呼び出しなら完了までまつ」なんて配慮を一切しない関数ですのでfor文を使ってかかねばならないのです。「for文書かないとだめなのかぁ」とげっそりするかも知れませんが、map的な高階関数を使いたい場合、非同期関数を同期的に適用してくれるような特別な関数でなければなりません。そういうmap/forEach関数は今のところ標準のArrayにはないと思います。
 ```js
 ...
-Array.prototype.asyncMap = async function (op) {
+Array.prototype.asyncForEach = async function (op) {
+  for (let i = 0; i < this.length; i++) {
-  for (let i in this) {
+    if (i in this) {
-    await op(this[i], i)
+      await op(this[i], i)
+    }
   }
 }
 async function run(array) {
-  array.asyncMap(async funcion(ns) { await concurrentMain(ns) })
+  array.asyncForEach(async funcion(ns) { await concurrentMain(ns) })
   log('completed')
 }
 run(array)
 ```
-無理やり書くならこんな感じになってしまうでしょう。
+無理やり書くならこんな感じになってしまうでしょう。
+追記: **スミマセン。最初の回答に書いたasyncMapはmapとは言えませんし定義も不適切でした**。単に繰り返しをするだけのものなので関数名をasyncForEachに変更し実装もそれなりにしました。なおオリジナルのforEachに比べ仕様を簡略化しています。)

誤記、コードの間違い

2019/03/06 12:00

投稿

KSwordOfHaste

スコア18404

answer CHANGED Viewed

@@ -95,7 +95,7 @@
 `.then(() => concurrentMain([...]))`
 としなければならない点に注意してください。thenの引数は前段の非同期処理が完了する以前に評価されますので直接concurrentMainを呼び出してしまうと意味がなくなります。thenには「非同期処理が完了したときに実行したい関数」を指定するようにしてください。
-ご質問の最初の段階のthenチェーンの例はPromise.allの引数の実際の書き方を省略しているおつもりなのでしょうが、その書き方にこそ重要な前提知識があるわけで、そこは省略せずに書いて実際に動かしてから質問を発した方がよいかったと思います。
+ご質問の最初の段階のthenチェーンの例はPromise.allの引数の実際の書き方を省略しているおつもりなのでしょうが、その書き方にこそ重要な前提知識があるわけで、そこは省略せずに書いて実際に動かしてから質問を発した方がよかったと思います。
 ```js
 Promise.all([1,2,3])
@@ -120,7 +120,7 @@
 array = divide(array, 3)
 cf = array.reduce(
-  (pre, cur) => () => pre().then(() => cmain(cur)),
+  (pre, cur) => () => pre().then(() => concurrentMain(cur)),
   () => Promise.resolve())
 cf()

日本語訂正

2019/03/06 11:09

投稿

KSwordOfHaste

スコア18404

answer CHANGED Viewed

@@ -1,10 +1,10 @@
 配列のmapなどで処理を書く前にもっと注意深くご自分のコードを検証(例えば実際に動かして)してみるべきだと思います。
-逆に「メッセージがいつ表示されたか」は重要な情報です。以下の例をまず動かしてみてください。
+なおここでは「メッセージがいつ表示されたか」は重要な情報です。以下の例をまず動かしてみてください。
 ```js
 const start = +new Date()
-function log(...m) {
+function log(...m) { // メッセージを時刻とともに表示するデバッグ用関数
   let t = '' + (+new Date() - start)
   t = '0'.repeat(Math.max(0, 6 - t.length)) + t
   console.log(`${t}:`, ...m)