这段代码有办法更优雅一点吗？

[waiaan]

  function traverseTree (tree, cb, isNodeFirst) {
    if (isNodeFirst) {
      for (let i = 0; i < tree.length; i++) {
        cb(tree[i], i, tree)
        if (tree[i].children && tree[i].children.length > 0) {
          traverseTree(tree[i].children, cb)
        }
      }
    } else {
      for (let i = 0; i < tree.length; i++) {
        if (tree[i].children && tree[i].children.length > 0) {
          traverseTree(tree[i].children, cb)
        }
        cb(tree[i], i, tree)
      }
    }
  }

[breezeFP]

function traverseTree(tree, cb, isNodeFirst) {
for (let i = 0; i < tree.length; i++) {
if(isNodeFirst) {
cb(tree[i], i, tree)
}
if (tree[i].children && tree[i].children.length > 0) {
traverseTree(tree[i].children, cb)
}
if(!isNodeFirst) {
cb(tree[i], i, tree)
}
}
}

[waiaan]

@breezeFP 多谢

[powerfj]

@waiaan traverseTree(tree[i].children, cb) 这个递归调用, 是不是漏了 isNodeFirst 的参数

[waiaan]

@powerfj 是

[autoxbc]

用 xpath 可以一次性获取全部节点，不用递归查找

const arr = [];
for( let result = document.evaluate('//node()', document ) , next ; next = result.iterateNext() ; arr.push(next) );
console.log(arr);

[waiaan]

@autoxbc
不是遍历 DOM 的。