Question

我做了一个基于promise的函数，该函数爬取层次结构直到到达顶部，然后使用包含该结构的对象进行解析。我对代码的唯一抱怨是，我在函数主体之外修改了变量，这意味着它不是一个纯函数。我研究了JavaScript闭包，并且完全掌握了它们的琐碎用法。但是我正在努力弄清它们如何/是否可以使我的功能纯净。到目前为止，我进行封闭的尝试仅覆盖了变量，而没有修改它们。这是使用全局变量的代码：

/* I want to move these variables inside function body to purify 'getPriorRows'*/
let priorRows = {}, level = 0;

const getPriorRows = id => new Promise(resolve => {
  fetch(`/api/org/${id}`).then(result => {

    /* global varaiables are modified here */
    priorRows[level++] = result;

    if (result.parentID) resolve(getPriorRows(result.parentID));
    else resolve(priorRows);

  });
});

getPriorRows('123432').then(result => console.log(result));

对此事的任何投入，我们将不胜感激。

Answer 1

您应该能够将整个函数及其“外部”变量包含在新函数中：

function getPriorRows(id) {

  let priorRows = {}, level = 0;
  const getNext = id => new Promise(
     ...
  );

  return getNext(id);
}

也就是说，您在每次迭代中创建一个明确的new Promise是Promise反模式：

function getPriorRows(id) {

  let priorRows = {}, level = 0;

  const getNext = id => fetch(`/api/org/${id}`).then(result => {
    priorRows[level++] = result
    if (result.parentID) {
      return getNext(result.parentID));
    } else {
      return priorRows;
    }
  });

  return getNext(id);
}

无论哪种方式，这样包装状态的优点是您现在可以并行进行多个对getPriorRows的调用，而不会互相干扰。

编辑第二个代码已被编辑以修复递归中的复制和粘贴错误-您必须递归调用内部函数，而不是外部函数。

Answer 2

将值作为参数传递：

function getPriorRows(id, priorRows = {}, level = 0) {
  return fetch(`/api/org/${id}`).then(result => {
    /* global varaiables are modified here */
    priorRows[level] = result;

    if (result.parentID) return getPriorRows(result.parentID, priorRows, level+1);
    else return priorRows;
  });
}

getPriorRows('123432').then(result => console.log(result));

您可以使用默认参数或包装函数，甚至不需要关闭：

function getAll(id) { return getPriorRows(id, {}, 0); }

我也删除了Promise constructor antipattern。

使用闭包来净化通过递归构建对象的函数-JavaScript

2 个答案: