代码review

这是公司一位小朋友的代码,他很疑惑为什么info是undefined;

    let list = [
        {
            type: 5,
            name: '创建节点',
            ondid: 'd8b30f2e',
            flowNodeType: 'node',
            propObj: { bgType: 5, topName: '创建节点', contentName: '申请人', flag: -1, list: [] },
        },
        {
            ondid: 'f97f3da8',
            flowNodeType: 'branchbox',
            list: [
                { flowNodeType: 'branch', ondid: '71edab98', propObj: { list: [] } },
                { flowNodeType: 'branch', ondid: '5936d6f4', propObj: { list: [] } },
            ],
        },
    ];
    
    let obj = {
        flowNodeType: 'branch',
        ondid: '5936d6f4',
    };
    
    let obj = {
        flowNodeType: 'branch',
        ondid: 'tmp-5936d6f4-ddfb-4254-8902-69198e10f8db-branch',
    };
    
    async function getIndex(ondid, list) {
        let index = list.findIndex((item) => item.ondid == ondid);
        return index;
    }
    
    /**
     * 分支消息
     * @param {*} obj
     * @param {*} list
     */
    async function getBranchInfo(obj, list) {
        let index = getIndex(obj.ondid, list);
        if (index >= 0) {
            return list[index];
        }
        list.forEach((item) => {
            if (item.flowNodeType == 'branchbox') {
                item.list.forEach((row) => {
                    getBranchInfo(obj, item.list);
                });
            } else if (item.flowNodeType == 'branch') {
                item.propObj.list.forEach((item) => {
                    getBranchInfo(obj, item.propObj.list);
                });
            }
        });
    }
    
    // 递归循环获取对象数据 先获取 { { { a=1} } }
    async function serachInfo(obj, list) {
        if (obj.flowNodeType == 'node') {
            // ...
        } else if (obj.flowNodeType == 'branch') {
            let info = await getBranchInfo(obj, list);
            console.log(info); // ??? 为什么是undefined
        }
    }
    
    serachInfo(obj, list); 

首先这段代码里有几个明显的问题?

1、forEach中使用return是无效的。
2、还有一个问题,async function xxx 的返回值是promise,需要用await来接收,并且都是同步代码,不需要用async/await来包裹。
3、getBranchInfo函数职责不单一

改造意见:

1、getBranchInfo职责单一化,可以将查找list的逻辑抽离到searchInfo中,getBranchInfo只用来判断当前传入的list是否含有符合条件的节点。
2、findIndex获取index,然后又从list[index]获取节点,其实也不必这么麻烦,可以用Array.find实现,不用这么麻烦,第一个就可以直接忽略了
3、searchInfo中,利用方式广度优先的策略,将list的所有子list合并成一个数组,作为参数,递归传入searchInfo中,递归结束条件:调用getBranchInfo获取到符合条件的节点。

改造以后的代码

function searchInfo (obj, list) {
    let info = list.find(item => {
        return item.ondid === obj.ondid
    })
    if (info) {
        return info
    }
    let nextLevel = []
    for (let item of list) {
        if (item.flowNodeType === 'node') {
            nextLevel = nextLevel.concat(item.propObj.list)
        } else {
            nextLevel = nextLevel.concat(item.list)
        }
    }
    return searchInfo(obj, nextLevel)
}

console.log(searchInfo(obj, list))   
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 215,463评论 6 497
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,868评论 3 391
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 161,213评论 0 351
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,666评论 1 290
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,759评论 6 388
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,725评论 1 294
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,716评论 3 415
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,484评论 0 270
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,928评论 1 307
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,233评论 2 331
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,393评论 1 345
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 35,073评论 5 340
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,718评论 3 324
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,308评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,538评论 1 268
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,338评论 2 368
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,260评论 2 352

推荐阅读更多精彩内容