Пройдите регистрацию для игры в 1d Battleship.Использование массива для записи предыдущих пользовательских вводов, затем перекрестная проверка их с текущим вводом

Question

Я начинающий программист.Я начал учить себя JavaScript.Я сделал рудиментарную линкорную игру.Проблема в том, что если пользователь входит в одно и то же место (если это попадание) 3 раза, линкор тонет.Чтобы избежать этого, я добавил массив «userchoices», чтобы записать вводимые пользователем данные, а затем перепроверить, повторяя цикл for.цикл for, в свою очередь, содержит оператор If, который должен предупредить пользователя, если он уже выстрелил в этом месте ранее.Проблема в том, что оператор if выполняется каждый раз.

Пожалуйста, просмотрите приведенный ниже код и предложите исправления.Спасибо.

var randomloc = Math.floor(Math.random() * 5);
var location1 = randomloc;
var location2 = location1 + 1;
var location3 = location2 + 1;
var guess;
var userchoices = [];
var hits = 0;
var guesses = 0;
var issunk = false;

function battleship() {
while(issunk == false)
{
    guess = prompt("Ready,Aim,Fire! (Enter a number 0-6):");

    console.log("users input = " + guess);

    if (guess == null)
        break;

    if (guess < 0 || guess > 6){
        alert("Please enter a valid cell number. No of guesses has been 
    incremented.")
    }
    else{
        guesses++;
        userchoices[guesses] = guess;
        console.log("users choices = " + userchoices);
        }

   /* for(var i = 0; i <= guesses; i++)
        {
            if(userchoices[guesses] = guess)
            console.log("you have already fired at this location");
        } */

    if (guess == location1 || guess == location2 || guess == location3){
        alert("Enemy Battleship HIT");
    hits = hits + 1;

    if (hits == 3){
        issunk = true;
        alert("Enemy battleship sunk")
    }
        }
    else{
        alert("You Missed");
    }
    }
    if (issunk){var stats = "you took " + guesses + " guesses to sink the battleship. You accuracy was " + (3/guesses);alert(stats);}
    else{alert("You Failed!"); issunk = false;}
    }

Эта часть вызывает ошибку

for(var i = 0; i<=guesses; i++)
{
if (userchoices[guesses] = guess){
console.log("you have fired at this location already");
}}

Оператор if должен выполняться только тогда, когда пользователь вводит номер сетки, по которой у него уже есть огонь,неважно, попал или не попал.

Answer 1

После столь необходимой помощи от Авина Кавиша и небольшого количества попыток самостоятельно я теперь могу представить ответ на свой вопрос будущим зрителям.

Редактировать: Больше похоже на мою финальную программу

function battleship() 
{
var guess; //Stores user's guess
var userchoices = []; //records user's guess until ship is sunk or user chickens out
var issunk = false; //status of ship
var hits = 0; //number of hits
var guesses = 0; //number of guesses
var randomloc = Math.floor(Math.random() * 5); //Random Number Generator
var location1 = randomloc; 
var location2 = location1 + 1;
var location3 = location2 + 1;

while(issunk == false)
{
    guess = prompt("Ready,Aim,Fire! (Enter a number 0-6):");
    console.log("users input = " + guess);

    if(guess == null) // If users presses 'OK' without entering anything or the 'Cancel' this would break the loop.
        break;

    if (guess < 0 || guess > 6){
        alert("Please enter a valid cell number. No of guesses has been incremented.");
guesses++; //Gotta punish the player.
    }
    else if (userchoices.includes(guess) == false) /*instead of doing what i did yo u 
can change this line to "else if (userchoices.includes(guess)) and then put the 
following oprations in its else clause. */
    {
        guesses++;
        userchoices[guesses] = guess;
        console.log("User choices = " + userchoices);

        if (guess == location1 || guess == location2 || guess == location3)
        {
            alert("Enemy Battleship HIT");
            hits = hits + 1;
            if (hits == 3)
            {
                issunk = true;
                alert("Enemy battleship sunk");
            }
        }
            else
            {
                alert("You Missed");
            }
    }
         else
        {
            alert("you have already fired at this location.")
        }
    if (issunk) //writing issunk == true is overkill
    {
        var stats = "you took " + guesses + " guesses to sink the battleship. You 
accuracy was " + (3/guesses);
        alert(stats);
    }
}
if(guess == null && issunk == false)
console.log("You failed");  //Humiliate the user for chickening out.
userchoices = []; //Empties the array so user can start over again without relaoding the page
issunk = false; //sets issunk to false for a new game
var randomloc = Math.floor(Math.random() * 5); //creates new random numbers for ship coordinates
}

2D версия 7X7 в ближайшее время.Опубликуем здесь.

Answer 2

Вы обращаетесь к массиву по неправильному индексу.Попробуйте userchoices[i] вместо userchoices[guesses].Также сравнение равенства выполняется с использованием 2 знаков равенства ==:

for(var i = 0; i<=guesses; i++)
{
  if (userchoices[i] == guess){
    console.log("you have fired at this location already");
  }
}

Это также может быть выражено как:

  if (userchoices.includes(guess)){
    console.log("you have fired at this location already");
  }

Также догадки следует увеличивать после добавления первого значения:

    else{
      userchoices[guesses] = guess;
      guesses++;
      console.log("users choices = " + userchoices);
    }

РЕДАКТИРОВАТЬ

Здесь есть логическая ошибка, поскольку вы проверяете массив для элемента после вставки его в массив, выполните проверку в операторе else перед вставкой элемента.Сочетая все вышеперечисленное:

else if (userchoices.includes(guess)){
  console.log("you have fired at this location already");
} else {
  userchoices[guesses] = guess;
  guesses++;
  console.log("users choices = " + userchoices);
}