チェックリストが空のときのアラート

Question

チェックリストでチャックを入れた項目だけ、DBからカラムを表示させるアプリを作っています。
３つのチェックリストがあり、全てをチェックしなかった場合アラートで送信を中断させたいのですが
nullpointExceptionでアラートがうまく動きません。
他に必要なコードがあればご指摘願います。

JS

<form id="fm" name="fmname" action="/test_Servlet" method="post">
　　<input type="checkbox" name="check" value="NO" id="check1" checked>NO
　　<input type="checkbox" name="check" value="NAME" id="check2" checked>NAME
　　<input type="checkbox" name="check" value="AGE" id="check3" checked>AGE
    <input id="fm" type="button" name="search" value="検索" onclick="return blank_check()">
</form>

HTML

<script type="text/javascript">
	function blank_check() {
		check1 = document.fmname.Check1.checked;
        check2 = document.fmname.Check2.checked;
        check3 = document.fmname.Check3.checked;

		if (check1==false && check2==false && check3==false) {
			alert("空です");
			return false;
		}
		else {
		return true;
		}
	}
</script>

###java

try{
StringBuffer sb = new StringBuffer("select ");
if(columns.length == 1){
						sb.append(columns[0]);
						sb.append(" from TEST");
			}
			else if(columns.length == 2) {
						sb.append(columns[0]);
						sb.append(",");
						sb.append(columns[1]);
						sb.append(" from TEST");
			}
			else if(columns.length == 3) {
						sb.append(columns[0]);
						sb.append(",");
						sb.append(columns[1]);
						sb.append(",");
						sb.append(columns[2]);
						sb.append(" from TEST");
			}
			else if(columns.length == 0) {
				throw new NullPointerException();
			}
}

Comments

[m.ts10806]

NullPointerException
ですね。こういうのはコピペしましょう。
ですが、提示のコードのどこでそのExceptionがthrowされますか？Javaが関係するのでしたらJavaのコードを提示してください

[m.ts10806]

(２回目)のどこでそのExceptionがthrowされますか？

エラーも全文コピペが原則です。質問の際に要約したりしてはいけません。

[azuapricot]

columnsがNullならぬるぽでますけどこれだけじゃなんとも...

[azuapricot]

Javaで出てるのかjavaScriptでコンソールエラーでてるのか。。。(Javaの方なのかな）

[dorei]

エラーはjavaのif文で出ていました
後ほど修正します。
質問が不慣れでご迷惑おかけしました。

Answer 1

Javaは関係ないのではと。

定義が
check1なのに
Check1で呼び出そうとしているからと思います。
dとbが別物であるようにaとAも多くのケースで別物です。
中にはPHPの関数のように大別しないものも一部ありますが、こういうのは手打ちではなくコピペですると間違いありません。

Comments

[dorei]

気付かなかったです・・・直すとアラートが出るようになりましたが実効が中断されず
画面遷移してしまいました

[m.ts10806]

エラー出てるならエラーをコピペしてください
なにもでてないことはないはずです。

Answer 2

id=fmが競合しています

<script>
window.addEventListener('DOMContentLoaded', ()=>{
  document.querySelector('#fm [type=button]').addEventListener('click',(e)=>{
    const flg=e.target.closest('form').querySelectorAll('[type=checkbox]:not(:checked)').length>0;
    if (flg) {
      alert("空のチェックボックスがあります");
    }
  });
});
</script>
<form id="fm" method="post">
<label><input type="checkbox" name="check" value="NO" id="check1" checked>NO</label>
<label><input type="checkbox" name="check" value="NAME" id="check2" checked>NAME</label>
<label><input type="checkbox" name="check" value="AGE" id="check3" checked>AGE</label>
<input type="button" name="search" value="検索" >
</form>