GLSLを適用したコードで頂点座標が変更される原因が知りたい。

Question

提示コードの////コメントの内部のコードです　正方形となる頂点座標を入力して単純なシェーダーを使って描画を行いたいのですが線が描画されてしまいます。この原因がわかりません。頂点情報の位置情報は変わっていないのですが**どうった原因で位置情報が変更されているのでしょうか？**頂点座標を変更するコードはこのプログラムにはありません

試したこと [ 頂点シェーダーファイルのvec4変数の第三引数や第四引数をいじりましたが表示は変わりません。またフラグメントシェーダーは単純に色を描画するだけなので問題ないです。また色が青になることシェーダーファイル適用されています。]

	struct Vertex
	{
		GLfloat position[2];
		//GLfloat color[4];
	};

#include "DrawTest.hpp"

//コンストラクタ
DrawTest::DrawTest()
{

////////////////////////////////////////////////////////////////////////////////////////////////////
	//頂点座標
	Vertex rectangleVertex[4] =
	{
		//頂点、頂点色
		{-0.5f,-0.5f},
		{0.5f,-0.5f },
		{0.5f,0.5f  },
		{-0.5f,0.5f }
	};

	/*
	
	//頂点座標
	Vertex rectangleVertex[4] =
	{
		//頂点、頂点色
		{-0.5f,-0.5f,   0.0,0.0,1.0,1.0},
		{0.5f,-0.5f ,   0.0,0.0,1.0,1.0},
		{0.5f,0.5f  ,   0.0,0.0,1.0,1.0},
		{-0.5f,0.5f ,   0.0,0.0,1.0,1.0}
	};

	*/

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

	//vbo
	glGenBuffers(1, &vbo);
	glBindBuffer(GL_ARRAY_BUFFER, vbo);



	//頂点
	glBufferData(GL_ARRAY_BUFFER, 4 * sizeof(Vertex), rectangleVertex, GL_STATIC_DRAW);
	glVertexAttribPointer(0, 2, GL_FLOAT,GL_FALSE, 4 * sizeof(Vertex), (GLvoid*)0);
	glEnableVertexAttribArray(0);


	/*
	//頂点カラー
	glBufferData(GL_ARRAY_BUFFER, 4 * sizeof(Vertex), rectangleVertex, GL_STATIC_DRAW);
	glVertexAttribPointer(1, 4, GL_FLOAT, GL_FALSE, 4 * sizeof(Vertex),(GLvoid*)(2 * sizeof(GLfloat)));
	glEnableVertexAttribArray(1);
	*/

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

}

//更新
void DrawTest::Update()
{

}

//描画
void DrawTest::Draw()
{
	glBindVertexArray(vao);
	glDrawArrays(GL_LINE_LOOP, 0, 4);
}

//デストラクタ
DrawTest::~DrawTest()
{
	glDeleteVertexArrays(1, &vao);
	glDeleteBuffers(1, &vbo);

}

#include "Shader.hpp"
#include "DrawTest.hpp"


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,3);
	glfwWindowHint(GLFW_OPENGL_FORWARD_COMPAT,GL_TRUE);
	glfwWindowHint(GLFW_OPENGL_PROFILE,GLFW_OPENGL_CORE_PROFILE);

	glClearColor(1.0, 0.0, 0.0, 1.0);	//背景色
// #############################################################################

	DrawTest test;


	Shader shader("Test.vert","Test.frag");

	shader.setBindAttribVertex(0, "Position");
	shader.setBindAttribFragment(0, "fragment");


	while (window)
	{
		glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);	//カラーバッファをクリア
		shader.Active();
		
		test.Draw();
		
		window.SwapBuffers();	//ダブルバッファリング
	}

}

#version 330 core

in vec2 position;


void main()
{
	gl_Position =  vec4(position.x,position.y,1.0,1.0);	//座標
}

#version 330 core

out vec4 fragment;


void main()
{
	fragment = vec4(0.0,0.0,1.0,1.0);
}

Comments

[Zuishin]

アカウント変更してもまた質問削除してるけど、誰かに注意されたんだろう。

そういうのを読まずに捨てるからなぜ低評価を受けるかわからないんだよ。
簡単に削除できるような質問をするんじゃない。もっと考えてから質問しろ。

Answer 1

おかえりなさいませ．

glVertexAttribPointer(0, 2, GL_FLOAT,GL_FALSE, 4 * sizeof(Vertex), (GLvoid*)0);

ここの 4 * sizeof(Vertex) を 0 にする，すなわち
glVertexAttribPointer(0, 2, GL_FLOAT,GL_FALSE, 0, (GLvoid*)0)
としたら改善しませんでしょうか？

（あるいは，4 * を取り除いて， sizeof(Vertex) にするのでも同じかな）

Comments

[fana]

この引数は，
「各頂点用のデータの（の先頭の）位置が rectangleVertex の中で何byteおきに配置されているのか？」
という意味だから，
この場合，sizeof(Vertex) を指定する．

で，本件のVertex型の定義を見るに，｛多分，おそらく，変なことがなければ｝sizeof(Vertex) は 2*sizeof(GLfloat) と同じだろうから，
0を指定して「密に並んでいる」という指定をしても良いだろう，と．

[退会済みユーザー]

なるほど。頂点カラーがある際のデータですが頂点二次元、float * 4 (頂点カラー),なのですがこの場合どうやってストライドすればいいのでしょか？

//頂点
glBufferData(GL_ARRAY_BUFFER, 4 * sizeof(Vertex), rectangleVertex, GL_STATIC_DRAW);
glVertexAttribPointer(0, 2, GL_FLOAT,GL_FALSE,0, (GLvoid*)0);
glEnableVertexAttribArray(0);

//頂点カラー
glBufferData(GL_ARRAY_BUFFER, 4 * sizeof(Vertex), rectangleVertex, GL_STATIC_DRAW);
glVertexAttribPointer(1, 4, GL_FLOAT, GL_FALSE, 0,(GLvoid*)(2 * sizeof(GLfloat)));
glEnableVertexAttribArray(1);

[fana]

・各頂点用の座標データが rectangleVertex の中で何バイトおきの位置にあるのか？
・各頂点用の色のデータが rectangleVertex の中で何バイトおきの位置にあるのか？
という話．

そういうのはちゃんと考えてみればわかる．
Vertex型は（理想的(?)には）GLfloat型6個が以下のような並びになっている．

頂点(4) 頂点(4) 色(4) 色(4) 色(4) 色(4)

※ここで「(4)」とかいうのは各要素のサイズ（byte）を示していて，sizeof(GLfloat)が4なのだと仮定して書いている．

さらにここでは，Vertexにはたまたま余計な余白じみた物が追加されたとして考えよう（「余計な余白…」については構造体のパディングとかの話で別途調べてください．）
例えば（実際にはこうはならないだろうけども例えとして）Vertex型が，コンパイラの気まぐれによって

頂点(4) 頂点(4) 余白(8) 色(4) 色(4) 色(4) 色(4)

とかいう事態になっているとしよう．
この場合，sizeof(Vertex)は32[byte]になる．

rectangleVertex は Vertex型の配列だから，以下のようにこいつが並んでいるイメージだ．

頂点(4*2) 余白(8) 色(4*4) 頂点(4*2) 余白(8) 色(4*4) 頂点(4*2) 余白(8) 色(4*4) ...

記述が長くなるから頂点２個と色4個は纏めて書いたけど，この上で考えれば，
・各頂点用の座標データが rectangleVertex の中で何バイトおきの位置にあるのか？
・各頂点用の色のデータが rectangleVertex の中で何バイトおきの位置にあるのか？
がわかるよね．

[fana]

座標データも色データも，この例だと 32[byte] おきに存在している．
もちろん，この32という値は，このてきとーに考えた一例の値でしかないから，実際のコードに 32 って書くのは間違いだ．

様々なパディング具合に対応できるようにするために
sizeof( Vertex )
と書くことになるだろう．

また，こういうことを考えると，色側の glVertexAttribPointer の最後の引数の値を
> 2 * sizeof(GLfloat)
としてしまうことにも不安があるね．
実際の色情報データがVertex型の先頭から何バイト目に位置するかはわからない前提でコードを書いた方がいい．
このことには offsetof が使えるんじゃないかな．

[fana]

で，余談だけども， offsetof について調べれば，「PODじゃないとどうの…」とか何とかいう話が見えたりもする．
（Vertex型は今は POD かもしれないけども，将来までずっとそうなのかはわからない．
コンストラクタとか付けたくなるかもしれないし，もっと複雑な機能をあれこれ持つ型に変えていくことがあるかもしれない　……とか考えると，PODの制限はちょっと面倒な話に思える）

私なら，そういった Vertex型の offestof のあれこれに思いを巡らせるのは面倒だから
glBufferData には，(rectangleVertexみたいな)構造体の配列を渡すんじゃなくて，単純に GLfloat型の配列に全データをきちきちに詰めて入れてそれを渡すようにするかなぁ．
（それなら別に，座標と色とを一個の配列に同居させてストライドやオフセットの値を明示的に与えなくとも，別個の配列で渡してストライドやオフセットは単純に0を与えれば済むし．）

[退会済みユーザー]

なるほど。プログラムはわかりやすく組むべきあることを忘れていました。

[fana]

うむ．
なので，私なら，
Vertex型の配列 rectangleVertex でデータを持っている場合，

わざわざ glBufferData に渡すためだけの配列を （glBufferData を呼ぶ箇所の手前で）用意して，
その配列に rectangleVertex からデータ値をせっせとコピーして，
その配列を glBufferData に渡す

というコードをまずは書くと思う．
その様相は他者から見たら非常にナンセンスでかっこ悪い雑魚丸出しなコードに見えるかもしれないけども，誰かのために書いているんじゃなくて自分のためのコードなんだから，まずは自分にとって把握しやすい（間違いが起きにくい）ことが一番だと思うので．

[fana]

ストライドやオフセットについては，まずは自身で

> 頂点(4*2) 余白(8) 色(4*4) 頂点(4*2) 余白(8) 色(4*4) 頂点(4*2) 余白(8) 色(4*4) ...

みたいなのを絵にでも描いて考えるべきだし，
そのためには少なくとも，このイメージを描ける必要がある．

glVertexAttribPointer のリファレンスを見てストライドの説明がわからない場合，質問するにしても
その自分が描いた絵を示して，「メモリイメージはこうなっているハズだけども，ストライドとはどこからどこまでのバイト数を意味するのか？　自分はここからここまでだと思ってこのようにコードを書いたのだが，ダメなようだ．解釈が違うのだろうか？」
とかいう形で質問するとよろしかろう．

[fana]

丸投げと評されるのは，そういうのを書かないためであろう．

また，書かれてないと，書かない理由が
「めんどくせぇから自分では考えない（：丸投げ）」
なのか，
「書けないから」
なのか（それとも別の何かなのか）というのが，読み手に解釈されてしまうことになる．
（「書けない」と解釈した者は「基礎からやれ」と言うだろう．）