代碼review

這是公司一位小朋友的代碼,他很疑惑為什么info是undefined;

    let list = [
        {
            type: 5,
            name: '創(chuàng)建節(jié)點',
            ondid: 'd8b30f2e',
            flowNodeType: 'node',
            propObj: { bgType: 5, topName: '創(chuàng)建節(jié)點', 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);
                });
            }
        });
    }
    
    // 遞歸循環(huán)獲取對象數(shù)據(jù) 先獲取 { { { 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函數(shù)職責不單一

改造意見:

1、getBranchInfo職責單一化,可以將查找list的邏輯抽離到searchInfo中,getBranchInfo只用來判斷當前傳入的list是否含有符合條件的節(jié)點。
2、findIndex獲取index,然后又從list[index]獲取節(jié)點,其實也不必這么麻煩,可以用Array.find實現(xiàn),不用這么麻煩,第一個就可以直接忽略了
3、searchInfo中,利用方式廣度優(yōu)先的策略,將list的所有子list合并成一個數(shù)組,作為參數(shù),遞歸傳入searchInfo中,遞歸結(jié)束條件:調(diào)用getBranchInfo獲取到符合條件的節(jié)點。

改造以后的代碼

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))   
最后編輯于
?著作權(quán)歸作者所有,轉(zhuǎn)載或內(nèi)容合作請聯(lián)系作者
【社區(qū)內(nèi)容提示】社區(qū)部分內(nèi)容疑似由AI輔助生成,瀏覽時請結(jié)合常識與多方信息審慎甄別。
平臺聲明:文章內(nèi)容(如有圖片或視頻亦包括在內(nèi))由作者上傳并發(fā)布,文章內(nèi)容僅代表作者本人觀點,簡書系信息發(fā)布平臺,僅提供信息存儲服務。

相關(guān)閱讀更多精彩內(nèi)容

  • 原文地址:http://www.itdecent.cn/p/fd742b49a25c 1.1、什么是HTML語義...
    Daydream_許多閱讀 736評論 0 0
  • 1.1、什么是HTML語義化?有什么好處? 根據(jù)內(nèi)容的結(jié)構(gòu)化(內(nèi)容語義化),選擇合適的標簽(代碼語義化)便于開發(fā)者...
    傾城一夢1123閱讀 2,580評論 1 11
  • 一、CSS問題 1.flex布局 display:flex; 在父元素設置,子元素受彈性盒影響,默認排成一行,如果...
    陳二狗想吃肉閱讀 793評論 0 9
  • jquery介紹 jQuery是目前使用最廣泛的javascript函數(shù)庫 據(jù)統(tǒng)計,全世界排名前100萬的網(wǎng)站,有...
    就是這么帥_567e閱讀 1,340評論 0 0
  • 2019.3.38 比較兩個文件 英文 detect: 檢測2019.3.27 如何找到第一個bug出現(xiàn)的comm...
    饒家俊閱讀 2,682評論 0 1

友情鏈接更多精彩內(nèi)容