2016-04-30 21 views
-3

に私は私が入力中に2列、SRCDESTを持っている、と私はに再帰的にSRCを連結する必要があるC.を連結文字列を再帰的にC

を再帰を使用して文字列を連結する助けが必要destを入力し、連結された文字列をの宛先に格納します。

src="house"dest="clock"の場合、出力は"chlooucske"になります。これは私のコードです:

EDITしかしSRCへのdestからそれがコピー単語全体を

char* str_concatenate(char dest[], char src[], int index){ 
    char temp[256]; // temporaty variable 
    temp[index]=src[index]; //should copy each letter from src to temp 
    temp[index+1]=dest[index]; //should copy each letter from dest to temp 
    dest[index]=temp[index]; //should store the concatenated string into dest 
    if (src[index]=='\0'){ //base case 
     return dest; 
    } 
    else 
     return str_concatenate(dest,src,index+1); 
} 

int main(){ //test 
     char dest[]="house"; 
     char src[]="clock"; 

     char* ris=str_concatenate(dest,src,0); 

     printf("dest= %s\n", ris); //should print "chlooucske" 
    return 0; 
    } 

と、それを印刷し、それが文字を連結していません。

+2

'dest'は文字列リテラルを指します。文字列リテラルは変更できません。書き込み可能なバッファを割り当てる必要があります。例: 'char dest [MAX_LEN] =" home ";' – kaylum

+2

この関数が 'NULL'ポインタ以外の何かを返すことを期待していますか?あなたは 'printf()'に 'NULL'を渡すことはできません。 – EOF

+1

@ marco2012、みんな今日あなたに迷惑をかけているようです。彼らはあなたに投票し、建設的な助けを必要とせずすべてを提供しています。あなたのコードは2つの文字列をインターリーブしたいと思うのですか?私は連結が起こっているとは思わない。あなたにはいくつかの問題があります。 1つは、あなたの関数を再帰的に入力するたびに、新しい一時配列を割り当てることです。もう1つは、src [index]にヌルターミネータがあると、ベースケースにヌルポインタを戻すことです。私は、あなたの望む結果が何であるか、他の人々があなたがそこに着くのを手伝ってくれることを教えてください。 – nicomp

答えて

2

宛先ポインタが文字列定数を指しています。あなたはそれを変更しようとしており、プログラムがクラッシュします。配列を使用し、それを宛先文字列として初期化することができます。

これをご覧ください。これはあなたの問題を説明します。 What is the difference between char s[] and char *s?