C 中的 Int 变量在未调用时发生变化

Question

提前道歉，我对此很陌生。

我正在尝试根据具有关联分值的 POINTS 数组对“拼字游戏”进行评分。

1. 我首先获取word1字符串的长度。
2. 然后，我创建一个 for 循环，该循环将小写字母的值减去 97（以达到索引 0）并将其添加到数组中。

您可以忽略其余代码，因为这就是问题所在。虽然 stringlength 变量仅定义一次，但我发现它在 for 循环的第二个循环期间发生了某种变化。四字母单词最初的 stringlength 为 4，但是 for 循环将其更改为 1第二次运行。

#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 main(void) {
    // Get input words from both players
    string word1 = get_string("Player 1: ");
    string word2 = get_string("Player 2: ");

    int stringlength = strlen(word1);

    int array[] = {};
    int array2[] = {};

    //Create an array with each letter
    for (int index = 0; index < stringlength; index++) {
        //THE BELOW LINE IS WHERE "STRINGLENGTH" CHANGES TO "1" ON THE SECOND LOOP
        array[index] = word1[index] - 97;
        array2[index] = POINTS[array[index]];
    }

    int score = 0;

    for (int index2 = 0; index2 < stringlength; index2++)
        score += array2[index2];

    printf("%i", stringlength);

    printf("\n");
}

我知道我可以创建一个辅助 stringlength 变量，但我知道这是糟糕的编程，我很想知道我做错了什么。

任何帮助将不胜感激。谢谢！

Answer 1

代码中存在多个问题：

• int array[] = {};是语法错误。数组定义必须至少有一个初始值设定项。在您的情况下，您必须定义长度为 stringlength 的 array：

  int 数组[字符串长度];
  

  您的编译器似乎接受定义并创建一个空数组，该数组的长度不足以存储字母分数：您在两个 for 循环中都有未定义的行为，访问超出其边界的数组元素。未定义的行为意味着任何事情都可能发生，包括您观察到的情况。

• 在 word1[index] - 97 中，您对 'a' 的 ASCII 值进行硬编码。这是不好的做法，你应该这样写：

  word1[索引] - 'a'
  

  但请注意，您在此表达式中还做出了 2 个更无声的假设：

  • 您假设 word1 仅包含小写字母，应在用户可以键入任何字符串时对其进行测试。
  • 您假设小写字母形式在执行字符集中是连续的，这对于 ASCII 来说是正确的，但 C 标准不保证。由于所有现代系统都使用 ASCII 作为标准字符，因此这个假设是可以的。

• printf("%i", stringlength); 输出字符串长度，而不是分数。

这是一个没有附加数组的修改版本：

#include <cs50.h>
#include <stdio.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 main(void) {
    // Get input words from both players
    string word1 = get_string("Player 1: ");
    string word2 = get_string("Player 2: ");

    //Compute the score for player 1
    int score1 = 0;
    for (int index = 0; word1[index] != '\0'; index++) {
        unsigned char c = word1[index];
        if (c >= 'a' && c <= 'z') {
            // assuming ASCII character set
            score1 += POINTS[c - 'a'];
        }
    }

    //Compute the score for player 2
    int score2 = 0;
    for (int index = 0; word2[index] != '\0'; index++) {
        unsigned char c = word2[index];
        if (c >= 'a' && c <= 'z') {
            // assuming ASCII character set
            score2 += POINTS[c - 'a'];
        }
    }
    printf("Player 1: %d\n", score1);
    printf("Player 2: %d\n", score2);
    return 0;
}

如果您了解函数，您可以编写一个函数 intcompute_score(const char *word) 以避免重复代码并提高可读性。