私のstrcatコードでprintfの何が問題になっていますか？ここでは...</p> <p>を、私はこのプログラムはstrcatの機能をエミュレートするために作られてきたが、私は理解していないのprintfとの誤差がある

Question

はコードです：

#include <stdio.h> 

char *mystrcat(char *s1, char *s2); 

int main(void) 
{ 
    char *s1,*s2; 
    s1="asdad"; 
    s2="asdad"; 
    s1=mystrcat(s1,s2); 
    printf(s1); 

    return 0; 
} 
char *mystrcat(char *s1,char *s2) 
{ 
    int i,j; 
    for(i=0;s1[i]<'\0';i++) ; 
    for(j=0;s2[j]!='\0';j++) s1[i+j]=s2[j]; 
    s1[i+j]='\0'; 
    return s1; 
} 

Answer 1

最初の問題は、s1にs2を追加するのに十分な領域がないことです。 s1で指されるバッファーのサイズは、少なくともstrlen(s1) + strlen(s2) + 1（NULターミネーターである+ 1）にする必要があります。

第2の問題は、文字列リテラルが読み取り専用であることです。 s1を"asdad"に割り当てると、（潜在的に）読み取り専用メモリへのポインタが作成されます。もちろん最初の問題は、書き込み可能であっても最後に追加するための十分なスペースがないことを意味しますが、これはCの一般的な落とし穴の1つであり、言及する価値があります。 （すでにanother answerで述べた）

第3の問題は、比較s1[i] < '\0'が間違っていると、ループが1つでも反復処理を実行されませんので、あなたが正しくs1の長さを見つけることができませんということです。正しい条件は、2番目のループの場合と同じです。!= '\0'です。 （これは最初からs1を誤って上書きしてしまったので、問題1をマスクします。）

Answer 2

少なくとも、s1[i] < '\0'は同じですs1[i] < 0として、常にfalseです。