0

我正在尝试编写棋盘游戏“Mastermind”（作业）的代码。我快写完了主要部分。不幸的是，我有一个不稳定的 if 语句，我无法通过使用 Netbeans 进行调试来解释。

在下面的代码中，第 58 行if的语句并不总是有效。游戏是这样的：

  1. 用户输入想要“随机”生成的颜色数量
  2. 用户尝试猜测（破解密码）颜色。
  3. 如果 N=4（例如），则用户输入 4 种颜色的序列来尝试猜测它们。4.程序检查他是否正确并反馈反馈。

6 种颜色是：W：白色、B：蓝色、G：绿色、Y：黄色、P：紫色和 R：红色。

以下代码只是整体的一部分。请原谅我使用gets 和声明这部分中未使用的变量。还有一些“愚蠢”的部分，但在代码正常运行后它们会被更改:) 你能帮我调试一下第 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);
}
4

1 回答 1

4

这是超出数组范围的访问，导致未定义的行为：

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

和A尺寸一样[5][3]。数组索引运行 from0到N - 1whereN是数组中元素的数量。还有其他的越界访问：

A[5][0]=82;

并在函数内color()：

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

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

str需要大一（不确定为什么temp甚至存在）。

A使用聚合初始化器代替循环将数组中的所有元素设置为零：

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

请参阅为什么 get 函数如此危险以至于不应该使用它？

于 2013-01-10T08:58:32.263 回答