1 / width の計算を行って その値掛ける画像のピクセル数というUV座標のプログラムでは誤差が生じるのか？

Question

提示画像ですが白い矩形の砲弾の画像がおかしくなっています。右の白い縦線の部分が謎なのですがこれは「1 / width の計算を行って その値掛ける画像のピクセル数というUV座標」とう誤差なのでしょうか？　提示コードのDrawGraph()関数部の頂点情報の関数ですがこれが悪いのでしょうか？画像を再度確認しましたがそんな線は入っていません。また画像サイズは０から始まるので引く-1をしてみましたが変わりませんでした。

提示コードのDrawGraph();関数は座標と。その画像の描画する座標の始まりと終わりを指定して描画する関数です。

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

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

	byte* data = LoadTexture(FileName,width,height,channels);	//テクスチャを取得
	
	//printf("widht: %f\n", width);
	//printf("height: %f\n", height);

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

	//1ドットのUV数値
	mUVpos.x = 1.0 / width;	//U
	mUVpos.y = 1.0 / height;//V	
	

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

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



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

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

	vertex[3] = VertexAttribute_2D{ -mSize.x / 2.0f,mSize.y / 2.0f,	start.x * mUVpos.x,start.y * mUVpos.y };
	vertex[4] = VertexAttribute_2D{ mSize.x / 2.0f,-mSize.y / 2.0f,	end.x * mUVpos.x,end.y * mUVpos.y };
	vertex[5] = VertexAttribute_2D{ mSize.x / 2.0f,mSize.y / 2.0f,		end.x * mUVpos.x,start.y * mUVpos.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);

}

Comments

[fana]

単純に Sprite::DrawGraph() の引数の値が間違ってる等の可能性は無いのですか？
（引数の仕様も処理の意味も何もかも不明なので，他者には判断不能ですが）

[退会済みユーザー]

しっかりと確認しました。１引いたり足したりもしましたｗ

[fana]

何を確認したのですか？

> １引いたり足したり

とか言っている時点で，その対象のデータの仕様がまともに定まっていないということに他ならないのでは？
各データには「この場合にはこの値でなければならない」という確固たる決まり（仕様）があるハズ．
確認すべきは，
・実際のデータ値がその仕様通りの値であること
・「その仕様の上で」行わんとしている演算内容が妥当であること
・上記演算内容が間違いなく実装されていること
等ではないのでしょうか？
これらが全て真っ当であると確認されていない段階で，演算精度がどうのという話を疑うのはおかしいし，
既に諸々確認して疑うべき事柄が演算精度くらいしかないとかいう状態なのであれば，実際の演算結果値を見てみれば良いのでは？

[fana]

…ということなので，せめて
「これこれこういう演算を行う処理であって，入力値がXであれば演算結果値はYとなるべき話であるのに，実際の演算結果値を見てみるとZになっている．なぜなのか？　浮動小数点演算の精度によるものなのだろうか？」
みたいな形で問うべきでは？

何の説明も無しに単にコードだけ示しても，他者にとっては何が正しくて何が違うのかすら不明なわけです．その状態で「これは演算精度のせいですかね？」とか問うても誰も判断できないのではないでしょうか．

Answer 1

浮動小数点での演算は、精度以下の小数点部分が切り落とされるため、誤差がつきまといます
できるだけ整数演算で済ますような配慮が必要となります

Comments

[退会済みユーザー]

ではこの場合はどうするのが妥当なのでしょうか？
UV座標は１から０までなので小数点を使わずにはいられませんｗ

[y_waiwai]

実際に、手計算でそのコードのとおりに出してみればわかると思います
1 / widthを出して、ピクセル数を掛けるのではダメです

全体のピクセル数/部分のピクセル　というように、浮動小数点での演算部分は最小にする必要があります

[退会済みユーザー]

続きの数式も教えてくれますでしょうか？