構造体の配列を渡してメンバの部分を加算する

Question

前提・実現したいこと

C言語の初心者です。
独自の練習で構造体の配列を渡してモンスター3体のhpのみを100ずつ上げるプログラミングをつくってるんですがエラーが起きてしまいます。やり方がわからないです。どなたか教えてくれないでしょうか。

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

28行目　hpplus[i]　ポインター型を宣言してください

エラーメッセージ

該当のソースコード

Ｃ```ここに言語名を入力
ソースコード

/*パラメータの設定*/
typedef struct Param {
	int hp = 0;
	int attack = 0;
}Param;

void add(int* hpplus);

int main(void)
{
	Param monster[125];
	
	monster[0] = { 33,88 };
	monster[1] = { 44,22 };
	monster[2] = { 29,80 };

	add(monster);
for (i = 0; i < 3; i++)
	{
		printf("モンスターのhpは加算してそれぞれ%dです。\n", monster[i].hp);
	}
	return 0;
}

void add(int* hpplus)
{
	Param hpplus[120];
	int i;
	for (i = 0; i < 3; i++)
	{
		hpplus[i]->hp += 100;
	}
}


### 試したこと

仮引数の内容を配列にする
実引数に&をつけるなど

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

多分基礎的なことがわかってないようなので詳しく説明してくれると嬉しいです。

Comments

[yumetodo]

markdownのソースコードを書く記法で記述してください。

Answer 1

void add(int* hpplus)

この関数は引数で構造体Paramへのポインタを受け取ってメンバにアクセスしたいのでしょうから、
void add(struct Param *hpplus)
ですね。

Param hpplus[120];

引数にhpplusがあるのに関数内で同名のデータを定義していますね。これはおかしい（余計）ですね？

add(monster);

addには構造体Paramへのポインタ（つまり構造体のアドレス）を渡すので、たとえば add(&monster[0]);のようでなければなりません。

（追記）

関数addは引数で受け取った構造体Paramへのポインタhpplusのメンバにアクセスするのですから、
hpplus[i]->hp += 100;の部分はhpplus->hp += 100;となります。

Comments

[itagagaki]

低評価した人は理由を書いてほしいですね

[yumetodo]

追加するなら
hpplus[i]->hp
は
hpplus[i].hp
ですよね。

そもそも
hpplus[i].hp
は
(*(hpplus + i)).hp
のことです。
hpplus[i]->hp
だと
(**(hpplus + i)).hp
になっておかしいですね。

[itagagaki]

修正すべき箇所を見落としていましたので回答に追記しました。
私の回答では add が構造体の配列のアドレスを受け取ることは想定していません。

[vaiton.jp]

回答ありがとうございます。仮引数(int* hpplus)の部分を訂正したのですがまだ同じようなエラー(hpplus[i]->hp += 100;の式にはポインター型が必要ですというエラーが出ます。
#include<stdio.h>

typedef struct Param {
int hp = 0;
int attack = 0;
}Param;

void add(struct Param* hpplus);

int main(void)
{
int i = 0;
Param monster[125];

monster[0] = { 33,88 };
monster[1] = { 44,22 };
monster[2] = { 29,80 };

add(monster);
for (i = 0; i < 3; i++)
{
printf("モンスターのhpは%dです。\n", monster[i].hp);
}
return 0;
}

void add(struct Param* hpplus)
{
int i;
for (i = 0; i < 3; i++)
{
hpplus[i]->hp += 100;
}
}

[vaiton.jp]

すみません解決しましたアロー演算子のところと仮引数でしたね。

[kazuma-s]

本当に解決したのでしょうか？
C では、struct Param {　int hp = 0; のように構造体のメンバの宣言で初期化はできないはずです。
また、monster[0] = { 33,88 }; のようなこともエラーになるはずです。
エラーにならないのは、ソースファイルの拡張子を .c ではなく、.cpp にして C++ として
コンパイルしているからではありませんか？

[itagagaki]

kazuma-sさん、確かに、私が指摘したところを修正しても、まだあちこちおかしいですね。
よく見たらaddでループを組んでいるのも、私の前提（addは単一の構造体にアクセスする）ではおかしい。
yumetodoさんが指摘したかった意味がわかりました。
本当は add はどういう設計にしたかったのか？？
そうですね、本当に解決したのかは気になります…
monster[125] とか大きく確保していながら初期化やループは３つだけだったりとかしていますし。
質問者さんはC初心者ということで、いろいろ勉強中なのだと思いますが。

[yumetodo]

125なのはもっとデータあるけど削ってもよかろうで削ったと見てた。なぜなら

> モンスター3体のhpのみを

とあるので。

---

> 構造体のメンバの宣言で初期化はできないはずです。

手元のコンパイルうっかりC++でやってましたorz見落とした。