Javascript 正しいループ処理

Question

promptへ文字列を入力させ、
タイプミスした場合は、最初のif文へループするようにしたいです。
下記のように自分で書いてみたのですが、そのようには動かないので（ループしない）、
アドバイスいただけますと助かります。

function startPrompt(){
 startPrompt = prompt('"yes" or "no"を入力してください');
 startPrompt = startPrompt.toLowerCase();
 while (true){
 if ( startPrompt === 'yes'){
  goShopping()
 }else if (startPrompt === 'no'){
  alert('さようなら')
  break;
 }else {
  return true;
 }

Answer 1

苦戦してますね。
このくらいのコード量ならば必要性は感じないかもしれませんが、
機能を関数で分割するとパッと見て分かりやすくなりますよ。

function startPrompt(){
  let ans;
  while(true) {
    ans = getCommand();
    if (ans == '') contine;
    if (ans == 'yes') {
      goShopping();
    } else {
      alert('さようなら');
    }
    break;
  }
}

function getCommand() {
  const cmd = prompt('"yes" or "no"を入力してください');
  let ret = cmd.toLowerCase();
  if (ret != 'yes' && ret != 'no') ret = '';
  return ret;
}

Comments

[hiro_ike]

ありがとうございます。
初心者なため、大変助かります。
こういう書き方もあるのですね、確かに理解しやすいし、見やすいです。
勉強になりました。

[takasima20]

(･∀･)b

Answer 2

間違いがたくさんあります。

まず、startPromptをfunctionとして定義したのに、
その中でstartPromptを変数？として使っています。
別の名前にしなければなりません。
たとえばanswerとしましょう。

そしてanswerという変数を使うには、
var answer = prompt('"yes" or "no"を入力してください');
などのようにしなければなりません。
この場合varではなくletでもよいです。
varとletの違いは後で調べてください。ついでにconstも。

goShoppingを呼ぶのはいいですが、goShoppingから帰ってきた後の処理が抜けているようです。
このままではwhileでループして無限にgoSHoppingを呼ぶことになります。

answerが'yes'でも'no'でもなかったとき、returnしてしまっています。
それでは当然入力のやり直しにはなりません。

このループが、'yes'でも'no'でもなかったときに入力をやり直すようにしたいなら、
promptの呼び出しはwhileループの中に無ければならないですね。
そうでないとanswerは一生変化しないので無限ループになります。

whileのブロック{を閉じる}がありません。

えーと、とりあえず、このくらいかな？

Answer 3

文法エラーにはなってないのですよね？であれば、質問文へのコピペに失敗して途中で切れてますよ。

本題：
おちついて、処理を最初から追ってみましょう。声に出して読むのも有効です。
どんな処理を繰り返しているでしょうか？promptは繰り返さなくて良いのでしょうか？
"yes"の場合にどうしたいのでしたっけ？
"no"の場合にどうしたいのでしたっけ？
"yes"でも"no"でもない場合にどうしたいのでしたっけ？

追加のアドバイス：
・関数名と同名の変数を使うと混乱の元です。
・varとかletとかを学びましょう。

Answer 4

ご丁寧にありがとうございます。
ご指摘いただいた箇所を下記のように訂正したら、やりたかったことは実現できました。

function startPrompt(){
while (true){
let answer = prompt('"yes" or "no"を入力してください');
answer = answer.toLowerCase();
if ( answer === 'yes){
goShopping()
}else if (answer === 'no'){
alert('さようなら')
break;
}else {
startPrompt ();
}
}}

1. 入力のやり直しは、else で再度startPrompt ()を呼ぶで良いのでしょうか？
2. goShopping 関数から返ってきたときは、何を書くか決めてませんがgoShopping関数内でstartPrompt() に行くように書こうと思ってました。(初心者なので、どうすべきが最適なのか分かっておりませんので、ご指摘いただけますと幸いです。)

Comments

[itagagaki]

1) 2) いずれも、それでは再帰呼び出しになってしまいます。
関数について基本を学びましょう。