関数は一度だけ実行しますC

Question

こんにちは私はCを使用して文字列に位置Nで文字列を挿入しようとしています。だから私はうまく機能する機能を開発しましたが、その問題はその作業が一度です。 これは私のコードです：

char *substring(char *string, int position, int length) 
{ 
    char *pointer; 
    int c; 

    pointer = (char*) malloc(length+1); 

    if(pointer == NULL) 
     exit(EXIT_FAILURE); 

    for(c = 0 ; c < length ; c++) 
     *(pointer+c) = *((string+position-1)+c); 

    *(pointer+c) = '\0'; 

    return pointer; 
} 
void insert_substring(char *a, char *b, int position) 
{ 
    char *f, *e; 
    int length; 

    length = strlen(a); 

    f = substring(a, 1, position - 1); 
    e = substring(a, position, length-position+1); 

    strcpy(a, ""); 
    strcat(a, f); 
    free(f); 
    strcat(a, b); 
    strcat(a, e); 
    free(e); 

} 


int main(void) { 



char username[UNLEN+1]; 
DWORD username_len = UNLEN+1; 
GetUserName(username, &username_len); 


char msg1 [] ="Good morning mr "; 

char msg2 [] ="Good evening mr "; 

insert_substring(msg1,username,17); 
printf("%s\n",msg1); 
insert_substring(msg2,username,17); 

printf("%s\n",msg2); 



    return 0; 
} 

プログラムディスプレイ1メッセージ：

おはようMR XXXXXプログラムは後に任意の命令を実行していないことを私が発言何

、 insert_substringの2回目の呼び出し。エラーは表示されません。たぶんローカル変数の問題です。関数で使用されます

Answer 1

コードを見ると、Win32 API関数を呼び出して現在のユーザーのユーザー名を取得し、時刻に基づいて挨拶メッセージを出力しようとしているようですまだこの部分を実装する）。

2つのヌル終了文字列を連結する場合、ターゲットバッファに連結する2つの文字列を保持するのに十分な領域があることを確認する必要があります。

あなたの場合、UNLENマクロを使用して、ユーザー名を保持するために必要な最大容量をすでに知っています。

したがって、サイズがsizeof(msg1) + UNLENのバッファをmallocを介して割り当てることができます。このような何か：

char *target_msg1 = (char *)malloc(sizeof(msg1) + UNLEN); 

あなたもGetUserNameのlpnSizeパラメータを使用し、できるだけ少ないスペースを割り当てるが、あなたはWin32プログラミングを行っているので、私は、メモリのカップルのバイトは懸念の多くではないと思うことができます。

あなたのコードは次のように変更します

strcpy(target_msg1, msg1); 
insert_substring(target_msg1,username,17); 
printf("%s\n",target_msg1);