c - CS50 拼字游戏 | 代码只有在“QUESTION!”时才会出错!单词
问题描述
编程小伙伴。我希望每个人都没事。
我正在做 CS50 的拼字游戏练习,程序运行良好。
但是......当你输入“问题!” 或“问题?” 作为第一个单词,它不再将“Q”识别为字母,因此将其设为零。
难道我做错了什么?
我试图在将字母从单词分配到点之后立即打印输出,似乎那里有问题,但我不知道是什么。
#include <ctype.h>
#include <cs50.h>
#include <stdio.h>
#include <string.h>
// Points assigned to each letter of the alphabet
int POINTS[] = {1, 1, 3, 3, 3, 3, 2, 2, 1, 1, 4, 4, 2, 2, 4, 4, 1, 1, 8, 8, 5, 5, 1, 1, 3, 3, 1, 1, 1, 1, 3, 3, 10, 10, 1, 1, 1, 1, 1, 1, 1, 1, 4, 4, 4, 4, 8, 8, 4, 4, 10, 10, 0};
char LETTERS[] = {'A', 'a', 'B', 'b', 'C', 'c', 'D', 'd', 'E', 'e', 'F', 'f', 'G', 'g', 'H', 'h', 'I', 'i', 'J', 'j', 'K', 'k', 'L', 'l', 'M', 'm', 'N', 'n', 'O', 'o', 'P', 'p', 'Q', 'q', 'R', 'r', 'S', 's', 'T', 't', 'U', 'u', 'V', 'v', 'W', 'w', 'X', 'x', 'Y', 'y', 'Z', 'z', '\0'};
int compute_score(string word);
int main(void)
{
int lcounter = 0;
int counter;
int index1;
int index2;
// Get input words from both players
string word1 = get_string("Player 1: ");
string word2 = get_string("Player 2: ");
// FIND LENGTH OF WORD STRINGS
int length_word1 = strlen(word1);
int length_word2 = strlen(word2);
// FIND CHARS OF STRINGS IN ARRAY AND ASSIGN VALUE
string cword1 = word1;
string cword2 = word2;
int word1_counter = length_word1 - 1;
int word2_counter = length_word2 - 1;
int w1index[word1_counter];
int w2index[word2_counter];
for (counter=0; counter < 53; counter ++)
{
if (cword1[word1_counter] == LETTERS[counter])
{
w1index[word1_counter] = counter;
word1_counter--;
counter = -1;
}
else if (word1_counter < 0)
{
counter = 54;
}
else if (counter == 52)
{
w1index[word1_counter] = 52;
word1_counter--;
counter = -1;
}
}
for (counter=0; counter < 53; counter ++)
{
if (cword2[word2_counter] == LETTERS[counter])
{
w2index[word2_counter] = counter;
word2_counter--;
counter = -1;
}
else if (word2_counter < 0)
{
counter = 54;
}
else if (counter == 52)
{
w2index[word2_counter] = 52;
word2_counter--;
counter = -1;
}
}
// Score both words
int score1;
int score2;
for (counter = 0; counter < length_word1; counter ++)
{
score1 = score1 + POINTS[w1index[counter]];
}
for (counter = 0; counter < length_word2; counter ++)
{
score2 = score2 + POINTS[w2index[counter]];
}
// TODO: Print the winner
if (score1 > score2)
{
printf("Player 1 wins!");
}
else if (score1 < score2)
{
printf("Player 2 wins!");
}
else if (score1 == score2)
{
printf("Tie!");
}
}
解决方案
好的,我想我理解了你的算法。
有两个错误。第一个在这里:
int word1_counter = length_word1 - 1;
int word2_counter = length_word2 - 1;
int w1index[word1_counter];
int w2index[word2_counter];
您没有为预期的大小保留足够的空间。所以第二个循环(填充的那个w2index
)实际上会在w1index
填充第一个成员时覆盖最后一个成员 from w2index
。将这些行更改为:
int word1_counter = length_word1 - 1;
int word2_counter = length_word2 - 1;
int w1index[word1_counter + 1];
int w2index[word2_counter + 1];
另一个错误在这里:
// Score both words
int score1;
int score2;
变量未初始化。改成:
// Score both words
int score1 = 0;
int score2 = 0;
变量lcounter
和index1
不index2
使用。删除它们。
这些名称word*_counter
不是很好,因为您实际上将它们用作索引...您可以使用它break
来退出循环并进一步分解/简化代码。但我看到你还在开始,所以很容易过于复杂......不过没关系。这需要时间。美妙之处在于,有了更多的经验,我们总是倾向于让这些事情变得更简单!
推荐阅读
- c - 带有管道的非终止 C 程序
- asp.net - 如何在 IIS 中为 asp.net 核心应用程序创建一个具有所有必要设置的新网站?
- arrays - 计算二维数组中从 arr[0,0] 到 arr[N,M] 的路径数,其中只能向右和向下移动
- keras - 使用 opencv 将 RGB 转为 GREY 移除 1 个通道
- mongodb - Robo 3T MongoDB 不是主要的
- java - 在可运行 jar 中包含 src/main/resources 中的文件
- lotus-notes - 防止 NotesDocument 在打开时寻找冗余服务器
- laravel - 完全自定义 Laravel Nova 界面
- python - 无法弄清楚如何在 JSON 中进一步展平嵌套以输出到 CSV
- asp.net-web-api2 - WebAPI 2 方法对一个方法说 404,但不是另一种方法