0

I am using following script to check that a browser is firefox and it is mac os. Right now i am on linux chrome but still the following if statement works. http://jsfiddle.net/RjyW6/3/

var BrowserDetect = {
    init: function () {
        this.browser = this.searchString(this.dataBrowser) || "An unknown browser";
        this.version = this.searchVersion(navigator.userAgent)
            || this.searchVersion(navigator.appVersion)
            || "an unknown version";
        this.OS = this.searchString(this.dataOS) || "an unknown OS";
    },
    searchString: function (data) {
        for (var i=0;i<data.length;i++) {
            var dataString = data[i].string;
            var dataProp = data[i].prop;
            this.versionSearchString = data[i].versionSearch || data[i].identity;
            if (dataString) {
                if (dataString.indexOf(data[i].subString) != -1)
                    return data[i].identity;
            }
            else if (dataProp)
                return data[i].identity;
        }
    },
    searchVersion: function (dataString) {
        var index = dataString.indexOf(this.versionSearchString);
        if (index == -1) return;
        return parseFloat(dataString.substring(index+this.versionSearchString.length+1));
    },
    dataBrowser: [
        {
            string: navigator.userAgent,
            subString: "Chrome",
            identity: "Chrome"
        },
        {   string: navigator.userAgent,
            subString: "OmniWeb",
            versionSearch: "OmniWeb/",
            identity: "OmniWeb"
        },
        {
            string: navigator.vendor,
            subString: "Apple",
            identity: "Safari",
            versionSearch: "Version"
        },
        {
            prop: window.opera,
            identity: "Opera",
            versionSearch: "Version"
        },
        {
            string: navigator.vendor,
            subString: "iCab",
            identity: "iCab"
        },
        {
            string: navigator.vendor,
            subString: "KDE",
            identity: "Konqueror"
        },
        {
            string: navigator.userAgent,
            subString: "Firefox",
            identity: "Firefox"
        },
        {
            string: navigator.vendor,
            subString: "Camino",
            identity: "Camino"
        },
        {       // for newer Netscapes (6+)
            string: navigator.userAgent,
            subString: "Netscape",
            identity: "Netscape"
        },
        {
            string: navigator.userAgent,
            subString: "MSIE",
            identity: "Explorer",
            versionSearch: "MSIE"
        },
        {
            string: navigator.userAgent,
            subString: "Gecko",
            identity: "Mozilla",
            versionSearch: "rv"
        },
        {       // for older Netscapes (4-)
            string: navigator.userAgent,
            subString: "Mozilla",
            identity: "Netscape",
            versionSearch: "Mozilla"
        }
    ],
    dataOS : [
        {
            string: navigator.platform,
            subString: "Win",
            identity: "Windows"
        },
        {
            string: navigator.platform,
            subString: "Mac",
            identity: "Mac"
        },
        {
               string: navigator.userAgent,
               subString: "iPhone",
               identity: "iPhone/iPod"
        },
        {
            string: navigator.platform,
            subString: "Linux",
            identity: "Linux"
        }
    ]

};
BrowserDetect.init();
console.log(BrowserDetect.browser);
console.log(BrowserDetect.OS);
if((BrowserDetect.browser)==="Firefox" && (BrowserDetect.OS)==="Mac");
                      {
                            alert('this');
                      }

any help?

4

3 回答 3

3

删除分号：

// ------------------------------------------------------------------v
if((BrowserDetect.browser)==="Firefox" && (BrowserDetect.OS)==="Mac");
{
  alert('this');
}

否则if块看起来是空的，即这是计算机看到的：

if((BrowserDetect.browser)==="Firefox" && (BrowserDetect.OS)==="Mac") {
  // Do nothing
}
{
  // Just another block
  alert('this');
}
于 2013-09-09T12:52:03.037 回答
2

你;在错误的地方。

;从末尾删除if((BrowserDetect.browser)==="Firefox" && (BrowserDetect.OS)==="Mac");

于 2013-09-09T12:52:09.317 回答
1

我认为你有一个额外的分号导致问题。尝试这个：

if((BrowserDetect.browser)==="Firefox" && (BrowserDetect.OS)==="Mac")  // this had the semicolon
{
  alert('this');
}
于 2013-09-09T12:52:51.537 回答