为什么这个JavaScript原型函数会破坏jQuery？

Question

为什么这个JavaScript原型函数会破坏jQuery？

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');
        // https://dev59.com/MWs05IYBdhLWcg3wQvyq

        $("#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原型冲突，但这根本没有帮助。

- Stefan Steiger

可能是重复问题 https://dev59.com/d3I-5IYBdhLWcg3wfoa1 - zs2020

你的问题已经得到了充分的回答。请随意接受其中一个答案 :) - benekastah

3个回答

3

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

这样做，是为了防范不良编码实践。他们不想让代码因此变得笨重，而是要求用户遵循良好的实践，例如永远不要将可枚举属性放在 Object.prototype 上。

换句话说......除非你希望所有代码都针对这些属性运行保护，而且你从不希望枚举继承的属性，否则不要将可枚举属性添加到 Object.prototype 中。

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

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保持不变，因此纯对象仍然是安全的。在枚举您的O对象时，只需要使用.hasOwnProperty()守卫即可。

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

关于解析JSON数据，您可以使用一个修复器函数来将普通的对象替换为您的O对象。

var parsed = JSON.parse(jsondata, function(k, v) {
    if (v && typeof v === "object" && !(v instanceof Array)) {
        return O(v)
    }
    return v
});

- the system

好的，但是我的可枚举关联数组通过JSON生成的类型似乎是对象。 - Stefan Steiger

@Quandary：我不确定你的意思。当您使环境中的所有对象继承一个属性（这就是当您扩展“Object.prototype”时发生的情况）时，所有继承属性的迭代都会遇到它。 - the system

是的，这就是原型的意义。但你实际上所说的是，我必须编写一个静态方法，将对象作为变量传递给它，并因此采用糟糕的编码实践，只是因为jQuery使用了糟糕的编码实践，而不是相反。 - Stefan Steiger

1

@Quandary：将对象传递给函数怎么会是一种不好的做法呢？你如何认为期望所有代码必须测试每个对象的每个枚举属性，以查看该属性是否直接在对象上，是一件好事情呢？采用有效消除继承枚举可能非常有用的做法又怎么是一件好事呢？采用良好的实践可以解决所有这些问题，例如不导致环境中的所有对象都继承可枚举属性。 - the system

3

您可以通过使用defineProperty来设置描述符，从而克服此副作用。

Object.defineProperty(Object.prototype, 'propName', {value: 'your value', enumerable: false});

- marekful

True。在JavaScript 1.8.5中引入。 - marekful

+1 - 不错的发现。然而在我的情况下，如果我只使用更新的浏览器，我仍然可以使用object.keys函数。但是我不仅仅使用更新的浏览器[虽然我很想这样做] :) - Stefan Steiger

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- benekastah · Accepted Answer

因为这将为每个对象添加一个可枚举的项。 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