🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

解決済

2回答

461閲覧

入力フォームのチェック機能を作りました。変な所の指摘をお願いします。

nekoma3

総合スコア3

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

0クリップ

投稿2021/02/08 15:34

編集2021/02/08 15:38

前提・実現したいこと

javasctiptでユーザー登録機能を作りました。
ユーザーの入力内容をチェックして、正しく入力されていないと送信を取り消す処理を書きました。
初学者のためこれで良いのかわかりません!評価して欲しいです。
おかしな所や、改善した方が良い書き方、もっとスマートな書き方、等ありましたら教えてくださるとありがたいです。
どんな細かい事であっても指摘して頂けたら嬉しいです。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

<!DOCTYPE html> <html lang="ja"> <head> <meta charset="UTF-8"> <meta name="viewport" content="width=device-width, initial-scale=1.0"> <title>Document</title> </head> <body> <article> <p id="add-user">ユーザー登録</p> <form method="get" id="user-text" onsubmit="return Check();"> <input type="text" id="name" placeholder="ユーザー名"> <p id="name_p"></p> <input type="text" id="mail" placeholder="メールアドレス"> <p id="mail_p"></p> <input type="text" id="pass" placeholder="パスワード"> <p id="pass_p"></p> <input type="text" id="pass2" placeholder="パスワード(確認)"> <p id="pass2_p"></p> <input id="button" type="submit" value="ユーザー登録"> </form> </article> <script> let nameFlag = false; let mailFlag = false; let passFlag = false; let pass2Flag = false; function nameCheck(){ let name = document.getElementById('name').value; let nameP = document.getElementById('name_p'); let regName = /^[^a-zA-Z0-9]/g; let nameResult = name.match(regName); if(name === ''){ nameP.innerHTML='名前を入力して下さい'; }else if(name <= 8){ nameP.innerHTML = '8文字以内で入力して下さい'; }else if(nameResult === null){ nameP.innerHTML = '英数字以外で入力して下さい'; }else{ nameP.innerHTML = '入力完了'; nameFlag = true; } } function mailCheck(){ let mail = document.getElementById('mail').value; let mailP = document.getElementById('mail_p'); let regMail = /^[A-Za-z0-9_.-]*@{1}[A-Za-z0-9_.-]{1,}.[A-Za-z0-9]{1,}$/g; let mailResult = mail.match(regMail); if(mail === ''){ mailP.innerHTML='メールアドレスを入力して下さい'; }else if(mailResult === null){ mailP.innerHTML = '不正なメールアドレスです'; }else{ mailP.innerHTML = '入力完了'; mailFlag = true; } } function passCheck(){ let pass = document.getElementById('pass').value; let passP = document.getElementById('pass_p'); let regPass = /^[a-zA-Z0-9]{10,18}$/g; let passResult = pass.match(regPass); if(pass === ''){ passP.innerHTML = 'パスワードを入力して下さい'; }else if(passResult === null){ passP.innerHTML = '半角英数字10-18文字で入力して下さい'; }else{ passP.innerHTML = '入力完了'; passFlag = true; } } function pass2Check(){ let pass = document.getElementById('pass').value; let pass2 = document.getElementById('pass2').value; let pass2P = document.getElementById('pass2_p'); if(pass2 !== pass){ pass2P.innerHTML='パスワードをもう一度入力して下さい'; }else{ pass2P.innerHTML = '入力完了'; pass2Flag = true; } } let button = document.getElementById('button'); button.addEventListener('click',nameCheck,false); button.addEventListener('click',mailCheck,false); button.addEventListener('click',passCheck,false); button.addEventListener('click',pass2Check,false); function Check(){ if(nameFlag===true && mailFlag===true && passFlag===true && pass2Flag===true){ return true; }else{ return false; } } </script> </body> </html>

試したこと

ここに問題に対して試したことを記載してください。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

m.ts10806

2021/02/08 21:23 編集

要件通り動いてるならそれでいいのでは。 「とりあえず全部見て」ではやろうと思えばどこまででも出来てしまうし 好みや現場のルールもある。面倒でだれも手を出さないと思います。 観点を明示してください。
guest

回答2

0

ベストアンサー

おかしなところがあるので別の質問を」と別の回答で書いたのは私ですが、前の質問文中にあった明らかな間違いは既に直っているようです。

■ 動作の不具合

あるフィールドが、一度「入力完了」になってからユーザが再編集してチェックが通らない状態になったとき、「入力完了」状態から抜けることがありません。nameFlagなど何とかFlag変数がfalseに戻ることがないからです。各チェック関数で「入力完了」ではないときに 何とかFlag = false; する必要があります。

<input>要素にname属性がないと、フォームで値が送信されません。

メールアドレスの正規表現が厳しすぎて、規格に準拠したメールアドレスも拒絶してしまいます。たとえば、foo+bar@example.com は規格準拠です。

■ コーディングスタイルなど

インデントをきちんと揃えましょう。読みにくいです。

if の後にはスペースを1つ入れましょう。{ の前にスペースを1つ入れましょう。行末ではない } の後にスペースを1つ入れましょう。行末ではない , の後にスペースを1つ入れましょう。=== の前後にスペースを入れましょう。

イベントハンドラ属性 onsubmitaddEventLisetner() が混在していて統一感がありません。現在は、どちらかというと後者が推奨されています。

何とかCheck() 関数の結果は Check() でしか使わないのですから、何とかCheck() 関数はCheck()の中で呼べばよいです。何とかCheck()関数は return truereturn false をするように変更すると、Check() は以下のようになります。ボタンのclickイベントを使う必要はありません。

javascript

1function Check() { 2 const nameFlag = nameCheck(); 3 const mailFlag = mailCheck(); 4 const passFlag = passCheck(); 5 const pass2Flag = pass2Check(); 6 return nameFlag && mailFlag && passFlag && pass2Flag; 7}

<input>required 属性 と pattern 属性を使ったり、<input type=email> を使ったりするとコードを少々簡略化できます。

投稿2021/02/09 00:17

編集2021/02/09 06:33
int32_t

総合スコア21679

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

nekoma3

2021/02/09 15:28

ご丁寧にありがとうございます。 すごく参考になりました!
guest

0

パスワード再入力の機能がおかしいですね。
submit時のチェック処理より各要素ごとにinputで
チェックしたほうがスマートかもしれません

投稿2021/02/09 00:14

yambejp

総合スコア116661

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問