0

下記のように、3つの文字列を入力させた上で、その3つの文字列を連結した状態のものを出力させるプログラムを入力したいのですが、自分のプログラムではうまく行きませんでした。

実行例

文字列1を入力してください:One
文字列2を入力してください:Two
文字列3を入力してください:Three
連結した結果:OneTwoThree

ソースコード

int main(void)
{
    char *s1[128],*s2[128],*s3[128];
    printf("文字列1を入力してください: ");
    scanf("%s",s1[128]);
    printf("文字列2を入力してください: ");
    scanf("%s",s2[128]);
    printf("文字列3を入力してください: ");
    scanf("%s",s3[128]);

    printf("%s\n",s1[128]);
    printf("%s\n",s2[128]);
    printf("%s\n",s3[128]);


    return 0;
}

コンパイル結果

$ ./a.out
文字列1を入力してください: One
文字列2を入力してください: 文字列3を入力してください: Two
Segmentation fault (コアダンプ)

なぜ「文字列2を入力してください」と「文字列3を入力してください」の部分が識別されていないのかもわかりません。どのようにプログラムすればうまくコンパイルできるようになるのでしょうか。

1

2 件の回答 2

1

ほぼすべての行でダメなんですが、なぜダメかを順番に解説していくと:

以下の変数定義は、文字列(128個の文字)を格納する領域では無くて、文字へのポインタを128個格納する領域を定義しています。

char *s1[128],*s2[128],*s3[128];

以下のパラメータは、s1という文字へのポインタの配列の(0番目から始まるので存在しないはずの)128番目のポインタ(さらに初期化していないから不定な値)を指定しています。

scanf("%s",s1[128]);

こちらも上記と同様ですね。実際にはこの行までは実行されずにSegmentation fault (コアダンプ)が発生しているわけですが。それから「連結」せず入力値を1つ1行で表示していますね。連結処理まで手が回らなかったということでしょうか。

printf("%s\n",s1[128]);

本来なら存在しないはずのポインタの(しかも初期化していない)アドレス値を指定しているので、文字列1を入力してください: Oneの入力が完了した時点でSegmentation fault (コアダンプ)が発生しても不思議では無いです。
それ以上動作して文字列2を入力してください: 文字列3を入力してください: Twoまで実行されたのは偶然です。


使う関数は以下の様な安全を考えた新しいものを使ったほうが良いでしょう。(VC++の資料ですが)
scanf_s、_scanf_s_l、wscanf_s、_wscanf_s_l
strcpy_s、wcscpy_s、_mbscpy_s、_mbscpy_s_l
strcat_s、wcscat_s、_mbscat_s、_mbscat_s_l

実際にはエラー発生を検出して対処したりする処理があった方が良いのですが、プログラムとしては以下のようになります。概要をコメントで記述しています。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <errno.h>
#define MAX_INPUT_LEN 128  /* 最大入力長 + 1 */
#define MAX_CONCAT_LEN 384 /* 最大連結長(3倍した値) */

int main(void)
{
    char s1[MAX_INPUT_LEN] = { 0 }; /* 入力文字の格納用の文字配列領域:初期化する */
    char s2[MAX_INPUT_LEN] = { 0 }; /* 同上 */
    char s3[MAX_INPUT_LEN] = { 0 }; /* 同上 */
    char s4[MAX_CONCAT_LEN] = { 0 }; /* 連結結果格納用の文字配列領域:初期化する */

    printf("文字列1を入力してください: ");
    scanf_s("%127s", s1, MAX_INPUT_LEN); /* 各入力処理:scanf_sを使い、かつ最大長を指定する */
    printf("文字列2を入力してください: ");
    scanf_s("%127s", s2, MAX_INPUT_LEN);
    printf("文字列3を入力してください: ");
    scanf_s("%127s", s3, MAX_INPUT_LEN);

    strcpy_s(s4, MAX_CONCAT_LEN, s1); /* 連結処理の最初は文字列コピー */
    strcat_s(s4, MAX_CONCAT_LEN, s2); /* 2個目、3個目は連結 */
    strcat_s(s4, MAX_CONCAT_LEN, s3);
    printf("連結した結果:%s\n", s4);

    return 0;
}
0

単純に直すならこんな感じです。

#include <stdio.h>

int main(void)
{
    char s1[128],s2[128],s3[128];
    printf("文字列1を入力してください: ");
    scanf("%s",s1);
    printf("文字列2を入力してください: ");
    scanf("%s",s2);
    printf("文字列3を入力してください: ");
    scanf("%s",s3);

    printf("%s",s1);
    printf("%s",s2);
    printf("%s\n",s3);


    return 0;
}

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。