float playerX, playerY; //プレイヤー座標x, y。txtファイルから読み込んでいるので、値は入れてあります。
void GameInit(); //ステージの初期化関数のプロトタイプ宣言
void GameInit(){
memset( &playerX, (int)playerX, sizeof(float) );
memset( &playerY, (int)playerY, siseof(float) );
}すみません。h2so5 さんが書きました:ビルドの通るコードを貼りましょう。
返信ありがとうございます。ISLe さんが書きました:デバッグ情報は容量がデカいだけじゃなく、個人情報が漏れる可能性もありますよ。
学校のPC使ってるっぽいので学校の関係者くらいにしか役に立たない情報かもしれませんが。
返信、そして何度も答えていただきありがとうございます。softya（ソフト屋) さんが書きました:不用意にファイルがデカくなるのでRENSYU.sdfやipchフォルダとDebugフォルダは圧縮ファイルには不要ですので取り除いてから圧縮して下さい。
GameInit関数の問題点を指摘しますと
１．memsetは必要ありません。
普通に代入できるのにmemsetを使う意味はまったくなくバグの危険性が増えるだけです。
２．値が不定のものを代入しても初期化にはなりませんので、デバッガで確認して下さいね。
コレは何度も説明したはずですが、動作がおかしかったらまずデバッガで確認が基本です。
どういう問題があったか報告をしてください。
３．初期化の仕方がすごく間違っています。
プログラムの流れを意識してと言うのも何度も書いたはずですので、stage.Wallがどんな流れで初期化されるか説明してみて下さい。
※ すごく大事な基本の基本が分かっていないとしか思えない組み方をしています。
Ｃ言語の入門書を読みなおしたほうが良いかも知れません。
表示出来る出来ない以前の確認をまずお願いします。
主に３の問題なのですが関数の実行順番や何処の関数でどういう値なのかの情報が抜け落ちています。C.C さんが書きました:返信、そして何度も答えていただきありがとうございます。
1.ほかの方法を探します。
2.txtファイルでのプレイヤーの座標はx 576, y 360なのですが、GameInitではx 3.003925, y 4.3900599e+024に設定されています。
3.txtファイルのx座標, y座標, 幅, 高さをStageInfo構造体に入れています
そこが分からないということでしょうか。C.C さんが書きました:すみません。InitGameでは画面の状態をリセットし、その後GameMapLoadで初期化した位置に表示したいと考えています。
遅くなって申し訳ありません。保存用の変数を用意すればよいのでしょうか？softya（ソフト屋) さんが書きました:そこが分からないということでしょうか。C.C さんが書きました:すみません。InitGameでは画面の状態をリセットし、その後GameMapLoadで初期化した位置に表示したいと考えています。
GameMapLoad()の時の情報を何処かに保存しておいて、InitGameで毎回保存しておいた値から復元するのが一般的です。
現状のコードは、GameMapLoad()で保存していない ＆ InitGameでどこからともなく不定な値を持ってきている という状態です。
すみませんが、GameMapLoad関数を見ていただけませんか？softya（ソフト屋) さんが書きました:上書きたり書き換えれたら元の状態に戻せませんので、まるまる複製した元の状態をとっておくしかありません。
必要なのは、元の型に合わせたそっくり同じ型の別の名前の変数です。
wallXなど単なるスカラ変数は複数の要素を記憶することはできません。C.C さんが書きました:すみませんが、GameMapLoad関数を見ていただけませんか？softya（ソフト屋) さんが書きました:上書きたり書き換えれたら元の状態に戻せませんので、まるまる複製した元の状態をとっておくしかありません。
必要なのは、元の型に合わせたそっくり同じ型の別の名前の変数です。
sscanf_sで変数に値を読み込み、これを構造体変数に代入しているのですが、(プレイヤーも同じようにやるつもりです)sscanf_sで使っている変数をそのまま保存用の変数としてはつかえないのですか？
#include <stdio.h>
int main(void)
{
int i,w;
for( i=0; i<3; i++ ) {
w = i*3;
printf( "%d-%d\n",i,w );
}
printf( "final:%d\n",w );
}おはようございます。softya（ソフト屋) さんが書きました:配列を用意するなら、同じ構造のものを用意したほうが混乱がなく素直に処理できます。
なので、stageと同じ型を用意したほうがシンプルで分かりやすくなります。
playerも構造体にまとめたほうが良いでしょう。
struct StageInfo{
int timerstart; //待機タイマーの開始時刻
int gamestarttime; //ゲームの開始時間
int gametime; //ゲーム開始からの経過ミリ秒
float playerX;
float playerY;
float mapsize_w, mapsize_h; //マップサイズ
Character Wall[MAXMAPCHARA];
Ball Ball[MAXBALLCHARA];
int num_mapchara;
bool isheroleft; //主人公の向き
bool isontheground;
};
//保存用の構造体
struct Retain{
int timerstart; //待機タイマーの開始時刻
int gamestarttime; //ゲームの開始時間
int gametime; //ゲーム開始からの経過ミリ秒
float playerX;
float playerY;
float mapsize_w, mapsize_h; //マップサイズ
Character Wall[MAXMAPCHARA];
Ball Ball[MAXBALLCHARA];
int num_mapchara;
bool isheroleft; //主人公の向き
bool isontheground;
};//マップの読み込み
int GameMapLoad(char *filepath){
int f; //ファイルハンドル
char buf[1024]; //テキスト読み込みバッファ
f = FileRead_open(filepath);
if (f == 0) return -1; //読み込みエラー
//マップサイズと主人公初期位置読み込み
if(FileRead_gets( buf, 1023, f ) == -1) return -1;
sscanf_s(buf, "%f, %f, %f, %f", &retain.mapsize_w, &retain.mapsize_h, &retain.playerX, &retain.playerY);
stage.playerX = retain.playerX;
stage.playerY = retain.playerY;
stage.mapsize_w = retain.mapsize_w;
stage.mapsize_h = retain.mapsize_h;
//地形オブジェクト読み込み
if(FileRead_gets( buf, 1023, f ) == -1) return -1;
int imax;
sscanf_s(buf, "%d" , &imax); //行数取得
float wallX, wallY, wallW, wallH, angle;
int id;
for(int i = 0; i < imax; i++){
if(FileRead_gets( buf, 1023, f ) == -1) return -1;
sscanf_s( buf, "%f, %f, %f, %f, %f, %d", &wallX, &wallY, &wallW, &wallH, &angle, &id);
if((int)wallW == 48){
ch.ID = WALL_48;
ch.ID_X = (int)wallX;
ch.ID_Y = (int)wallY;
ch.ID_W = (int)wallW;
ch.ID_H = (int)wallH;
ch.ID_Angle = angle;
}
if((int)wallW == 192){
ch.ID = WALL_192;
ch.ID_X = (int)wallX;
ch.ID_Y = (int)wallY;
ch.ID_W = (int)wallW;
ch.ID_H = (int)wallH;
ch.ID_Angle = angle;
}
if((int)wallW == 284){
ch.ID = WALL_284;
ch.ID_X = (int)wallX;
ch.ID_Y = (int)wallY;
ch.ID_W = (int)wallW;
ch.ID_H = (int)wallH;
ch.ID_Angle = angle;
}
if((int)wallW == 568){
ch.ID = WALL_568;
ch.ID_X = (int)wallX;
ch.ID_Y = (int)wallY;
ch.ID_W = (int)wallW;
ch.ID_H = (int)wallH;
ch.ID_Angle = angle;
}
if(id == 1){
ch.ID = GOAL_FLAG;
ch.ID_X = (int)wallX;
ch.ID_Y = (int)wallY;
ch.ID_W = (int)wallW;
ch.ID_H = (int)wallH;
ch.ID_Angle = angle;
}
ch.used = true;
stage.Wall[stage.num_mapchara] = ch;
retain.Wall[stage.num_mapchara] = stage.Wall[stage.num_mapchara];
stage.num_mapchara++;
retain.num_mapchara = stage.num_mapchara;
}
FileRead_close(f);
return 0;
}//初期化関数
void GameInit(){
ZeroMemory(&stage, sizeof(stage)); //StageInfo構造体の変数をすべて0にする
stage.playerX = retain.playerX;
stage.playerY = retain.playerY;
stage.mapsize_w = retain.mapsize_w;
stage.mapsize_h = retain.mapsize_h;
for(int i = 0; i <= retain.num_mapchara; i++){
stage.Wall[i] = retain.Wall[i];
}
stage.gamestarttime = g_lasttime;
}自己解決しました。softya（ソフト屋) さんが書きました:ZeroMemory(&stage, sizeof(stage)); //StageInfo構造体の変数をすべて0にする
で全部0にしているのからでは？
ステージ最初の状態に戻すなら他にも設定すべきものがあるのではないでしょうか。
上にも書きましたがstruct StageInfoだけで十分でstruct Retainは不要です。
まぁ、その手はあることはあるんですが他にも初期化するものがあるような・・・。C.C さんが書きました:自己解決しました。softya（ソフト屋) さんが書きました:ZeroMemory(&stage, sizeof(stage)); //StageInfo構造体の変数をすべて0にする
で全部0にしているのからでは？
ステージ最初の状態に戻すなら他にも設定すべきものがあるのではないでしょうか。
上にも書きましたがstruct StageInfoだけで十分でstruct Retainは不要です。
GameInitの中でLoadGameMapを呼び出すとうまく表示されました。
ありがとうございました。
初期化が必要なものがでてきたら、自分で考えてみます。まぁ、その手はあることはあるんですが他にも初期化するものがあるような・・・。
ここのルールですので、最終コードの貼り付けをお願いします。
//初期化関数
void GameInit(){
ZeroMemory( &stage, sizeof(stage) );
stage.playerX = retain.playerX;
stage.playerY = retain.playerY;
stage.mapsize_w = retain.mapsize_w;
stage.mapsize_h = retain.mapsize_h;
for(int i = 0; i <= retain.num_mapchara; i++){
stage.Wall[i] = retain.Wall[i];
}
GameMapLoad("media\\stage01.txt");
stage.gamestarttime = g_lasttime;
}
メニュー
Clock
統計データ
Link to us
カウンター
カレンダー
最新メンバー
投稿数