[glfw]将来性のためキー入力はどういった形で実装するべきか？オブジェクト指向が知りたい。

Question

キー入力についてなのですが場当たりなことはせず将来性を考えた設計をする際にどうやってキー入力を実装したらいいコードになるのか知りたいです。現状はWindowクラスにウインドウコンテキスト当を管理しているのですが"ここに実装するべき"コメント部に実装するべきなのでしょうか？またゲーム内容とフレームワーク部でオブジェクト指向を分けたいのでゲームプログラム内でwindowコンテキスト変数を直接触りたくなりません。現在は描画物は一つですがGame クラスを作ってそこにゲーム内容を記述してくといった規模の大きなプロジェクトになる場合どういった手法でプロ人は実装するのでしょうか？

//#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();

		shader.setUniform2fv("location",window.getLocation());
		shader.setUniform1f("scale",window.getScale());
		shader.setUniform2fv("size",window.getSize());



		shape->Draw();	//表示物





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

}

#include "Window.hpp"

Window::Window(int width, int height, const char* title)
	:window(glfwCreateWindow(width,height,title,NULL,NULL)),
	scale(100.0f),
	location{ 0.0f,0.0f }

{
	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);		//マウスホイール操作時に呼び出す処理

	Resize(window, width, height);

}

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

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

	if (instance != NULL)
	{
		instance->size[0] = (GLfloat)width;
		instance->size[1] = (GLfloat)height;
	}
}

const glm::vec2 Window::getSize() const 
{
	return size;
}

GLfloat Window::getScale() const
{
	return scale;
}

const glm::vec2 Window::getLocation()const
{
	return location;
}


Window::~Window()
{
	glfwDestroyWindow(window);
}

void Window::Wheel(GLFWwindow* win, double x, double y)
{
	Window* const instance = (Window*)glfwGetWindowUserPointer(win);

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

Window::operator bool()
{
	glfwPollEvents();	//イベントを取り出す
//////////////////////////////////////////////////////////////////
//ここに書くべきか？
//////////////////////////////////////////////////////////////////

	//マウスをクリックしたとき
	if (glfwGetMouseButton(window, GLFW_MOUSE_BUTTON_1) != GLFW_RELEASE)
	{
		//マウスカーソルの座標を取得
		double x, y;
		glfwGetCursorPos(window, &x, &y);

		//マウスカーソルの正規化デバイス座標系上での座標を求める
		location.x = (GLfloat)x * 2.0f / size.x - 1.0f;
		location.y = 1.0f - (GLfloat)y * 2.0f / size.y;
	}

	//ESCキーを押したら強制終了
	if (glfwGetKey(window, GLFW_KEY_ESCAPE))
	{
		return false;
	}

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

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

#ifndef ___WINDOW_HPP
#define ___WINDOW_HPP

#include <iostream>
#include <glew/include/GL/glew.h>
#include <glfw/include/GLFW/glfw3.h>
#include "glm/glm.hpp"


class Window
{
public:
	Window(int width = 640, int height = 480, const char* title = "Hello!");
	virtual ~Window();

	explicit operator bool();
	void SwapBuffers()const;	//ダブルバッファリング
	static void Resize(GLFWwindow* const window, int width, int height);	//サイズ変更
	const glm::vec2 getSize() const;
	GLfloat getScale() const;
	const glm::vec2 getLocation()const;
	static void Wheel(GLFWwindow* win, double x, double y);

	GLfloat scale;		//ワールド座標系に対するデバイス座標系の拡大率

private:

	GLFWwindow *const window;	//ウインドウコンテキスト

	glm::vec2 size;		//ウインドサイズ
	glm::vec2 location;	//図形の正規化デバイス座標上での位置


};

#endif

Comments

[K_3578]

コピペして作るのが精一杯、なんならコピペですら作れてない貴方が心配するような事じゃないでしょ。
将来性を心配するなら一度基礎から教材とかで体系的に学び直そう。
時間が本当に勿体ないよ。

[退会済みユーザー]

調べたことを理解しながら手打ちで実装したいのですが...

[K_3578]

貴方、2017年からずーっと同じような質問してるでしょ？殆どはその500件以上の質問についた回答の中に
答えが詰まってる。
今でも回答してくれてる人いるけど、その人達が何を言ってるのかも理解出来てないから、
ずーっと同じようなところで躓き続ける。そりゃそうだ、基礎がわかってないのに動かして遊べる訳がない。

プロフィールに書いてるとおりだとしたら年齢も近いし、
恐らく学びたい、出来るようになりたいという気持ちだけはわかるから伝える。
頼むから基礎からやり直してくれ。多分数ヶ月もあれば学び直せるし、
間違いなく今より回答者の回答が理解出来るようになる。

[m.ts10806]

他人に説明出来て初めて「理解できる」わけですが、
コード自体を適切に説明されたコメントを見たことがありません。
「やりたいこと」だけ書いてある質問ばかりです。

[K_3578]

結局説得も通じなかった。
何年泳げもしない河を泳いで渡ろうとしてるんだ。
橋を架けろ。三回ぐらい飛び込んで渡ろうとしている間に作れる。

Answer 1

私は

プロ人

とかいう者ではないので，実際上の方法を述べられる立場にありませんから，単なる思い付きのアイデアを書くことになりますが…

---

「ゲーム」のロジックの側から見て，（ウィンドウやら何やらが実装されている世界の側に対して）要求したい事柄を考えて（整理して）みれば良いのではないでしょうか．

例えば，

• キー入力状態が知りたい
• ゲーム側の都合でプログラムを終了に向かわせたい（ゲームオーバーの時とかにプログラムが終わるようにしたい）
• ゲーム側に必要な描画手段を伝えたい（それを使って描画処理を行ってもらう）
• etc...

だとか．

そういった要求を行うための手段（それらのための関数群であったり，インタフェースクラスだったり）にゲームロジック側からアクセスできるならば，それらが具体的にどのように実装されているのか？なんてことは，ゲームロジック側は知る必要が無いです．例えば，キー入力について言えば

//引数で指定したキーが押されているかどうかを知る手段
bool GetKeyState( キーの種類 );

みたいな問い合わせ手段があれば，「キーの状態を実際に取得している手段は実は glfwGetKey() なる関数で，その引数には GLFWwindow* 型の何かを与える必要がある」とかいう事情を，この関数を呼ぶゲームロジック側は全く知らなくても済む．
知らなくて済むということは，ゲームロジック側の実装コードには GLFWwindow* みたいな物が現れないということ．
そしたら，

ゲームプログラム内でwindowコンテキスト変数を直接触りたくなりません

といった話が意図通りの方向に進むんじゃないでしょうか．

Comments

[退会済みユーザー]

ありがとうございます。質問ですがその関数をどうやって使えるようにするかが知りたいです。、
windowクラスに記述していいのか？それとも別のやり方がるのか？それとも場合によるのか？
その辺の実装周りが知りたいです。

[fana]

ゲームロジックを実装している個所から，

> bool GetKeyState( キーの種類 );

の宣言が見えるならば，あとは，「どこかに」実装があればいいだけの話．

例えばあなたのwindowクラスがメンバ関数としてこれを提供するのだとしたら，
ゲームロジック側がwindowクラスを直接触る形にしてもいいかもしれないし，
そうではなくてゲームロジック側はwindowクラスなんて物の存在すら知らずに済む形にしてもよい．
どこはどうしたいか次第．

後者の例は，（良し悪しは別として）最も簡素な例は，
「windowクラスのインスタンスに触ることができる場所に GetKeyState() という単なる関数の実装がある」
みたいな形：

static window windowInst; //window型インスタンス
bool GetKeyState( キーの種類 key ){ return windowInst.GetKeyStateMethod(Key); } //この関数は単にインスタンスのメンバ関数を呼ぶだけ

関数GetKeyState()を呼ぶ側はwindowという型を知らずに済む．

[退会済みユーザー]

windowクラスに実装することにしました。

[fana]

とりあえず，そのコードの「将来」のことを考えるなら，
GetKeyState()みたいなのをゲームロジック側に提供するためのインタフェースクラスを用意して，
windowクラスそのものとかをゲームロジック側から隠すようにしとけばいいんじゃないかな．
（最も単純には，windowクラスがそのインタフェースクラスを継承する）

[fana]

class XXXInterface{ ... }; //GetKeyState()とかをゲームロジック側に提供する用

class window : public XXXInterface { ... }; //XXXInterfaceの提供するメソッドの具体実装はwindowクラスにある

とか．
で，ゲームのロジック側は，プログラムのメインループから毎度呼ばれるメソッドを

class GameLogic{ void Update( XXXInterfaceX & ); };

とか何とかしとけばいいんじゃない？