2016-07-21 8 views

答えて

2

あなたの質問で引用されたコードは、不必要にStringオブジェクトを作成するため、JavaScriptに堪能でない人が作成したようです。

var IMG = 'img'; 
var DOT = '.'; 
var CLASS = 'class'; 

か、それはあなたを可能にするということであるとES2015に+

const IMG = 'img'; 
const DOT = '.'; 
const CLASS = 'class'; 

については、「なぜこれを行う、」最善の正当化は、私が思い付くことができます:それは全く存在している場合、それは単にする必要がありますあなたが使用した場所のすべてを見つけようと、のタグをimgタグの目的に使用したことがあります。'img'の場所を完全に見つけることはできません。

ではありません(または少なくとも1つは、それが意図されていないことを期待しています)。後でIMGを名前を変更せずに'image'に変更した場合、そのコードで作業している後続の作者を積極的に誤解を招きます。

+0

意見:マジックナンバーに定数を使うのは合理的で便利ですが、一般的にリテラルに定数を使うのは意味がありません。 –

+0

明確な答えです、ありがとうございます。コピーキャットの落とし穴を私にもっと認識させる – Richard

関連する問題