std::copyでエラー

Question

実現したいこと

DirectX12の頂点バッファに可変長配列データをコピーしたい

前提

VisualStadio2022
C++でDirectX12のサンプルコードを試しています

発生している問題・エラーメッセージ

C++ オーバーロードされた関数 のインスタンスが引数リストと一致しません
引数の型: (DXApplication::Vertex *)

該当のソースコード

　　　struct Vertex
{
DirectX::XMFLOAT3 pos;
DirectX::XMFLOAT2 uv;
};

　　　　　//頂点の数はその都度変わるのでvectorを使用して頂点バッファにコピー

　　　　　std::vector<Vertex>Myvertices{};

	Vertex v;

	// 前面
	v = { { -1.0, -1.0,  1.0}, { 1.0f, 1.0f } };
	Myvertices.push_back(v);
	v = { {  1.0, -1.0,  1.0}, { 0.0f, 1.0f } };
	Myvertices.push_back(v);
	v = { {  1.0,  1.0,  1.0}, { 0.0f, 0.0f } };
	Myvertices.push_back(v);
	v = { { -1.0,  1.0,  1.0}, { 1.0f, 0.0f } };
	Myvertices.push_back(v);
	// 背面
	v =	{{ -1.0, -1.0, -1.0,}, { 1.0f, 1.0f }};
	Myvertices.push_back(v);
	v =	{{ -1.0,  1.0, -1.0,}, { 0.0f, 1.0f }};
	Myvertices.push_back(v);
	v =	{{  1.0,  1.0, -1.0,}, { 0.0f, 0.0f }};
	Myvertices.push_back(v);
	v =	{{  1.0, -1.0, -1.0,}, { 1.0f, 0.0f }};
	Myvertices.push_back(v);
	// 上面
	v =	{{ -1.0,  1.0, -1.0}, { 1.0f, 1.0f }} ;
	Myvertices.push_back(v);
	v =	{{ -1.0,  1.0,  1.0}, { 0.0f, 1.0f }};
	Myvertices.push_back(v);
	v =	{{  1.0,  1.0,  1.0}, { 0.0f, 0.0f }} ;
	Myvertices.push_back(v);
	v =	{{  1.0,  1.0, -1.0}, { 1.0f, 0.0f }} ;
	Myvertices.push_back(v);
	//	// 底面
	v =	{{ -1.0, -1.0, -1.0}, { 1.0f, 1.0f }} ;
	Myvertices.push_back(v);
	v =	{{  1.0, -1.0, -1.0}, { 0.0f, 1.0f }} ;
	Myvertices.push_back(v);
	v =	{{  1.0, -1.0,  1.0}, { 0.0f, 0.0f }}  ;
	Myvertices.push_back(v);
	v =	{{ -1.0, -1.0,  1.0}, { 1.0f, 0.0f }} ;
	Myvertices.push_back(v);
	// 右側面
	v =	{{  1.0, -1.0, -1.0}, { 1.0f, 1.0f }} ;
	Myvertices.push_back(v);
	v =	{{  1.0,  1.0, -1.0}, { 0.0f, 1.0f }} ;
	Myvertices.push_back(v);
	v =	{{  1.0,  1.0,  1.0}, { 0.0f, 0.0f }} ;
	Myvertices.push_back(v);
	v =	{{  1.0, -1.0,  1.0}, { 1.0f, 0.0f }} ;
	Myvertices.push_back(v);
	//	// 左側面
	v =	{{ -1.0, -1.0, -1.0}, { 1.0f, 1.0f }} ;
	Myvertices.push_back(v);
	v =	{{ -1.0, -1.0,  1.0}, { 0.0f, 1.0f }} ;
	Myvertices.push_back(v);
	v =	{{ -1.0,  1.0,  1.0}, { 0.0f, 0.0f }} ;
	Myvertices.push_back(v);
	v =	{{ -1.0,  1.0, -1.0}, { 1.0f, 0.0f }} ;
	Myvertices.push_back(v);

　　　　　//MyverticesのVertexをverticesにコピー
Vertex* vertices= nullptr;

	vertices = (Vertex*)malloc(sizeof(Vertex) * Myvertices.size());

	int MyCount = Myvertices.size();

	//ループでコピー
	for (int i=0; i < MyCount; i++) {
		vertices[i] = Myvertices[i];
	  }

            //省略


	Vertex* vertexMap = nullptr;

	ThrowIfFailed(vertexBuffer_->Map(0, nullptr, (void**)&vertexMap));
	
	std::copy(std::begin(vertices), std::end(vertices), vertexMap); ！！！ここでエラ－！！！！

	
	エラ－メッセージ

	C++ オーバーロードされた関数 のインスタンスが引数リストと一致しません
        	引数の型: (DXApplication::Vertex *)	

試したこと

固定長配列では成功し、表示されます。
座標点を動的に増減し表示しようと上記vectorにしたのですが

補足情報（FW/ツールのバージョンなど）

visualstudio2022 C++

Answer 1

式中に配列が現れた場合にはその配列の先頭要素を指すポインタに型変換されます。 変換後のポインタはポインタの先頭要素を指しているだけで配列全体の情報を失ってしまうので終端位置はわからなくなります。

しかしこの暗黙の変換には例外があります。 配列の参照の初期化子である場合はそのひとつです。 std::begin や std::end は参照として配列を受け取ることでポインタに変換される前の配列として受け取る仕組みになっています。

https://timsong-cpp.github.io/cppwp/n3337/iterator.range

なので固定長の配列である場合はその型から配列の大きさを知り、大きさがわかるということは終端位置もわかるということです。

---

固定長の配列ではない場合はにプログラマはメモリを確保したときに確保した大きさを記録しておき、必要に応じて与える形にしなければなりません。 結論としてはこうです。

std::copy(vertices, vertices+MyCount, vertexMap);

配列に対してポインタはイテレータとしての性質を満たすのでこの場合は std::begin や std::end を使わずともポインタ自体をイテレータとして渡してかまいません。

Comments

[nakaT]

表示されるようになりました。
添削コードも表示していただき、ありがとうございます。

Answer 2

この回答はGPT-4による自動生成回答です

エラーが発生している箇所は、std::copyの部分です。std::end(vertices)が問題を引き起こしています。verticesはポインタであり、std::endはコンテナに対して使用する関数です。std::vectorの場合は問題ありませんが、ポインタに対しては使用できません。

解決策として、std::copyの第2引数をvertices + MyCountに変更してください。これにより、ポインタの範囲を正しく指定できます。

std::copy(std::begin(vertices), vertices + MyCount, vertexMap);

また、verticesをmallocで確保していますが、C++ではnewを使用することが推奨されます。mallocはC言語で使用されるメモリ確保関数で、C++ではnewが型情報を持っているため、より安全にメモリ確保ができます。

Vertex* vertices = new Vertex[Myvertices.size()];

そして、メモリ解放時にはdelete[]を使用してください。

delete[] vertices;

Answer 3

std::copy(std::begin(vertices), std::end(vertices), vertexMap);

strd::end(vertices) が使えるのは vertices が固定長配列(Vertex vertices[N])のとき。
なので std::copy(vertices, vertices+Myvertices.size(), vertexMap); が正解。

Comments

[nakaT]

表示されるようになりました。
添削コードも表示していただき、ありがとうございます。