我该如何修复'重构此功能以一致地使用“返回”。”来自声纳js

Question

我将SonarJS分析应用于我的项目。在下面的代码中发现错误“重构此函数以一致地使用“返回””。我该如何解决？

var filter1Depth = function(attr0, attr1, val){
        var id;
        if(val === "vAuto"){
            id = "AirConditioner.Indoor.Ventilator";
            val = "Auto";
        }
        return { // Error is generated from this line.
            field : attr0,
            operator: function(item, value){
                if(value == "") return true;
                if(!item || !item.length) return;

                var i, length = item.length;
                if(value == "undefined"){
                    for( i = 0; i < length; i++ ){
                        if(typeof item[i][attr1] === 'undefined'){
                            return true;
                        }
                    }
                }else{
                    for(i = 0; i < length; i++){
                        if(item[i][attr1] == value){
                            if(id){
                                 if(id === item[i]["id"]) return true;
                                 return;
                            }
                            return true;
                        }
                    }
                }

                return;
            },
            value : val
        };
    };

另一个示例如下。

var filter1DepthNone = function(attr0, attr1, val){
        return { field : attr0, operator: function(item, value){ // error!!
            if(value == "") return true;
            if(!item || !item.length) return;

            var length = item.length;
            for(var i = 0; i < length; i++){
                if(item[i][attr1] != value){
                    return;
                }
            }

            return true;
        }, value : val};
    };

我的产生此错误的代码具有类似的模式。

Answer 1

[在某些块中，您什么都不返回（return;，所以undefined），但是在同一函数的其他块中，您返回布尔值（true / false）。

[您需要确保函数内的返回值是一致的-也就是说，如果在某个地方返回布尔值，则在同一函数内返回值的任何地方，也都返回布尔值。