JavaScriptでのフォーム未入力チェックについて

Question

前提・実現したいこと・発生している問題

JavaScriptでフォームの未入力チェックシステムを作っています。
Submitが押された際に未入力の項目がある場合、アラートが表示されるようにしたかったのですが、全ての項目を入力した際もアラートが表示されるようになってしまい画面遷移できなくなってしまいます。

該当のソースコード

function check(){
	if(document.form.year.value == ""){
		alert("年が未入力です");
		return false;
	}else if(document.form.month.value == ""){
		alert("月が未入力です");
		return false;
	}else if(document.form.day.value == ""){
		alert("日が未入力です");
		return false;
	}else if(document.form.name.value == ""){
		alert("名前が未入力です");
		return false;
	}else{
		return true;
	}
} 

<form action="confirm.php" method="post" name="form">
	<p>年月日をご入力ください</p>
	<p><input type="text" name="year">年
	<input type="text" name="month">月
	<input type="text" name="day">日</p>
	<p>名前<br>
	<input type="text" name="name"></p>
	<p>電話番号<br>
	<input type="text" name="phonenumber"></p>
	<p>メールアドレス<br>
	<input type="text" name="mail"></p>
	<input type="submit" value="送信" onclick="return check()">
</form>

試したこと

プログラミング初心者なのでインターネットで調べてみたりはしたのですが、同じような形式でできるという情報もあり、どこをどう間違っているのかすら理解できていない状態です。JavaScriptとHTMLはファイルを分けていますが、他のシステムは動いているのでJavaScript自体が読み込めていないということはないと思います。

Comments

[退会済みユーザー]

再現しないですね。
check()の戻り値を確認するように一部ソースを書き替えていますが問題なさそうですよ。
https://jsfiddle.net/q0eL6spc/1/

[terayabasu]

リンク先でソースを書き換えていただいている部分は動作にはあまり関係ないものですか。
質問する際に必要のないソースは削ってあるのですが、そこが問題なのですかね。

[退会済みユーザー]

リンク先は常にfalseで画面遷移しないようにしていますが、元に戻すと404エラーが出るようになります。
少なくとも提示いただいたcheck()のつくりとは別の問題と思った方がいいと思います。

Answer 1

name属性のdayが問題あったようで、そこを変えたら動作しました。みなさん丁寧に教えてくださってありがとうございました。

Comments

[m.ts10806]

これでは意味がわかりません。
現状で問題再現しないのですけど、何がどう問題でどのように修正したのでしょうか。

あと質問「受付中」のままになっています。

[m.ts10806]

これでは意味がわかりません。
現状で問題再現しないのですけど、何がどう問題でどのように修正したのでしょうか。

Answer 2

とりあえず必須項目にrequired属性をつけるだけでも勝手に飛ぶのはさけられます

Answer 3

全ての項目を入力した際も

アラート出ずにconfirm.phpに遷移しましたよ。問題再現しません。
どのような入力をしてなんのアラートが出たのでしょうか。画面キャプチャも添えてもらったほうが良いです。

あと、提示された以外のコードもあれば。

Comments

[m.ts10806]

仕様的に１個１個出るのは使いづらいかと。invalidになった全項目一気にだしましょう。