ポインタvector型への変換

Question

C++ ポインタvector配列への詰め替え

vector型から ポインタvector型への変換を行いたいです。

• const std::vector<T>&からconst std::vector<T*>&への変換

前提

C++の使用歴が浅く、不勉強であることは承知しております。
社内でC++がわかる人もおらず、前任者もいないため困り果てています。

社内の遺産的なコードに手を入れている関係で大きく作り直すことは不可能であり、
今回、このような対応をすることとなりました。

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

const std::vector<T>&から、const std::vector<T*>&への変換方法がわからない。

該当のソースコード

void ClassA::MethodA(const std::vector<T> & arg1)
{
      const std::vector<T*> arg2;
      /*
          ここで、arg1からarg2への詰め替えを行いたい。
　　　この詰め替え方法がわからないです。
       */     

　　// 詰め替えたarg2をMethodBの引数として使用したい。
     MethodB(arg2);
}

void ClassA::MethodB(const std::vector<T*>& arg3)
{
   //引数arg3を使用して処理を行う
}

試したこと

arg2に対して、push_backを行うときにnewなどを行いましたが、メモリリークを引き起こしてしまい、
できませんでした。
何をどうすれば、新しいvectorに詰め替えることができるのか見当もつかず困り果てています。

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

ツールはvisual studio 2012、言語はC++です。

Comments

[ianaka3]

すいません。
私の書き方がいけなかったのですが、vector<T>のTが独自クラスの場合、どうすれば良いのか教えて頂けると助かります。

[episteme]

独自クラスの場合、何か問題がありますか?

Answer 1

すでに解決方法は出たようですので、補足的に解説をします。

vector は実質的に内部に配列を持っており、そこに要素を格納するという構造です。

質問中の

push_back を行うときに new などを行いましたが

というのはおそらく以下のような要領でしょうが、個々の要素の管理は vector は行わないので寿命が尽きる前に個々に delete をしないと回収されずにメモリリークとなってしまいます。

#include <iostream>
#include <vector>

int main() {
    std::vector<int> src = {1, 2, 3};
    std::vector<int*> dst;

    // int -> int*
    for (int& item : src) {
        // あらたにメモリを割りあてると……
        dst.push_back(new int(item));
    }

    for (int* item : dst) {
        std::cout << (void*)item << " " << *item << std::endl;
    }

    // 解放する必要がある
    for (int* item : dst) {
        delete item;
    }
}

図で言えばこういう構造ですが……

詰め替え後の vector の寿命が尽きても個々の要素の管理はまた別だということです。

episteme さんが提示する方法は元の要素をそのまま指すというものです。

質問文に書かれている状況だとおそらくこれで良いとは思うのですが、 vector は状況によって内部で管理する配列の再配置をすることがあります。 詳細な条件はここでは述べませんがたとえば要素をひとつ追加するというだけでも再配置が起こる可能性があります。

もしも再配置が起こってしまうと再配置が起こる前の場所を指しているポインタは全て無効になってしまいます。 つまり再配置が起こらないように配慮する必要があり、よく理解できていないならポインタの vector を使っている間は元の配列には全く触れないのが無難です。

Answer 2

#include <iostream>
#include <vector>

int main() {
  std::vector<int>  src = { 1, 2, 3 };
  std::vector<int*> dst;
  
  // int -> int*
  for ( int& item : src ) {
    dst.push_back(&item);
  }
  
  // できたかな?
  for ( int* item : dst ) {
    std::cout << (void*)item << " " << *item << std::endl;
  }
}

[追記]

vector<T>のTが独自クラスの場合、どうすれば良いのか教えて頂けると助かります。

同じです。

#include <iostream>
#include <vector>

class foo {
  int value_;
public:
  foo(int v=0) : value_(v) {}
  int value() const { return value_; }
};

int main() {
  std::vector<foo>  src = { 1, 2, 3 };
  std::vector<foo*> dst;
  
  // foo -> foo*
  for ( foo& item : src ) {
    dst.push_back(&item);
  }
  
  // できたかな?
  for ( foo& item : src ) {
    std::cout << (void*)&item << " " << item.value() << std::endl;
  }
  std::cout << std::endl;	
  for ( foo* item : dst ) {
    std::cout << (void*)item << " " << item->value() << std::endl;
  }
}

Comments

[ianaka3]

episteme様
回答いただき、ありがとうございます。
記載の方法で試してみたいと思います。