我在执行此javascript程序时遇到问题

我在执行此javascript程序时遇到问题,javascript,Javascript,每次运行程序并输入桩号和圈号时,我都会得到一个TypeError:piles[pileChoice]是未定义的错误。我试过几次调试它,但仍然无法使它正常工作 var piles = [ {name: 'Pile A', circles: 'ooooo'}, {name: 'Pile B', circles: 'ooooo'}, {name: 'Pile C', circles: 'ooooo'} ]; function boardPrint(){ console.log("NI

每次运行程序并输入桩号和圈号时,我都会得到一个TypeError:piles[pileChoice]是未定义的错误。我试过几次调试它,但仍然无法使它正常工作

var piles = [
  {name: 'Pile A', circles: 'ooooo'},
  {name: 'Pile B', circles: 'ooooo'},
  {name: 'Pile C', circles: 'ooooo'}
];

function boardPrint(){
  console.log("NIM");
  for(var i = 0; i < piles.length; i++) {
    console.log(piles[i].name + ": " + piles[i].circles);
  }
}

function getUserInput(){
  return prompt("Enter the letter for the pile (A-C) and the number of stones you want to remove (1-5). Example: A3").toLowerCase();
}

function userMove(){
  var pileIdx = 0;
  var valid = false;
  var numToRemove = 0;

  while(!valid) {
    var gameIns = getUserInput(); // This will now get called multiple times until user enters valid input
    var pileChoice = gameIns[0]; // This makes 'A' turn into 'a', which makes further logic easier.

    // I rebuilt this part of the function to be a bit cleaner and to show you how switch statements could be used
    switch(pileChoice){
      case 'a':
      pileIdx = 0;
      valid = true;
      break;
      case 'b':
      pileIdx = 1;
      valid = true;
      break;
      case 'c':
      pileIdx = 2;
      valid = true;
      break;
      default:
      alert('Error! Invalid input.');
    }
    numToRemove = Math.min(gameIns[1],piles[pileChoice].circles.length); // This way, they can't select a number that is greater than the number remaining in the pile.
  }

  piles[pileIdx].circles = piles[pileIdx].circles.slice(numToRemove);
}

function computerMove(move){
  // Task 1: pick a pile

  var pileIdx = 0;

  if(piles[0].circles.length > 0) { // tests for whether there are circles left in pile A
    piles[0].circles = piles[0].circles.slice(pileIdx);  // do something

  } else if(piles[1].circles.length > 0) {
    piles[1].circles = piles[1].circles.slice(pileIdx);  // do something

  } else if(piles[2].circles.length > 0) {
    piles[2].circles = piles[2].circles.slice(pileIdx);  // do something
  }

  // Task 2: pick a number to remove from the pile

  // Optional: see how many piles are left and base your number to remove on that
  //var pilesCount = 0;

  // [some logic for counting piles]

  // Otherwise, just remove all that are remaining from a pile
  //var numToRemove = 0;

  if (pilesCount > 1){
    // select a number to remove
  }
  else {
    // select another number to remove
  }

  piles[pileIdx].circles = piles[pileIdx].circles.slice(numToRemove);
}

while(true) {
  boardPrint();
  userMove();

  if (boardEmpty() === true) {
    boardPrint();
    console.log("You win!");
    break;
  }

  boardPrint();
  computerMove();

  if (boardEmpty() === true) {
    boardPrint();
    console.log("Computer wins!");
    break;
  }
}

function boardEmpty() {
  // Check if the board is empty
}

我认为问题在于:

你有

var piles = [
  {name: 'Pile A', circles: 'ooooo'},
  {name: 'Pile B', circles: 'ooooo'},
  {name: 'Pile C', circles: 'ooooo'}
];
后来

numToRemove = Math.min(gameIns[1],piles[pileChoice].circles.length);
我想你的意思是让它成为一堆[皮莱德]?pileChoice将是一个字母,并且您有一个对象数组,因此您需要使用in index访问正确的对象。

在您的userMove函数中,在while循环结束时,您尝试使用以下行设置numToRemove:

numToRemove = Math.min(gameIns[1],piles[pileChoice].circles.length); // This way, they can't select a number that is greater than the number remaining in the pile.

但PileChoice不是“a”、“b”就是“c”,对吧?这就是为什么你在你的交换机里计算了pileIdx。您需要改用它,因为您定义桩的方式无法用“a”、“b”或“c”索引。

您试图引用桩[Pileechoice],如游戏[0]中的桩['a']、桩['b']等,但桩数组的格式不是这样的,因为桩是一个对象数组

您可以尝试重新考虑对桩建模的方式,或者尝试以下解决方法来获取圆。长度:

// define pileCircleCount
var pileCircleCount;

// go over your piles array
piles.forEach(function(pile) {

 // get the pile name value and retrieve the last char which is A/B/C
 // and convert it to lowercase
 var pileId = pile.name.substr(pile.name.length - 1).toLowerCase();

 // now match it up with the pileChoice, if a match is found get its
 // circle length and set it to the pileCircleCount
 if(pileId === pileChoice) {
   pileCircleCount = pile.circles.length;
 }
});

numToRemove = Math.min(gameIns[1], pileCircleCount); 
gameIns[0]只会给出提示响应的第一个字符。提示返回一个字符串。在字符串上运行数组表示法时,它类似于String.charAt。另外,numToRemove=Math.mingameIns[1],piles[pileChoice].circles.length;,有什么意义;?一个数字的Math.min无论如何都将是该数字。在piles[pileChoice]前面的行中添加此控制台。logpileChoice,piles;这可以帮助您更好地了解出了什么问题,您可能需要将其更新为piles[pileIdx]