[GLFW] glfwSetKeyCallback()関数を使ってキーが押された瞬間を検出したい。

Question

提示コードの///コメント部内の部のコードですがglfwSetKeyCallback();イベント処理関数を使ってキーが押された瞬間１と返す方法が知りたいです。提示画像ですが提示コードのように実装したのですがなぜか１という値が数フレーム返って来てしまいます。本来押された瞬間であれば１は一回だけで後は２の値が来るはずなのですがログを見ると数フレーム来てしまっています。これは何が原因なのでしょうか？

参考サイト: https://www.glfw.org/docs/latest/group__input.html#ga1caf18159767e761185e49a3be019f8d

//#define GLEW_STATIC	//スタティックリンク
#include <iostream>
#include <fstream>
#include <cstdlib>
#include <vector>
#include <glew/include/GL/glew.h>
#include <glfw/include/GLFW/glfw3.h>



#include "Window.hpp"
#include "Shape.hpp"
#include "Shader.hpp"

Object::Vertex rectangleVertex[4] =
{
	{-0.5f,-0.5f},
	{0.5f,-0.5f},
	{0.5f,0.5f},
	{-0.5f,0.5f}
};


int main()
{
	if (glfwInit() == GL_FALSE)
	{
		std::cerr << "glfw初期化失敗。" << std::endl;
		return -1;
	}


	atexit(glfwTerminate);	//プログラム終了時の処理を登録
	Window window;

	//OpenGL Verison 3.2 Core Profile　を選択する
	glfwWindowHint(GLFW_CONTEXT_VERSION_MAJOR,3);
	glfwWindowHint(GLFW_CONTEXT_VERSION_MINOR,2);
	glfwWindowHint(GLFW_OPENGL_FORWARD_COMPAT,GL_TRUE);
	glfwWindowHint(GLFW_OPENGL_PROFILE,GLFW_OPENGL_CORE_PROFILE);

	Shape* shape = new Shape(2, 4, rectangleVertex);
	
	Shader shader("Test.vert","Test.frag");
	shader.setBindAttribVertex(0, "Position");
	shader.setBindAttribFragment(0, "fragment");


	glClearColor(1.0, 0.0, 0.0, 1.0);	//背景色
	while (window)
	{
		glClear(GL_COLOR_BUFFER_BIT);	//カラーバッファをクリア
		shader.Active();
		

		
		printf(": %d\n", window.getKeyInput(GLFW_KEY_SPACE));

		





		window.SwapBuffers();	//ダブルバッファリング
	}

}

#include "Window.hpp"

//コンストラクタ
Window::Window(int width, int height, const char* title)
	:window(glfwCreateWindow(width, height, title, NULL, NULL)),
	wheel(0)
{
	std::fill(std::begin(keyStatus),std::end(keyStatus),0);

	if (window == NULL)
	{
		std::cerr << "ウインドウ生成失敗" << std::endl;
		exit(1);
	}

	glfwMakeContextCurrent(window);	//コンテキストを作成

	glewExperimental = GL_TRUE;
	if (glewInit() != GLEW_OK)
	{
		std::cerr << "GLFW 初期化失敗" << std::endl;
		exit(1);
	}

	atexit(glfwTerminate);	//プログラム終了時の処理を登録
	glfwSwapInterval(1);	//垂直同期


	//イベント処理
	glfwSetWindowUserPointer(window, this);	//このインスタンスのthis
	glfwSetWindowSizeCallback(window, Resize);	//ウインドウサイズを変更する時に呼び出す処理
	glfwSetScrollCallback(window,Wheel);		//マウスホイール操作時に呼び出す処理
//////////////////////////////////////////////////////////////////////////////////////////////
	glfwSetKeyCallback(window, KeyBoard);		//キー入力時に呼び出す処理
//////////////////////////////////////////////////////////////////////////////////////////////


	Resize(window, width, height);

}

//サイズ変更
void Window::Resize(GLFWwindow* const win, int width, int height)
{
	int fbWidth, fbHeight;
	glfwGetFramebufferSize(win, &fbWidth, &fbHeight);
	glViewport(0, 0, fbWidth, fbHeight);

	Window* const instance = (Window*)glfwGetWindowUserPointer(win);

	if (instance != NULL)
	{
		instance->size.x = (GLfloat)width;
		instance->size.y = (GLfloat)height;		
	}
}

//ウインドウサイズを取得
const glm::vec2 Window::getSize() const 
{
	return size;
}


//キー入力を取得
const int Window::getKeyInput(int key)const
{
	return keyStatus[key];
}

//マウスホイール
void Window::Wheel(GLFWwindow* win, double x, double y)
{
	Window* const instance = (Window*)glfwGetWindowUserPointer(win);

	if (instance != NULL)
	{
		instance->wheel += (GLfloat)y;
	}
}

/////////////////////////////////////////////////////////////////////////////////////////////////
//キー入力
void Window::KeyBoard(GLFWwindow* win, int key, int scancode, int action, int mods)
{
	Window* const instance = (Window*)glfwGetWindowUserPointer(win);

	if (instance != NULL)
	{	
		instance->keyStatus[key] = action;		
	}
}
////////////////////////////////////////////////////////////////////////////////////////////////

//bool 演算子
Window::operator bool()
{
	glfwPollEvents();	//イベントを取り出す

	//ウインドウを閉じる必要があれば false
	if (glfwWindowShouldClose(window) != 0)
	{
		return false;
	}
	else {
		return true;
	}
}

//ダブルバッファリング
void Window::SwapBuffers()const
{
	glfwSwapBuffers(window);
}

//デストラクタ
Window::~Window()
{
	glfwDestroyWindow(window);
}

Comments

[退会済みユーザー]

文章を修正しました。

Answer 1

GLFW なら callback を使えばどうでしょう．

glfwSetKeyCallback で設定した callback 関数の引数にイベント情報（押した，離した，etc）が来るから，
それで判定するので十分ならば，自前で押された履歴みたいなのを記憶するよりは楽かも．

---

編集された件について：

なぜか１という値が数フレーム返って来てしまいます。本来押された瞬間であれば１は一回だけで後は２の値が来るはずなのですがログを見ると数フレーム来てしまっています。これは何が原因なのでしょうか？

1を複数回表示しているのはあなたのメインループです．
callback関数の引数に1が複数回連続で来ているわけではありません．

あなたの言う「数フレーム」の間に，callback関数が何回呼ばれているのかを確認すべきです．
というか，キーを押しっぱなしにした時のcallback関数の呼ばれ具合を最初に確認すべき．
キーリピート（：テキストエディタ上でキーを押しっぱなしにしたときの入力具合）と同様の呼び出しタイミングになるはず．

キーを押した際に callback関数が呼ばれて，値が1になった後（キーを押しっぱなしにしたとして），
次に callback関数が呼ばれて値が2になるまでの間に，「数フレーム」が経過しています．

Comments

[退会済みユーザー]

イベント処理では無理だということなので単純にglfwGetKey();を呼ぶことにしました。

[fana]

> 無理

という結論がどこから来たのかわからないが…
callback関数が引数値1で呼ばれた際に，その旨をどこか（キーが押された瞬間を知りたい側）に向けて伝達すればよいだけの話ではないのだろうか？
それこそcallbackの仕組みにするとかで．

[退会済みユーザー]

ちょっと自分には難しいのでわからないのでglfwGetKey();で行います。提案ありがとうざいました。

[dodox86]

あなたは一体何を言っているのか。そもそもの質問が「glfwSetKeyCallback()関数を使ってキーが押された瞬間を検出したい」でしょうに。それに則って回答をいただいているのが分かりませんか。

[fana]

質問内容を丸ごと書き換えるからこういうことになる．
（根本から書き換えるのをやめろ，という旨の指摘は何度かされたと思うのだが）

元々は，「キーが押された瞬間が知りたい」みたいな話だったハズ．
で，callbackを使えば，キーが押されたタイミングである旨が引数でご丁寧に渡されてくる話になっているのだから，それで解決できるのでは？という旨の回答をした．
押されたタイミングで通知が来るというのに，これ以上何を望むというのか？

[fana]

自分で勝手に書いたループ内の処理の進捗単位である「フレーム」と
その他の処理が何もせずとも勝手にタイミング同期してくれるなんてことはないわけで．
そこの調整くらいは自分で必要十分な形にやろうよ，っていう．

[fana]

余談：

glfwGetKey()が実際どういう実装になっているのか知らないけども，
もしもそれを使うことで，キーの（物理的な？）状態を「フレーム」の間隔でポーリングする形になるのだとしたら，キー押下時間がポーリング間隔よりも短い場合に入力を取りこぼし得る，ということは知っておくべきかもしれない．
（例えば，「フレーム」の処理が過度に重くなると，操作困難になるかもしれない．）
（GLFWには，そういうことに対処できる方法として(?) GLFW_STICKY_KEYS なんていうのがあるみたい）

[退会済みユーザー]

この質問者に必要なのは、場当たり的な解決法の提示よりも自分で考えさせる事じゃないかなと思ったり。
他から拾ってきたコードや、他人に提示された提案を自分で理解・検証したりせずそのまま自分のプロジェクトに持ってきて、そこで不具合を起こしても理解出来ないしデバッグもしないから、また質問してデバッグ作業を丸投げするの繰り返し。
過去の質問の傾向がずっとそうです。

[fana]

「ログとしてどうの」とかなんとかいう建て前を盾に取れば，
回答は質問者のみに向けたものというわけではないから，「場当たり的な解決法」を示すことにも一定の意味はあるのではなかろうか．
（その意味においては，この質問者の問題が解決しなかろうが，自分で一切考えなかろうが，どうでもいい．）

まぁ，現状の質問内容：
> これは何が原因なのでしょうか？
という話に答えるだけだと，これはこの質問者の現在のコード固有のバグの話にしかならないから，前記の建て前すら成立しないのだけども…

[退会済みユーザー]

今更質問ですがつまりイベント型ではキーが押された瞬間だけ‘１であとは２，３が返ってくるという処理はできないのでしょうか？実現不可能ということでしょうか？

[fana]

> キーが押された瞬間だけ

「瞬間」って何だよっていう．

> あとは２，３が返ってくる

とかいうけど，それを「返す側」から見たら，あなたが言うところの「(1を返すべき)瞬間」が「過ぎ去ったこと」はどうやってわかるの？
「返す値」をどんなタイミングで　1 から 2だの3だのに切り替えればいいの？ っていうところは，「返す側」に判断つく話なのか？

判断が付くのであれば，その判断処理を実装して適切な返せばいいだけ．
判断が付かないのであれば，その仕事は「その判断をすることが可能な別の場所」でやればいいんじゃないですかね．

---

っていうか，アカウントをまたいで話続ける形はどうなん？

[退会済みユーザー]

すいませんでした。

Answer 2

提示サイトですが「キーが押された１フレームだけ一瞬だけを取得する」関数はあるのでしょうか？自分は押している間ずーっとif文が通るコードしか作れず困っています。モード切り替えの再に一回だけif文の中を実行できるような関数がほしいのですがどうすればいいのでしょうか？

C++ならbool型の変数「key」を作り、キーが押した瞬間にkeyの値を変更すれば良いかと思います。

bool key = false;

if (glfwGetKey(Window, GLFW_KEY_F1) == GLFW_PRESS)
{
    if (key == false){
        key = true;
        //キーを押した瞬間の処理
    }
}else{
    key = false;
}

Comments

[退会済みユーザー]

自分もそれを思いつきましたが。ライブラリがあるだろ？と思ったのですがないものはないので自分作るしかないようですねわかりました。