JS 记一次工作中,由深度优先到广度优先的算法优化
壹 ❀ 引
坦白的说,本人的算法简直一塌糊涂,虽然有刷过一段时间的算法题,但依然只能解决不算复杂的问题,稍微麻烦的问题都只是站在能不能解决问题的角度,至于性能优化,算法方法的选择并没有过于深刻的理解。较巧的是,最近在工作中正好遇到了一个实际场景,整体修改下来也算感受颇深,便记录于此,做个小分享,那么本文开始。
贰 ❀ 乞丐版深度优先
需求其实很简单,比如企业微信中有人员组织架构,类似如下:
简单来理解就是,一个公司的员工会被划分到多个部门,比如一级部门A,B,C...,一级部门下也可能还有二级部门如A-1-1,同理二级部门A-1-1下可能还有三级部门。有些员工可能角色特殊,会同时属于多个不同层级的部门,现在需求就是,后端会返回一个该员工所属部门的ID数组,你需要在上述部门数据中利用ID数组查找出对应的部门名字(name),同时要按一级=>二级=>三级类似的顺序排列并组合成字符串作为展示。
如下提供了模拟数据,因为2对应的是UI,层级比5的前端二组高,所以期望得到的结果为UI;前端二组
,那么怎么做呢?
// 某用户所在的部门id
const departmentsId = [5,2];
// 部门模拟数据
const departmentTree = [
{
name: "技术部",
id: 0,
children: [
{
id: 1,
name: "前端",
children: [
{
id: 4,
name: "前端一组",
children: [],
},
{
id: 5,
name: "前端二组",
children: [
{
id: 9,
name: "前端-移动端一组",
children: [],
},
],
},
],
},
{
id: 2,
name: "UI",
children: [
{
id: 6,
name: "UI一组",
children: [{ id: 10, name: "视觉设计", children: [] }],
},
],
},
{
id: 3,
name: "后端",
children: [
{
id: 7,
name: "后端一组",
children: [],
},
{
id: 8,
name: "后端二组",
children: [],
},
],
},
],
},
];
我首先想到的就是遍历departmentIds
,每次拿一个部门ID到departmentTree
中查找,由于层级高的要排在前面,所以想到深度遍历,每往下一层都会记录当前的depth
属性,如果找到了,最终会到如一个包含部门name
和此部门depth
的对象,因为是深度优先,所以需要结合递归,我的实现是这样:
const getDeptNameFromTreeDepts = (treeDepartments, departmentid) => {
let departmentNameInfo = "";
function departmentTraversal(node, depth) {
// 这里利用find,找到了就没必要继续后续查找了
const targetDepartment = node.find((department) => {
// 判断当前部门id是否和提供的id相同
return department.id === departmentid
? true
: department.children && //如果不相同,判断有没有children,同时让depth加1
department.children.length &&
departmentTraversal(department.children, depth + 1);
});
// 记录当前部门name和depth属性
if (targetDepartment) {
departmentNameInfo = {
name: targetDepartment.name,
depth: depth + 1,
};
}
}
treeDepartments.forEach(({ children }) => {
departmentTraversal(children, 0);
});
return departmentNameInfo;
};
//用于记录每次查找到的结果
const departmentNames = [];
// 遍历人员所属部门id,一次查找一个。
departmentIds.forEach((departmentId) => {
const transformedDepartment = getDeptNameFromTreeDepts(
departmentTree,
departmentId
);
if (transformedDepartment) {
departmentNames.push(transformedDepartment);
}
});
// 根据depth属性来决定部门先后并做name拼接
const name = departmentNames
.sort((a, b) => {
return a.depth - b.depth;
})
.reduce((acc, cur) => {
return `${acc + cur.name}; `;
}, "");
console.log(name);// UI; 前端二组;
代码看着有点多,确实比较复杂,抛开递归不说,最终得到了目标数组,还需要根据depth
属性对部门name
排序,之后再做字符拼接,而且得到的结果是UI; 前端二组;
,我们想要的是UI; 前端二组
,尾部还多了一个;
,理论上来说还要做一次额外处理。在发版前提测测试通过没问题,很遗憾,在code review环节未通过(我算法烂,其实心里也感觉通过不了),得到的反馈如下:
- 效率太低,一次只能查询一个部门id,应该支持批量查询
- 遍历次数过多,最后为什么不用
join
直接拼接name
。 - 不应该是深度优先,改为广度优先
叁 ❀ 广度优先优化
我看到反馈其实心里也有疑虑,就找到了review的前辈,说之所以没用join
是因为最终得到的数组并不是包含纯部门的name
,而是多个对象数组,它们之前并无先后顺序,所以需要根据depth
来做排序最后做拼接。
前辈说不应该啊,你在查找的时候不是已经知道了多个目标部门的先后顺序了吗,为什么还要利用depth
呢?我当时还没反应过来,问他难道在查找的时候顺便做一次插入排序?这样就能保证返回的结果已经带有顺序。他说广度优先不是从上到下一层一层的找吗?你直接把Tree遍历一次,每到了一个节点,看这个节点在不在departmentsId
里面,如果在,那说明是你想要的,由于广度优先是一层层向下,所以你查找出来的结果已经自带层级排序了,写算法之前一定要先设计好自己的算法思路,这样才能少走弯路。我顿时恍然大悟!!!立马回去改了代码!!!
一番修改,于是得到了广度优先的实现代码:
const getDeptNameFromTreeDepts = (treeDepartments, departmentIds) => {
const departmentNames = [];
// 浅拷贝一份,不然在做队列操作时会影响原数据
const queue = [...treeDepartments];
while (queue.length > 0) {
// 每次从对了头部取一个用于做对比
const department = queue.shift();
// 判断当前节点的部门uuid在不在departmentUuid数组中,在的话就是我们想要找的部门,并提取name属性
if (departmentIds.includes(department.id)) {
departmentNames.push(department.name);
}
// 将当前部门的部门的子部门加到队列尾部
department.children && queue.push(...department.children);
}
return departmentNames.length > 0 ? departmentNames.join("; ") : "";
};
const name = getDeptNameFromTreeDepts(departmentTree, departmentIds);
console.log(name);// UI; 前端二组
所以实现到这,我整个人都傻了,代码量直接少了一大半不说,得到的结果也不需要做额外处理。其次,我们在前面的实现中,虽然可以保证不同层级的部门排序,但却无法保证同级部门的先后排序。比如第一层级部门顺序为前端、UI、后端
,在前面的实现中,我们是依次拿departmentsId
中的id去查找,所以同级的先后顺序其实被departmentsId
中id的先后顺序给决定了,在前的会被先找到,例如前面的实现,假设departmentsId
为[3,2]
,查询出来的结果就是后端;UI
,与Tree的顺序并不一致。
而在后续的修改中,同级顺序的问题也不需要考虑了,因为我们一共就遍历了一次Tree,同级节点如果满足,自然会被先找到,所以这段修改不仅解决了不同层级的先后排序,同时也满足了同级部门顺序与Tree一致,大功告成。
肆 ❀ 总
虽然这并不是一次很难的需求,但是在改完代码之后,老实说我心里感触很大。本人算法确实一塌糊涂,很多场景只是能达到实现的角度,缺少了对于算法选择的大局观。正如前辈所说,有时候的你的算法选择应该更去贴合需求,如果你期望结果是ABAB,很明显你应该用深度优先,但如果你要的是AABB,这时候广度优先会更明智,之所以你写了那么多非必要代码,就是因为在广度优先的场景下你用了深度优先。
我虽然之前了解广度深度的概念,但确实缺少实际场景的经验,就像你学了知识,该用到的时候你完全就没有这个概念。所以这一次也算是自己对于广度深度的一次不错的理解了,那么本文结束。