HPゼロでオブジェクトを破棄する仕様を作りたいです

Question

前提・実現したいこと

C++でRPGじみた簡単なゲームを作りたいです。
そこでオブジェクト内のprivateメンバ変数（ここではHP）が０になったら、そのオブジェクトを破棄するという仕様を作りたいです。別クラスのオブジェクト内のメンバ関数（attack関数）でHPがゼロ以下になったらその関数内で、参照中のオブジェクトを破棄したいです。

今回はHPがゼロになる可能性のある関数に、オブジェクトを破棄する仕組みを加えるつもりですが、そもそもクラスの中に、メンバ変数が一定の条件を満たしたときに、オブジェクトを破棄する仕組みがあれば教えてほしいです。

該当のソースコード

言語はc++です
#include<iostream>

using namespace std;

class test{
int HP;
public:
test(int hp){cout <<"コンストラクタ"<<endl; HP = hp;}
~test(){cout <<"デストラクタ"<<endl;}
int getHP(){return HP;}
int setHP(int hp){HP = hp;}
};

class systems{
public:
void damage(int d, test &ob){
int hp;
hp = ob.getHP() - d;
ob.setHP(hp);

    if(hp <= 0){
        cout << d << " ダメージ受けた!!\n";
        cout <<"HPがゼロになった\n";
        /* 
        ここで参照中のオブジェクトを破棄　
        */
    }else{
        cout << d << " ダメージ受けた!!\n";
    }
}

};

int main(){
test *t, t1(50);
systems s1;

t = &t1;

s1.damage(20,*t);
cout << t1.getHP() <<endl;
s1.damage(30,*t);


cout <<"終了\n";
return 0;

}

### 試したこと

ob.~test()で直接デストラクタを呼び出してもうまくいきませんでした。

Comments

[BeatStar]

この手の場合は別のクラスオブジェクトが管理して、間接的にやるんじゃないかな。

[fana]

> オブジェクトを破棄

する意義がわかりせん．
（仮にそのmain()関数内で t1が唐突に解体されたとして，一体何がうれしいのですか？）

オブジェクトを「破棄」することが何らかの意味を持つようなコード例を示すことはできませんか？

[Spe3_0624]

testクラスに敵の情報の基を設定して、main関数内で敵を生成したいときに、testクラスを使って、仮引数（敵のHPなど）とともに敵を生成するという仕様で行こうと思ってました。そこで敵のHP（testのオブジェクト内のメンバ変数）が０になったら、そのままオブジェクトが残っていると気持ち悪いので、その時点で破棄したいと考えていました。
確かに今考えると、オブジェクトを破棄する場合は別のクラスなり関数なりを使用して破棄するのも手だと思いました。

[fana]

何と言うか，オブジェクトが唐突に「破棄」されるのだとして，その「破棄」結果があなたのプログラムの動作(?)に何かしらの良い影響を与えてくれるような【仕組み】が無いと意味ないですよね．
【あるオブジェクトを「破棄」したならば，あれがこうなってここがああなるから，それによってこういう動作になってうれしい】みたいな．

> オブジェクトが残っていると気持ち悪い

とかじゃなくて．

[Spe3_0624]

仕様としては、メンバ変数のHPがゼロになったら、デストラクタが起動して、「敵が倒れた」的な文とともに、次のステージに段階を進める操作を加えたいと考えていました。
仮にここでオブジェクトがそのままになっていた場合、main関数内に新たにHPが０になったかどうかを見極める関数を作らなければならず、その関数を使うタイミングなどが、プレイヤーからの操作によって実行手順が変わるプログラムにおいて少し不便だなと感じたので、HPが０になった時点でオブジェクトを処理する機能を加えたいです。

確かに現状のような小規模なプログラムでは別の方法でもなんとかなるかもしれませんが、ここからシステムを拡大していったときに支障が出ないとも言い切れません。

[Serbonis]

＞オブジェクトを破棄
これは、1)オブジェクトのインスタンスを破棄する、2)オブジェクトを再利用できるようにする、のどちらを意図されているでしょうか？

[Spe3_0624]

１）です。testクラスを基に敵（インスタンス）を生成し続け、そのインスタンス内のメンバ関数（HP）が０になり次第破棄するという仕様にしたいと考えています。

[Serbonis]

まず、t1はmain関数を抜けるまで領域を占有しています。デストラクタを明示的に呼び出しでも、それは単にデストラクタというメンバ関数を呼び出すだけで、領域の解放とは直接関係がありません(main関数を抜けるときにもう1回呼ばれます)。1)の意図であれば、敵のインスタンスはnewで適宜動的に確保するのがよいかと思います。そしてインスタンスの破棄はインスタンス自身が行うのではなく、インスタンスを管理する別のクラス(ex.エネミーマネージャー)に行わせる方がよいかと思います。

[fana]

> testクラスを基に敵（インスタンス）を生成し続け

（繰り返しになる感ですが）
刻々と生成なり破棄されていくインスタンス群をどうやって管理するのか？　という話の想定がまず存在していないと，そもそも「破棄」の効能が不明瞭ですよね．

・生成とはどうやって行うのか？
・生成したからには以降でいつかそいつにアクセスしたくなるだろうけど，それはどうやってやるのか？
・破棄したからには以降はそいつにはアクセスしてはいけないのだろうけど，そのことはどうやって担保されるのか？
・etc...

（例えば，既についている回答では std::vector<test> によって管理していますよね）

Answer 1

#include <iostream>
#include <algorithm>
#include <vector>

class test {
  int HP;
public:
  test(int hp) { std::cout <<"コンストラクタ\n"; HP = hp;}
 ~test()       { std::cout <<"デストラクタ\n"; }
  int  getHP() const { return HP; }
  void setHP(int hp) { HP = hp; }
};

class damage {
  int d_;
public:
  damage(int d) : d_(d) {}
  bool operator()(test& ob) const {
    int hp = ob.getHP() - d_;
    ob.setHP(hp);
    if  ( hp <= 0 ) {
      std::cout << d_ << " ダメージ受けた!!\n";
      std::cout <<"HPがゼロになった\n";
      return true;
    } else {
      std::cout << d_ << " ダメージ受けた!!\n";
      return false;
    }
  }
};

int main() {
	
    std::vector<test> obs;
    obs.reserve(10);
    for ( auto hp : { 50, 15, 60, 40 }) {
      obs.emplace_back(hp);
    }
    std::cout << obs.size() << " remains.\n\n";

    obs.erase(std::remove_if(obs.begin(), obs.end(), damage(20)), obs.end());
    std::cout << obs.size() << " remains.\n\n";

    obs.erase(std::remove_if(obs.begin(), obs.end(), damage(30)), obs.end());
    std::cout << obs.size() << " remains.\n\n";
}

Answer 2

#include <iostream>
using namespace std;

class test{

    int HP;
    public:
        test(int hp){ cout <<"コンストラクタ"<<endl; HP = hp; }
        ~test(){ cout <<"デストラクタ"<<endl; }
        int getHP(){ return HP; }
        void setHP(int hp){ HP = hp; }
};

class systems{

    public:
        void damage(int d, test *ob){
        
            int hp;
            hp = ob->getHP() - d;
            ob->setHP(hp);
    
            if (hp <= 0){

                cout << d << " ダメージ受けた!!\n";
                cout <<"HPがゼロになった\n";
                
                delete ob;
                
            } else {
                cout << d << " ダメージ受けた!!\n";
            }
        }
};

int main(){

    test *t = new test(50);

    systems s1;
    
    s1.damage(20, t);
    cout << t->getHP() <<endl;
    s1.damage(30, t);
    
    cout <<"終了\n";
    return 0;
}

Serbonis さんご指摘のとおり、new で動的にオブジェクトを宣言しないと delete できません。

それから、↑のように、new する場所と delete する場所が違うのは大変気持ち悪いです。
systems クラスの中に、敵の生成 (new) するメソッドを作るとかした方がよいと思います。

Comments

[episteme]

オブジェクトの集合を用意すれば「new で動的にオブジェクトを宣言しないと delete できません」
じゃなくなる。(回答した)