javascriptの勉強中です

質問

javascriptについて勉強中で、switch文とif文を組み合わせようと考えています。

やりたいことは、数値を入力してもらい、入力した文字をnumberに表示します。
また、nyuryokuにキャンセルや入力されていないことを通知します。

ここまではできたのですが、入力の確認画面でキャンセルした場合「もう一度入力してください」とアラートと記述を行いたいのですが、trueが返っているのか数値が記載され、「ありがとうございます」の記述が行われます。

この場合はどうすれば入力確認のアラートを出すことができるのでしょうか。

下記をご確認ください。

<script>

function putin() {
var num = prompt('数値入力');
switch (num) {
case '': document.getElementById('nyuryoku').innerText =('入力してください。');
alert ('入力してください');
str = ('入力してください');
break;
case null: document.getElementById('nyuryoku').innerText =('キャンセルされました');
str = ('キャンセルされました');
break;
default: confirm('入力内容はこちらでよろしいですか？');
if (true) {
document.getElementById('number').innerText = num;
document.getElementById('nyuryoku').innerText =('ありがとうございます');
}
else {
alert('入力内容を確認しもう一度入力してください');
document.getElementById('nyuryoku').innerText('入力内容を確認しもう一度入力してください');
}
}
}
</script>

<a onclick="putin()">数値はここで入力が必要</a>
<br>
<span id="number">ここに数値</span>
<br>
<span id="nyuryoku"></span>

ヒントでもよいので、何卒よろしくお願いします。

回答No.2

Ogre7077 · ベストアンサー

confirm の戻り値を参照せずに if(true) としているので、
かならず「ありがとうございます」になってしまいます。
正しくは
var confirmResult = confirm(...);
if (confirmResult) { ... } else { ... }

余談ではありますが

JavaScript にて null/undefined/0/'' の比較は、面倒くさい問題が起きがちです。
今回の範囲では大丈夫なのですが、できれば以下の様にすることをオススメします。

if (num === null) { // 厳格に null と判定する
...
} else if (num === '') { // 厳格に 空の文字列と判定する
...
} else {
...
}

お礼

回答ありがとうございます。

はじめに自分で色々と試していた時に、
nullでも ' ' 空の文字列でもうまくいかなくてNaN等、全部試していました。

今回のこのJavaScriptには当てはまりませんが、今後作る際に
色々なケースでの判定を予め追加しておけばいいんですね。

確かに、参考にしたサイトでも
null/undefined/0/''
ここの質問や不具合が多かったです。

詳しく説明していただきありがとうございました。

回答No.1

maiko0318

なら

default:
if(confirm('入力内容はこちらでよろしいですか？')){
:
else

にしたらどうですか？

お礼

回答ありがとうございます。

defaultで先にconfirmを出すのではなく、if文にすればよかったんですね。
confirmが先に処理されていたからif文はalertしか読み込まれていなかったんですね。

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