シェーダーファイルの読み込みで関係ないchar型文字が一つ代入される理由が知りたい。

Question

提示画像ですがシェーダーファイルを読み込むとエラーが出ます不明なchar　- が入っていますとのエラーですが頂点、フラグメント両方なんな文字を入っていません。これはなぜでしょうか？提示コード二つ目はシェーダーファイルを読み込むコードですが何が悪いのでしょうか？

// ##################################### プログラムオブジェクト作成 ##################################### 
GLuint FrameWork::Shader::CreateProgram(std::shared_ptr<GLchar*> vert, std::shared_ptr<GLchar*> frag)
{
	const GLuint program = glCreateProgram();	//シェーダープログラムを作成

	if (vert != nullptr)
	{
		const GLuint vobj = glCreateShader(GL_VERTEX_SHADER);
		
		GLchar *v[] = { NULL };
		v[0] = *vert;

		glShaderSource(vobj, 1,v, NULL);
		glCompileShader(vobj);
		CompileInfoLog(vobj, "Complie Error: Vertex Shader");
		glAttachShader(program, vobj);
		glDeleteShader(vobj);
	}
	else 
	{
		std::cerr << "頂点シェーダー読み込み失敗" << std::endl;
	}

	if (frag != nullptr)
	{
		const GLuint fobj = glCreateShader(GL_FRAGMENT_SHADER);
		GLchar *v[] = {NULL};
		v[0] = *frag;

		glShaderSource(fobj, 1, v, NULL);
		glCompileShader(fobj);
		CompileInfoLog(fobj, "Complie Error: Fragment Shader");
		glAttachShader(program, fobj);
		glDeleteShader(fobj);
	}
	else 
	{
		std::cerr << "フラグメントシェーダー読み込み失敗" << std::endl;
	}

	glLinkProgram(program);		//リンクプログラム
	ProgramInfoLog(program);	//リンク時のログを表示
	
	return program;
}

// ##################################### シェーダーを読み込む ##################################### 
void FrameWork::Resource::LoadShader(Asset assetName)
{
	
	std::shared_ptr<Shader> data = std::make_shared<Shader>();

	data->fileData = std::make_shared<GLchar*>();

	for (std::vector<Resource_File>::const_iterator itr = resourceFile->begin(); itr != resourceFile->end(); itr++)
	{
		if (itr->assetName == assetName)
		{

			std::ifstream file(itr->fileName, std::ios::binary);
			if (file.fail() == true)
			{
				std::cerr << "シェーダーファイルを読み込めません: " << itr->fileName << std::endl;

				file.close();
				assert(0);
				break;
			}
			else
			{	
				std::vector<GLchar> buffer(0);
				file.seekg(0L, std::ios::end);
				GLsizei length = static_cast<GLsizei>(file.tellg());
				buffer.resize(length + 1);
				file.seekg(0L, std::ios::beg);
				file.read(buffer.data(), length);
				buffer.at(length) = ('\0');
				*data->fileData = buffer.data();
				file.close();

				shader->push_back(std::make_shared<Shader_Data>(Shader_Data{ data,assetName }));				
			}

			file.close();
			break;
		}
	}
	
}

/*#########################################################################
# 単色の二次元描画　頂点シェーダー
###########################################################################*/
#version 330											//バージョン
#extension GL_ARB_explicit_attrib_location: enable		//


// ###################### 頂点属性 ###################### 
layout(location = 0) in vec2 vertexPosition;	//頂点座標

// ###################### 出力 ###################### 
layout(location = 2) out vec4 vFragment;	//頂点カラー

// ###################### Uniform ###################### 
uniform mat4 uScale;			//スケール
uniform mat4 uRotate;			//回転
uniform mat4 uTranslate;		//平行移動
uniform mat4 uViewProjection;	//ビュープロジェクション行列
uniform vec4 uFragment;			//色


void main()
{
	vec4 vertex = vec4(vertexPosition,0.0,1.0);		//頂点座標
	mat4 model = uTranslate * uRotate * uScale;		//モデル行列

	gl_Position =  (uViewProjection * model) * vertex;
	
	vFragment = uFragment;				//カラー
}

Answer 1

2点問題があります。

問題点その1　シェーダプログラムに日本語が入っている事
コメント部分だけならば問題ないことになっているらしいですが、
VisualStudioで日本語ソースを保存した場合おそらくBOM付きUTF-8になるため先頭にBOMが付与される

問題点その２　FrameWork::Shader::CreateProgramが呼び出された段階でfileDataの中身の寿命が切れている可能性が高い
実際の処理順が分かりませんが、FrameWork::Resource::LoadShaderを抜けた段階でbufferの寿命が切れていますのでおそらくは

そもそもstd::shared_ptr<GLchar*>は以前も指摘しましたが意味が違います。
Shaderのポインタを管理するためのstd::shared_ptr<Shader>なのと同様に
GLcharへのポインタを管理するためにはstd::shared_ptr<GLchar>(もしくはstd::shared_ptr<GLchar[]>)を用います。
(追記: deleterの問題があるのでC++17以降ではstd::shared_ptr<GLchar[]>を用いた方がいいですね。)
std::shared_ptr<GLchar*>はGLchar*へのポインタを管理しますのでGLchar*自体は寿命が切れている事もありえますしメモリリークもします。
また、所有権を他と共有しないのであればstd::vector<GLchar>を持った方がいいです。
個人的にはシェーダーファイルの所有権はShaderのみが持てばよいように感じます

Comments

[fana]

全然関係ない話で恐縮ですが，
「シェーダのコードを外部ファイルから動的に読み込む」という形の｛意義，利点，etc｝がどうにもわからない……

動的に読み込む　ってことは，読み込んでみるまでどんな内容のコードがやってくるかわからない，ってことだけど，
C++側としては，そのシェーダコードについての全てを知ってなければまずまともに使えないと思うんですよね．
結局は，「attribute やら uniform やらにどんなやつがあって，それらは何なのか？」みたいな仕様を完全把握しているC++側コードと対になって初めて仕事ができる．

つまり
・シェーダのコード
だけが有っても意味が無いわけで
・そのシェーダのコードに対応した固有のPC側処理実装
が必要になるんだけど，
この両者を物理的に分離することはただただ弊害（面倒）が生じるだけにしか思えない．

もちろん，【何らかの仕様をきっちり決めてしまえば】
C++側を全く変えずに（リコンパイルせずに）シェーダ処理だけを差し替える　ことはできるだろうけども……
（そんなことがしたい場面ってあるのかなぁ？）
動的にファイルから読み込んだシェーダコードがその仕様に従う物なのか？っていうチェック処理はどうするのか？　とか考えるだけで，ただただ面倒な仕組みに思える．

[asm]

あまりシェーダについては詳しくないのですがそうなのですか

[fana]

（こちらもあまり詳しくないからこそ疑問であるわけですが）
シェーダの処理に必要なデータ値は転送してやなきゃならない．
例えばシェーダの処理で何か座標変換を行うための変換マトリクスの値が必要とか．
↓
つーことは，それを転送する側としては，シェーダが何をどんな形で必要とするのかを知っている必要がある．
変換マトリクスの例で言えば
・マトリクスの値を外界から設定してやる必要があること
・そのマトリクスの大きさや要素の型
・それがシェーダコード上で何という変数名なのか
・それが何を意味するものなのか（何の変換であって，妥当な値とは何か）
くらいを知らないとならないハズ．

※シェーダコード文字列を自前で解釈するとかすれば上から３つまではわかるかもしれないが，最後のは無理だよね．

[退会済みユーザー]

質問ですがメモ帳でbomなしで上書きしましたが同じエラーが出ます。これはなぜでしょうか？

[asm]

> fanaさん
よくわかりませんが、コンパイル済みシェーダを使った方がいいという主張でしょうか？
それはOpenGLには存在しない(かった)機能です

> saijou_chanさん
提示した問題点両方を解決しないと寿命が切れているオブジェクトには何の保証もありません。

[fana]

テキストファイルから実行時に読むんじゃなくて，最も単純には
const GLchar Code[] = "シェーダコード";
とかでいいんじゃないのか的な．
どうせ使う側の実装とセットで用意するんだし．

[asm]

なるほど。raw string literal使うのも悪くないですね。