セグフォの原因を教えていただけないでしょうか

Question

前提

C言語のポインタを学習中です。
理解を深めようと、2つの配列をポインタにより結合するコードを書いたのですが、セグフォが発生してしまいます。
原因は、どこなのでしょうか？

発生している問題・エラーメッセージ

文字列を入力:あい
文字列を入力:うえお
Segmentation fault (core dumped)

該当のソースコード

#include <stdio.h>

int main(){
	char str1[100],str2[100],*p;
	printf("文字列を入力:");
    scanf("%s",str1);
    printf("\n文字列を入力:");
    scanf("%s",str2);
    p = str1;
    
    for(int i = 0; str1[i] != '\n' ; i++){
        p++;
    }
    
    for(int j = 0 ; str2[j] != '\n'; j++ ){
        *(p+j) = str2[j];
    }
    printf("\n%s",p);
    return 0;
}

補足情報（FW/ツールのバージョンなど）

実行環境:https://www.tutorialspoint.com/compile_c_online.php

Answer 1

for(int i = 0; str1[i] != '\n' ; i++){

この場合、str1 に改行コードは入ってないと思います。'\n' のかわりに文字列の終端である 0 をチェックしてください。

また、こういう問題はデバッガでステップ実行すればすぐに原因がわかるので、デバッガが使えるような開発環境を用意することをおすすめします。

Comments

[bbiiq]

回答ありがとうございます。
教えていただいた通り、終端文字に0を指定したコードに書き換えたのですが、同じくセグフォが発生してしまいます。

#include <stdio.h>

int main(){
char str1[100],str2[100],*p;
int num = 0;
printf("文字列を入力:");
scanf("%s",str1);
printf("\n文字列を入力:");
scanf("%s",str2);
p = str1;

for(int i = 0; str1[i] != '0' ; i++){
num++;
}
for(int i = 0; str2[i] != '0' ; i++){
p[num + i] = str2[i];
}
printf("\n%s",p);
return 0;
}

[bbiiq]

eclipseで行っていたのですが、謎のエラーが起きてしまって使っていません。

[int32_t]

'0' じゃなくて 0、と
str2 の内容をコピーした直後に 0 で文字列を終端しないと printf() でクラッシュすると思います。

int i;
for (i = 0; str2[i] != 0; i++) {
p[num + i] = str2[i];
}
p[num + i] = 0;
printf(....

Answer 2

本質問の回答としてはint32_t様の回答で解決すると思います。
ただ、そこだけの修正だと質問者様の出したい結果が得られないと思いますので補足します。

for(int i = 0; str1[i] != '\n' ; i++){
        p++;
    }

ここでpのアドレスはインクリメントされています。
そのため、最後のprintfでもインクリメントされたアドレスから先の中身を出力してしまいます。

正しくはpの大元のアドレスを渡してあげるべきです。

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

Comments

[bbiiq]

回答ありがとうございます。
確かに、int32_t様の回答でセグフォ自体のエラーは解決しました(感謝)。
そしてALOHAMS様の回答をもとに以下のコードに書き換えたのですが、入力した2つの文字列を結合することができません。
#include <stdio.h>

int main(){
char str1[100],str2[100],*p;
printf("文字列を入力:");
scanf("%s",str1);
printf("\n文字列を入力:");
scanf("%s",str2);
p = str1;

for(int i = 0; str1[i] != '0' ; i++){
p++;
}

for(int j = 0 ; str2[j] != '0'; j++ ){
*(p+j) = str2[j];
}
printf("\n%s",str1);
return 0;
}

そもそも配列を一気に出力することはできないと記憶しているのですが、
printf("\n%s",str1);
という書き方でも出力されるのでしょうか？

[ALOHAMS]

>printf("\n%s",str1);
>という書き方でも出力されるのでしょうか？

出力されます。%sを指定している時、終端文字までを出力します。
参考:http://wisdom.sakura.ne.jp/programming/c/c57.html

[ALOHAMS]

質問者様のコードですと、２個目のfor文でセグフォが発生するはずです。
というのも'0'は終端文字ではないからです。
'0'と0は全く違うものです。一度調べてみるとよいと思います。

それと、配列は0などで初期化しておくとよいと思います。

[bbiiq]

回答ありがとうございます。
終端文字なのですが、やはり'0'ではなく'\n'なのでしょうか？

[ALOHAMS]

'\0'は終端文字であり、8進数の0です。 //YUKI_DAYO様の指摘により修正しました
また、０も8進数の0であるので終端文字として見れます。（多分この理解であってると思う...）

しかし'0'は全く違うものですので注意してください。

[YUKI_DAYO]

揚げ足を取るようで申し訳ないのですが、'\n' は改行では？
'\0' と間違われていませんか？

[ALOHAMS]

オワー！大変申し訳ありません。
YUKI_DAYO様の言う通り¥0が終端文字です失礼しました...
修正しておきます...