別関数の配列の値をmain関数で表示させたい

Question

input関数で入力された配列の値をmain関数で表示させたいのですが、コアダンプ（セグメンテーション違反）でうまくいきません。
input関数のreturnで配列の先頭アドレスがmainに渡され、main関数で表示させようとしています。なぜ意図した通りに動かないのでしょうか？

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

#define MAXDATA 255
#define DATA 3

char* input(void);

int main(void) {
  char *xy[DATA] = { };

  *xy = input();
  printf("座標：%s\n", *xy);

  return 0;
}

char* input(void) {
  char in_xy[DATA] = { };

  // A1などを入力(正しく入力されると仮定)
  printf(">>");
  fgets(in_xy, MAXDATA, stdin);

  // 配列の最後を¥0に変更
  in_xy[strlen(in_xy) - 1] = '\n';

  return in_xy;

}

```![イメージ説明](e1dd5e84ee101c82228249d81e7e0025.png)

Answer 1

• アドレスを返したいなら、戻り値の型はcharではなくchar*です。
• 関数input内で宣言したローカル変数in_xy[]の寿命は関数内で終わりです。

関数を抜けた後、このアドレスは別のことに使われてしまう可能性があります。
mainで宣言して引数で渡すなどしましょう。

Comments

[gyoruo]

ただいま混乱中なので少しゆっくりと考えてみます！
ご回答ありがとうございます！

[ozwk]

こんな感じ
int main(){
char xy[DATA];
input(xy);
...

[gyoruo]

わかりやすい例までありがとうございます！
やってみます！

Answer 2

• input関数内のin_xy[]の寿命は，この関数から抜けた時点で尽きています．
• それはそれとして，input関数の戻り値の型もおかしい

Comments

[fana]

というか，もっといろいろと異常だな．

[gyoruo]

本当でした。。。
ありがとうございます！

[fana]

main側での戻り値の受け方：
*xy = input();
も謎めいている．

[fana]

* inputにそれ用の引数を設けて，main側から「ここに結果をいれてくれ」と指定する形にする
* input側から寿命が尽きない場所を返す形にする

等といった方向に修正するとよいのでは．

[gyoruo]

少し混乱してきたので考えてみます！
ご回答ありがとうございます！