这怎么评价为假呢？

Question

当我单步执行此代码时，图中的值会导致 cssLoaded = 0;据我所见，长度大于 0 并且 cssStylesheet.sheet 具有属性，因此应该为 true，因此 cssLoaded 应设置为 1，但它没有发生。我一定在这里遗漏了一些东西..？

function _cssIsLoaded(cssStylesheet, stylePath) {
var cssLoaded = 0;
if (tryCount == 3){
    console.log('oops');
}
if (cssStylesheet.href == stylePath){
    try {
        if ( cssStylesheet.sheet && cssStylesheet.sheet.cssRules.length > 0 ){
            cssLoaded = 1;}
        else if ( cssStylesheet.styleSheet && cssStylesheet.styleSheet.cssText.length > 0 ){
            cssLoaded = 1;}
        else if ( cssStylesheet.innerHTML && cssStylesheet.innerHTML.length > 0 ){
            cssLoaded = 1;}
        }
        catch(ex){ }
}
    if(cssLoaded) {
        //alert('cssloadedcomplete');
        resetPops();
        $('#video-overlay').show();
        positionElements();
        saveBizzmail();
        console.log('try:' + tryCount)
    } else {
        tryCount+=1;
        console.log('try:' + tryCount);
        setTimeout(function() { this._cssIsLoaded(cssStylesheet); }, 2000);
    }
}

Answer 1

您的代码可以通过三种方式获得 cssLoaded == 0;

1. if (cssStylesheet.href == stylePath) 计算结果为 false。
2. 所有三个 if/else 语句的计算结果均为 false。
3. 其中一个 if/else 语句引发异常，因此其他 if/else 语句不会执行。

在解决问题时，您应该验证前两个不是问题，因为这些可能只是比较中的逻辑错误，或者进入函数的数据不是您所期望的。

对于第三个问题，您可以实现防御性编码，以确保通过将代码更改为此来检查所有三个 if/else 语句（您甚至不再需要 try/catch，因为它允许您的代码绕过一些测试）：

function _cssIsLoaded(cssStylesheet, stylePath) {
    var cssLoaded = 0;
    if (tryCount == 3){
        console.log('oops');
    }
    if (cssStylesheet && cssStylesheet.href == stylePath) {
        if ( cssStylesheet.sheet && cssStylesheet.sheet.cssRules && cssStylesheet.sheet.cssRules.length > 0 ) {
            cssLoaded = 1;
        }
        else if ( cssStylesheet.styleSheet && cssStylesheet.styleSheet.cssText && cssStylesheet.styleSheet.cssText.length > 0 ) {
            cssLoaded = 1;
        }
        else if ( cssStylesheet.innerHTML && cssStylesheet.innerHTML.length > 0 ) {
            cssLoaded = 1;
        }
    }
    if(cssLoaded) {
        //alert('cssloadedcomplete');
        resetPops();
        $('#video-overlay').show();
        positionElements();
        saveBizzmail();
        console.log('try:' + tryCount)
    } else {
        tryCount+=1;
        console.log('try:' + tryCount);
        setTimeout(function() { this._cssIsLoaded(cssStylesheet); }, 2000);
    }
}

Answer 2

前一个检查可确保您避免类似 NPE 的情况（通过检查 cssStylesheet.sheet 不为 null）。

Answer 3

您必须检查该元素是否存在，否则可能会发生错误（cssStylesheet.sheet is undefined）。在以下情况下，表达式的计算结果为 false：

• 变量不存在（undefined、null、void 0）
• 等于 0 code>（零）
• 等于 false
• 等于空字符串 (""、'')
• 非数字值 (0/0，NaN）

请注意，空对象 {} 的计算结果为 true，因为它是一个对象。

另一个例子：

if(window.addEventListener) window.addEventListener("load", function(){}, true);
else if(window.attachEvent) window.attachEvent("onload", function(){});
else window.onload = function();

如果不存在 if 条件，则代码将在旧版本的 IE 中抛出错误，因为旧版本的 IE 没有名为 window.addEventListener 的方法。