スマートポインタを使っているのにも関わらずメモリリークの理解が正しいのか知りたい。

Question

提示コードですがstd::shared_ptr<>は最後の参照がなくなった時にメモリが解放させるという処理ですが。この場合while();文の中では変数の命が尽きないので実質メモリリークしており。while();文を抜けたときメモリが解放されるという理解で正しいのでしょうか？

参考サイト: https://qiita.com/hmito/items/db3b14917120b285112f

#ifndef ___TEXTURE_HPP_
#define ___TEXTURE_HPP_
#include <iostream>
#include "glm/glm.hpp"
#include "glew/include/GL/glew.h"

/*#########################################################################
# テクスチャ構造体

説明
	テクスチャのデータ
###########################################################################*/
namespace FrameWork
{
	//テクスチャデータ構造体
	typedef struct
	{
		glm::ivec2 size;				//画像サイズ
		std::shared_ptr<unsigned char*> fileData;		//画像データ
		int channel;					//画像のチャンネル数
		GLuint ID;						//テクスチャID
		GLuint textureUnitNumber;		//テクスチャユニット番号
	}TextureData;

	std::shared_ptr<FrameWork::TextureData> LoadTexture(const char* fileName);	//テクスチャーロード
}
#endif

#include "Texture.hpp"

#define STB_IMAGE_IMPLEMENTATION
#include "stb/stb_image.h"

// ##################################### 画像読み込み ##################################### 
std::shared_ptr<FrameWork::TextureData> FrameWork::LoadTexture(const char* fileName)
{
	//std::shared_ptr<FrameWork::TextureData> data = std::make_shared<FrameWork::TextureData>();
	TextureData data;// = std::make_shared<FrameWork::TextureData>();
	data.fileData = NULL;
	data.fileData = std::make_shared<unsigned char*>(stbi_load(fileName, &data.size.x, &data.size.y, &data.channel, 0));	//画像読み込み
	assert(data.fileData && "画像ファイルがありません。");

	return std::make_shared<FrameWork::TextureData>(data);
}

	while (*window)
	{
		window->FrameUpdate(glm::vec4(0.0f, 0.0f, 255.0f, 255.0f));	//フレーム更新

		std::shared_ptr<FrameWork::TextureData> textureData = FrameWork::LoadTexture("Assets/tile.png");


		window->SwapBuffers();		//ダブルバッファリング
		window->Wait();				//フレームレート制御
	}

Answer 1

std::make_shared<unsigned char*>(stbi_load(fileName, &data.size.x, &data.size.y, &data.channel, 0));

これはあなたの想定したようには動作しないでしょう。

data.fileData = std::shared_ptr<unsigned char>(stbi_load(fileName, &data.size.x, &data.size.y, &data.channel, 0), stbi_image_free);

にて動作するかもしれません。

Comments

[退会済みユーザー]

エラーになります。
エラーコードを見て
data.fileData = std::shared_ptr<unsigned char*>(stbi_load(fileName, &data.size.x, &data.size.y, &data.channel, 0), stbi_image_free);
としましたがエラーが出るのですが？
重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー (アクティブ) E0289 コンストラクター "std::shared_ptr<_Ty>::shared_ptr [代入_Ty=unsigned char *]" のインスタンスが引数リストと一致しません FrameWork C:\Users\yw325\Desktop\ActionGame\FrameWork\FrameWork\Texture.cpp 13

と表示されるのでそもそも違うのではないでしょうか？

[asm]

そうですね。型の方も修正が必要ですね

[int32_t]

stbi_load() が https://github.com/nothings/stb/blob/master/stb_image.h これなら、shared_ptr<unsigned char*> も shared_ptr<unsigned char> もダメなのでは。stbi_image_free() で開放する必要があるようですから。

[asm]

私の提示コードではdeleter指定していますが問題ありますか？
(構造体に入れようとすると面倒くさいかもしれないが)

[int32_t]

あ、コードの後ろの方をスクロールして見てませんでした。失礼しました。

[退会済みユーザー]

以下のうに(unsigned char*)型にキャストすると例外エラーが出るのですがそもそも無理なのでしょうか?
glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA, textureID.back().size.x, textureID.back().size.y, 0, GL_RGBA, GL_UNSIGNED_BYTE, (unsigned char*)*tex->fileData);

[asm]

tex->fileData.get()
です

Answer 2

whileループ内に書かれている

std::shared_ptrFrameWork::TextureData textureData = ...

の寿命はループの末尾で尽きます．
（つまりこいつは毎度作られては破棄されてる）

---

メモリリークしているのだとすれば

std::shared_ptr<unsigned char*> fileData; //画像データ

これが怪しいように思うが．