JavaScript函数不搜索数组

时间:2011-11-01 11:45:38

标签: javascript browser browser-detection

我在JavaScript中编写了一些代码,使用window.navigator.languagenavigator.browserLanguage对象来搜索包含这些可能值的数组。

我想要实现的是检查数组以查看其中的任何值是否与浏览器语言相匹配。我知道您可以使用PHP并检查HTTP标头但我需要在客户端执行此检查。无论如何,我已经编写了下面的代码但由于某种原因它似乎没有返回我想要的值。我写的函数是基于每次浏览器语言与数组中的值匹配时它将返回true,否则它将返回false。但是每次运行该函数时它都会返回true!我究竟做错了什么?代码如下。

JavaScript的:

var langArray = new Array("en","en-GB","en-US","fr","de","en-gb","en-us"); 

function checkforIE() {
    if (navigator.browserLanguage) { //if the navigator.browserLanguage is supported (IE only)
        for(var i=0;i<langArray.length;i++) {
            if(this[i] = navigator.browserLanguage) {
                return true;
            }
            else {
                return false;
            }
        }
    }
}

function checkforFirefox() {
    if (window.navigator.language) { //if the window.navigator.language is supported 
        for (var i=0;i<langArray.length;i++) {
            if(window.navigator.language==this[i]) {
                return true;
            }
            else {
                return false;
            }
        }
    }
}

HTML:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<script type="text/javascript" src="test-browser.js"></script>
<title>Test Browser support</title>
</head>

<body>
<h2>Test browser support</h2>
<a href="#" onclick="checkforIE(); checkforFirefox();">Click here</a>

</body>
</html>

提前致谢。

2 个答案:

答案 0 :(得分:1)

this[i]应为langArray[i]。您希望this引用langArray,但这不正确。 this是指window

此外,您没有对返回值执行任何操作。要合并您的函数行为,请使用:

var lang = checkforIE() || checkforFirefox(); //If IE, use IE, otherwise, use FF
if(lang) alert('Language supported');

答案 1 :(得分:1)

  • 您要分配而不是比较
  • 您应该从数组中获取,而不是从this
  • 获取
  • 如果全部失败,您应该只返回false。目前,您在第一次迭代后总是返回。如果第一个元素不匹配,则函数不应该直接返回false,而是继续检查第二个元素,等等。

例如:

function checkforIE() {
    if (navigator.browserLanguage) { //if the navigator.browserLanguage is supported (IE only)
        for(var i=0;i<langArray.length;i++) {
            if(langArray[i] == navigator.browserLanguage) { // == for comparison
                                                            // and fetch from the array
                return true;
            }
        }
    }
    return false; // only return false if all failed
}