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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

推荐关注

› Meteor

› JSLint - a JavaScript code quality tool

› jsFiddle

› D3.js

› WebStorm

推荐书目

› JavaScript 权威指南第 5 版

› Closure: The Definitive Guide

这是一个创建于 1664 天前的主题，其中的信息可能已经有所发展或是发生改变。

  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)
      }
    }
  }

6 条回复 • 2021-06-23 09:48:09 +08:00

breezeFP

2021 年 6 月 22 日

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

2021 年 6 月 22 日

@breezeFP 多谢

powerfj

2021 年 6 月 22 日

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

waiaan

2021 年 6 月 22 日 via Android

@powerfj 是

autoxbc

2021 年 6 月 23 日

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

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

waiaan

2021 年 6 月 23 日

@autoxbc
不是遍历 DOM 的。