メモリリークではないがstd::vectorで一定数までメモリ使用量が増える理由が知りたい。

Question

提示コードですがコメント内部の///内部のコードですが開放しているのにメモリ使用量が一定量まで上昇してあとは一定になるというバグが発生しますこれはなぜでしょうか？画面に現在の要素が増えるならわかりますがそれが画面の外に出てeraseしているのも関わらずメモリ使用量が減りません。**メモリリークではないので一定数まで上昇してあとは一定になります。**これはなぜでしょうか？

#include "../../Header/Game/Player.hpp"

Player::Player(Entry *e)
{
	Owner = e;	//Entry クラス
	mPosition = glm::ivec2(500,-500);	//座標
	mVector = VECTOR_UP;				//方向

	mSprite = std::make_shared<Sprite>(e,LoadTexture("Assets/Player.png"));	//スプライト
	UV = TextureUV{ glm::vec2(0,0),glm::vec2(CELL,CELL) };					//UV座標	
	InputKey = std::make_shared<Input>(e);									//キー入力
	text = std::make_shared<Text>(e);										//文字表示

	//バレットのスプライト
	bullet_sprite[0] = LoadTexture("Assets/Bullet_UP.png");
	bullet_sprite[1] = LoadTexture("Assets/Bullet_DOWN.png");
	bullet_sprite[2] = LoadTexture("Assets/Bullet_LEFT.png");
	bullet_sprite[3] = LoadTexture("Assets/Bullet_RIGHT.png");


}


// プレイヤー更新
void Player::Update()
{
	//移動
	mSpeed = 10;
	if (InputKey->KeyDownHold(GLFW_KEY_LEFT) == true)
	{
		mVector = VECTOR_LEFT;
	}
	else if (InputKey->KeyDownHold(GLFW_KEY_RIGHT) == true)
	{
		mVector = VECTOR_RIGHT;
	}
	else if (InputKey->KeyDownHold(GLFW_KEY_DOWN) == true)
	{
		mVector = VECTOR_DOWN;
	}
	else if (InputKey->KeyDownHold(GLFW_KEY_UP) == true)
	{
		mVector = VECTOR_UP;
	}
	else
	{
		mSpeed = 0;
	}

	//バレット発射
	if (InputKey->KeyDown(GLFW_KEY_SPACE) == true)
	{
		bullet.push_back(Bullet(Owner,bullet_sprite,mVector,mPosition));
	}
	
	mPosition += mVector * mSpeed;

	Bullet_Update();	//バレット更新
}

//////////////////////////////////////////////////////////////////////////////////////////////////////
//バレット更新
void Player::Bullet_Update()
{
	for (std::vector<Bullet>::iterator itr = bullet.begin(); itr != bullet.end();)
	{
		if (itr->getPostion().x > SCREEN_WIDTH || itr->getPostion().x < 0 
			|| itr->getPostion().y < -SCREEN_HEIGHT || itr->getPostion().y > 0 )
		{
//			printf("あああ\n");
			itr = bullet.erase(itr);
		}
		else
		{
			itr->Update();
			itr++;
		}
	
		
	}
}
//////////////////////////////////////////////////////////////////////////////////////////////////////

//プレイヤー描画
void Player::Draw()
{
	Bullet_Draw();

	mSprite->DrawGraph(mPosition,UV.start,UV.end);
	text->RenderText(0, 0,1.0,glm::vec3(0,1,0),"Position: %d , %d",mPosition.x, mPosition.y);
}

//バレット描画
void Player::Bullet_Draw()
{
	
	for (std::vector<Bullet>::iterator itr = bullet.begin(); itr != bullet.end();)
	{
		itr->Draw();
		itr++;
	}
}


Player::~Player()
{

}

#ifndef ___PLAYER_HPP
#define ___PLAYER_HPP

#include "../Actor_2D.hpp"
#include "../Text.hpp"
#include "../Sprite.hpp"
#include "../LoadTexture.hpp"
#include "../Entry.hpp"
#include "../Input.hpp"
#include "../Collision.hpp"
#include "Bullet.hpp"

#include <iostream>
#include <vector>

class Input;
class Text;
class Bullet;

class Player : public Actor_2D
{
public:
	Player(Entry* e);
	~Player();

	void Bullet_Update();
	void Bullet_Draw();

	void Update() override;
	void Draw() override;

	Box_Collision_2D mCol;

private:

	float radian = 0;
	int mSpeed = 10;

	std::shared_ptr<Text> text;

	// スプライト
	std::shared_ptr<Sprite> mSprite;

	TextureUV UV;	
	std::shared_ptr<Input> InputKey;


	std::shared_ptr<TextureData> bullet_sprite[4];	//弾のスプライト

	std::vector<Bullet> bullet;	//スプライト
	
};

#endif

#ifndef ___BULLET_HPP
#define ___BULLET_HPP

#include "../Actor_2D.hpp"
#include "../Text.hpp"
#include "../Sprite.hpp"
#include "../LoadTexture.hpp"
#include "../Entry.hpp"
#include "../Input.hpp"
#include "../Collision.hpp"
#include "../Circle.hpp"

#include <iostream>
#include <vector>

class Input;
class Text;
class Bullet : public Actor_2D
{
public:
	Bullet(Entry* e, std::shared_ptr<TextureData> Data[4],glm::ivec2 vec, glm::ivec2 pos);
	~Bullet();


	void Update() override;
	void Draw() override;

	Box_Collision_2D mCol;

private:

	float radian = 0;
	int mSpeed = 10;
	std::shared_ptr<Text> text;

	// スプライト
	std::shared_ptr<Sprite> mSprite_up;
	std::shared_ptr<Sprite> mSprite_down;
	std::shared_ptr<Sprite> mSprite_left;
	std::shared_ptr<Sprite> mSprite_right;

	TextureUV UV_UD;	//上下
	TextureUV UV_LR;	//左右

	glm::ivec2 Size;
};

#endif

#include "../../Header/Game/Bullet.hpp"

//コンストラクタ
Bullet::Bullet(Entry* e, std::shared_ptr<TextureData> Data[4], glm::ivec2 vec, glm::ivec2 pos) : Actor_2D()
{
	Owner = e;	//Entryクラス

	mVector = vec;		//方向
	
	//スプライト	
	mSprite_up = std::make_shared<Sprite>(Owner, Data[0]);
	mSprite_down = std::make_shared<Sprite>(Owner, Data[1]);
	mSprite_left = std::make_shared<Sprite>(Owner, Data[2]);
	mSprite_right = std::make_shared<Sprite>(Owner, Data[3]);

	//UV座標
	UV_UD = TextureUV{ glm::ivec2(0,0), glm::ivec2(Data[0]->mSize.x,Data[0]->mSize.y) };
	UV_LR = TextureUV{ glm::ivec2(0,0), glm::ivec2(Data[0]->mSize.y,Data[0]->mSize.x) };



	// 座標修正
	if (mVector == VECTOR_UP)
	{
		mPosition.x = pos.x + CELL / 2 - Data[0]->mSize.x / 2;
		mPosition.y = pos.y;
	}else if (mVector == VECTOR_DOWN)
	{
		mPosition.x = pos.x + CELL / 2 - Data[1]->mSize.x / 2;
		mPosition.y = pos.y;
	}else if (mVector == VECTOR_LEFT)
	{
		mPosition.x = pos.x;
		mPosition.y = pos.y - CELL / 2 - (-Data[2]->mSize.y / 2);
	}
	else if (mVector == VECTOR_RIGHT)
	{
		mPosition.x = pos.x;
		mPosition.y = pos.y - CELL / 2 - (-Data[2]->mSize.y / 2);
	}
	






	
	text = std::make_shared<Text>(e);		//テキスト表示


	
}



void Bullet::Update()
{

	mPosition += mVector * mSpeed;	//速度
	mCol.setMove(mVector * mSpeed);	//移動座標
	mCol.setPosition(mPosition);	//当たり判定の移動
}

void Bullet::Draw()
{

	//スプライトの角度を変更
	if (mVector == VECTOR_LEFT)
	{
		mSprite_left->DrawGraph(mPosition, UV_LR.start, UV_LR.end);
	}
	else if (mVector == VECTOR_RIGHT)
	{
		mSprite_right->DrawGraph(mPosition, UV_LR.start, UV_LR.end);
	}
	else if (mVector == VECTOR_DOWN)
	{
		mSprite_down->DrawGraph(mPosition, UV_UD.start, UV_UD.end);
	}
	else if (mVector == VECTOR_UP)
	{
		mSprite_up->DrawGraph(mPosition, UV_UD.start, UV_UD.end);
	}



	//text->RenderText(0, 0, 1.0, glm::vec3(0, 1, 0), "Position: %d , %d", mPosition.x, mPosition.y);

}


Bullet::~Bullet()
{

}

Answer 1

• vectorで
• eraseしてもメモリ使用量が減らない

ということであれば，こういう話かな．

Answer 2

動的メモリ管理の実装は、メモリ使用量だけでなく、実行コストも考慮する必要があります。

OSへリクエストしてメモリを取ってくるのには一定のコストがかかるので、ある程度のブロックでOSからもらってきて、それをプログラム内部で小分けにして使う、というような実装がよく行われます。

Comments

[退会済みユーザー]

メモリリークではないので実装としてはどうなのでしょうか？

[maisumakun]

> 実装としてはどうなのでしょうか？

今どきのコンピューター（おそらくx64用にコンパイルされているでしょうが）でたった64MBの話です。問題だとお考えなのですか？

[maisumakun]

（「どうなのでしょうか？」のニュアンスを取り間違えていた場合もお知らせください）