初心者ですが、画像を動かせません。

[MASSA]

VC++とDXライブラリを使用して、
「画像をメモリに読み込んで表示し、上キーと下キーを押すと画像が上下に移動する」というプログラムを作りたくて、下のようなプログラムを書いてみたのですが

code

#include "DxLib.h"
int Key[256];
int GetHitKeyStateAll_2(int GetHitKeyStateAll_InputKey[]){
char GetHitKeyStateAll_Key[256];
GetHitKeyStateAll( GetHitKeyStateAll_Key );
for(int i=0;i<256;i++){
if(GetHitKeyStateAll_Key==1) GetHitKeyStateAll_InputKey++;
else GetHitKeyStateAll_InputKey=0;
}
return 0;
}
int func_state = 0;
int x =220;
void draw_picture();
int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance,LPSTR lpCmdLine, int nCmdShow ){
ChangeWindowMode(TRUE);//ウィンドウモード
if(DxLib_Init() == -1 || SetDrawScreen( DX_SCREEN_BACK )!=0) return -1;//初期化と裏画面化

while(ProcessMessage()==0 && ClearDrawScreen()==0 && GetHitKeyStateAll_2(Key)==0 && Key[KEY_INPUT_ESCAPE]==0){
//↑ﾒｯｾｰｼﾞ処理 　 ↑画面をｸﾘｱ 　　　 ↑入力状態を保存　　　　　　　↑ESCが押されていない

switch(func_state){
case 0:
draw_picture();
default:
break;
}
ScreenFlip();
}

DxLib_End();
return 0;
}

void draw_picture(){
int image;
image = LoadGraph( "picture.png" ) ;
DrawGraph( 60 , x , image , FALSE ) ;
if(Key[KEY_INPUT_UP]==1) x=x-5;
if(Key[KEY_INPUT_DOWN]==1) x=x+5;
}

/code

上キーや下キーを押し続けても一度移動するだけなんです。
押している間は動かし続けたいのですが、どうすればよいでしょうか・・

初歩的な質問で申し訳ありません

[ひよこ]

このプログラムのKeyは押されていたら１を足し、押されてなかったら０になるというものなので
押されていたら１、２と増えますので、＝＝１ではなく＞＝１です。
if(Key[KEY_INPUT_UP]==1)
if(Key[KEY_INPUT_DOWN]==1)の部分を
if(Key[KEY_INPUT_UP]>=1)
if(Key[KEY_INPUT_DOWN]>=1)にします。
これで大丈夫だと思います。

[MASSA]

>ひよこ　　さん

ありがとうございます。解決しました！！

本当に助かりました。

[wing]

解決してしまったようですが、
気になることを少し書きます。

case 0:
draw_picture();
のあとにbreak; がありません。

void draw_picture(){
int image;
image = LoadGraph( "picture.png" ) ;
DrawGraph( 60 , x , image , FALSE ) ;
if(Key[KEY_INPUT_UP]==1) x=x-5;
if(Key[KEY_INPUT_DOWN]==1) x=x+5;
}
ここでは毎フレームごとにDrawGraphされているんですが、
LoadGraphも同じように毎回なされています。
危険なので、
int image;
image = LoadGraph( "picture.png" ) ;
は、ループの外に移しましょう。
while(ProcessMessage()==0 &&.....の上の方にということです。

あと、y座標のところをXと書くのも変だと思います。

[みけCAT]

int image;
image = LoadGraph( "picture.png" ) ;
は、ループの外に移しましょう。
while(ProcessMessage()==0 &&.....の上の方にということです。

これをやる際は、int image;はWinMain関数の外に出しておかないと、
imageが定義されていないと怒られると思います。
注意してください。
勘違いだったらすみません。

[MoNoQLoREATOR]

？？
もっと簡単に書けるのでは？

コード:

#include "DxLib.h"

int image;
char key[256];
int x,y;

int WINAPI WinMain(HINSTANCE hI, HINSTANCE hP, LPSTR lpC, int nC){
	ChangeWindowMode(TRUE);				
	if(DxLib_Init() == -1) return(-1);
	SetDrawScreen(DX_SCREEN_BACK);
	
	image = LoadGraph("image.png");
	
	while(ProcessMessage() == 0 && CheckHitKey(KEY_INPUT_ESCAPE) == 0){
         	ClsDrawScreen();
		
		GetHitKeyStateAll(key);
		
		//キーが押されていれば1が、
		//キーが押されていなければ0が格納されているので
		y -= key[KEY_INPUT_UP] * 5;
		y += key[KEY_INPUT_DOWN] * 5;
		x -= key[KEY_INPUT_LEFT] * 5;
		x += key[KEY_INPUT_RIGHT] * 5;

		DrawGraph(x,y, image, TRUE);
		
		ScreenFlip();
	}

	DxLib_End();
	return(0);
}

これだけでOK
ついでに四方向に移動できるようにしました。

[ISLe]

#GetHitKeyStateAll_2を使う場合も考慮して

コード:

        x += ((Key[KEY_INPUT_RIGHT] >= 1) - (Key[KEY_INPUT_LEFT] >= 1)) * 5;
        y += ((Key[KEY_INPUT_DOWN] >= 1) - (Key[KEY_INPUT_UP] >= 1))  * 5;

こんなふうにも書けますよ。

この書き方を懐かしいと感じるひとがプログラムを始めたころには掛け算がひとつふたつ減ることに大きな意味があったものですが。