CS50 拼字游戏 - 我自己的实验室方法

cs50 scrabble- my own approach to the lab

提问人:h.jr11 提问时间:10/30/2023 更新时间:11/1/2023 访问量:31

问:

所以我在 CS50 中做我的拼字游戏实验室,

我已经查看了发布的其他问题,但似乎我正在尝试一种与其他人不同的方法,或者至少我还没有找到。

我创建了一个函数来转换为大写,我称之为大写,并且它编译了,但现在我要回去尝试构建“compute_score”函数。我不能让两者一起工作。我不知道我做错了什么。我尝试了几行我认为可行的不同代码。我需要在“compute_score”函数上方写“大写”函数吗?还是我试图错误地调用“大写”?我确定那里的某个地方存在转换错误,由于我目前遇到的错误,我还无法解决?它说,当它在上面的代码中声明时,我有未声明的标识符,而且代码的数学行也不正确,我想我正在尝试调用数组中的数组,这似乎是错误的。但是,由于我无法克服“未声明”的方面,因此我还无法解决数学问题。“debug50”对我没有帮助,因为我无法编译代码。

这是我目前所拥有的完整代码......

#include <ctype.h>
#include <cs50.h>
#include <stdio.h>
#include <string.h>

// Points assigned to each letter of the alphabet
int POINTS[] = {1, 3, 3, 2, 1, 4, 2, 4, 1, 8, 5, 1, 3, 1, 1, 3, 10, 1, 1, 1, 1, 4, 4, 8, 4, 10};

int compute_score(string word);
string uppercase(string word);

int main(void)
{
// Get input words from both players

string word1 = get_string("Player 1: ");
string word2 = get_string("Player 2: ");

// Score both words
int score1 = compute_score(word1);
int score2 = compute_score(word2);

// TODO: Print the winner
if (score1 > score2)
{
    printf("Player 1 wins!\n");
}
else if (score1 == score2)
{
    printf("Tie!\n");
}
else
{
    printf("Player 2 wins!\n");
}
}


int compute_score(string word)
{
// TODO: Compute and return score for string
int i;
string s = uppercase(word[i]);// <-- it givees me undeclared identifier
int score = 0;

if (word[i] >= 65 && word[i] <= 90 ) // <-- again it gives me undeclared but it's declared above
{
   score = score += POINTS[word[i]]; //<-- here I'm getting an error about the array being a "char" type
                                     //how to do I properly convert it to the int that I need it to be
}
return 0;
}


string uppercase(string word)
{
string s = word;
for (int i = 0; i < strlen(s); i++)
{
    printf("%c", toupper(s[i]));
}
return 0;
}

另外,我确信我可能遗漏了其他错误,所以请随时启发我。我只是无法通过我被卡住的部分。任何帮助将不胜感激。

数组 数学 CS50 拼字游戏

评论


答:

0赞 DinoCoderSaurus 10/30/2023 #1

最根本的问题是将字符串 () 设置为字符 ()。
注意,IDE 编译器 () 给出 ,然后退出。
即使它确实编译了,它也是一个等待发生的运行时错误。 声明为 int,但它未初始化为值,因此会产生不可预知的结果。
string s = uppercase(word[i]);sword[i]makeerror: incompatible integer to pointer conversion passing 'char' to parameter of type 'string'....note: passing argument to parameter 'word' here string uppercase(string word);iword[i]

既然程序需要遍历每个字符来计算分数,为什么要重新发明轮子(即)什么时候会做这项工作呢?uppercasetoupper

评论

0赞 h.jr11 11/1/2023
谢谢你,你的帮助让我达到了现在可以编译代码的地步。
0赞 chux - Reinstate Monica 11/1/2023 #2

问题包括:

  • 在需要时通过。@DinoCoderSaurus 未使用,因此请使用指针调用并让它更新字符串charstringsuppercase()word
  // string s = uppercase(word[i]); // <-- it givees me undeclared identifier
  uppercase(word);
  • 缺少序列点。简化代码
// score = score += 
score += POINTS[(unsigned char) word[i]];
  • 无需重新计算字符串长度乘以。简化。
  // for (int i = 0; i < strlen(s); i++) {
  for (int i = 0; word[i]; i++) {
  • 不要打印大小写转换。分配结果
    // printf("%c", toupper(s[i]));
    word[i] = toupper((unsigned char) word[i]);
  • 节省空间,因为值只有 1 到 10。
//int POINTS[] = {1, 3, 3, 2, 1, 4, 2, 4, 1, 8, 5, 1, 3, 1, 1, 3, 10, 1, 1, 1, 1, 4, 4, 8, 4, 10};
const signed char POINTS[] = {1, 3, 3, 2, 1, 4, 2, 4, 1, 8, 5, 1, 3, 1, 1, 3, 10, 1, 1, 1, 1, 4, 4, 8, 4, 10};
  • 使用字符常量可更清晰。
  // if (word[i] >= 65 && word[i] <= 90)
  if (word[i] >= 'A' && word[i] <= 'Z')
  • 形成一个 0 碱基索引。减法结果为 类型 。int
    score += POINTS[word[i] - 'A'];
  • 代码需要循环并返回分数
  // if (word[i] >= 'A' && word[i] <= 'Z') {
  //   score += POINTS[ word[i] - 'A'];
  // }
  // return 0;

  for (int i = 0; word[i]; i++) {
    if (word[i] >= 'A' && word[i] <= 'Z') {
      score += POINTS[ word[i] - 'A'];
    }
  }
  return score;

例如重写也进行大写转换。compute_score()

int compute_score(string word) {
  int score = 0;
  while (*word) {
    char letter = *word;
    if (letter >= 'a' && letter <= 'z') {
      letter += 'A' - 'a';
    }
    if (letter >= 'A' && letter <= 'Z') {
      score += POINTS[letter - 'A'];
    }
    word++;
  }
  return score;
}