C - scanf 在没有直接访问的情况下更改数组的值

Question

这是我第一次来 stackoverflow。我希望我的问题适合。 我们这个学期开始在大学里编写 C 语言。不幸的是，只有少数在线讲座。但我们仍然需要解决这些任务。

这次我们应该编写一种 Hang-Man。换句话说，猜一个隐藏的单词。 我有以下问题。我得到一个字符，但输入它后，谜语数组的内容发生了变化。如果我省略输入，它会起作用。我不明白为什么会发生这种情况，因为 scanf 实际上并没有访问谜语。 我自己不知道在这里做什么。我希望有人能告诉我代码有什么问题。

//sry some variables and texts are in german
char* createRiddle(char const* const str){ 

    int laenge = strlen(str);
    char temp[laenge+1];
    char *t = temp;
    strcpy(temp, str);
    int te = strcmp(temp, str);     
    if (te != 0){
    printf("ERROR: Bei der Speicherreservierung ist ein Fehler aufgetreten");
        exit(0);
    }
    int i;
    for (i=0; i < (int)strlen(temp);i++){
        if (str[i] > 65 && str[i] < 90){ //ASCII Großbuchstaben-Bereich prüfen 
            char verdeckt = '*';
            temp[i] = verdeckt;
        } else {
            temp[i] = str[i];
        }
    }

    return t;


 }

//----------------------------
int uncoverLetter(char *riddle, const char *solution, char letter){ 
    printf("RD3: %s\n",riddle);
    letter = toupper(letter);

    int i; 
    int treffer = 0;
    for (i=0; i < (int)strlen(solution); i++) {

        if (letter == solution[i]) { // Buchstabe im Wort?
            if (letter != riddle[i]) { //Buchstabe schon aufgedeckt?
                riddle[i] = solution[i];
                treffer = treffer + 1;
            }
        }

    }
    return treffer; 
}

//----------
int gamingLoop(const char* solution){ 

    int punkte;  //points
    printf("Lets GO!\n\n"); 
    char *riddle = createRiddle(solution);
    printf("Gesuchtes Wort: %s\n\n",riddle); //Word: *-******* ( = C-Compiler )
    int highscore = 0;

 while ((strcmp(riddle, solution)) != 0) {

        printf("RD1: %s\n",riddle);  //Test: What does Riddle look like?
        printf("Bitte geben Sie einen Buchstaben ein: "); // pls enter letter
        char eingabe; 
        scanf(" %c", &eingabe);  //-----!!Here is the point where things go wrong!!------
        printf("RD2: %s\n",riddle); //Test2
        int treffer = uncoverLetter(riddle, solution, eingabe);


        //----------- probably unimportant for the problem ----------------

        //Zufallszahl 

        int zufz = (rand() % 11) + 1;
        int ii = 1;
        for (ii=1; ii < 11 ; ii++){
            if ( zufz == ii) {
            punkte = zufz*100;
            }
        }
        //------------
        if (treffer != 0) {
            printf("Du hast %d richtige Treffer.\n", treffer);
            highscore = highscore + (treffer*punkte);
            printf("Punkte: %i\n\n", highscore);
        } else {
            printf("Du hast leider keinen Treffer.\n");
            highscore = highscore - punkte; 
            printf("Punkte: %d\n\n", highscore);
        }
        printf("%s\n\n",riddle);
    }

    return highscore;

}

输出： 抱歉没有图片，因为我没有 10 个代表 :(

链接：https://imgur.com/UIeltVR

//函数中的R3 unlockLetter

我强烈怀疑我犯了一个非常愚蠢的错误，但不幸的是我自己看不到/还看不到。 我期待着建议和帮助。

谢谢。

Answer 1

您的问题出在createRiddle，您在其中创建了*** 模式：

char* createRiddle(char const* const str){ 

    int laenge = strlen(str);
    char temp[laenge+1];
    char *t = temp;

    // ... create pattern ...

    return t;   
 }

您返回一个本地数组。 （t 只是数组 temp 的别名。）当函数退出时，该数组将超出范围，因此无效。

有几种可能的解决方案。

让调用者提供空间

传入一个调用者可以填充的数组：

void createRiddle(char *temp, char const* const str)
{ 
    // ... create pattern in temp ...
 }

然后这样称呼它：

char riddle[MAX];
createPattern(riddle, solution);

这里不需要返回数组，因为它是你提供的同一个数组，只是填充了。 （如果它使调用更容易，您可以返回它。您也可以返回错误代码。使用您的良好判断力。）

当然，函数和调用者需要就必须提供多少空间达成一致。 （可能是另一个函数参数或全局常量。）

动态分配内存

动态内存在堆上分配，保证不被其他人使用：

char *createRiddle(char const* const str)
{ 
    int laenge = strlen(str);
    char *temp = malloc(laenge + 1);

    // ... create pattern in temp ...

    return temp;
}

然后像这样使用它：

char *riddle = createRiddle(char const* const str);

// ... play the game ...

free(riddle);  // be nice and clean up

静态数组

制作数组static。

    static char temp[laenge+1];

这里，static 关键字意味着只有一个数组在调用之间保留其值。就好像您已将数组声明为函数外部的全局数组，但另外它的名称只有您的函数知道。

这是一个快速简单的解决方案，但是当您的函数是递归的或当您的代码的其他部分使用相同的函数时，它会失败。 （不过，您的游戏中可能并非如此。）