プログラマ専用SNS ミクプラ

このようなプログラムを書いたのですが以下のエラーが出て実行されません。

(5) : error C4430: 型指定子がありません - int と仮定しました。メモ: C++ は int を既定値としてサポートしていません
(5) : error C2086: 'int img' : 再定義されました。
(3) : 'img' の宣言を確認してください。

コンパイラはVC2008EEを使用しています。

すでに定義している変数と同じ名前の関数を定義することはできません。

関数名を変えましたが同じエラーが出ました

こうしてみてはどうでしょうか。
int imgをimg２にして関数のとこで名前をimgにしてみました。これでエラーは出ないと思います

同じエラーが出ます

もう一度エラー内容を教えていただけませんか？
もしかしたら微妙に変わっているかもしれないんでw

DrawGraph( 0, 0, img, TRUE );の場合下のエラーが増えました。
(8) : error C2664: 'DxLib::DrawGraph' : 3 番目の引数を 'void (__cdecl *)(void)' から 'int' に変換できません。(新しい機能 ; ヘルプを参照)

DrawGraph( 0, 0, img2, TRUE );の場合同じエラーです。

紫電 さんが書きました:こうしてみてはどうでしょうか。

そのコードが正しくなさそうなので、質問者さんは混乱されているかもしれません。

紫電 さんが書きました:

コード:

int img2;
img2 = LoadGraph( "../dat/img/menu/menu.png" );
void img(){
    DrawGraph( 0, 0, img, TRUE );
}

変数名をimg2にしたのであれば、
DrawGraph()の第3引数もimg2にしないとまずいですよね？私は詳しくありませんけれど。

そもそも関数外に代入文があるのが大問題だと思いますけど。

box さんが書きました:

紫電 さんが書きました:こうしてみてはどうでしょうか。

そのコードが正しくなさそうなので、質問者さんは混乱されているかもしれません。

紫電 さんが書きました:

コード:

int img2;
img2 = LoadGraph( "../dat/img/menu/menu.png" );
void img(){
    DrawGraph( 0, 0, img, TRUE );
}

変数名をimg2にしたのであれば、
DrawGraph()の第3引数もimg2にしないとまずいですよね？私は詳しくありませんけれど。

ああ、そうでしたww

DrawGraph( 0, 0, img２, TRUE );

ってかんじですねw

関数内に書いたら正常に動作しました。
ありがとうございます。

softya（ソフト屋) さんが書きました:そもそも関数外に代入文があるのが大問題だと思いますけど。

そりゃそうですね。

それとLoadGraph()とDrawGraph()を同じ関数内に書くとメモリリークを起こします。
徐々にメモリ消費量が増えていき、やがてＤＸライブラリの管理個数の限界を超えて画像が消えます。
「ゲームプログラミングの館」のあちこちに書いてあると思いますが読み込みと表示の関数は必ず分けるようにして下さい。

softya（ソフト屋) さんが書きました:それとLoadGraph()とDrawGraph()を同じ関数内に書くとメモリリークを起こします。
徐々にメモリ消費量が増えていき、やがてＤＸライブラリの管理個数の限界を超えて画像が消えます。
「ゲームプログラミングの館」のあちこちに書いてあると思いますが読み込みと表示の関数は必ず分けるようにして下さい。

そうなんだw

勉強になりましたw

紫電 さんが書きました:

softya（ソフト屋) さんが書きました:それとLoadGraph()とDrawGraph()を同じ関数内に書くとメモリリークを起こします。
徐々にメモリ消費量が増えていき、やがてＤＸライブラリの管理個数の限界を超えて画像が消えます。
「ゲームプログラミングの館」のあちこちに書いてあると思いますが読み込みと表示の関数は必ず分けるようにして下さい。

そうなんだw

勉強になりましたw

より正確に書くとLoadGraph()した画像はＤＸライブラリの管理ハンドルを１つ消費してDeleteGraph()や画面モードを切り替えない限りメモリ内に常駐します。
だから、いつでもDrawGraph()で表示できるのですが、もしこのプログラムのように毎回呼び出す関数img()の中にimg2 = LoadGraph( "../dat/img/menu/menu.png" );があるとimg2;に毎回新しい画像ハンドルが設定されます。
これだと前の画像ハンドルは見かけ上は消えたように見えますが、ＤＸライブラリ内には前の画像ハンドル・画像データは残ったままなので秒間６０個づつ画像ハンドルとメモリを消費して行くことになります。
まぁ、毎回ファイルを読み込むので遅いですし、30KB程度の画像だとしても１分も立たない内に30KBx60フレームx60秒=約１００ＭＢを消費します。