ベクトルを使用するクラスオブジェクトをpush_backします。

Question

クラスのオブジェクトを別のクラスのベクトルに追加する方法。

#include <iostream> 
#include <vector> 
#include <string> 

using namespace std; 

class info{ 
    private: 
     int id; 
     string name; 
    public: 
     info(int extId, string extName) { 
      this->id = extId; 
      this->name = extName; 
     } 
}; 

class db { 
    private: 
     vector<info> infoVector; 
    public: 
     void pushData(info * data) { 
      this->infoVector.push_back(&data); 
     } 
}; 

int main(){ 
    info * testData = new info(123, "nice"); 

    db database; 
    database.pushData(testData);  

    return 0; 
} 

私はinfoクラスのオブジェクトを作成しています。オブジェクトには、1つのintと1つのstringの変数が含まれています。それから私はdbオブジェクトを作成していて、そこにはtestDataオブジェクトを渡しています。

プロジェクトの作成中にエラーメッセージが表示されました。

main.cpp: In member function ‘void db::pushData(info*)’: 
main.cpp:23:44: error: no matching function for call to ‘std::vector<info>::push_back(info*&)’ 
      this->infoVector.push_back(data); 
              ^
In file included from /usr/include/c++/5/vector:64:0, 
       from main.cpp:2: 
/usr/include/c++/5/bits/stl_vector.h:913:7: note: candidate: void std::vector<_Tp, _Alloc>::push_back(const value_type&) [with _Tp = info; _Alloc = std::allocator<info>; std::vector<_Tp, _Alloc>::value_type = info] 
     push_back(const value_type& __x) 
    ^
/usr/include/c++/5/bits/stl_vector.h:913:7: note: no known conversion for argument 1 from ‘info*’ to ‘const value_type& {aka const info&}’ 

私は間違っていますか？

Answer 1

info *タイプのアドレスをvector<info>::push_back（only accepts types of const info & or info &&）に渡そうとしているようです。あなたがpush_backを呼び出すときdereference operator * instead of the address-of operator &を使用してみてください：

this->infoVector.push_back(*data); 

しかしこれは、ポインタを使用するのに最適な方法ではない、とdataは、ベクターからか、削除された場合、削除された場合は、メモリリークやセグメンテーション違反につながる可能性があります。その後、削除

std::vector<info*> infoVector; 

：

class db { 
    private: 
    vector<info> infoVector; 
    public: 
    void pushData(info data) {   // note: not a pointer 
     this->infoVector.push_back(data); // note: not address-of 
    } 
}; 

int main(){ 
    info testData(123, "nice"); // note: not a pointer 
    db database; 
    database.pushData(testData); 
    return 0; 
} 

をそれ以外の場合は、あなたが本当にinfoVectorは、ポインタを含むようにしたい場合として、それを宣言し、：それはあなたの代わりにこれをやって検討するかもしれないので、そのメンバーを所有するvectorのためのより良いですアドレス - 演算子。

P.S.、avoid using namespace std whenever possible！

Answer 2

あなたはvector<info>を持っていて、info *を置きたい、やってみてください：

int main(){ 
    info testData(123, "nice"); 

    db database; 
    database.pushData(testData);  

    return 0; 
}