構造体とポインタについて

Question

前提・実現したいこと

構造体を用いて、氏名と身長・体重を順に表示したい。

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

invalid initializer PHYSICAL *pdata[3] = &data[3];

該当のソースコード

#include <stdio.h>
#include <math.h>

typedef struct physical {
  char name[64];
  float height;
  float weight;
} PHYSICAL;

int
main(void)
{
  int i;

  PHYSICAL data[] = {
    {"Yamada 0taro", 168.9, 61.8},
    {"Yamada 1taro", 163.6, 70.1},
    {"Yamada 2taro", 177.8, 58.7}


  };
   PHYSICAL *pdata[3] = &data[3];


  for (i = 0; i < sizeof data / sizeof data[0]; i++) {
    printf("%s %e %e\n", pdata[i]->name, pdata[i]->weight, pdata[i]->height);
  }
  return 0;
}

試したこと

エラーメッセージの箇所をどのように変更すればよいかわかりません。

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

Answer 1

pdataは大きさ3の配列なので、

PHYSICAL *pdata[3] = {なにか, なにか, なにか};

のように初期化します。1－2個しか書かないときは残りには0が入ります。

何を入れたいのか不明ですが、例えば、

  PHYSICAL *pdata[3] = {&data[0],&data[1],&data[2]};

ただ、これだとわざわざpdataを作る意味が無いですが。

Comments

[os-t]

PHYSICAL *pdata[3] = {&data[0],&data[1],&data[2]};の部分をfor文で書くことは可能でしょうか

[otn]

初期化構文じゃなくて、普通に代入文で代入すればいいです。

[os-t]

投稿するためにデータの数を減らしました。今自分が実行しようとしているデータは100個のデータがあるのでなるべく手間を省こう思ったのですが、代入していくしか方法はないのでしょうか・・・
pdataを用いているのは、後々にひとりずつのBMIから全員の平均BMIを求める関数と全員の身長と体重の測定値の標本平均と標準偏差の推定値を求める関数を定義しmain 関数内で呼び出して使うためです。

[otn]

> 代入していくしか方法はないのでしょうか・・・
forを使ってループにすればいいのでは？
というか、
> pdataを用いているのは、
dataをそのまま使えない理由が、質問のコードからは不明です。

Answer 2

もう回答は出揃っているようですし「pdata要らないよ」が正答な気がしますが
ソースコードを見る限り「ポインタの扱いが分からないまま雰囲気で書いた」感が強いのでポインタの扱いについて1つ。

PHYSICAL *pdata[3];

と書いた場合これは「PHYSICALのポインタ型の要素を持つ長さ3の配列」を意味します。
つまり各要素はPHYSICAL*になります。
対して

&data[3];

は「dataの4番目の要素のポインタ」の意味になります。元々dataは長さ3なのではみ出ていますし、
仮に配列dataの先頭要素のポインタを取得したい場合は単にdataもしくは&data[0]になります。

つまりもしpdataを使うのであれば

PHYSICAL *pdata = data;

が正解ですが今回の場合全く必要がありません。
ちなみにこれはポインタを代入しているだけなので中身は複製されていません(同じものを見ている)

ポインタの概念は難解かもしれませんが、重要なので一度きっちり理解しておくと後々のために良いかと思います。

Answer 3

Cでは、「配列」はデータが密に並んだデータ構造です。data[N]とdata[N+1]は必ずお隣さん同士なので、data[N]の場所がわかればdata[N+1]の場所もわかる...つまりdata[0]の場所さえ知っていれば、任意のdata[N]は計算でわかるでしょ、だから配列を扱うときは先頭要素へのポインタだけ受け渡しすれば最低なんとかなるよね! と割り切ってしまったのがCの配列の扱いの原則です。(ただし、その割り切りの結果配列を関数に渡すとサイズの情報は失われますから、必要なら別途渡す必要はあります。)

つまり、単純にN個のデータを関数に渡してごにょごにょするための施策としては、PHYSICAL *pdata[3]のようなものを設ける意味は全くありません。

ひとりずつのBMIから全員の平均BMIを求める関数と全員の身長と体重の測定値の標本平均と標準偏差の推定値を求める関数を定義し

というのが本来の目的であったなら、

#include <stdio.h>

typedef struct physical {
  char name[64];
  float height;
  float weight;
} PHYSICAL;

void printData(PHYSICAL dt[], size_t sz) {//本番ではBMI計算してどーたら、して下さい
  for (int i = 0; i < sz; i++) {
    printf("%s %e %e\n", dt[i].name, dt[i].weight, dt[i].height);
  }
}

int main(void) {
  PHYSICAL data[] = {{"Yamada 0taro", 168.9, 61.8},
                     {"Yamada 1taro", 163.6, 70.1},
                     {"Yamada 2taro", 177.8, 58.7}};
// 配列を単独で記述したら配列の先頭要素へのポインタと解釈される
// 配列を引数に渡すと、ポインタと解釈されるので要素数の情報は別途渡さないといけない
// 要は、これで配列の情報が全て関数に伝えられるということ
  printData(data, sizeof data / sizeof data[0]);
  return 0;
}

このようなことがあなたがやるべきことじゃないでしょうか。

PHYSICAL *を導入しなきゃと考えてしまったところで道を誤ったと思います。まぁ、この段階でいろいろ歩き回ってみるのはいい勉強になるでしょうから、「誤った」といっても必ずしも悪いことじゃありませんね。

PHYSICAL *を導入したらどういうことが出来るかと言うと、例えば元の配列をいじらずに順番の変更が出来ます(ソートしたりとか)。こんなプログラムなら、PHYSICAL *が有効に使えるでしょう。

#include <math.h>
#include <stdio.h>

typedef struct physical {
  char name[64];
  float height;
  float weight;
} PHYSICAL;

int main(void) {
  int i;

  PHYSICAL data[] = {{"Yamada 0taro", 168.9, 61.8},
                     {"Yamada 1taro", 163.6, 70.1},
                     {"Yamada 2taro", 177.8, 58.7}

  };
  PHYSICAL *pdata[sizeof data / sizeof data[0]];  // sizeof演算子はコンパイル時に静的に値が決まっているのでVLA扱いにならない
  for (i = 0; i < sizeof data / sizeof data[0]; i++) {//いわゆる初期化
    pdata[i] = &data[i];
  }
  //最初の状態を表示
  for (i = 0; i < sizeof data / sizeof data[0]; i++) {
    printf("%s %e %e\n", pdata[i]->name, pdata[i]->weight, pdata[i]->height);
  }
  printf("\n");
  //データを入れ替えてみる
  PHYSICAL *tmp = pdata[1];
  pdata[1] = pdata[0];
  pdata[0] = tmp;
  //入れ替えたものを表示
  for (i = 0; i < sizeof data / sizeof data[0]; i++) {
    printf("%s %e %e\n", pdata[i]->name, pdata[i]->weight, pdata[i]->height);
  }
  printf("\n");
  //元データは変更されていない
  for (i = 0; i < sizeof data / sizeof data[0]; i++) {
    printf("%s %e %e\n", data[i].name, data[i].weight, data[i].height);
  }
  printf("\n");

  return 0;
}

Comments

[os-t]

ありがとうございます。頑張ってみようと思います。

Answer 4

配列名がポインタになるので、関数にそれを渡せます。

#include <stdio.h>
#include <math.h>

typedef struct physical {
	char name[64];
	float height;
	float weight;
} PHYSICAL;

void print(const PHYSICAL *pdata, int n)
{
#if 1  // #if 0  にすると、#else のコードになります。
	for (int i = 0; i < n; i++)
		printf("%s %e %e\n",
			pdata[i].name, pdata[i].weight, pdata[i].height);
#else
	for (int i = 0; i < n; i++, pdata++)
		printf("%s %e %e\n",
			pdata->name, pdata->weight, pdata->height);
#endif
}

int main(void)
{
	PHYSICAL data[] = {
		{"Yamada 0taro", 168.9, 61.8},
		{"Yamada 1taro", 163.6, 70.1},
		{"Yamada 2taro", 177.8, 58.7},
	};
	print(data, sizeof data / sizeof data[0]);
	return 0;
}

Answer 5

PHYSICAL *pdata[3] = &data[3];

この文が意味不明です
pdataはポインタの配列ですが、右辺の初期化式は配列になってません

Comments

[os-t]

実際は100個のデータを使うのですが、どのようにすればポインタに渡すことが出来ますか？

[y_waiwai]

初期化式でやろうとせず、実行文で１００個代入しましょう。
forループでもやれば一発ですぜ

[os-t]

ありがとうございます

[y_waiwai]

そもそも
PHYSICAL *pdata = data;
...
pdata[99]=なんたらかんたら；
としたい、というはなしでしょうか？
またこれは違う、といわれそーですがｗ

そもそもなにをするためにそれをしたいんでしょうか

[os-t]

{"Yamada 0taro", 168.9, 61.8},のようなデータが100個あるという事です。
各個人bmiの平均を出す関数を定義し、その中で使うためポインタに渡そうと考えています。

[y_waiwai]

ポインタを定義したいってことなら、
PHYSICAL *pdata = data;
でいけませんか

Answer 6

エラーから、初期化の必要がありそうですね。

struct PHYSICAL data[3] = {
    {"Yamada 0taro", 168.9, 61.8},
    {"Yamada 1taro", 163.6, 70.1},
    {"Yamada 2taro", 177.8, 58.7}
};