OpenGL ウインドウコンテキストをextern化することは正解なのかどうか知りたい。

Question

タイトル通りですがOpenGLでいろいろな場所で使うウインドウコンテキスト変数をextern化する行為は正解なのでしょうか？
参考サイトのextern宣言が不要なシステム設計を目指せ！項目のグローバル変数をファイル間で共有する設計は危険の文章を読んだのですがそれを踏まえて下記のコード設計はどうなのでしょうか？
extern変数 Data.hpp部のextern std::shared_ptr<FrameWork::Window> windowContext;はMain.cppで定義されていて
Player.cpp,Enemy.cppでテクスチャサイズに合わせて画像を拡大縮小させるためにウインドウサイズの情報が必要なのでいちいち引数に取っては面倒なのでextern化したいのですがこれはどうなのでしょうか？ちなみにこのプロジェクトではウインドウは一つの予定なのです。またゲーム制作です。

参考サイト：https://monozukuri-c.com/langc-funclist-extern/

Data.hpp

#ifndef ___DATA_HPP___
#define ___DATA_HPP___

#include "Window.hpp"
#include <memory>

extern std::shared_ptr<FrameWork::Window> windowContext;



#endif

Main.cpp

#include "Window.hpp"
#include "Init.hpp"
#include "Sprite.hpp"
#include "Camera.hpp"
#include "Shader.hpp"

#include <memory>
#include <iostream>

#include "Data.hpp"
#include "Player.hpp"
#include "Enemy.hpp"

std::shared_ptr<FrameWork::Window> windowContext =nullptr;

int main()
{

	if(glfwInit() != GLFW_TRUE)
	{
		assert(0 && "glfw 初期化失敗");
	}

	windowContext = std::make_shared<FrameWork::Window>(glm::ivec2(800,600),"Hello World");    
	windowContext->setCurrentContext();

	// OpenGL Verison 4.5 Core Profile
	glfwWindowHint(GLFW_CONTEXT_VERSION_MAJOR, 4);
	glfwWindowHint(GLFW_CONTEXT_VERSION_MINOR, 5);
	glfwWindowHint(GLFW_OPENGL_FORWARD_COMPAT, GL_TRUE);
	glfwWindowHint(GLFW_OPENGL_PROFILE, GLFW_OPENGL_CORE_PROFILE);

	if(glewInit() != GLEW_OK)
	{
		assert(0 && "glew 初期化失敗");
	}

	std::shared_ptr<FrameWork::Camera> camera = std::make_shared<FrameWork::Camera>(windowContext);
	//camera->shader->Load("ast/sdr/FrameBuffer.vert", "ast/sdr/FrameBuffer.frag");
	camera->setPosition(glm::vec3(0,0,1));
	camera->setLook(glm::vec3(0,0,-1));

	std::shared_ptr<Player> player = std::make_shared<Player>();
	std::shared_ptr<Enemy> enemy = std::make_shared<Enemy>();
	while(*windowContext)
	{
//		camera->ClearBuffer();

		player->Update();
		enemy->Update();




		camera->View(player);	
		camera->View(enemy);		





		camera->RendererBuffer();
		windowContext->SwapBuffers();
	}
	return 0;
}

Player.cpp

#include "Player.hpp"
#include "Shader.hpp"

#include "Data.hpp"


#include <iostream>
#include <glm/glm.hpp>

#define SPRITE_NUMBER ((int)(1))



Player::Player()
{
    sprite = std::make_shared<FrameWork::Sprite>();

    sprite->shader = std::make_shared<FrameWork::Shader>();
    sprite->shader->Load("ast/sdr/3D/texture/BasicTexture.vert","ast/sdr/3D/texture/BasicTexture.frag");

    texture = FrameWork::LoadTexture("ast/tex/debug.png",glm::vec2(8,8));

//    std::cout<<texture.at(1).position.start.x<<std::endl;
//    std::cout<<texture.at(1).position.start.y<<std::endl;

}


void Player::Update()
{
	transform.setPosition(glm::vec3(-0.3,0,-1));		                                    //座標
	transform.setSizeScale(texture.at(SPRITE_NUMBER).position.start,texture.at(SPRITE_NUMBER).position.end);    //画像サイズスケール
	transform.setRotate(glm::vec3(1,0,0),glm::vec3(1,0,0));                             //回転  
}



void Player::Render(glm::mat4 view)const
{
    sprite->setAttributePosition("vertexPosition");
    sprite->setAttributeUV("vertexUV");

    sprite->setTexture(0,texture.at(SPRITE_NUMBER));

    sprite->shader->setEnable();

    sprite->shader->setUniformSampler2D("uImage",0,texture.at(SPRITE_NUMBER).id);
    sprite->shader->setUniformMatrix4x4fv("uViewProjection", view);
	sprite->shader->setUniformMatrix4x4fv("uScale", transform.getMatSizeScale(windowContext));
	sprite->shader->setUniformMatrix4x4fv("uTranslate",transform.getMatTranslation());
	sprite->shader->setUniformMatrix4x4fv("uRotate", transform.getMatRotate());    

    sprite->Render(view);

	sprite->shader->setDisable();
}

Player::~Player()
{
    
}

Enemy.cpp

#include "Enemy.hpp"
#include "Shader.hpp"

#include "Data.hpp"


#include <iostream>
#include <glm/glm.hpp>

#define SPRITE_NUMBER ((int)(3))

Enemy::Enemy()
{
    sprite = std::make_shared<FrameWork::Sprite>();

    sprite->shader = std::make_shared<FrameWork::Shader>();
    sprite->shader->Load("ast/sdr/3D/texture/BasicTexture.vert","ast/sdr/3D/texture/BasicTexture.frag");

    texture = FrameWork::LoadTexture("ast/tex/debug.png",glm::vec2(8,8));

//    std::cout<<texture.at(1).position.start.x<<std::endl;
//    std::cout<<texture.at(1).position.start.y<<std::endl;

}


void Enemy::Update()
{
	transform.setPosition(glm::vec3(0.3,0,-1));		                                    //座標
	transform.setSizeScale(texture.at(SPRITE_NUMBER).position.start,texture.at(SPRITE_NUMBER).position.end);    //画像サイズスケール
	transform.setRotate(glm::vec3(1,0,0),glm::vec3(1,0,0));                             //回転  
}



void Enemy::Render(glm::mat4 view)const
{
    sprite->setAttributePosition("vertexPosition");
    sprite->setAttributeUV("vertexUV");

    sprite->setTexture(0,texture.at(SPRITE_NUMBER));

    sprite->shader->setEnable();

    sprite->shader->setUniformSampler2D("uImage",0,texture.at(SPRITE_NUMBER).id);
    sprite->shader->setUniformMatrix4x4fv("uViewProjection", view);
	sprite->shader->setUniformMatrix4x4fv("uScale", transform.getMatSizeScale(windowContext));
	sprite->shader->setUniformMatrix4x4fv("uTranslate",transform.getMatTranslation());
	sprite->shader->setUniformMatrix4x4fv("uRotate", transform.getMatRotate());    

    sprite->Render(view);

	sprite->shader->setDisable();
}

Enemy::~Enemy()
{
    
}

Answer 1

参考サイトのextern宣言が不要なシステム設計を目指せ！項目のグローバル変数をファイル間で共有する設計は危険の文章を読んだのですがそれを踏まえて下記のコード設計はどうなのでしょうか？

「extern宣言が不要な形にしようぜ」っていう話を読んだけども extern std::shared_ptr<FrameWork::Window> windowContext; という実装をしているっていうことなら，「踏まえて」なくない？

リンク先は「それ関数でよくね？」っていう話になってるんだから，その話を「踏まえて」の設計をするというのであれば，

ウインドウサイズの情報

を返す関数を定義するような方向にいくんじゃない？

Comments

[fana]

FrameWork::Window　なる型の正体が不明だけども，コピーしても問題ない程度のブツなのであれば，この型を戻り値とした関数を１個定義する，っていう形も考えられるかな．

---

個人的には「ふつーに引数で渡せばよくね？」って思うけど．
「引数だと面倒」って言うけど，コード記述作業をちょっとだけ楽にすることと引き換えに
余計な制約（固定の依存関係というか）を作っちゃう，ってのは，なんだかなぁと．

（「面倒」について言えば，引数としないようにするために今現在余計な面倒事が起きてるという点はどうなん？）

Answer 2

「グローバル変数をファイル間で共有する設計は危険」
と
「ウインドウサイズの情報が必要なのでいちいち引数に取っては面倒」
とを天秤にかけて、重い方を選べばいい。

※ 僕は前者に重きを置きます。

Comments

[退会済みユーザー]

質問なのですがなぜ「グローバル変数をファイル間で共有する設計」は危険なのでしょうか？

[episteme]

参考サイトを読んでいないのですか? (読んだうえでの質問なら)なにがわからんですか?

[退会済みユーザー]

値が変わらないなら問題ないと思うのですが？どうなのでしょうか？

[episteme]

そう思ってるならそれでいいし、windowContextを返す関数を用意すれば値が変わる心配をしなくていい。

[fana]

まぁ，極論(?)，まともに動きさえすれば実装形態なんてどうでもいいわけだし．
そのコードをあなた以外の誰かが触るなんてことは未来永劫無いのだろうし，あなた自身が問題無いと思う（自分は問題を起こさないと思う）ならいいんじゃない？
質問タイトルに「正解なのか」とか書いてるけども，「コレが正解」っていうのは無いわけだし．

参考サイトの内容に異を唱えるのであれば，そもそも「このサイトの話を踏まえての実装としてどうなん？」とか言い出す意味がわからんけども．

[dodox86]

こういうのは経験から導き出されて進化してきたことでもあるから、自分で考えて設計して実装するなり他人のコードを利用してても保守などで面倒見るなりして失敗して自分で苦労しないとその良さ、考え方の妥当性が理解できないのだと思います。