2

我正在开发一个刽子手程序，我想在每次猜测后向用户输出状态报告。当我使用类时，我需要使用“朋友”关键字。我对班级和朋友的概念很满意，但是我正在努力在我的程序中正确实现它。

主要问题是没有计算元音的数量，并且用户可以尝试的字母数量没有更新。每次都显示完整的字母表。

我的程序代码很丰富，我不打算在这里全部发布。我遇到的问题是功能remainingLetters()；和vowelCount()。我附上了相关的代码片段。这段代码显然不会编译。我只是希望有人可能会看到我错过的明显错误。

**Hangman.h:**
{
public:
...
int vowelCount()const; //to be implemented
char* remainingLetters()const;//to be implemented
friend ostream& operator<< (ostream &out, Hangman &game);

private:
...
int vowel_count;
char secretWord[MAXWORDLENGTH];
char* remaining_letters;
}

**hangman.cpp:**
{
...
int Hangman::vowelCount()const
{
  int vowel_count = 0;
  int i;
  secretWord[i];

  for(i=0; i<strlen(secretWord); i++)
  {


if(secretWord[i]=='a'||secretWord[i]=='e'||secretWord[i]=='i'||secretWord[i]=='o'||secretWord[i]=='u')
    {
      vowel_count++;
    }
  }
  return vowel_count;
}

char* Hangman::remainingLetters()const
{
  char alphabet[26]={'a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z'};
  char *remaining_letters=new char[ALPHABETSIZE];

  for(int i=0; i<strlen(secretWord); i++)
  {
  for(int j=0; j<ALPHABETSIZE; j++)
    {
       if(secretWord[i]!=alphabet[j])
       {
     remaining_letters[j]=alphabet[j];
       }
    }
  } 
 return remaining_letters;
}

ostream& operator<< (ostream &out, Hangman &game)
{
    out << "Number of guesses remaining is: " << game.numGuessesAllowed-game.numWrongGuesses << endl
     << "Number of incorrect guesses so far is: "<<game.numWrongGuesses <<endl
     << "Letters remaining are: "<<game.remaining_letters <<endl
     << "Hint: The secret word contains "<<game.vowel_count <<" vowels"<<endl;

    return out;
}
}

**main.cpp**
{
...
 cout << game;
...
return 0;
}
4

2 回答 2

2

您需要调用函数vowelCount()并remainingLetters()计算相应的变量（以便您可以输出它们）。

此外，char *remaining_letters=new char[ALPHABETSIZE];将为字母分配内存并用 0 初始化每个位置——这也恰好是字符串终止值。所以如果第一个位置（字母'a'）没有设置，out<<game.remaining_letters就不会输出任何东西

此外，该函数vowelCount()定义了隐藏类中的局部变量vowel_count（成员变量），因此成员变量不会被更新（除非您将返回值显式分配vowelCount()给该成员变量）我建议您删除局部变量声明从vowelCount()函数中，它将使用成员变量，这是您所需要的

上一段也适用于成员变量remaining_letters

还有一件事：分配的内存new不会自动释放，一旦你从函数调用返回，分配的内存就会丢失（因为没有办法再次访问它，但它仍然使用内存），除非你分配它当函数返回时。您需要将每个new[]呼叫与其对应的delete[]呼叫相匹配。更好的是：在类的构造函数中分配内存并在成员变量的析构函数中删除它

注意：此逻辑中还有另一个错误：

for(int i=0; i<strlen(secretWord); i++) 
{ 
  for(int j=0; j<ALPHABETSIZE; j++) 
  { 
    if(secretWord[i]!=alphabet[j]) 
    { 
      remaining_letters[j]=alphabet[j]; 
    } 
  } 
}

您在字母表中循环，每次密码中的当前字母不匹配时，您分配给remaining_letters. 每次都会发生这种分配，因为字母表中会有一个字母与当前的 secretWord 字母不匹配。

要解决此问题，您需要执行以下操作（注意循环的反转）：

int secret_len = strlen(secretWord); // cache length, so we don't recalculate it
                                     // every time: secretWord does not change
for(int j=0; j<ALPHABETSIZE; j++) 
{ 
  bool found = false; // assume the current ABC letter is not in secretWord
  for(int i=0; i<secret_len; i++) 
  { 
    if(secretWord[i]!=alphabet[j]) 
    { 
      found = true; // but it was
    } 
  } 
  if (!found) // if it wasn't
  {
    remaining_letters[j]=alphabet[j]; 
  }
}
于 2012-04-11T17:37:55.933 回答
1

您的主要问题是您没有在班级成员中保留元音计数vowel_count。您将它存储在局部变量中vowel_count并返回它，但类成员永远不会更新。

于 2012-04-11T17:38:17.997 回答