片方のみ処理されてしまう

Question

・チェックボックス未チェックのままsubmitボタンを押下した場合はエラー(javascript:alert)を表示し、投稿を中止
という条件をJavaScriptのみで処理したいんですが、アラート表示の条件がうまく処理できません
下記コードで処理を行うとラジオボタンのID属性を代入した変数up・deの内deのみ処理されてしまいます
変数upも同様に行いたいのでelse ifをもう1つ追加してupとdeを分けて処理してみても改善しませんでした

何か抜けている処理があるんでしょうか

//up又はdeのラジオボタン選択でtrue
else if (up.checked == true || de.checked == true) {

//チェックボックスのname属性取得
var ca = document.getElementsByName('chkid[]');

//変数boxは配列
var box = [];

//変数caは配列なので、ループの中で 一つずつチェックされているか判定
for(var cb = 0; cb < ca.length; cb++) {

    //チェックされていれば、変数boxを押した時に変数cbをカウントした変数caを代入
	if (ca[cb].checked) {
		box.push(ca[cb]);
	}
}
     //1つもチェックされていない時
   if (box == 0) {
	  window.alert("ERROR5");

       //falseで処理を返す
	  return false;
   }
}

Comments

[kunai]

なぜ同じ内容で何度も質問を立て直されているのでしょうか。
回答する側としては、今まで行われた回答や質問の中で、ここに記載のない有用な情報があったりもして、余計な質問等もすることなく回答出来るようになると思うのですが。。

[NatsumiOki]

コードの形式がHTMLになっていますが、中身がJavaScriptなのでハイライトがおかしくなっちゃってます。JavaScriptに変えたほうが良いですよ。

[jm1156]

else if (up.checked == true || de.checked == true) {
↑の行のまえのif文はどうなってますか？
以前のコードのままなら、最初のif文にup.checkedをチェックするコードが
はいっていたはずなので、このコードにはたどり着かないはずです。

あと↓の質問、
https://teratail.com/questions/46751
Kosuke_Shibuyaさんのコードはみましたか？
ちゃんと内容を理解して「解決」にしましたか？

Answer 1

「else if」だからです。
http://alphasis.info/2013/01/javascript-control-else_if/

Answer 2

if (box == 0) {

box は配列なのに == 0 で比較するのがおかしい。

Answer 3

この手の処理は１個でもチェックされていればいいので
フラグで処理するのが妥当。
checkedが見つかったらあとの処理は不要なのでbreakすること

var ca = document.getElementsByName('chkid[]');
var ca_flg=false;
for(var i = 0; i < ca.length; i++) {
    if (ca[i].checked) {
        ca_flg=true;
        break;
    }
}
if (ca_flg) {
   alert("ERROR5");
   return false;
}