このプログラムが思い通りにいかない

Question

###最終的な目標
金額を打ち込むと税込の金額が出るようなものにしたいです。今はその過程で、税率を決めるプログラムを作ってます。
###目指している動作
window.confilm[税込を計算します。税率は10%に設定してあります。税率を変えますか?]
OK>>>
_________window.prompt[税率を変更します。何%ですか?]
_________**0以上の数字だった場合>>>**その数字を税率に代入
_________**0未満の数字または数字でなかった場合>>>**window.prompt[0以上の数字にしてください。]
________________________________________________※繰り返し
**キャンセル>>>**window.alert[これからやる]
###実際のコード

var tax = 0.1;

if(window.confirm('税込を計算します。税率は' + tax * 100 +'%に設定してあります。税率を変えますか?')) {
    tax = parseInt(window.prompt('税率を変更します。何%ですか?')) / 100;
    while(! tax >= 0) {
        tax = parseInt(window.prompt('税率は0以上の数字のみにしてください。何%ですか?')) / 100;
    }
    window.alert('これからやる');
}   else {
    window.alert('これからやる');
}

###実際の動作

• 税率を0以上の数字で設定したのに、[税率を0以上の数字のみにしてください。]と出てくる。
• 最初のconfilmでキャンセルを押すと、[これからやる]とでるが、そのあと[税率を0以上の...]がでてくる。

###環境
FireFoxでやってます。VScodeで記述しました。

Comments

[mather]

「税率を0以上の数字で設定したのに」とは具体的にどんな値で試しましたか？

Answer 1

税率を0以上の数字で設定したのに、[税率を0以上の数字のみにしてください。]と出てくる。

! tax >= 0 は (! tax) >= 0 と解釈されます。 ! 1 は false で、 false >= 0 は true となります。これはJavaScriptの仕様です。
tax < 0 と書きましょう。

最初のconfilmでキャンセルを押すと、[これからやる]とでるが、そのあと[税率を0以上の...]がでてくる。

再現しません。 window.prompt の方でキャンセルしませんでしたか？

Comments

[suzunox]

tax < 0としたら、きちんと動作してくれました！！
しかし、taxが文字列だった場合、普通にスルーされてしまうのですが、どのように対処すべきでしょうか?
僕なりに、while(tax < 0 || typeof tax === 'string')と入力して実験してみたのですが、やってみたところ、[これからやる]もなぜかスルーされてしまいました。

[mather]

「スルーされてしまう」ではなく「〇〇が実行されない、表示されない」と正確に書いてください。
文字列の場合、 parseInt("hoge") が NaN になり、 NaN / 100 も NaN で、 NaN < 0 がfalseになるので while の中身は処理されません。
表示されないのは「これからやる」ではなくて、「数字のみにしてください」の方ではありませんか？

[suzunox]

ありがとうございます、parseInt(文字列)だった場合、NaNを返すんですね！
試しにwhile(tax < 0 || Number.isNaN(tax))と入力したらきちんと動作してくれました！！！

Answer 2

while文の条件を tax < 0　にしてみてはどうでしょうか。

Comments

[suzunox]

tax < 0としたら、きちんと動作してくれました！！
しかし、taxが文字列だった場合、普通にスルーされてしまうのですが、どのように対処すべきでしょうか?
僕なりに、while(tax < 0 || typeof tax === 'string')と入力して実験してみたのですが、やってみたところ、[これからやる]もなぜかスルーされてしまいました。

[stdio]

taxを事前に型変えてみればどうでしょうか？

[Tanukiandou]

ParseInt()は文字列で入力された文字列の数値も整数に変換します。
その上で文字列から数値への変換ができないときはNaNが帰ってくるみたいなので、
Tax ===NaNを加えてみてはいかがでしょう。

[suzunox]

tax === NaNでもできました。
ありがとうございます!

[suzunox]

すいません、後で試したんですが、while(tax < 0 || tax === NaN)だと、文字列を入力した際に、whileの処理をしなかったです。([0以上の数字にしてください。]のところをとばして、[これからやる]にいってしまった。)
while(tax < 0 || Number.isNaN(tax))だときちんと動作しました。

Answer 3

###分かったこと

• parseInt(文字列)はNaNを返す事。
• 論理式で、! 何か >= 0　は、!(何か >= 0)　ではなく、!(何か) >= 0と解釈されてしまうことで、0以上の数字以外にtrueを示すという事ではない。

###解決方法
(上のプログラムで)
while(tax < 0 || Number.isNaN(tax))とすること。