6

问题：

只要将以下代码添加到我的 html 页面，我就会得到：

Line: 4
Error: Object doesn't support the property or method "exec".

这是导致错误的原型：

    Object.prototype.allKeys = function () {
        var keys = [];
        for (var key in this) 
        {
            // Very important to check for dictionary.hasOwnProperty(key) 
            // otherwise you may end up with methods from the prototype chain.. 
            if (this.hasOwnProperty(key)) 
            {
                keys.push(key);
                //alert(key);
            } // End if (dict.hasOwnProperty(key)) 

        } // Next key

        keys.sort();
        return keys;
    }; // End Extension Function allKeys

这是重现错误所需的最少代码（有问题的浏览器：IE9）：

<!DOCTYPE html>
<html>
<head>
    <title>TestPage</title>

    <script type="text/javascript" src="jquery-1.9.1.min.js"></script>


    <script type="text/javascript">
        /*
        Object.prototype.getName111 = function () {
            var funcNameRegex = /function (.{1,})\(/;
            var results = (funcNameRegex).exec((this).constructor.toString());
            return (results && results.length > 1) ? results[1] : "";
        }; // End Function getName
        */

        Object.prototype.allKeys = function () {
            var keys = [];
            for (var key in this) 
            {
                // Very important to check for dictionary.hasOwnProperty(key) 
                // otherwise you may end up with methods from the prototype chain.. 
                if (this.hasOwnProperty(key)) 
                {
                    keys.push(key);
                    //alert(key);
                } // End if (dict.hasOwnProperty(key)) 

            } // Next key

            keys.sort();
            return keys;
        }; // End Extension Function allKeys

    </script>

</head>
<body>

    <select id="selLayers" name="myddl">
      <option value="1">One</option>
      <option value="2">Twooo</option>
      <option value="3">Three</option>
      <option value="4">Text1</option>
      <option value="5">Text2</option>
    </select>


    <script type="text/javascript">

        //var dict = { "de": { "Text1": "Ersetzung 1", "Text2": "Ersetzung 2" }, "fr": { "Text1": "Replacement 1", "Text2": "Réplacement 2" }, "it": { "Text1": "Replacemente 1", "Text2": "Replacemente 2" }, "en": { "Text1": "Replacement 1", "Text2": "Replacement 2"} };
        /*
        var languages = dict.allKeys();


        for (var j = 0; j < languages.length; ++j) 
        {
            var strCurrentLanguage = languages[j];
            var dictReplacements = dict[strCurrentLanguage]
            var keys = dictReplacements.allKeys();

            //alert(JSON.stringify(dictReplacements));
            //alert(JSON.stringify(keys));


            for (var i = 0; i < keys.length; ++i) {
                var strKey = keys[i];
                var strReplacement = dictReplacements[strKey];

                alert(strKey + " ==> " + strReplacement);
                //alert('#selLayers option:contains("' + strKey + '")');
                //$('#selLayers option:contains("' + strKey + '")').html(strReplacement);
                //$('#selLayers option:contains("Text1")').html("foobar");


            }    
        }
        */


        $('#selLayers option:contains("Twooo")').text('Fish');


        //alert(dict.allKeys());        
        //alert(dict["de"]["abc"]);




        /*

        $('#selLayers option[value=2]').text('Fish');
        $('#selLayers option:contains("Twooo")').text('Fish');
        $('#selLayers option:contains("Twooo")').html('&Eacute;tage');
        // http://stackoverflow.com/questions/7344220/jquery-selector-contains-to-equals

        $("#list option[value=2]").text();

        $("#list option:selected").each(function () {
            alert($(this).text());
        });  

        $("#list").change(function() {
            alert($(this).find("option:selected").text()+' clicked!');
        });

        */

    </script>

</body>
</html>

我尝试重命名原型函数，以防它与任何 jquery 原型冲突，但这根本没有帮助。

4

3 回答 3

6

因为这将为每个对象添加一个可枚举项。Sizzle（jQuery 使用）使用对象文字来配置它们的选择器解析。当它循环这些配置对象以获取所有令牌时，它并不期望您的功能。在这种情况下，它可能会尝试将您的函数用作RegExp.

想象一下这个场景：

var obj = { a: 1, b: 2, c: 3 };
var result = 0;
for (var prop in obj) {
  // On one of these iterations, `prop` will be "allKeys".
  // In this case, `obj[prop]` will be a function instead of a number.
  result += obj[prop] * 2;
}
console.log(result);

如果您在Object的原型中添加了任何不能用作数字的东西，您将得到NaN您的结果。

解决此问题的一个好方法是将allKeys函数添加到Object而不是Object.prototype. 这模仿Object.keys：

Object.allKeys = function (obj) {
    var keys = [];
    for (var key in obj) 
    {
        // Very important to check for dictionary.hasOwnProperty(key) 
        // otherwise you may end up with methods from the prototype chain.. 
        if (obj.hasOwnProperty(key)) 
        {
            keys.push(key);
            //alert(key);
        } // End if (dict.hasOwnProperty(key)) 

    } // Next key

    keys.sort();
    return keys;
}; // End Extension Function allKeys
于 2013-02-18T17:13:58.180 回答
3

.hasOwnProperty()因为 jQuery在枚举对象时不会检查代码。

这样做是为了防止不良的编码实践。他们没有权衡他们的代码来适应这种做法，而是要求他们的用户遵守良好的做法，比如永远不要将可枚举的属性放在Object.prototype.

换句话说...不要添加可枚举属性，Object.prototype除非您希望所有代码都针对这些属性运行防护，并且您永远不想枚举继承的属性。

FWIW，如果您真的想在普通对象上调用方法，只需创建一个构造函数，以便您拥有一个可以安全扩展的中间原型对象。

function O(o) {
    if (!(this instanceof O))
        return new O(o)
    for (var p in o)
        this[p] = o[p]
}

O.prototype.allKeys = function() {
    // ...
};

现在你像这样创建你的对象：

var foo = O({
    foo: "foo",
    bar: "bar", 
    baz: "baz"
});

......并且Object.prototype遗骸未受影响，因此普通物体仍然是安全的。您只需要.hasOwnProperty()在枚举O对象时使用防护。

for (var p in foo) {
    if (foo.hasOwnProperty(p)) {
        // do stuff
    }
}

对于被解析的 JSON 数据，您可以使用 reviver 函数将普通对象替换为您的O对象。

var parsed = JSON.parse(jsondata, function(k, v) {
    if (v && typeof v === "object" && !(v instanceof Array)) {
        return O(v)
    }
    return v
});
于 2013-02-18T17:14:21.933 回答
3

您可以通过使用definePropertywhich 设置描述符来克服这种副作用。

Object.defineProperty(Object.prototype, 'propName', {value: 'your value', enumerable: false});
于 2013-02-18T17:21:05.273 回答