c - c 中的奇怪结果

Question

我想问一下我的节目。这个的主要目的是用从“tab1”和“tab2”数组收集的数据填充“result”数组。有人可以检查一下，为什么结果如此奇怪？谢谢。

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

void laczenie(char tab1[],char tab2[])
{
int i;
char* result =(char*) malloc(100*sizeof(char));

for(i=0;i<30;i++)
{
    if(tab1[i] != '\0') tab1[i]==result[i];
     else if (tab2[i] !='\0')tab2[i]==result[i];
     else printf(" ");

}
for(i=0;i<30;i++)printf("%c",result[i]);
free(result);
}


int main() 
{
char x[10]={'n','a','p','i','s','1'};
char y[10]={'n','a','p','i','s','2'};
//char x[10] = {"napis1"};
//char y[10] = {"napis2"};
laczenie(x,y);

return 0;}

Answer 1

除了上面 LihOs 的回答之外，这个 block 看起来是错误的:

if(tab1[i] != '\0') 
  tab1[i]==result[i];
else if (tab2[i] !='\0')
  tab2[i]==result[i];
else printf(" ");

你的意思是不是要把值赋给tab1[i]或tab2[i]至result[i]像这样吗？

if(tab1[i] != '\0') 
  result[i] = tab1[i];
else if (tab2[i] !='\0') 
  result[i] = tab2[i];
else printf(" ");

还使用magic numbers就像循环中一样:for(i=0;i<30;i++)这是非常糟糕的做法，您可能应该使用一个常量作为大小值(然后您可以在循环和数组声明中使用它。当数组只有 10 个元素时，为什么要循环到 30？