dxlib キー入力で押した時と押されている間を実装したい。

Question

提示コードですがここからどうすればキーが押された時と押されている間の取得出来るのでしょうか？それぞれを関数化したいです。
実装方法を含めどうしたらいいかわからないです。どうすればいいのでしょうか？

参考サイト　https://www.ay3s-room.com/entry/dxlib-key-input

#include "Input.hpp"




static int Key[256];
// キーの入力状態を更新する
int gpUpdateKey() {
    char tmpKey[256];               // 現在のキーの入力状態を格納する
    GetHitKeyStateAll(tmpKey);      // 全てのキーの入力状態を得る
    for (int i = 0; i < 256; i++) {
        if (tmpKey[i] != 0) {       // i番のキーコードに対応するキーが押されていたら
            Key[i]++;               // 加算
        }
        else {                      // 押されていなければ
            Key[i] = 0;             // 0にする
        }
    }
    return 0;
}

bool getKeyDown(int keycode)
{
    if (Key[keycode] == 1)
    {
        return true;
    }
    else
    {
        return false;
    }

}

Comments

[dodox86]

> どうすればキーが押された時と押されている間の取得出来るのでしょうか？それぞれを関数化したいです。

このままでは単なる作業依頼です。まず考えて自分で実装してください。1フレームごとにgpUpdateKey()を呼ぶとして、キーが押されている間、キーごとのtmpKeyの各要素がインクリメントされ続けるのですから、そこから判断することができるはずです。ただし、安易にインクリメントし続けると、255から0になったときに判断を誤ることになります。

[退会済みユーザー]

なるほど。わかりました。

[dodox86]

> tmpKeyの各要素がインクリメントされ続けるのですから、
この点、誤りのある記述でした。tmpKeyは単なるチェック用で、インクリメントはkey[]の方で行う操作ですね。

[dodox86]

[2021/02/16 11:32]の質問の編集を読んで：

> 実装方法を含めどうしたらいいかわからないです。どうすればいいのでしょうか？

．．．そう来ましたか。（汗）　この質問に限らないのですが、もはや、自ら考えることを放棄してませんか？

[退会済みユーザー]

よく考えたのですがどうしたらいいのでしょうか？本当にどうしたらいいかわからないのです。、

[dodox86]

「DXライブラリ」のタグはあるので、追加した方が良いと思います。

[fana]

方法論がわからないのに gpUpdateKey() とか Key[256]; とかいう実装を持ってくる，その行動の意味がわからない．
使い方が不明なコードを何故かどこぞからパクって，パクり元とは別の所で「わからん」とか……

[dodox86]

(fanaさんの指摘を読んで）もしかすると、そもそもDXライブラリが提供するGetHitKeyStateAll()関数と、それを利用した自作関数であるgpUpdateKey()がどういう働きをしているか理解されていない、ということでしょうか。もしそうだとすると、コードを提示しても恐らく理解できないですね。（コピッて終わり）

[退会済みユーザー]

質問ですがクラスにしても間違えではないのでしょうか？

[dodox86]

C++でクラス化しようがC言語の関数であろうが、適切に、要望の動きができれば別に間違えではないと思いますが、クラス化するかどうかは解決の本質ではないと思います。

[kazuma-s]

> キーが押されている間、キーごとのtmpKeyの各要素がインクリメントされ続けるのですから、そこから判断することができるはずです。ただし、安易にインクリメントし続けると、255から0になったときに判断を誤ることになります。

インクリメントされるのは、char の tmpKey[i] ではなく、int の Key[i] ですから、
4294967296/60秒 = 828.5日　推し続けないと 0 になりません。

[dodox86]

＞@kazuma-sさん
ご指摘のとおりですね。Key[]の方の型を見誤っていました。どうもありがとうございます。

[fana]

押され続けた時間の長さに関する情報の取得が不要ならば，
・前回の状態（押されてたかどうか）
・今回の状態
だけあればよいわけで．
256種類のキーを扱う必要があるとしても，覚えとく情報は256bit(=32byte)で済む話．
そこに800日稼働できるカウンタを用意するという発想自体がそもそもどうかしてる感．

Answer 1

以下のコードにより実装、解決しました。クラスにすることにしました。
落ち着いて思考して理解出来ました。

ボタンを押している間のコードはそんなに長い間押していることないため取り得ずint 型にしました。

#include "Input.hpp"

// コンストラクタ
Input::Input()
{
    //キー状態配列を初期化
    for (int i = 0; i < 256; i++)
    {
        Key[i] = 0;
    }
}

//キー入力状態を更新
void Input::Update()
{    
    char tmpKey[256];   // 現在のキーの入力状態を格納する
    GetHitKeyStateAll(tmpKey);  // 全てのキーの入力状態を得る
    for (int i = 0; i < 256; i++) {
        if (tmpKey[i] != 0) {   // i番のキーコードに対応するキーが押されていたら
            Key[i]++;   // 加算
        }
        else {              
            // 押されていなければ
            Key[i] = 0;   // 0にする
        }
    } 
}

//キーを押した時だけ
bool Input::getKeyDown(unsigned char KeyCode)
{
    if (Key[KeyCode] == 1)
    {
        return true;
    }
    else
    {
        return false;
    }
}

//キーを押している時
int Input::getKeyDownHold(unsigned char KeyCode)
{
    return Key[KeyCode];
}

//　デストラクタ
Input::~Input()
{

}

#ifndef ___INPUT_HPP_
#define ___INPUT_HPP_

#include "dxlib.h"

class Input {

public:
	Input();	//コンストラクタ
	~Input();	//デストラクタ
	
	void Update();								//キー入力状態を更新

	bool getKeyDown(unsigned char KeyCode);		//キーが押された瞬間だけ
	int  getKeyDownHold(unsigned char KeyCode);	//キーが押されている間

private:
	int Key[256];	//キー状態

};



#endif

#include "Player.hpp"

//コンストラクタ
Player::Player() : Actor()
{
	Sprite = LoadGraph("Assets/Player.png");
	input = std::make_shared<Input>();
}



//計算
void Player::Update()
{
	input->Update();
}

//描画
void Player::Draw()
{
	if (input->getKeyDownHold(KEY_INPUT_LEFT) > 0) {
		DrawGraph(100, 100, Sprite, true);
		printf("ああああ\n");
	}
	else
	{
		printf("いいいいい\n");
	}
}