Question

避免无法读取...undefined的属性

2016-12-10

943

javascript jquery

我使用此函数将标题属性附加到表的每个 tr 。表的内容来自数组。这很好用。但我还有一个函数可以手动将行添加到表中。对于这些行，我得到了一个异常。很明显，因为它们不在数组中。但我该如何避免这些异常呢？我不需要为这些行添加标题。

它说：

Cannot read property 'dictCanon' of undefined(…)

function postBody() {
    // add title to tr
    var trs = $table.find('tbody').children();
    for (var i = 0; i < trs.length; i++) {
        $(trs[i]).mouseover(function(e) {
            index = $(e.currentTarget).data('index');
            var d = (diagnosis[index].additionalParameters);
            console.log('d', d);
            dt = $(e.currentTarget).parent().parent().find('thead').find('th')
                                   .eq($(e.currentTarget).data('index')).data();
            //console.log(dictCanon);
            if (d != undefined || d !== 'null') {
                 var dictCanon = diagnosis[index].additionalParameters.dictCanon;
                 var icd = diagnosis[index].additionalParameters.icd;
                 $(this).attr('title',icd + ' ' + dictCanon);
             }
        });                
    };
};

Answer 1

此表达式触发了错误 “无法读取未定义的属性‘dictCanon’” ：

diagnosis[index].additionalParameters.dictCanon

...这意味着您在 diagnosis 中有一些条目没有 additionalParameters 属性。您尝试保护代码免受该错误的影响，但使用了错误的布尔运算符。使用 && 而不是 || ，并且不要将 null 放在引号中。我还建议调整 for 循环中的条件，以确保您在 diagnosis 中拥有必要的条目：

function postBody() {
    // add title to tr
    var trs = $table.find('tbody').children();
    for (var i = 0; i < trs.length && i < diagnosis.length; i++) {
        $(trs[i]).mouseover(function(e) {
            var index = $(e.currentTarget).data('index'); // use `var`
            var d = diagnosis[index].additionalParameters // parentheses not needed
            console.log('d', d);
            dt = $(e.currentTarget).parent().parent().find('thead').find('th')
                                   .eq(index).data(); // you have `index`, use it
            //console.log(dictCanon);
            if (d !== undefined && d !== null) { // <--- changed!
                 var dictCanon = d.dictCanon; // <-- you have `d`, use it
                 var icd = d.icd; // <-- idem
                 $(this).attr('title',icd + ' ' + dictCanon);
             }
        });                
    };
};

另请注意我所做的其他一些更改...请参阅代码中的注释。