【C言語】"scanf"から得た２つの文字列の結合方法

Question

２つの文字列を引数として入力し、それらを結合した文字列を返り値として返す関数を
作成しなさい。

[実行結果]
文字列を入力してください。

1つ目:aaa

2つ目aaaa
Segmentation fault (コアダンプ)

［質問］
なぜエラーが起きるのか？またこの改善方法を教えてください。

#include<stdio.h>

char string(char *,char *);/*プロトタイプ宣言*/

int main(void)
{
        char str1[100],str2[100];

        /*文字列の代入*/
        printf("文字列を入力してください。\n");
        printf("\n1つ目:");
        scanf("%s",str1);
        printf("\n2つ目:");
        scanf("%s",str2);

        printf("%s",string(str1,str2));
        return 0;
}

/*関数の定義*/
char string(char a,char b)
{
        char c;
        sprintf(c,"%s%s\n",a,b);
        return a;
}

Comments

[fana]

貼られているコードでは実行まで行き着けるように思えませんが……

（「引数として返す」という言葉が怪しい気がしないでもないが）
とりあえず引数の個数について考えれば

* 入力のために２個
* 「引数として返す」とかいう行為のための引数が１個

で，計３個ほど必要なんじゃないかと思えますが，どうなんですかね．
まずは「何はともあれ題意に見合う形のものを作る」ということを考えてはどうです？

[fana]

> 文字列を返り値として返す

に変わった様子だが，それならそれで　【「文字列を」返せや】　って話なのだから，
戻り値の型が char では題意に沿っていないのではないか？

てきとーなコードを書く前に，方法論の側：
【「文字列」っていうのは，どうやって返せばよいのか？】
という話をまずは固めるべきだし，
もしも固まっているのなら「俺はこういう方法で　文字列を返す　を実現しようとしているのだが」という話をちゃんと述べるべき．
（あなたが採用する方法次第で実装が変わるハズだから）

[退会済みユーザー]

コンパイラ何使ってるかは知らないけど、実行以前にコンパイルエラーになると思いますが。char* とcharの区別はついていますか？

Answer 1

char string(char a,char b)
{
        char c;
        sprintf(c,"%s%s\n",a,b);
        return a;
}

charは文字であり、文字列ではない

その関数stringは 「文字をふたつもらって文字を返す」になっている。
「文字列をふたつもらって文字列を返す」に改めるべし。

※ 「文字がcharなのは知ってるが、文字列はなんだかわかんない」なら教本読むべし

Answer 2

char* astrcat(const char* a, const char* b)
{
  size_t na = strlen(a), nb = strlen(b);
  char* c = (char*)malloc(na + nb + 1);
  memcpy(c, a, na);
  memcpy(c + na, b, nb);
  c[na + nb] = '\0';
  return c;
}

チキンマルナゲット君のために、元の文字列は非破壊にしてあるぞい。ヘッダぐらいは自力で調べ給へ。