質問をすることでしか得られない、回答やアドバイスがある。

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

ただいまの
回答率

89.23%

フォームの項目チェックで、未入力→その後入力した際にエラーの文字が消えてくれない

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 3,187

yuzuka

score 7

前提・実現したいこと

フォームの項目が未入力の時に送信したらその箇所に対してエラーの文字が出るようにしたいと思っています。
また、メールアドレスと名前の欄、両方にエラーの文字が出ていたとして、どちらかだけを入力して送信した際、送信は出来ないのですが、入力した箇所のエラー文字は消える。
という動きにしたいです。

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

メールアドレス、名前の項目を未入力の状態にして送信→未入力の箇所にエラーの文字が出る。
メールアドレスだけを入力して、もう一度送信を押すと、メールアドレスは入力されているため、エラーの文字が消えます。
ですが、同様の操作を名前の箇所で行うとエラーの文字が消えてくれません。
つまり、
メルアド、名前の項目共に未入力→両方共エラー文字が出る→名前の項目のみを入力して送信→名前の項目のエラーが消えてくれない。
という問題が発生しております。

該当のソースコード

<form id="register" action="#">
  <th>メールアドレス<span id="mailerror"></span></th><br>
   <input type="text" name="mail"><br><br>

  <th>名前<span id="nameerror"></span></th><br>
   <input type="text" name="namae"><br><br>

  <input type="submit" value="送信">
</form>
var mailE = '  (メールアドレスを入力して下さい)';
var nameE = '  (名前を入力して下さい)';
document.getElementById('register').onsubmit = function() {
  var mailCheck = document.getElementById('register').mail.value;
  var nameCheck = document.getElementById('register').namae.value;
  //入力してあればtrue 未入力ならfalse
  var MC = jg(mailCheck);
  var NC = jg(nameCheck);

 if ( !MC && !NC ) {
   document.getElementById('mailerror').textContent = mailE;
   document.getElementById('nameerror').textContent = nameE;
   return false;
 }

 if ( !MC ) {
   document.getElementById('mailerror').textContent = mailE;
   return false;
 } else {
   document.getElementById('mailerror').textContent = "";
 }
  if ( !NC ) {
   document.getElementById('nameerror').textContent = nameE;
   return false;
 } else {
   document.getElementById('nameerror').textContent = '';
 }
}
function jg(mn) {
 var mn;
  if ( mn.length != 0 ) {
    mn = true;
    return mn;
  } else {
    mn = false;
    return mn;
  }
}

JavaScriptの

if ( !MC ) {
   document.getElementById('mailerror').textContent = mailE;
   return false;
 } else {
   document.getElementById('mailerror').textContent = "";
 }
  if ( !NC ) {
   document.getElementById('nameerror').textContent = nameE;
   return false;
 } else {
   document.getElementById('nameerror').textContent = '';
 }


ここのif文の上の方しか動かないようなのですが、それの理由がわかりません。

これの解決方法をご教示いただきたいと思っております。
よろしくお願い致します。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

関数内で return false; とすると、それ以降の処理は行われません。なので、変数に状況を入れておいて最後にそれを return すると良いと思います。

var mailE = '  (メールアドレスを入力して下さい)';
var nameE = '  (名前を入力して下さい)';
document.getElementById('register').onsubmit = function() {
  var mailCheck = document.getElementById('register').mail.value;
  var nameCheck = document.getElementById('register').namae.value;
  //入力してあればtrue 未入力ならfalse
  var MC = jg(mailCheck);
  var NC = jg(nameCheck);
  var flag = true; // 追加

 if ( !MC && !NC ) { // 多分無くてもよい。
   document.getElementById('mailerror').textContent = mailE;
   document.getElementById('nameerror').textContent = nameE;
   flag = false; // 変更
 }

 if ( !MC ) {
   document.getElementById('mailerror').textContent = mailE;
   flag = false; // 変更
 } else {
   document.getElementById('mailerror').textContent = "";
 }
  if ( !NC ) {
   document.getElementById('nameerror').textContent = nameE;
   flag = false; // 変更
 } else {
   document.getElementById('nameerror').textContent = '';
 }
 return flag; // 追加
}
function jg(mn) {
 var mn;
  if ( mn.length != 0 ) {
    mn = true;
    return mn;
  } else {
    mn = false;
    return mn;
  }
}

動くサンプル:https://jsfiddle.net/0ru37knt/

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/07/29 16:28

    ご回答頂きありがとうございます。
    大変よく分かりました!
    ありがとうございました!!

    キャンセル

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

  • ただいまの回答率 89.23%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る