5

我处于以下情况：

我必须检查传递给函数的对象参数是否有效：

示例：

function (opt) {
     if (opt && opt.key && opt.key2) {
         // make something
     }
}

有没有更好的方法来进行这种检查？

4

6 回答 6

1

并不真地。

除非你可以使用opt.pleaseReadMyMind();-)

不过，您可以创建一个方法来检查所有字段的值是否与 不同null。

于 2012-05-23T20:39:56.810 回答
1

这是最紧凑的方式。

最正确的方法是：

if( typeof opt !== "undefined" && typeof opt.key !== "undefined" && typeof opt.key2 !== "undefined") {

但正如你所看到的那样，这是非常拗口的，并不是真正必要的。

于 2012-05-23T20:40:35.897 回答
1

只需编写一个简单的测试例程来验证给定属性名称列表的对象：

// usage: testProps(object to test, [list, of, property, names])
// returns true if object contains all properties
function testProps(obj, props)
{
  if (obj === null)
    return false;

  var i;
  for (i=0; i<props.length; ++i) 
  {
    if (!(props[i] in obj)) 
      return false;
  }
  return true;
}

然后在你的函数中：

if (!testProps(obj, ['key', 'key2'])
   return;
于 2012-05-23T20:49:42.523 回答
0

是的，但如果你有很多钥匙要检查，而不仅仅是三个，那只会“更好”。像这样的东西：

function opt(opt) {
    for(var i = 0; i<3; i++) {
        if(typeof opt["key"+((i > 0) ? "" : i + 1))] === "undefined") {
            return;
        }
    }

    // create object
}

如果opt未定义，它的所有键也将是，所以也有一个隐式检查。

您还可以定义要在数组中检查的变量名称，如下所示：

var propsToCheck = ["key", "key1", "key2"];

function(opt) {
    for(var i = 0, ii = propsToCheck.length; i<ii; i++) {
        if(typeof opt[propsToCheck[i]] === "undefined") {
            return;
        }

        // create object
    }
}

并不是一个更好的解决方案，但如果您计划检查三个或四个以上的属性，它确实可以减少输入。

于 2012-05-23T20:42:16.903 回答
0

你总是可以这样做：

function validate(o, args) {
    if (typeof(o) == 'object' && args instanceof Array) {
        for (var i = args.length - 1; i >= 0; --i) {
            if (typeof(o[args[i]]) === 'undefined') return false;
        }
        return true;
    } else {
        return false;
    }
}

function myFunction(obj) {
    if (validate(obj, ['foo', 'bar'])) {
        // Your code goes here.
    } else {
        // Object passed to the function did not validate.
    }
}

这是给你的小提琴：http: //jsfiddle.net/reL2g/

于 2012-05-23T20:49:27.367 回答
0

你所做的是有效的，但它确实有缺陷。

if (opt && opt.key && opt.key2) {

如果opt.key有错误的值 [0,null,false, 等等]，此检查将失败

在这种情况下，您必须进行 typeof 检查以确保它不是未定义的。

if (opt && typeof opt.key !== "undefined" && opt.key2) {
于 2012-05-23T20:55:16.360 回答