进行CS50实验室时的分割故障（核心倾倒）错误

Question

当我输入 ./scrabble 并输入玩家 1 和 2 的单词时，结果是分段错误（核心已转储）。我查看了其他类似的解决方案，他们没有错误，那么我哪里出错了。

 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);
    
    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("player 2 Wins\n");
        }
        else if (score1 == score2)
        {
            printf("Tie\n");
        }
    }
    
    int compute_score(string word)
    {
        // TODO: Compute and return score for string
        int score= 0;
        for (int i = 0; i < strlen(word); i++)
        {
            if islower(word)
            {
                score = POINTS[(word[i])-65];
            }
            if isupper(word)
            {
                score = POINTS[(word[i])-97];
            }
        }
        return score;
    }

Answer 1

您正在从字符代码中减去错误的值，因为65是a的ASCII代码，而不是a。通过使用角色文字而不是ASCII代码来避免这样的问题。

islower（）和isuper（）的参数必须是单词word [i]的当前字符，而不是整个字符串Word。

您也不会增加分数，而是在循环中覆盖它。因此，得分将只是单词中的最后一个字母。使用+=添加到变量而不是=进行分配。

    int compute_score(string word)
    {
        int score= 0;
        for (int i = 0; i < strlen(word); i++)
        {
            if islower(word[i])
            {
                score += POINTS[(word[i])-'a'];
            }
            else if isupper(word[i])
            {
                score += POINTS[(word[i])-'A'];
            }
        }
        return score;
    }