2011-07-03 15 views
1

6抽象化、1構築

典型的な

if (this.sun.hidden === true && 
    this.moon.visible === false) { 
    print "its daytime" 
} 

可能性のある不要な抽象

私が好きないくつかの構文

if (isSunHidden() && 
    isMoonVisible()) { 
    print "its daytime" 
} 

を削除

if (isSunHidden() === true && 
    isMoonVisible() === false) { 
    print "its daytime" 
} 

これは非常に読みやすい、だけでなく、ハードコード化された「日中」の文字列を必要と

if (timeOfDay() === "daytime") { 
    print "its daytime" 
} 

これは、一日の時間が良くない、チェックして

printDaytime(); 

とにかくを

if (isItDaytime()) { 
    print "its daytime" 
} 

ミックス印刷を読むことも容易ですこれは個人的な好みですが、他のものよりも好都合な理由もあります。その理由を聞いてみたいと思います。ヒンジ。

ありがとうございます!あなたのオプションの

答えて

3

、私が好む:

if (isItDaytime()) { 
    print "its daytime" 
} 

この1つはまた、ひどいではありません。

if (timeOfDay() === "daytime") { 
    print "its daytime" 
} 

...私は列挙または類似の構造の代わりに、文字列リテラルを使用しますが、 like:

if (timeOfDay() === TimeOfDay.DAY_TIME) { 
    print "its daytime" 
} 
関連する問題