Javascriptにおけるループについて。

Question

はじめまして。

ある配列をループして、ある条件式に等しい場合、'Right!'と表示され、全ての配列が条件式に合わなかった場合、一回のみ'False!'と表示されるようなコードを実装したいのですが、
配列のループ分'False!'と表示されてしまいます。

for(var k in array){
		
		if(条件式){
		document.write('Right!');
		}else{  //elseがfor内に入ってしまっているので、配列の数だけ'False!'と表示されてしまう。
		document.write('False!');
		}
		
}

このような場合、上記のコードをどのように書き換えれば良いでしょうか?
何かお分かりの方は、是非ともご教授の程よろしくお願いします。

Answer 1

flag を使用する方法もありますが、document.write を何度も呼び出すとリフロー回数が増えてパフォーマンス的によろしくない為、配列にキャッシュして一度に document.write する方法をお勧めします。
**for-in は index の列挙順を保証しない(ランダム)**なので for 文を使用しました(今回の例では支障は生じませんが、基本的に配列に使うべきではないと思います)。

var array = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9],
    results = [];

for (var i = 0, l = array.length, value; i < l; ++i) {
  value = array[i];

  if (value % 2 === 0) {
    results.push('Right!');
  }
}

document.write(results.length > 0 ? results.join('\r\n') : 'False!');
console.log(results); // [0, 2, 4, 6, 8]

Answer 2

外側に、「一度でも条件をクリアしたか」を記録するフラグを入れましょう。

var cleared = false;

for(var k in array){
        
        if(条件式){
          document.write('Right!');
          cleared = true;
        }
}

if (!cleared) {
  document.write('False!');
}

なお、2点ほどアドバイスがあります。

• document.writeを使うと、ブラウザの動作が一時中断したり、ロード後に実行するとブラウザの中身が全クリアされたりするなど、現代では使いにくいものとなっています。DOMで出力しましょう。
• 配列をfor-inで回すと、余計なものが引っかかったり遅かったりで綺麗に動作しません。普通のforループでインデックスを操作しましょう。

Comments

[Shinog]

親切かつ丁寧なご回答ありがとうございます!!
clearedのフラグを立てることで、無事解決出来ました。
2つのアドバイスもこれから参考にさせて頂きます。

Answer 3

一例です

var f = 0;

for(var k in array){
        
    if(条件式){
        document.write('Right!');
        f = 1;
        break;
    }
}

if( f==0 ){
    document.write('False!');
}

Answer 4

break;を利用すればいいのではないでしょうか?