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

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

新規登録して質問してみよう
ただいま回答率
86.12%
JavaScript

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

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

解決済

入力回数が規定回数までなったらボタンが非表示になるようにしたい

Hello.world
Hello.world

総合スコア9

JavaScript

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

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

1回答

0グッド

0クリップ

261閲覧

投稿2022/10/22 13:23

追加ボタンを押すと"tuikanumber"に1足すようにし、"tuikanumber"が3以上になるとボタンが非表示になるように記述しているのですが上手くいきません。
処理の順番が間違っているのでしょうか?

html

<body> <div class="base container"> <div class="row"> <h1>アカウント登録</h1> </div> <div class="row"> <div class="card" id="card"> <div class="card-block"> <form id="form-area" class="form-inline" method="post"> <div class="form-group"> <div class="input-group"> <input id="namae" name="namae" type="text" class="form-control" placeholder="名前"> </div> </div> <div class="form-group"> <div class="input-group"> <input id="nickname" name="nickname" type="text" class="form-control" placeholder="ニックネーム"> </div> </div> <div class="form-group"> <input id="tuikaBtn" type="button" name="touroku" value="追加"> </div> </form> <div id="tuikaMoto"> </div> <div id="result"></div> </div> </div> </div> </div> </div> <div id="overLay"> </div>

JS

<script> let result = document.getElementById("result"); let tuikanumber = 0; let tuikaBtn = document.getElementById("tuikaBtn"); tuikaBtn.addEventListener('click',function() { let namae = document.getElementById("namae"); let nickname = document.getElementById("nickname"); if(namae.value == "" || nickname.value == "")return false; if(window.confirm( namae.value + "さん" + nickname.value + "を登録します。よろしいですか?" )) { return result.textContent = `${namae.value} ${nickname.value}`, window.alert( namae.value + "さん" + nickname.value + "にて登録しました。" ); tuikanumber++; } else { return false; } if(tuikanumber >= 3) { tuikaBtn.style.visibility = "hidden"; return false; } }); </script>

以下のような質問にはグッドを送りましょう

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

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

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

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

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

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

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

適切な質問に修正を依頼しましょう。

回答1

1

ベストアンサー

不具合の原因

  • tuikanumberのカウントがいっさい増えていません。

    • その理由は、tuikanumber++;のまえにreturnしているからです。
    • returnをすると、return以降のそのスコープの処理はいっさい実行されません。
    • ですのでしなければならない処理はreturnの前に書きましょう。
  • また、result.textContent = ...returnする意味がありませんので、returnなしでそのまま書いたらいいと思います。

  • またnamae.value, nickname.valueを何回も書いているのが気になりました。

    • 繰り返し利用する値は、定数や変数に格納しましょう。
  • if (window.confirm(namae.value + "さん" + nickname.value + "を登録します。よろしいですか?")) {...}が非常に可動性が悪いです。

    • 一旦定数や変数に格納して(例 const response = window.confirm(...)
    • if (response) {...}と書くことをおすすめします。
  • その他不要なところでreturnreturn false;をしていましたので、削除しました。

コード例

javascript

1const result = document.getElementById("result"); 2const tuikaBtn = document.getElementById("tuikaBtn"); 3let tuikanumber = 0; 4 5tuikaBtn.addEventListener('click', function() { 6 const namae = document.getElementById("namae"); 7 const nickname = document.getElementById("nickname"); 8 9 const namaeVal = namae.value; 10 const nicknameVal = nickname.value; 11 12 if (namaeVal == "" || nicknameVal == "") return; 13 14 const response = window.confirm(`${namaeVal}さん(${nicknameVal})を登録します。よろしいですか?`); 15 16 if (response) { 17 tuikanumber++; 18 result.textContent = `${namaeVal} ${nicknameVal}`; 19 window.alert(`${namaeVal}さん(${nicknameVal})にて登録しました。`); 20 } else { 21 return; 22 } 23 24 if (tuikanumber >= 3) { 25 tuikaBtn.style.visibility = "hidden"; 26 } 27});

投稿2022/10/22 14:40

編集2022/10/22 14:41
Cocode

総合スコア2096

退会済みユーザー👏を押しています

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

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

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

このような回答には修正を依頼しましょう。

回答へのコメント

Hello.world

2022/10/22 21:17

returnの仕組みをよく理解しないまま使っていたのが原因ですね。 丁寧に教えてくださりありがとうございました!

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

ただいまの回答率
86.12%

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

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

質問する

関連した質問

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

JavaScript

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

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。