自作strcat

関数strcatを自分で作ってみました。

#include<stdio.h>

char *my_strcat(char *s1, char *s2){
int i = 0;
int n = 0;
while(s1[i] != '\0'){
i++;
}
while(s2[n] != '\0'){
s1[i] = s2[n];
i++;
n++;
}
s1[i] = '\0';
return s1;
}

int main(){
char *s1 = "abc";
char *s2 = "vwxyz";
my_strcat(s1, s2);
printf("s1:%s s2:%s\n", s1, s2);
return 0;
}

これを実行したところ、s1はちゃんとs1にs2を連結した形で表示されました。しかし、s2は一文字少なく表示されてしまいます。自分ではどこをどう直せばいいかわかりません。よろしくお願いします。

No.5 回答者： kenji_aki 回答日時： 2009/01/23 04:42

s1→[0]

s2→[4]

[0]番地
|0|1|2|3 |4...
|a|b|c|\0|...
[4]番地
|4|5|6|7|8|9 |A...
|v|w|x|y|z|\0|...

my_strcat(s1, s2);
|0|1|2|3|4|5|6|7|8 |9 |A...
|a|b|c|v|w|x|y|z|\0|\0|...
|↑s1　|↑s2

文字列定数がどのようにメモリに配置されているのか
環境によって違います。
今回は、たまたま問題なく実行できただけです。

No.4 回答者： aris-wiz 回答日時： 2009/01/21 13:28

> char *s1 = "abc";

> char *s2 = "vwxyz";
上記二つは s1 と s2 に「どのような値」が入ってくるのか、
s1とs2が何を指すのかきちんと理解してください。

文字列「定数」がどのようなものなのか考えれば、
何故、これでダメなのか考えればすぐに分かります。

No.3 回答者： chie65535 回答日時： 2009/01/21 11:50

＞char *s1 = "abc";

＞char *s2 = "vwxyz";
＞my_strcat(s1, s2);
問１．この「s1」の後ろに文字列を連結した時、s1の後ろはどうなりますか？以下のＡ～Ｃから正しいと思う物を１つ選びなさい（２５点）

選択肢
Ａ．何も起きない
Ｂ．文字列"vwxyz"が書き込まれる
Ｃ．何かが書き込まれるが、何が書き込まれるか判らない

問２．また、連結前のs1の後ろには、何がありますか？以下のＡ～Ｃから正しいと思う物を１つ選びなさい（２５点）

選択肢
Ａ．文字列s2を書き込むだけの充分な開き領域がある
Ｂ．たぶん文字列s2がある
Ｃ．そこに何があるか判らない

問３．文字列"abc"と"vwxyz"が、下図の図１ようにメモリに配置されていた場合、関数my_strcatを呼び出した後の、メモリ内の状態はどうなりますか？以下のＡ～Ｃから正しいと思う物を１つ選びなさい（２５点）

図１
'a'←ポインタs1が指すアドレス
'b'←ポインタs1が指すアドレス＋１
'c'←ポインタs1が指すアドレス＋２
0←ポインタs1が指すアドレス＋３
'v'←ポインタs1が指すアドレス＋４、ポインタs2が指すアドレス
'w'←ポインタs1が指すアドレス＋５、ポインタs2が指すアドレス＋１
'x'←ポインタs1が指すアドレス＋６、ポインタs2が指すアドレス＋２
'y'←ポインタs1が指すアドレス＋７、ポインタs2が指すアドレス＋３
'z'←ポインタs1が指すアドレス＋８、ポインタs2が指すアドレス＋４
0←ポインタs1が指すアドレス＋９、ポインタs2が指すアドレス＋５

選択肢
Ａ．
'a'←ポインタs1が指すアドレス
'b'←ポインタs1が指すアドレス＋１
'c'
'v'
'w'
'x'
'y'
'z'
0
'v'←ポインタs2が指すアドレス
'w'
'x'
'y'
'z'
0

Ｂ．
'a'←ポインタs1が指すアドレス
'b'←ポインタs1が指すアドレス＋１
'c'
'v'←ポインタs2が指すアドレス
'w'←ポインタs2が指すアドレス＋１
'x'
'y'
'z'
0

Ｃ．
'a'←ポインタs1が指すアドレス
'b'←ポインタs1が指すアドレス＋１
'c'
'v'
'w'←ポインタs2が指すアドレス
'x'←ポインタs2が指すアドレス＋１
'y'
'z'
0
0

＞printf("s1:%s s2:%s\n", s1, s2);
問４．問３の条件で関数my_strcatを呼び出した結果、上記のprintf関数で表示される文字列はどうなりますか？以下のＡ～Ｃから正しいと思う物を１つ選びなさい（２５点）

選択肢
Ａ．s1:abc s2:vwxyz
Ｂ．s1:abcvwxyz s2:vwxyz
Ｃ．s1:abcvwxyz s2:wxyz

すべて、正解は「Ｃ」です。

これで１００点を取れれば、s2がどうして１文字少なく表示されるかが判ります。

そして、何が悪かったのか、どう直せば良いかも判る筈です。

No.2 回答者： Tacosan 回答日時： 2009/01/21 01:25

よかったね～, 死ななくって.

my_strcat でどこに書き込んでますか?

No.1 回答者： koko_u_u 回答日時： 2009/01/21 01:24

>これを実行したところ、s1はちゃんとs1にs2を連結した形で表示されました。

たまたまです。よく実行できましたね。
strcat() のマニュアルをまずは読みましょう。