構造体の配列について

[platyくん]

構造体の中に配列を入れようとしているのですが、初期化でうまくいきません。

コード:

struct weapon_conf{
	float attack[32];
	int rensya[32];
	int speed[32];
	int shot_type[32];//0→不使用
	bool get[32];
};

extern weapon_conf *w;

コード:

#include "weapon_conf.h"
struct weapon_conf *w;
void weapon_config(){
	if(loop_time == 0){
		//初期武器データを構造体で代入
		w->shot_type[0] = 0;
		w->shot_type[1] = 0;
		w->shot_type[2] = 0;
		w->shot_type[3] = 0;
	}
}

GameProg.exe の 0x003cd6ce でハンドルされていない例外が発生しました: 0xC0000005: 場所 0x00000180 に書き込み中にアクセス違反が発生しました。
とエラー表示されてしまいます。
おそらくポインタにint型の数を代入しているからだと思うのですが、解決策が浮かびません。
どう修正すればいいのですか？

[h2so5]

おそらくポインタにint型の数を代入しているからだと思うのですが、

それが分かっているなら、ポインタを使わずに
struct weapon_conf w;
とすればOKではないでしょうか？

[beatle]

それが分かっているなら、ポインタを使わずに
struct weapon_conf w;
とすればOKではないでしょうか？

僕が想像するに、恐らくstruct wepon_conf* 型の変数wに何も設定しておらず、NULLポインタアクセスになっています。
（0x00000180 に書き込み中にエラーがでていることから推測しました。sizeof(float) == sizeof(int) == 4な環境では普通、shot_type[0]のオフセットは0x180になります。）

おそらくポインタにint型の数を代入しているからだと思うのですが、

というのは、多分

コード:

w->shot_type[0] = 0;
w->shot_type[1] = 0;
w->shot_type[2] = 0;
w->shot_type[3] = 0;

のことを言っているのだと思います。
この4行自体にはなんら変なところはありません。

問題なのは、wに何も代入しない状態でwepon_config()関数を呼び出していることにあります。

[non]

3行目と4行目目の間に次の行を入れる。
w = (struct weapon_conf *)malloc(sizeof(struct weapon_conf ));

[platyくん]

3行目と4行目目の間に次の行を入れる。
w = (struct weapon_conf *)malloc(sizeof(struct weapon_conf ));

一応解決しました。ですが、これだと関数を呼び出すたびにメモリー確保するみたいになりませんか？

[non]

一応解決しました。ですが、これだと関数を呼び出すたびにメモリー確保するみたいになりませんか？

もちろんそうですよ。これは、初期化する関数じゃないのかな？
いらなくなったらfreeを忘れないように。

[beatle]

3行目と4行目目の間に次の行を入れる。
w = (struct weapon_conf *)malloc(sizeof(struct weapon_conf ));

一応解決しました。ですが、これだと関数を呼び出すたびにメモリー確保するみたいになりませんか？

もちろんそうなります。

nonさんの方法を使わず、例えばこのようにしても解決できます。

コード:

int main()
{
    struct weapon_conf w0;
    w = &w0;
    weapon_config();
    ...
}

もしくは、weapon_config関数がstruct weapon_conf*を引数に取るようにして

コード:

void weapon_conf(struct weapon_conf* w)
{
    // 中身は前と同じ
}

int main()
{
    struct weapon_conf w0;
    weapon_config(&w0);
    w = &w0;
    ...
}

なーんて方法もあります。
こうすると、グローバル領域にwを作る必要はあまりなくなってきます。
グローバル変数を使わないことが、多くの場合に「良いコード」とされるので、参考にしてください。