javascriptでエラーを出力したい。

やりたいこと
jspでjavascript入力チェックをしたいです。
フォームの中が空のときに、エラーが表示されるはずなのに、表示されません。
(条件があってjavascriptが必須です。HTMLのみの記述はNGという制約があります。）
簡単な入力チェックも作れず、四苦八苦しています。

試してみたこと
doument.getElementById(xxx)の部分を排除してみました。

画面遷移はうまくいくので、javascriptが間違っているとしか思えませんが、お手上げ状態です。

java
1コード<script type="text/javascript">
2<!--
3
4function formcheck(){
5        var flag = 0;
6
7        //設定開始(必須にする項目を設定してください)
8        if(document.Form1.userId.value == ""){
9        flag = 1;
10        document.getElementById('notice-input-1');
11        }
12        else if(document.Form1.pass.value == ""){
13       	document.getElementById('notice-input-2');
14        flag = 1;
15        }
16
17        if(flag){
18        window.alert('ログインIDまたはパスワードが違います。');
19        return false;
20        }
21        else{
22        return true; //送信を実行
23
24        }
25}
26
27//  -->
28</script>
29<%@ page language="java" contentType="text/html; charset=UTF-8"
30	pageEncoding="UTF-8"%>
31<h1>業務報告システム</h1>
32</head>
33 <body>
34        <form name = "Form1" action="/Sample" method="post" onsubmit="return Check()"><br>
35
36     	ユーザーID:<input type="text" name="userId" id="notice-input-1"> <br>
37		パスワード:<input type="text" name="pass" id="notice-input-2"> <br>
38
39		<button type=submit>ログイン</button>
40	</form>
41</body>
42</html>

m.ts10806

2018/07/05 04:17

タイトルには要件を書いて下さい。 https://teratail.com/help/question-tips#questionTips3-1 タイトルから質問内容が想像できないもの、タイトルと質問内容が食い違っているものはスムーズな回答を得られなくなります。

m.ts10806

2018/07/05 04:17

「うまく動作しない」「うまくいかない」では何が起きているか伝わりません。「○○のようになるはずなのに××となってしまう」のような形で「ゴール」と「現象（エラーメッセージなども含む）」を明確にしてください。https://teratail.com/help/question-tips#questionTips3-4-1

m.ts10806

2018/07/05 04:18

formcheck()はどこからも呼び出されていないようですが？

退会済みユーザー

2018/07/05 04:39

formcheck()の呼び出しができていませんでした。　mts10806をベストアンサーにしたいのですが、どうしたらいいですか？

m.ts10806

2018/07/05 04:40

自身で回答を書いて自己解決としてください。私は現状確認をしただけです。

行動規範の内容に同意します

回答3件

htmlのみはダメってことらしいので。

onsubmit="return Check()"><br>をonsubmit="return formcheck()"><br>にすればいいのでは？

投稿2018/07/05 04:43

papinianus

総合スコア12705

papinianus

2018/07/05 04:44

mts10806さんのコメントで解決してましたね。すみません。

退会済みユーザー

2018/07/05 04:45

ありがとうございました！

行動規範の内容に同意します

必須にすればいいだけなら、JavaScriptなしで大丈夫です。required属性をつけるだけです。

html
1         ユーザーID:<input type="text" name="userId" id="notice-input-1" required> <br>
2        パスワード:<input type="text" name="pass" id="notice-input-2" required> <br>