JavaScript：要素のプロトタイプを拡張する

Question

要素を拡張することに関して多くの議論があります。私の知る限り、これらが主な問題は、次のとおりです。

• それは他のライブラリ
• と競合する可能性DOMはそれがレガシーIE
• とそれを動作しません
• ルーチンには文書化されていない機能が追加されています
  ：未来と競合する可能性は他のライブラリ、ドキュメントの変更を参照していない、と歴史的なブラウザの気にしていないプロジェクトを考えると

を変更

要素のプロトタイプを拡張しない理由がありますか。技術ここでは、これは便利である方法の例です：

Element.prototype.toggleAttribute=function(attribute,value) { 
    if(value===undefined) value=true; 
    if(this.hasAttribute(attribute)) this.removeAttribute(attribute); 
    else this.addAttribute(attribute,value); 
}; 

私は合理的な説明を提供することなく、プロトタイプを拡張するの弊害について、あまりにも多くのコメントを見てきました。

注1：toggleAttributeは今後追加される可能性のある種類のものであるため、上記の例はあまりにも明白です。議論のために、それがmanngoToggleAttributeと呼ばれると想像してください。

注2：メソッドがすでに存在するかどうかのテストを削除しました。たとえそのようなメソッドが既に存在していても、それをオーバーライドする方がより予測可能です。いずれにせよ、ここでのポイントは、メソッドがまだ定義されておらず、実装されているのではなく、であると仮定しています。それがここのポイントです。

ありがとうございました

Answer 1

あなたが所有していないオブジェクトは変更しないでください。

将来の標準がElement.prototype.toggleAttributeと定義されているとします。あなたのコードはあなたの関数を割り当てる前に本当の値を持っているかどうかチェックします。だから、将来のネイティブ関数で終わる可能性があります。これは、期待したものとは異なる動作をする可能性があります。

さらに、Element.prototype.toggleAttributeを読むだけでゲッターが呼び出され、望ましくない横効果を伴うコードが実行される可能性があります。たとえば、Element.prototype.idを取得したときにどうなるかを参照してください。

チェックをスキップして、機能を直接割り当てることができます。しかし、それはいくつかの望ましくない横向きの影響でセッターを実行する可能性があり、あなたの関数はプロパティとして割り当てられません。

プロパティ割り当ての代わりにプロパティ定義を使用できます。それは安全でなければなりません... Element.prototypeにはいくつかの特殊な[[DefineOwnProperty]]内部メソッド（プロキシなど）がない限り。

多くの方法で失敗する可能性があります。これをしないでください。

Answer 2

Elementプロトタイプを拡張しないような技術的な理由はありますか？

絶対にありません！

恩赦： 絶対にありません！

さらに、 .__proto__は、実際には昨日までの不正なプロトタイプ拡張でした。 - 今日、それは基準です。

---

p.s：あなたは（Element.prototypeで「toggleAttribute」）が行います場合は、任意の手段によってif(!Element.prototype.toggleAttribute)構文の使用は避けるべきです。

Answer 3

大丈夫ですか？技術的にはい。ネイティブAPIを拡張する必要がありますか？親指の規則としてはない。残念ながら答えはより複雑です。 EmberやAngularのような大きなフレームワークを作成している場合は、APIの利便性が向上すれば消費者がBenifitsを持つことができるので、そうすることをお勧めします。しかし、あなた自身のためにこれをやっているのであれば、親指のルールはノーです。

このようにすると、そのオブジェクトの信頼が不安定になることが考えられます。言い換えれば、ネイティブオブジェクトを追加、変更、変更することによって、他の誰か（あなたの将来の自己を含む）が期待するよく理解され、文書化された動作に従うことはもうありません。

このスタイルでは、気付かれない実装を隠します。 この新しい方法は何ですか？、それは秘密のブラウザのものですか？、どうしますか？、Googleやマイクロソフトにこれを報告してもらえますか？。少し誇張されていますが、APIの真実は今や変わってきており、この1つのケースでは予期せぬことです。メンテナンスは、独自の関数またはラッパーオブジェクトを使用した場合にはそうではないような余分な考えと理解を必要とします。また、変更が難しくなります。

関連のポスト：Extending builtin natives. Evil or not?

の代わりに誰か他の人の（または標準）をマックしようとしているコードだけで独自のものを使用します。

function toggleAttribute(el, attribute, value) { 
    var _value = (value == null ? true : value; 
    if (el.hasAttribute(attribute)) { 
    el.removeAttribute(attribute); 
    } else { 
    el.addAttribute(attribute, _value); 
    } 
}; 

これで、安全で、可搬性があり、持ち運びができ、保守が容易になりました。他の開発者（あなたの将来の自己を含む）は、標準APIまたはJS APIに記載されていないこの魔法の方法がどこから来たのか、彼らの頭を混乱させることはありません。