Question

我从ajax-request获得以下JSON响应。

var json = {
    "response": {
        "freeOfChargeProduct": {  
        "description": "Product",  
        "orderQty": 5,
        "productName": "XYZ",
        "qty": 6,
        "details": {
            "price": 55.5, 
            "instock": "true",
            "focQuantity": 1
        }
    }, 
    "orderLineId": 4788,
    "totalOrderLinePrice": "741.36",
    "totalOrderPrice": "1,314.92",
    "totalQty": 17
};

JSON并不总是返回“freeOfChargeProduct”属性。因此，如果我想获得“freeOfChargeProduct”价格，那么我必须执行以下操作：

var getFreeOfChargeProductPrice = function() { 
   var r = json.response;
   if (r && r.freeOfChargeProduct && r.freeOfChargeProduct.details) {
      return r.freeOfChargeProduct.details.price;         
   }
   return null;
};

没问题。但是检查对象中的每个属性都非常烦人，因此我创建了一个函数来检查对象中的属性是否已定义。

var getValue = function (str, context) {
    var scope = context || window,
        properties = str.split('.'), i;
    for(i = 0; i < properties.length; i++) {
      if (!scope[properties[i]]) {                       
         return null;
      } 
      scope = scope[properties[i]];        
    }
    return scope;
};

var price = getValue('json.response.freeOfChargeProduct.details.price');
// Price is null if no such object exists.

现在我的问题：这是检查对象中是否存在属性的好方法还是坏方法？有更好的建议/方法吗？

修改

我不想使用＆amp;＆amp; -operator。我很懒，而且我正在寻找一种可重用的方法来检查是否定义了一个对象（或对象的属性）。

:)谢谢！

Answer 1

使用保护模式：

if (json.response && json.response.freeOfChargeProduct && json.response.freeOfChargeProduct.details) {
    // you can safely access the price
}

这就是防护模式的工作原理。

if (a && a.b && a.b.c) { ... } else { ... }

第一项检查是“属性a是否存在？”。如果没有，则执行else分支。如果是，则进行下一次检查，即“对象a是否包含属性b？”。如果不是，则执行else分支。如果是，则进行最终检查：“对象a.b是否包含属性c？”。如果不是，则执行else分支。如果是（并且只有那时），if分支执行。

更新：为什么称为“警卫模式”？

var value = a && b;

在此示例中，成员b（右操作数）由&&运算符保护。仅当成员a（左操作数）是真实的（“值得”）时，才会返回成员b。但是，如果成员a是假的（“不值得”），那么它本身就会被返回。

顺便说一句，如果成员返回以下值，则会有误：null，undefined，0，""，false，NaN。在所有其他情况下，成员都是真实的。

Answer 2

if(x && typeof x.y != 'undefined') {
    ...
}

// or better
function isDefined(x) {
    var undefined;
    return x !== undefined;
}

if(x && isDefined(x.y)) {
    ...
}

这适用于JavaScript中的任何数据类型，甚至是零的数字。如果要检查对象或字符串，只需在if语句中使用x && x.y，或者如果您已经知道x是对象，if(x.y) ...

Answer 3

你可以这样做：

try{
    var focp = json.response.freeOfChargeProduct
    var text = "You get " + focp.qty + " of " +
        focp.productName +
        " for only $" + (focp.qty-focp.details.focQuantity)*focp.details.price +
        ", You save $" + focp.details.focQuantity*focp.details.price;
    $("order_info").innerText = text;
} catch(e) {
    // woops, handle error...
}

如果字段存在，它会从您提问的数据中生成这样的消息：

只需277,5美元即可获得6个XYZ，节省55.5美元

如果数据不存在，您将最终进入catch块。如果你无法想出一种处理错误的方法（也许可以对数据做一个新的AJAX请求？），你可以随时尝试Try，Catch，Forget here。

Answer 4

这不是语法问题，因为它是设计模式问题。

问题A. *你有控制json服务器吗？

如果答案是否定的，我认为，情况将全部在客户端。

请阅读： http://martinfowler.com/eaaDev/PresentationModel.html

由于服务器是源，在这种情况下，它将提供模型。此模式指定了另一个工件：表示模型（PM）。在javascript中，我会建议两个工件，转换器代码的附加工具。

根据这种设计模式，PM负责将模型转换为PM，并在必要时再次返回。在您的情况下，不会发生从PM到M的转换。

这意味着js对象有一个方法或构造函数，在转换器的帮助下（下面）消化模型并自行翻译。

这样做会导致PM看起来像这样：

var OrderlinePM = {
  "hasFreeOfCharge": false | true,
  "freeOfCharge" : {...}

enter code here this.getFreeOfCharge = function（）{ ... } this.fromModel = function（jsonEntry，convertor）{ //将此转换为转换器;）转换为此特定视图可用的OrderlinePM //也是inwith ... } “orderLineId”：0， “totalOrderLinePrice”： “741.36”， “totalOrderPrice”： “1,314.92” “totalQty”：17 };

function mySpecialFunctionPMConvertor { this.fromModel = functiono（）{ ... //用模型做一些奇怪的东西，然后用它把PM弄掉。 } }

好的，我放弃尝试在这个富文本编辑器中格式化代码：（

根据相同的模型对象，您可以使用多个PM：s来完成不同的任务。
此外，这将使转换器对象可以在可以自动执行的内容中进行测试....无论如何都可以手动执行，但无论如何。

因此，繁琐的反射代码问题确实不是问题。但凝聚力是一个问题，在JavaScript中是一个问题。

检查对象是否已定义，最佳做法。

4 个答案: