Javascript Question posted in
A very good W3school tutorial can be found here.

Javascript – prompt + loop on array – how to ask to only re-response the wrongly answered question and not all the loop?

Objectives :

  1. to have the user type the shown words in the input fields.
  2. to check if the typed word are exact copy of the ones that were asked.
  3. if yes, then go on to the next one
  4. if no, then ask to type again the wrongly typed word

All the words are included in an array.

What works so far :

objectives 1 to 3 work great !

What doesn’t work :

Objective 4 partially works. The problem is that it restarts the whole loop instead of keeping track of the progress so far and only restart from the wrongly answered question.

The code :

 wordList = ["Cachalot", "Pétunia", "Serviette"]
 score = 0
  for (let i=0; i<wordList.length; i++){
        let typedWord = prompt("Veuillez recopier le mot suivant : " + wordList[i])
        if(typedWord===wordList[i]){
            score+=1
            console.log(score)
        } else{
            console.log ("merci de recommencer")
            typeSmart()
        }

    }
}typeSmart()

I know calling the function is due to start the whole process again. But I’m a bit clueless as how to do otherwise. should I use a while instead or in complement of the current for loop ?

Tags:

2

Answers


  1. 0

    Add --i; in your else statement. This will keep the value of i for the for loop the same, untill the user enters the correct answer. So it will keep prompting the same prompt, until the user enters rhe correct answer, which serves your purpose.

    wordList = ["Cachalot", "Pétunia", "Serviette"]
 score = 0
  for (let i=0; i<wordList.length; i++){
        let typedWord = prompt("Veuillez recopier le mot suivant : " + wordList[i])
        if(typedWord===wordList[i]){
            score+=1
            console.log(score)
        } else{
            console.log ("merci de recommencer")
            --i;
        }

    }
    Login or Signup to reply.
  2. 0

    Yes, use a while loop instead. As you don’t know how many times you’re going to be asking for the word (ie: looping), a while loop is more appropriate. for loops are typically better for when you know the number of times in advance you need to loop, which isn’t the case here.

    When the user gets the word correct, you can advance the index of the current word you’re looking at (in the below snippet I’ve called that currentWordIdx). Also, make sure you declare your variables with let or const (var is also an option but not preferred), otherwise they become global variables and may impact other parts of your code outside of your function unintentionally:

    function typeSmart() {
  const wordList = ["Cachalot", "Pétunia", "Serviette"];
  let score = 0;
  let currentWordIdx = 0;
  while(currentWordIdx < wordList.length) {
    const currentWord = wordList[currentWordIdx];
    const typedWord = prompt("Veuillez recopier le mot suivant : " + currentWord);
    if (typedWord === currentWord) {
      score += 1;
      currentWordIdx++; // advance to the next word
      console.log(score);
    } else {
      console.log("merci de recommencer");
    }
  }
}
typeSmart();
    Login or Signup to reply.
Please signup or login to give your own answer.

Back To Top
Search