Question

我是C的新手，我的阵列出了问题。该应用程序是一个anagram游戏，应该将用户猜测与正确的单词进行比较。我试图记录所有不正确的答案，以及两个单独的数组中的答案的正确版本。但是，在程序结束时，当我尝试打印所有不正确的猜测和正确的版本时，它只打印数组中的第一个术语。我认为它可能是由数组引起的问题，因为它们是持有字符串，所以在C中它实际上是一个数组，我认为？为什么我的代码只打印数组中的第一个术语？

（代码更多，但这只是关键部分，我认为这是唯一需要查看的部分）

char correctWord[20];
char anagramWord[20];
int  guesses, score;
char* incorrectGuess[20];
char* correctVersion[20];

void userGuess(){
char userWordGuess[20];

printf("Anagram: ");
printf(anagramWord);
printf("\n Your Guess: ");
scanf("%s",userWordGuess); //Reads in user input
strtok(correctWord, "\n");
guesses++;

if(strcmp(userWordGuess, correctWord) == 0){
    printf("Congratulations, you guessed correctly! (Please wait for the 

next question...)\n");
        score++;
        Sleep(1600);
        system("cls");
    }else{
        printf("Incorrect, try harder!(Please wait for the next question...) \n");
        Sleep(1600);
        system("cls");
        int i = 0;
        incorrectGuess[i]=(userWordGuess);
        correctVersion[i]=(correctWord);
        i++;
    }
}

void finalScore(){
    int i;
    system("cls");
    int percentage = ((score/guesses) * 100);
    printf("Congratulations - Game Complete!");
    printf("\n Guesses: %d", guesses);
    printf("\n Score: %d", score);
    printf("\n Percentage Correct: %d", percentage);
    int numberOfIncorrect = (guesses-score);
    for(i=0;i<=numberOfIncorrect;i++){
        printf(incorrectGuess[i]);
        printf(correctVersion[i]);
    }
    getch();
}

Answer 1

在此代码中，您没有使用任何值初始化guesses和score，并尝试递增它们。你应该

int  guesses = 0 , score = 0 ;

然后，在你的else块中

else
{
    printf("Incorrect, try harder!(Please wait for the next question...) \n");
    Sleep(1600);
    system("cls");
    int i = 0;                             // you initialize i to 0 every time
    incorrectGuess[i]=(userWordGuess);
    correctVersion[i]=(correctWord);
    i++;
}

在注释行中，您每次都会i初始化为0。

Answer 2

首先，变量guesses和score并非初始化。你需要像

这样的东西

int  guesses=0, score=0;

在这一行中

incorrectGuess[i]=(userWordGuess);
correctVersion[i]=(correctWord);

你不复制字符串。你应该使用

incorrectGuess[i]=strdup(userWordGuess);
correctVersion[i]=strdup(correctWord);

代替。或者您可以像这样使用malloc和strcpy

incorrectGuess[i] = malloc(strlen(userWordGuess)+1);
strcpy(incorrectGuess[i], userWordGuess);
correctVersion[i] = malloc(strlen(correctWord)+1);
strcpy(correctVersion[i], correctWord);

在这种情况下，最后你需要释放分配的内存，如

free(incorrectGuess[i]);
free(correctVersion[i]);

Answer 3

此：

incorrectGuess[i]=(userWordGuess);

错误，不复制字符串。您所做的只是存储一个指向包含当前猜测的数组的指针，并且该数组始终是相同的。

您需要将实际字符复制到新分配的内存中才能记住它们。你也可以把它变成2D数组：

char incorrectGuess[20][100];

只是strcpy()当前的猜测。请注意不要覆盖，如果有，请使用snprintf()。

C程序调用包含循环内字符串的数组

3 个答案: