配列プッシュ関数を使用して要素を配列に追加しています
var Oben = document.getElementById("Oben")
var Unten = document.getElementById("Unten")
if( Oben ) {
objekte.push( Oben );
}
if( Unten ) {
objekte.push( Unten );
}
このコーディングスタイルを改善するにはどうすればよいですか?
配列プッシュ関数を使用して要素を配列に追加しています
var Oben = document.getElementById("Oben")
var Unten = document.getElementById("Unten")
if( Oben ) {
objekte.push( Oben );
}
if( Unten ) {
objekte.push( Unten );
}
このコーディングスタイルを改善するにはどうすればよいですか?
var Oben = document.getElementById("Oben");
var Unten = document.getElementById("Unten");
!Oben || objekte.push( Oben );
!Unten || objekte.push( Unten );
また
var Oben = document.getElementById("Oben");
var Unten = document.getElementById("Unten");
Oben && objekte.push( Oben );
Unten && objekte.push( Unten );
大丈夫だよ。しかし、あなたがしなければならない場合:
function pushIfFound(list, id){
var node = document.getElementById(id);
if (node){
list.push(node);
}
}
pushIfFound(objekte, 'Oben');
pushIfFound(objekte, 'Unten');
他の人が維持するのが最も直感的ではなく、1行で比較を行うため、私はおそらく実際にはこれを行いません。
var Oben = document.getElementById("Oben");
var Unten = document.getElementById("Unten");
objekte.push(Oben?Oben:Unten?Unten);
たとえば、次のようになります。
var Oben = document.getElementById("Oben"),
Unten = document.getElementById("Unten");