比较字符串时，下面的程序异常终止？

Question

我比较字符串的两个数组，但程序比较一些字符串后异常终止：

什么是错我的代码？

int main() 
{ 

    int N,Q; 
    printf("Enter no. of strings:"); 
    scanf("%d",&N); 
    char *a[N],*b[Q],k[50],*p; 
    int len; 

    //scanning first array of strings 
    for(int i=0;i<N;i++) 
    { 
     scanf("%s",k); 
     len=strlen(k); 
     p=(char*)malloc((len+1)*sizeof(char)); 
     strcpy(p,k); 
     a[i]=p; 
    } 
    printf("no. of Query:"); 
    scanf("%d",&Q); 

    //scanning second array of strings 
    for(int i=0;i<Q;i++) 
    { 
     scanf("%s",k); 
     len=strlen(k); 
     p=(char*)malloc((len+1)*sizeof(char)); 
     strcpy(p,k); 
     b[i]=p; 
    } 

    ***//comparing both the arrays of strings*** 
    for(int i=0;i<Q;i++) 
    { 
     for(int j=0;j<N;j++) 
     { 
      int i=strcmp(a[j],b[i]); 
      printf("%d\t",i); 
     } 
     printf("\n"); 
    } 
    return 0; 
} 

Answer 1

分配STRCMP的结果为不同的变量名称，而不是“我”，因为“我”是你的外循环变量，但我的b[i]是在栈

for(int i=0;i<Q;i++) 
{ 
    for(int j=0;j<N;j++) 
    { 
     int i=strcmp(a[j],b[i]); 
     printf("%d\t",i); 
    } 
    printf("\n"); 
} 

Answer 2

截至b[Q]Q声明初始化 - 在此之后可能发生任何事情。您至少需要将b[]的声明更改为Q。

可能有其他的问题 - 而这仅仅是一个显而易见的。

Answer 3

你正在输入字符串的数量，而不是正确地使用它。你应该使用动态内存分配。使用的malloc

printf("Enter no. of strings:"); 
scanf("%d",&N); 

和

char *a[N] 

地方，因为它应该是

a = malloc(sizeof(char*) * N); 

并返回，以避免内存韭菜之前释放它。 I am not sure that this is the problem but this can be a step in the right direction.

编辑基于@Clifford评论：C99标准允许这样，所以如果你的编译器支持C99，这不是一个问题。