【C++】vectorの初期化

Question

コンストラクタで、以下のメンバ変数をnullptrで初期化したいのですが、どのように書いたらいいですか？

 std::vector<class Cat*> cats;

Comments

[cateye]

初期vectorは、要素がありません（初期化するものがない）。分かりますか？

[episteme]

欲するものはCat*の列? それともCatの列?

[torimingo]

>>cateyeさん
なるほど、初期化するもの自体がないんですね・・・！
>>epistemeさん
Catクラスをnewした戻り値（オブジェクトのポインタ）をvectorで管理することを意図したのですが、スマートポインタというものがあることを知って、よくわからなくなってしまいました。

[episteme]

↑ちょっと言ってる意味がわからない。スマートポインタを知らなかったらできたのに? だったら質問してないよね。

Answer 1

こんにちは。

Catへのポインタを要素とするstd::vectorですね。
std::vectorはデフォルトでは要素数が0ですので、当然Catへのポインタは1つもありません。
従って、それを nullptr で初期化することはできないですよ。

それから、C++11以降、new や delete はほとんど書かずに済ませることが出来るようになりました。
これらはリークの元ですから、書かないで済む時は書かないようにするとバグを減らせます。
例えば、std::unique_ptrを使って次のような定義がお薦めです。

#include <vector>
#include <memory>
  :
typedef std::unique_ptr<Cat> CatPointer;
std::vector<CatPointer> cats;

Comments

[torimingo]

ご回答ありがとうございました。
unique_ptrの例をありがとうございます。
ユニークポインタの勉強をしたいと思います。

Answer 2

member変数がcatsってことですか？するとおかしいです。catsはvectorなので。

要素数0のvectorがほしければ

struct foo {
    std::vector<class Cat*> cats;
    foo() : cats{} {}
};

でいいですが。

それはそうとstd::vector<class Cat*>ってなんか変です。もしかしてどっかでnewとかdeleteとか書いてませんか？

newやdeleteを書くのは完全に誤っていて、スマートポインタを利用するべきです。

• std::unique_ptr
• std::shared_ptr

がありますので目的に合わせて書き換えたほうがいいでしょう。

Comments

[torimingo]

ご回答ありがとうございました。
最も早く正しい回答を頂けたので、ベストアンサーにさせて頂きました。
スマートポインタの勉強をしていきたいと思います。

[episteme]

nullptrで初期化されてないのに、なんでこれで解決なんよ?

[torimingo]

>>epistemeさん
ごめんなさい、なんか、間違えちゃいましたかね・・・？
要素があるからnullptrで初期化しようとしていたんですが、要素がないという回答を頂けたというわけでしたので・・・汗

[episteme]

よーするに 「cats のナカミをからっぽにしたい」だったのかな。だったら初期状態でからっぽだからなんもせんでえぇ、と。

[torimingo]

>>epistemeさん
はい、そんな感じです・・・！

Answer 3

無理やりやろうとすると・・・

#include <vector>
//
class Cat
{
    int a;
    int b;
  public:
    Cat() : a(0), b(0) {}
};

int main(void)
{
    std::vector<Cat *> cats;
    //
    Cat *catp = nullptr;
    for (int i = 0; i < 10; i++) {
        cats.push_back(catp);
    }
    //
    return 0;
}

・・・勝手に10個もベクター要素作って意味が有るのか？　この後どうするんだろう？