c - C语言中的if语句不稳定

我正在写棋盘游戏“主谋”（家庭作业）的代码。我差不多写完了主要部分不幸的是，我有一个不稳定的if语句，无法用Netbeans调试来解释。
在下面的代码中，第58行的if语句并不总是有效的。游戏是这样的：
用户输入要“随机”生成的颜色数
用户试图猜测（破坏代码）颜色。
如果N=4（例如），则用户输入一个由4种颜色组成的序列来尝试猜测它们。
4.程序检查他是否正确，并给出反馈。
这6种颜色是：W:白色，B:蓝色，G:绿色，Y:黄色，P:紫色和R:红色。
下面的代码只是整个代码的一部分。请原谅我使用get和声明此部分中未使用的变量。也有一些“愚蠢”的部分，但是在代码正常工作后它们将被更改：）您能帮我调试一下第58行的if语句吗？（你会看到旁边的评论）。
代码如下：

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

char *colour(char *xr, int N);
int RandomInteger(int low, int high);

int main()
{
    int i,j,q,N,w,z,k,A[6][4],win;
    char *xr,*xrx,temp[1],sur[]="SURRENDER";

    printf("Gimme number from 1-100");
    scanf("%d",&N);
    getchar();

    for(i=0; i<6; i++) A[i][1]=0;
    for(i=0; i<6; i++) A[i][2]=0;
    for(i=0; i<6; i++) A[i][3]=0;
    A[0][0]=87,A[1][0]=66,A[2][0]=71,A[3][0]=89,A[4][0]=80,A[5][0]=82;
    xr=malloc(N*sizeof(char));
    xrx=malloc(N*sizeof(char));
    xr=colour(xr,N);

    printf("\nxr: %s",xr);
    i=0;
    q=0;
    z=1;
    printf("\nGimme the color sequence: ");
    gets(xrx);
    //if(strcmp(xrx,sur)==0) goto end;

    while(xrx[i])
    {
   if(xrx[i]!=87 && xrx[i]!=66 && xrx[i]!=71 && xrx[i]!=89 && xrx[i]!=80 && xrx[i]!=82)
       {z=0;}
       if(z==0)
       {
           printf("\nGimme the color sequence: ");
           gets(xrx);
           i=0;
           z=1;
       }
       else i++;
    }

    i=k=q=0;
    while(xr[i])
    {
    if(xr[i]==xrx[i])
    {
            q++;
            for(w=0; w<=5; w++)
            {
                     if(xr[i]==A[w][0]) //SOMETIMES IT DOESN'T WORK
                     {
                             printf("\nhere:");
                             A[w][3]=1;
                             printf("%d",A[w][3]);
                             break;
                     }
            }
    }
    i++;
    }
    printf("\nColor and position correct: %d",q);
    for(i=0; i<=5; i++) printf("\n%d",A[i][3]); //PERIERGO


   // end:
   printf("!!BYE!!");
   return 0;
}

char *colour(char *xr, int N)
{
    int r,i,j,number;
    char *str,temp[2];

    str=calloc(N,sizeof(char));
    srand((int)time(NULL));
    for(j=0; j<N; j++)
    {
       r=RandomInteger(0,5);
       if(r==0)
       {
           temp[0]='W';
           temp[1]='\0';
           strcat(str,temp);
       }
       else if(r==1)
       {
       temp[0]='B';
       temp[1]='\0';
       strcat(str,temp);
       }
       else if(r==2)
       {
           temp[0]='G';
           temp[1]='\0';
           strcat(str,temp);
       }
       else if(r==3)
       {
           temp[0]='Y';
           temp[1]='\0';
           strcat(str,temp);
       }
       else if(r==4)
       {
           temp[0]='P';
           temp[1]='\0';
           strcat(str,temp);
       }
       else if(r==5)
       {
           temp[0]='R';
           temp[1]='\0';
           strcat(str,temp);
       }
    }
    return str;
}

int RandomInteger(int low, int high)
{
    int k;
    double d;
    d=(double)rand()/((double)RAND_MAX+1);
    k=(int)(d*(high-low+1));
    return (low+k);
}

最佳答案

这超出了数组的界限，导致未定义的行为：

for(i=0; i<5; i++) A[i][3]=0;

asA具有维度[5][3]。数组索引从0运行到N - 1，其中N是数组中的元素数。还有其他出现的越界访问：

A[5][0]=82;

在功能范围内：

char *str,temp[1];
str=malloc(N*sizeof(char)); /*Note sizeof(char) is guaranteed to be 1: omit.*/

/* snip */
temp[1]='\0';

color()需要更大一个（不知道为什么str甚至存在）。
使用聚合初始值设定项，而不是循环将数组temp中的所有元素设置为零：

int A[5][3] = { { 0 } };

见Why is the gets function so dangerous that it should not be used?

关于c - C语言中的if语句不稳定，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/14253993/