VAO VBOは複数あってもいいのかどうか？

Question

DrawGprah関数部のコメント部のコードですが毎フレームvao vbo を生成して消す処理と
Spriteクラスのインスタンスごとにvao vbo を生成する処理は正しいのでしょうか？

提示コードは画像を表示させるクラスです。

#include "../Header/Sprite.hpp"
#include "../Header/Shader.hpp"
#include "../Header/Game.hpp"
#include "../Header/Texture.hpp"
#include "../Header/VertexData.hpp"

// 数学ライブラリ
#include "glm/ext.hpp"
#include "glm/glm.hpp"

// OpenCV 
#include <opencv2/core.hpp>

class Game;
/*################################################################################################################
* 画像　描画クラス
################################################################################################################*/

//コンストラクタ
/*
* Game クラス
* テクスチャ パス
* 描画する画像の寸法
*/

Sprite::Sprite(class Game* g, const char* FileName) : Transform_2D(g)
{
	shader = new Shader(g,"Shader/2D/Sprite.vert", "Shader/2D/Sprite.frag");
	Owner = g;//Gameクラス
	
	Transform_2D::setTransfrom(glm::vec2(1,1),0,glm::vec2(0,0));	//トランスフォームを初期化

	//テクスチャを設定     
	glGenTextures(1, &TextureID);
	glBindTexture(GL_TEXTURE_2D, TextureID);

	int width = 0, height = 0;	//画像の寸法
	int channels = 0;			//画像のチャンネル数

	std::shared_ptr<byte[]> data = LoadTexture(FileName, width, height, channels);	//テクスチャを取得
	

	mPicSize.x = width;
	mPicSize.y = height;

	//printf("widht: %f\n", width);
	//printf("height: %f\n", height);

	if (data.get() != NULL) 
	{
		if (channels == 3)
		{
			glTexImage2D(GL_TEXTURE_2D, 0, GL_RGB, (GLsizei)width, (GLsizei)height, 0, GL_RGB, GL_UNSIGNED_BYTE, data.get());
		}
		else if (channels == 4)
		{
			glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA, (GLsizei)width, (GLsizei)height, 0, GL_RGBA, GL_UNSIGNED_BYTE, data.get());
		}
	}
	else {
		std::cerr << "Unable to load texture: " << FileName << std::endl;
	}

	 

//	mSize = end - start;	//画像のサイズを取得

//	printf("width: %.2f\n", mSize.x);
//	printf("height: %.2f\n",mSize.y);

	/*
	vertex[0] = VertexAttribute_2D{ -width / 2.0f,height / 2.0f,	0,0 };
	vertex[1] = VertexAttribute_2D{ -width / 2.0f,-height / 2.0f,	0,1 };
	vertex[2] = VertexAttribute_2D{ width / 2.0f,-height / 2.0f,	1,1 };

	vertex[3] = VertexAttribute_2D{ -width / 2.0f,height / 2.0f,	0,0 };
	vertex[4] = VertexAttribute_2D{ width / 2.0f,-height / 2.0f,	1,1 };
	vertex[5] = VertexAttribute_2D{ width / 2.0f,height / 2.0f,	1,0 };
	*/


	/*
	* 
	// 頂点情報を設定
	vertex[0] = VertexAttribute_2D{ -mSize.x / 2.0f,mSize.y / 2.0f,	start.x * uvWidth,start.y * uvHeight };
	vertex[1] = VertexAttribute_2D{ -mSize.x / 2.0f,-mSize.y / 2.0f,	start.x * uvWidth,end.y * uvHeight };
	vertex[2] = VertexAttribute_2D{ mSize.x / 2.0f,-mSize.y / 2.0f,	end.x * uvWidth,end.y * uvHeight };

	vertex[3] = VertexAttribute_2D{ -mSize.x / 2.0f,mSize.y / 2.0f,	start.x * uvWidth,start.y * uvHeight };
	vertex[4] = VertexAttribute_2D{ mSize.x / 2.0f,-mSize.y / 2.0f,	end.x * uvWidth,end.y * uvHeight };
	vertex[5] = VertexAttribute_2D{ mSize.x / 2.0f,mSize.y / 2.0f,		end.x * uvWidth,start.y * uvHeight };

	*/


	//VAO
//	glGenVertexArrays(1, &VAO);
//	glBindVertexArray(VAO);

	//VBO
//	glGenBuffers(1, &VBO);
//	glBindBuffer(GL_ARRAY_BUFFER, VBO);
//	glBufferData(GL_ARRAY_BUFFER, (sizeof(vertex) / sizeof(vertex[0])) * sizeof(VertexAttribute_2D), vertex, GL_STATIC_DRAW);

	//頂点座標
//	glEnableVertexAttribArray(0);
//	glVertexAttribPointer(0, 2, GL_FLOAT, GL_FALSE, sizeof(VertexAttribute_2D), NULL);

	//UV座標
//	glEnableVertexAttribArray(1);
//	glVertexAttribPointer(1, 2, GL_FLOAT, GL_FALSE, sizeof(VertexAttribute_2D), (void*)(sizeof(float) * 2));

	//ミニマップを設定
	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR_MIPMAP_LINEAR);
	glGenerateMipmap(GL_TEXTURE_2D);

	//異方性フィルタリングを設定
	GLfloat largest;
	glGetFloatv(GL_MAX_TEXTURE_MAX_ANISOTROPY_EXT, &largest);
	glTexParameterf(GL_TEXTURE_2D, GL_TEXTURE_MAX_ANISOTROPY_EXT, largest);


	glDeleteBuffers(1, &VAO);
	glDeleteVertexArrays(1, &VAO);

	glBindVertexArray(0);
	glBindTexture(GL_TEXTURE_2D, 0);


//	delete data;
//	data = nullptr;

}

//描画
void Sprite::DrawGraph(glm::vec2 pos, glm::vec2 start, glm::vec2 end)
{

	
	Transform_2D::UpdateTransform();		//トランスフォームを更新
	Transform_2D::setTransform_Move(pos);	//移動

	shader->Enable();	//シェーダーを有効にする
	shader->SetFloatUniform_3m("uViewMatrix", getViewMatrix());
	shader->SetFloatUniform_3m("uWorldMatrix", getWorldMatrix());


	glm::vec2 mSize;	
	mSize = end - start;	//画像のサイズを取得
//	printf("mSize.x  %f\n", mSize.x);
//`	printf("start.x. x%f\n", end.x);




	// 頂点情報を設定
	vertex[0] = VertexAttribute_2D{ -mSize.x / 2.0f,mSize.y / 2.0f,	    start.x / mPicSize.x   ,start.y / mPicSize.y };

//	printf("%f\n", start.y / mSize.y);

	vertex[1] = VertexAttribute_2D{ -mSize.x / 2.0f,-mSize.y / 2.0f,	start.x / mPicSize.x   ,end.y / mPicSize.y };
	vertex[2] = VertexAttribute_2D{ mSize.x / 2.0f,-mSize.y / 2.0f,	    end.x / mPicSize.x     ,end.y / mPicSize.y };

	vertex[3] = VertexAttribute_2D{ -mSize.x / 2.0f,mSize.y / 2.0f,	    start.x / mPicSize.x  ,start.y / mPicSize.y };
	vertex[4] = VertexAttribute_2D{ mSize.x / 2.0f,-mSize.y / 2.0f,	    end.x / mPicSize.x    ,end.y / mPicSize.y };
	vertex[5] = VertexAttribute_2D{ mSize.x / 2.0f,mSize.y / 2.0f,		end.x / mPicSize.x    ,start.y / mPicSize.y };

////////////////////////////////////////////////////////////////////////////////////////////

	//VAO
	glGenVertexArrays(1, &VAO);
	glBindVertexArray(VAO);

	//VBO
	glGenBuffers(1, &VBO);
	glBindBuffer(GL_ARRAY_BUFFER, VBO);
	glBufferData(GL_ARRAY_BUFFER, (sizeof(vertex) / sizeof(vertex[0])) * sizeof(VertexAttribute_2D), vertex, GL_STATIC_DRAW);
///////////////////////////////////////////////////////////////////////////////////////////
	//頂点座標
	glEnableVertexAttribArray(0);
	glVertexAttribPointer(0, 2, GL_FLOAT, GL_FALSE, sizeof(VertexAttribute_2D), NULL);

	//UV座標
	glEnableVertexAttribArray(1);
	glVertexAttribPointer(1, 2, GL_FLOAT, GL_FALSE, sizeof(VertexAttribute_2D), (void*)(sizeof(float) * 2));

	
	glBindVertexArray(VAO);
	glBindTexture(GL_TEXTURE_2D, TextureID);
	glDrawArrays(GL_TRIANGLES, 0, (GLsizei)std::size(vertex));	

	// バッファを解放
	glDeleteBuffers(1,&VAO);
	glDeleteVertexArrays(1,&VAO);

	//glBindVertexArray(0);
//	glBindTexture(GL_TEXTURE_2D, 0);

}



// 画像サイズを取得
glm::vec2 Sprite::getSize()
{
	return mPicSize;
}



// デストラクタ
Sprite::Sprite(const Sprite & sp) : Transform_2D(sp.Owner)
{
	shader = new Shader(sp.Owner, "Shader/2D/Sprite.vert", "Shader/2D/Sprite.frag");
	Owner = sp.Owner;//Gameクラス
	mPicSize.x = sp.mPicSize.x;
	mPicSize.y = sp.mPicSize.y;
	

}


// デストラクタ
Sprite::~Sprite()
{
//	printf("Sprite delete\n");


	
	delete shader;
	shader = nullptr;
	glDeleteTextures(1,&TextureID);

}

Comments

[y_waiwai]

なにをするコードなんでしょうか

[退会済みユーザー]

スプライト描画クラスです。

[fana]

> スプライト描画クラスです。

とだけ言われても，何をするのかわからんのですが…
画像の任意の部分矩形領域を表示する手段　みたいなものなのでしょうか？
そうであれば，DrawGraph()の引数の意味は，
・pos : 矩形を表示する場所を指定（矩形の中心の位置か）
・start, end : 画像のどこらへんを矩形に張り付けるのか，という領域を指定（大元の画像のpixel座標で指定する)
という話で合ってますか？

Answer 1

どんな形がベストなのかわかりませんが，
Sprite::DrawGraph()を実施するたびに【VAOやVBOを作って→使って→すぐ捨てる】というのは効率が悪そうに思います．

Sprite::DrawGraph()の引数によって
頂点座標とテクスチャ座標がちょっと変わるだけなのであれば，例えば
その変換のための uniform変数を用意して，そいつだけを更新する形にすれば良いのではないかと．

Comments

[fana]

あと，Spriteのコンストラクタに

glDeleteBuffers(1, &VAO);
glDeleteVertexArrays(1, &VAO);

とかありますが，必要な処理なんでしょうか？（やっても問題ないのか？）