2011-10-29 7 views
0

私はjQueryのソースからこのコードを乾燥するために乾燥しています:乾燥うちいくつかのコードを

cssNumber: { 
    "fillOpacity": true, 
    "fontWeight": true, 
    "lineHeight": true, 
    "opacity": true, 
    "orphans": true, 
    "widows": true, 
    "zIndex": true, 
    "zoom": true 
} 

私は次のことを示唆しています。それはより良いことができますか?

cssNumber: {} 
"fillOpacity fontWeight lineHeight opacity orphans windows zIndex zoom" 
    .split(" ") 
    .forEach(function() { 
     cssNumber.name = true; 
    }); 
+3

達成したい目標は何ですか?最初のバージョンは間違いなくより良いパフォーマンスを持っています。コードを減らすには、ミニファイアを見てください。 – Smamatti

+0

私は楽しい運動としてそれを取っています。 – Randomblue

+0

「true」を繰り返すことは、他の場所でコードが「false」と同じでない限り、自分自身を繰り返すとは思えません。その場合、それを関数に投げてください。それでも、それはちょうど効率的です。しかし、DRYはより大きなコードブロックのためのものです。 – Ryan

答えて

0

本当にバイトスペースを節約する必要がある場合は、私はこのために行っています。あなたがしたいことは、オブジェクトを値で埋めることです。あなたはまさにそれよりもすべきです。あなたの最初のコードはどれですか。

+1

私はまだJS minifyがうまくいくと思います。 – Smamatti

+0

もちろん、DRYされたコードも縮小することができます。 – HerrSerker

+0

心配しないでください。この場合、乾燥されたコードは、たとえ縮小されたとしてもより長くなります。 - コードを削除することは場合によっては良いことかもしれませんが、この場合はそうではありません。私はここでの質問に対する32bitkidのコメントに強く同意します。 – Smamatti

関連する問題