Async: 콜백이 없는 함수를 약속으로 처리

함수가 약속을 반환하는지 여부와 콜백을 미리 전달해야 하는지 여부를 안정적으로 감지할 수 있는 방법이 없습니다. ( Function.length 또는 Function.toString() 구문 분석으로는 충분하지 않습니다.) 이것이 asyncify 에 관한 것입니다. 약속을 반환하는 함수를 콜백을 받는 함수로 변환하는 것입니다. async 함수는 유형이 다르기 때문에 감지할 수 있으므로 자동으로 asyncify 할 수 있습니다(이것이 내부적으로 작동하는 방식입니다). 그러나 약속을 반환하는 일반 함수의 경우 Async가 호출하기 전에 알 수 있는 방법이 없습니다.

@aearly 흠, 그래서 아마도 최상위 수준에서 콜백을 감지하는 것과 같은 더 간단한 것이 있을까요?

async function parallel(o, cb) {
  if (!cb) { // promise behavior
    return Object.fromEntries(
      await Promise.all(Object.entries(o).map(async ([k, v]) => [k, await v()]))
    )
  }
  // else do the callback behavior
}

우리는 이미 그런 식입니다. async.parallel 대한 콜백을 생략하면 약속을 반환합니다. 사용자 의도를 가정할 방법이 없기 때문에 콜백을 생략하면 전달한 함수가 약속을 반환한다고 가정하지 않습니다. 내가 무슨 말인지 오해하고 있는 건 아닐까?

좋아, 구체적으로 말해 보자: https://repl.it/@caub/async -async

// npm.im/async expects all inner functions to be async,
// even when using no outer callback (as 2nd arg of async.parallel)
console.log(
  await async.parallel({
    foo: async () => delay(1000).then(() => 1),
    bar: async (rs) => {
      return 2
    },
  })
);

// Doesn't work without those async,
// but I'd like this to resolve just like above instead of freezing
console.log(
  await async.parallel({
    foo: () => delay(1000).then(() => 1),
    bar: () => {
      return 2
    },
  })
);

다음 주요 버전에서는 다음과 유사한 방식으로만 콜백을 지원하는 것이 흥미로울 수 있다고 생각합니다.

await Promise.all([
  { then(resolve){ setTimeout(resolve, 50, 1) } }, // callback way
  2,
  delay(45).then(())=>3)
])
// [1, 2, 3]

내부 함수에서 불필요한 async 키워드의 필요성 제거

장점은 더 명확합니다. 사용자 의도를 가정하지 않는 것에 대한 귀하의 요점을 이해하지만 여전히 사람들이 Promise 해결을 사용하려고 할 때 async 키워드를 잊지 않을 것이라고 가정하고 있습니다.

그것은 꽤 큰 방식으로 이전 버전과의 호환성을 깨뜨릴 것입니다. 예를 들어 Async.map(fileNames, fs.readFile) 와 같은 일을 더 이상 할 수 없습니다.

어디서 부러지는지 설명해주실 수 있나요?

현재 중단된 내용은 다음과 같습니다.

await async.map(['server.js', 'package.json'], fs.readFile) // works
await async.map(['server.js', 'package.json'], fs.promises.readFile) // works
await async.map(['server.js', 'package.json'], s => fs.promises.readFile(s)) // doesn't work
await async.map(['server.js', 'package.json'], async s => fs.promises.readFile(s)) // works

아이디어는 세 번째 문제를 해결하는 것입니다.

예를 들어 다음과 같이 썼습니다.

  // ...
  doc = await async.map(items, item => item.toDoc(currentUser, options));

어딘가에, 그리고 async item => item.toDoc.. 잃어버렸기 때문에 시간이 초과되었습니다.

그 경우에 나 혼자라면 가치가 없지만 더 많은 사람들이이 상황에 있다면 제안 된 변경 사항을 고려할 가치가 있다고 생각합니다

async.map 가 있는 이 예도 매우 대표적인 예입니다.

  doc = await Promise.all(items.map(item => item.toDoc(currentUser, options)));

async item => item.toDoc.. 필요 없이 작동

세 번째 문제를 안정적으로 고칠 수 있는 방법은 없습니다. 이것은 과거에 많이 논의되었습니다. 함수의 반환 값을 확인한 다음 소급하여 함수에 콜백을 전달하지 않을 수 없습니다. async 키워드는 그만큼 좋습니다.

@caub 뻔뻔한 플러그가 여기에 있지만 asyncp 에는 여기에서 찾고 있는 것을 처리할 수 있는 능력이 있다고 생각합니다.

이 문제가 종료된 상태에서도 위의 링크된 문제와 같이 여기서 @caub 문을 다시 강조하고 싶습니다.
현재의 행동은 나를 많이 혼란스럽게 했고 여기서 논의된 기술적 한계를 (일종의) 이해하지만 문서가 단순히 현재의 행동과 일치하지 않는다고 생각합니다.
예를 들어 mapLimit의 경우 "반환: 콜백이 전달되지 않은 경우 약속"이라고 표시됩니다. - 저는 콜백 버전으로 시작한 다음 기본적으로 콜백을 생략했고 약속을 얻지 못한 것에 놀랐습니다. '비동기' 키워드).
또한 iteratee 함수의 'callback' 문은 필요하지 않으며 대신 값을 '반환'해야 합니다.
따라서 혼동을 피하기 위해 동작을 변경(선호)하거나 최소한 문서와 예제를 내 관점에서 정렬해야 합니다.

@caub 이것이 이유입니다
https://github.com/caolan/async/blob/8aecf108b3922bc5211036706a0f6f75e02bd42b/lib/internal/wrapAsync.js#L3 :L5

function isAsync(fn) {
    return fn[Symbol.toStringTag] === 'AsyncFunction';
}

이렇게 하면 실제로 호출하지 않고 함수가 약속을 반환하는지 확인하는 것이 더 쉽습니다. 그러나 이것이 async/await 규칙에 위배된다는 점에는 여전히 동의합니다. 내부 await 없이 async 키워드를 추가하는 것은 실제로 대부분의 경우 나쁜 습관으로 간주됩니다.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function에서

async 함수의 본문은 0개 이상의 await 표현식으로 분할되는 것으로 생각할 수 있습니다. 첫 번째 await 표현식(있는 경우)까지 포함하는 최상위 코드는 동기식으로 실행됩니다.

... 그래서 쓸모없고 최악의 경우 코드가 babel에 의해 트랜스파일되는 경우 약간의 성능 오버헤드가 추가됩니다. 예를 들어 async 키워드가 있기 때문에 더 많은 코드와 단계가 추가되기 때문입니다.

예를 들면 다음과 같습니다.

async function foo() {
   return 1
}

... 다음과 동일합니다.

function foo() {
   return Promise.resolve(1)
}

하지만 이 라이브러리를 사용하면

async.mapLimit([1], 1, async v => v)

그리고

async.mapLimit([1], 1, v => Promise.resolve(v)))

동등하지 않습니다!