为什么我的 break / if 语句不起作用？

Question

我正在制作一个石头剪刀布游戏，在玩家玩够之前有重赛选项，如果玩家想停止玩，他只需键入“N”退出循环，但我的似乎不会工作，无论玩家输入如何，它都会不断重新匹配

这是我的代码：

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <time.h>

int regles(char joueur, char cpu) // rules of rock paper scissor "p" = rock, "f" = paper, c = scissor
{
    if (joueur == cpu) // if player == cpu its a draw
    {
        return 2;
    }
    if ((joueur == 'p' && cpu == 'c') || (joueur == 'f' && cpu == 'p') || (joueur == 'c' && cpu == 'f')) // player victory case
    {
        return 0;
    }
    if ((joueur == 'p' && cpu == 'f') || (joueur == 'f' && cpu == 'c') || (joueur == 'c' && cpu == 'p')) // player defeat case
    {
        return 1;
    }
}

int f_random(int max) // using s_rand and time to have a random number each time for the cpu to use to play
{
    time_t seconds;
    int result;
    seconds = time(0);
    srand(seconds);
    result = rand() % max;
    return result;
}

char f_jeu_cpu() // cpu "choosing" his move
{
    int n;
    char cpu;
    n = f_random(100);
    if (n < 33)
    {
        cpu = 'p';
    }
    else if (n > 33 && n < 66)
    {
        cpu = 'f';
    }
    else
    {
        cpu = 'c';
    }
    
    return cpu;
}

char f_jeu_joueur() // player input
{
    while ( getchar ( ) != '\n' );
    char joueur;
    printf("Entrez p pour PIERRE, f pour FEUILLE, c pour CISEAUX ");
    scanf("%c", &joueur);
    return joueur;
}

int f_affichage(int resultat, char joueur, char cpu) // result display
{
    if (resultat == 2)
    {
        printf("egalité\n"); // draw
    }
    else if (resultat == 0)
    {
        printf("vous remportez la partie\n"); // player win
    }
    else
    {
         printf("Vous perdez la partie\n"); // player defeat
    }
    printf("Vous avez choisi %c et l'ordi a choisi %c \n", joueur, cpu); // display player and cpu choice
    return 0;
}

int main() 
{
    char joueur, cpu;
    
    int resultat;
    
    int affichage;
    
    char choix;
    
    while(1) // while loop for a rematch
    {
        printf("Voulez vous rejouer?\n"); // ask for a rematch
        scanf("%c", &choix);
        if (&choix == "N")
        {
        break;
        }
        
        joueur = f_jeu_joueur();
    
        cpu = f_jeu_cpu();

        resultat = regles(joueur, cpu);
    
        affichage = f_affichage(resultat, joueur, cpu);
    }
}

知道为什么会这样吗？

Answer 1

这个 if 语句

if (&choix == "N")

比较两个指针：指向对象choix 的指针和指向字符串文字第一个字符"N" 的指针。正如字符串字面量和对象choix 一样声明

char choix;

占用不同程度的内存，则 if 语句始终评估为逻辑假。

你只需要写

if ( choix == 'N')

我还建议在调用scanf 时使用以下格式字符串

scanf(" %c", &choix);

见转换说明符%c之前的空白。

注意f_jeu_cpu函数可以在n等于33时返回未初始化的变量cpu。