strcpyでのセグメンテーション違反

Question

char型ポインタの配列にstrcpyで文字列をコピーして，最後にfor文でまとめて表示させようとしたのですが，セグメンテーション違反になってしまします．

strcpyの部分でセグメンテーション違反になるのですが，なぜでしょうか...?

#include <stdio.h>
#include <string.h>

int main(void) {
  char *str[16];
  char a[] = "Hello";
  char b[] = "World";
  char c[] = "This";
  char d[] = "is";
  char e[] = "PC";

  strcpy(str[0], a);
  strcpy(str[1], b);
  strcpy(str[2], c);
  strcpy(str[3], d);
  strcpy(str[4], e);

  for (int i=0; i<5; i++) {
    printf("%s ", str[i]);
  }
  puts("");

  return 0;
}

Answer 1

  char *str[16];
  char a[] = "Hello";
  char b[] = "World";
  char c[] = "This";
  char d[] = "is";
  char e[] = "PC";

strは１６個のポインタ配列です。
コピー先の実体がどこのアドレスを指しているかわからないので、たいていの場合Segmentation Faultが発生します。

char str[5][16];  /* 16バイトの配列を５個 */

として実行してみてください。

追記
加えてstrcpy関数の引数が逆です。
char *strcpy(char s1, const char S2);
文字列s1に文字列s2を\0までコピーします。

Comments

[RaitoN]

ありがとうございます．

[DreamTheater]

strcpy関数の引数が逆です。

[RaitoN]

解決しました！ありがとうございます

[hidezzz]

質問者の提示しているソースコードのstrcpy()の引数の順番は正しいと思います。

[DreamTheater]

ごめんなさい見間違えたようです。
RaitoNさん失礼しました。
hidezzzさんフォローありがとうございました。

Answer 2

char *str[16];

これは char へのポインタを 16 個用意しています。

用意するポインタは、この場合だと 5 個で良いので、

char *str[5];

としておきましょう。

次に、ポインタの指す先を用意します。各文字列の長さは最長 5 文字なので余裕を見て 8 文字用意するとして、

for (int i = 0; i < 5; i++) {
str[i] = malloc(8);
}

といった感じで用意します。malloc() を使うには stdlib.h が必要です。

これでコンパイルが通り、実行すると想定の通りの

Hello World This is PC

という出力が得られると思います。

注意するのは、各文字列の長さです。確保するサイズは余裕を持たないとメモリを破壊することがあります（物理的にではありません）。また、char の配列等で末端を判別するために NULL をつけるので見た目の文字列長より 1 バイト余計に必要になります。

strcpy の動作を理解したら、strncpy を理解するのが良いと思います。できれば strlcpy も。

C では「文字列」というものが無いので要注意ですね。

Comments

[RaitoN]

センキュー！

Answer 3

strcpy(str[0], a);

コピー先の領域が確保されてません
メモリ破壊を起こしてます

Comments

[RaitoN]

ありがとうございます．
```
size_t len;
len = strlen(a);
str[0] = (char *)malloc(len * sizeof(a));
```
と書いてみても同じようにセグメンテーション違反が起きるのですがこれはなぜでしょうか...?

[y_waiwai]

str[1]やstr[2] .. str[15]のメモリ確保も適切に行う必要があります

[DreamTheater]

横から失礼します。
strcpy関数の引数が逆ですね。（私の回答欄にも追記しました）

[y_waiwai]

んで、
sizeof(a)
と記述すると、aの占めるメモリサイズを返します
なので、aをコピーするのであれば
str[0] = (char *)malloc(sizeof(a));
でOKです。
ましかし、str[1]やstr[2]やそれ以降はどうするのかを考えると、いちいち配列名を変えるのはうざいので（もちろんそういう記述でも構いませんが）、ある程度大きなサイズで決め打ちで確保するのでもいいかと思います
for(i=0;i<16;i++) str[i]=(char*)malloc(256);

ましかし、決め打ちで確保すると、それ以上のサイズの文字列が来ると破綻する、ので注意しましょう