Windowsにエラーがあると言われる。

質問

ゲームでswitch文で難易度毎に敵の構造体にデータをセットしたいのですが、
方法によって起動後にWindowsにエラー送信云々と言われて終了してしまいます。

駄目だった時の書き方は
switch(level){
case LEVEL_EASY:
teki[0] = SetTeki(type,y,timing*(1));
teki[1] = SetTeki(TEKI_BANAYELOW2,0,timing*(2));
teki[2] = SetTeki(type,y,timing*(3.5));
teki[3] = SetTeki(type,y,timing*(4.5));
teki[4] = SetTeki(TEKI_BANAYELOW2,0,timing*(6));
teki[5] = SetTeki(TEKI_BANAYELOW2,0,timing*(7));
teki[6] = SetTeki(TEKI_BANAYELOW3,0,timing*(8.5));
teki[6] = SetTeki(TEKI_BANABLUE,0,timing*(10));
teki[7] = SetTeki(TEKI_BANABLUE,y,timing*(11));
teki[8] = SetTeki(type,y,timing*(12.5));
teki[9] = SetTeki(type,y,timing*(13.5));
teki[11] = SetTeki(TEKI_BANAYELOW2,0,timing*(15.5));
teki[12] = SetTeki(TEKI_BANAYELOW2,0,timing*(16.5));
teki[13] = SetTeki(TEKI_BANAYELOW3,0,timing*(19));
teki[14] = SetTeki(TEKI_BANAYELOW3,y,timing*(19));
teki[15] = SetTeki(TEKI_BANABLUE,y,timing*(20.5));
teki[16] = SetTeki(TEKI_BANABLUE,0,timing*(20.5));
teki[17] = SetTeki(TEKI_BANARED,y/2,timing*(20.5));
break;
//作成途中なのでこの下にもう2つLEVEL_NORMALとLEVEL_HARDで同じ内容の分岐があります
}

で、良かった時の書き方は
for(int i=0;i<TEKI_MAX;i++){
　　if(i==0){
　　　　teki[i] = SetTeki(TEKI_BANABLUE,0,timing*(i+1));
　　}else if(i==1){
　　　　teki[i] = SetTeki(TEKI_BANARED,y/2,timing*(i+1));
　　}else{
　　　　teki[i] = SetTeki(type,y,timing*(i+1));
　　}
}
でした。
自分には何で上が駄目で下が良いのか全く分かりません。
LEVEL_EASYなどはヘッダーで

enum { //レベル
LEVEL_EASY,
LEVEL_NORMAL,
LEVEL_HARD,
};

としてあり、分割してあるソースコードの内、これを使う他の関数は正常に機能しています。

多すぎるのかと思って半分や1つにもしましたが同様にエラーだと言われます。
VC++ExpressEditionではビルド時にはエラーがあるとは言われません。
何がなんだか分からないのでどなたか原因の可能性がある部分を教えてください。

回答No.4

Oh-Orange

★アドバイス
・エラーダイアログで『デバッグ』というボタンを押してデバックして調査して下さい。
　運がよければソース上のどの場所でエラーになるのかが分かります。
・以上。

回答No.3

Oh-Orange

★エラー場所を特定して下さい。
・SetTeki 関数を呼ぶ前と呼んだ後に MessageBoxなどを挿入してどちらで
　お馴染みのエラーダイアログが発生しますか？
　つまり
　CharaData teki[ 18 ];

　switch( level ){
　　case LEVEL_EASY:
　　　MessageBox( hWnd, "呼ぶ前", "確認1", MB_OK );
　　　SetTeki( &teki[0], … );
　　　MessageBox( hWnd, "呼んだ後", "確認2", MB_OK );
　　　break:
　　：
　}
　として２つのメッセージボックスの何処でお馴染みのエラーダイアログが
　表示されるか調査して下さい。
・組み合わせは
　(1)『確認1』より前にエラーダイアログが表示される
　(2)『確認1』と『確認2』の間でエラーダイアログが表示される
　(3)『確認2』の後でエラーダイアログが表示される
　これにより SetTeki 関数が原因か、それ以外かの切り分けが出来ると思います。
・(1)なら SetTeki 関数を呼ぶもっと手前の処理でバグっています。
　(2)なら SetTeki 関数を徹底的に見直すこと。
　(3)なら SetTeki 関数を呼んだ後の処理で代入、参照などの処理でバグっている可能性があります。
　きっとソースファイルは大きいでしょうからエラーダイアログが表示される場所を
　特定することを目標にして下さい。
・以上。

お礼

処理を追うためにあちこちにメッセージボックスを埋め込んだ所、
ある事に気づきました。

この敵をセットする関数はTeki1Init()という名前で、
メインとなる.cppファイルにて

case GAME_STAGE_1:
score = 0;
flagScore = TRUE;
JikiInit();
//※1
Teki1Init();
//※2
BakuenInit();
StopSoundMem(soundOpening);
PlaySoundMem(soundBgm, DX_PLAYTYPE_LOOP);
break;

とswitch文で呼ばれているのですが、
SetTeki()をswitch文で呼び出すと、
※1も※2も表示されませんでした。
しかし、switch文でもfor文でもその前後に仕込んだメッセージボックスは表示されます。
そしてTeki1Init()関数はここでしか呼ばれていません。
という事は、「呼ばれていないのに処理が実行されている」という事になるのでしょうか？
そんな事は起こりうるのでしょうか？

補足

確認したところ（３）のようです。
とりあえず処理を追って確認して見ます。

回答No.2

Oh-Orange

★アドバイス
・『teki』配列はどのように宣言していますか？
　(1)struct CharaData teki[ 18 ]
　(2)struct CharaData *teki[ 18 ]
　どっちですか？
・あと構造体の定義は typedef を使った方が分かりやすくなりますよ。
　typedef struct CharaData {
　　：
　　省略
　　：
　} CharaData;
　として
・CharaData SetTeki( int tekiType, int y, int timing )
　{
　　CharaData val;

　　return val;
　}
・CharaData teki[ 18 ];

　switch( level ){
　　case LEVEL_EASY:
　　　teki[ 0 ] = SetTeki( … );
　　　　：
　　　teki[ 17 ] = SetTeki( … );
　　　break:
　　：
　}
　特に SetTeki 関数に問題は無いように見えます。
　teki[0]～teki[17] はポインタの配列ではないですよね。
　ポインタの配列ではエラーになりますけど。
・どうせなら SetTeki 関数の第１引数に構造体へのポインタを渡すように
　すれば良いと思いますが…。つまり
・void SetTeki( CharaData *val, int tekiType, int y, int timing )
　{
　　CharaData val;　←不要になる

　　val->x = WINDOW_WIDTH; //初期座標
　　val->y = y;
　　val->timing=timing; //出現タイミング
　　　：
　　省略
　　　：
　　return val;　←不要になる
　}
　このように定義して
　CharaData teki[ 18 ];

　switch( level ){
　　case LEVEL_EASY:
　　　SetTeki( &teki[0], … );
　　　　：
　　　SetTeki( &teki[17], … );
　　　break:
　　：
　}
　とします。
・実行時のエラーとはどんなメッセージ(ダイアログ)が表示されますか？
　ここも補足して下さい。
・それでは。また。

お礼

ご指摘の通りに改良しましたがやはり同様に駄目でした。

補足

定義は(1)です。
SetTeki()関数の改良については追ってご報告致します。
実行時のエラーとは

問題が発生したため、Object.exe を終了します。 ご不便をおかけして申し訳ありません。

と言われてしまいます。
そしてエラー報告を送信しますか云々と聞かれます。
私はこの手のエラーになるのが応答なしのプログラムをタスクマネージャで強制終了させる時くらいしか見ないので
対策が全く分からなかったのです。

回答No.1

Oh-Orange

★補足要求します。
・SetTeki() 関数のプロトタイプか定義内容を見せて下さい。
　ついでに teki[0]～teki[17] はどんな配列ですか？
　構造体なども見せて下さい。
・以上。

補足

SetTeki()関数は

struct CharaData SetTeki(int tekiType,int y,int timing )
{
struct CharaData val;
val.x = WINDOW_WIDTH; //初期座標
val.y = y;
val.timing=timing; //出現タイミング
switch(tekiType)
{
case TEKI_BANAYELOW1:
val.life = 0;//出現フラグを立てる
val.animpat = 1;
val.ghandle = &imgTeki[0];
GetGraphSize(*(val.ghandle),&(val.image_w),&(val.image_h));
val.bounds_w=64;
val.bounds_h=64;
val.size=1.0;//敵の大きさ
val.hitmode=HIT_NORMAL;
val.pattern = 1; //どういう軌道を描くか
val.counter = 0;
break;
case TEKI_BANAYELOW2:
val.life = 0;//出現フラグを立てる
val.animpat = 1;
val.ghandle = &imgTeki[0];
GetGraphSize(*(val.ghandle),&(val.image_w),&(val.image_h));
val.bounds_w=64;
val.bounds_h=64;
val.size=1.0;//敵の大きさ
val.hitmode=HIT_NORMAL;
val.pattern = 2; //どういう軌道を描くか
val.counter = 0;
break;
case TEKI_BANAYELOW3:
val.life = 0;//出現フラグを立てる
val.animpat = 1;
val.ghandle = &imgTeki[0];
GetGraphSize(*(val.ghandle),&(val.image_w),&(val.image_h));
val.bounds_w=64;
val.bounds_h=64;
val.size=1.0;//敵の大きさ
val.hitmode=HIT_NORMAL;
val.pattern = 3; //どういう軌道を描くか
val.power = 1.0
val.counter = 0;
break;
case TEKI_BANABLUE:
val.life = 0;//出現フラグを立てる
val.animpat = 1;
val.ghandle = &imgTeki[0];
GetGraphSize(*(val.ghandle),&(val.image_w),&(val.image_h));
val.shot_flag = 0;
val.bounds_w=64;
val.bounds_h=64;
val.size=1.0;//敵の大きさ
val.hitmode=HIT_NORMAL;
val.pattern = 4; //どういう軌道を描くか
val.counter = 0;
break;
case TEKI_BANARED:
val.life = 0;//出現フラグを立てる
val.animpat = 1;
val.ghandle = &imgTeki[0];
GetGraphSize(*(val.ghandle),&(val.image_w),&(val.image_h));
val.shot_flag = 0;
val.bounds_w=64;
val.bounds_h=64;
val.size=1.0;//敵の大きさ
val.hitmode=HIT_NORMAL;
val.pattern = 5; //どういう軌道を描くか
val.counter = 0;
break;
}
for(int i=0;i<TEKITAMA_MAX;i++){
tekiTama[i].life = 0;
tekiTama[i].shot_flag = 0;
}
return val;
}

で、その元となる構造体CharaDataはヘッダーファイルにて

struct CharaData{
float x; //左上の座標
float y; //
int *ghandle; //画像ハンドルへのポインタ
int animpat; //アニメーションコマ数
int life; //
int image_w; //イメージのサイズ
int image_h; //
int bounds_w; //当たり判定のサイズ
int bounds_h; //
int hitmode; //当たり判定属性
int power; //攻撃力
double size; //0.0～1.0
int timing;
int pattern; //移動パターン
int shot_flag
int counter
double angle;//角度保存用。
};

と定義されていて、他の部分でも使っています。
これで大丈夫でしょうか？