2016-05-19 3 views
0

私はdocumentは、私はいくつかの他の場所から取得したオブジェクトであるため、ブラウザのドキュメントはありませんが、同じ名前を持つだけで任意のオブジェクト機能変数に名前を付けるために '文書'を使用するのは正当でしょうか?

function processDocument (document) { 
    if (document.name === "client-invoice") { 
     // Do something with the document 
    } 
} 

を持っています。私は、document.nameがいくつかの値に等しくなるのをチェックすると、渡されたパラメータオブジェクトを調べるのではなく、ブラウザのドキュメントオブジェクトを調べると思う。

もしそうなら、変数documentオブジェクトと矛盾しないように変数の名前を変更します。

質問は私の前提が正しいと私の変数の名前を避ける必要がありますdocument

+0

"私はそれを疑う..." - それは簡単にテストすることができます。なぜそれをテストする代わりにStackoverflowを尋ねるのですか? – Quentin

+0

私は働いている大きなアプリを持っていますが、私はそれに精通していませんので、他の開発者の間違いがないように変数を命名してください。 –

+0

大きなアプリでは、小さなスタンドアロンのテストケースを書くことができます。 – Quentin

答えて

0

いいえ、動作します。

しかし、あなたのコードでdocumentという名前の変数を使わないようにすることをお勧めします。それは将来混乱する可能性があります。

function processDocument (document) { 
 
    
 
    alert(document); 
 
    
 
    if (document.name === "client-invoice") { 
 

 
    return '' 
 
    } 
 
} 
 

 
processDocument('Test')

1

私の仮定が正しい

var document_copy = document; 
 

 
function myFunction(document) { 
 
    alert("Locally scoped document === global document_copy: " + (document_copy === document)); 
 
} 
 

 
myFunction("some value");

です

私の変数文書の名前は避けてください。

はい。あなたのコードを維持しなければならない人々を混乱させるような名前です。

関連する問題