B类中的A类初始化,包含指向A类的指针

时间:2012-05-01 16:25:32

标签: c++ pointers vector initialization

我希望A类包含一些数据,B类将包含指向该数据的指针。我通过返回对数据对象A的引用的函数提供对数据的访问。如果我创建对象B然后我可以访问对象A,但是如果创建指向B的指针则等效操作产生分段错误。像这样:

#include <iostream>
#include <vector>

class A {
public:
    A(const int pInt) {mInt = pInt;}
    void print() {std::cout << mInt << std::endl;}
private:
    int mInt; //some data
};

class B {
public:
   B() {mP1 = new A(1);} //initialise to 1
   ~B() {delete mP1;}
   A& access() {return *mP1;} //return reference to the data

private:
    A* mP1; //pointer to some data
};

int main() {
    B vB;
    vB.access().print(); //this works.

    B *vBptr;
    vBptr->access().print(); //Segmentation fault!

    std::vector<B*> vVec;
    vVec.resize(1);
    vVec[0]->access().print(); //Segmentation fault!
}

我想在创建B * vBptr时,B对象没有初始化? 那我怎么能创建一个指向自动初始化的B对象的指针向量?

干杯。

7 个答案:

答案 0 :(得分:3)

您需要初始化指针:

B *vBptr = new B;
vBptr->access().print();

同样适用于您的矢量 - 您必须遍历元素并初始化它们。

for ( int i = 0 ; i < vVec.size() ; i++ )
   vVec[i] = new B;

请注意,您必须手动释放内存

delete vBptr;

for ( int i = 0 ; i < vVec.size() ; i++ )
   delete vVec[i];

不要遇到内存泄漏。

答案 1 :(得分:2)

程序中的两个问题

B *vBptr;

您的指针尚未初始化,虽然它的指针类型为B但未指向类型B的有效对象。将其更改为

B *vBptr = new B();

vVec.resize(1);

此处调整大小不会为向量的每个元素分配存储空间。您可以通过向向量

的resize方法提供初始化来使用实际对象初始化元素
vVec.resize(1,new B());

答案 2 :(得分:1)

您实际上需要将指针设置为:

B *vBptr;          // bad -- uninitialized 
B *vBptr = new B;  // proper

答案 3 :(得分:1)

如其他答案中所述,指针变量不会自动为其分配内存,因此您需要手动分配和释放内存,如下所示:

B vB;
vB.access().print(); //this works.

B *vBptr = new B;//allocate memory for vBptr
vBptr->access().print();
delete vBptr;//clean up of vBptr
std::vector<B*> vVec;
vVec.push_back(new B);
vVec[0]->access().print(); 
delete vVec[0];

但是我建议不要使用原始指针来管理内存,因为它很容易导致内存泄漏和错误。例如。如果vBptr->access().print()抛出异常,上面的代码会泄漏内存。您的B类违反了规则3(手动管理资源时默认的复制构造函数/赋值操作)。因此,如果您复制B对象,可能会发生可怕的事情(双重释放并准确访问已经删除的内存)。

因此我建议你使用某种智能指针。如果您使用c ++ 11,则可以使用std::shared_ptrstd::unique_ptr,否则它们为std::tr1::shared_ptr。如果您的编译器没有tr1,则boost也有boost:shared_ptr。然后您的代码如下所示:

class B {
public:
   B():mP1(new A(1)) {} //initialise to 1
   ~B() {} //no extra managing necessary
   A& access() {return *mP1;} //return reference to the data

private:
    std::shared_ptr<A> mP1; //pointer to some data
};

int main() {
    B vB;
    vB.access().print(); //this works.

    std::shared_ptr<B> vBptr(new B);
    vBptr->access().print();

    std::vector<std::shared_ptr<B> > vVec;
    vVec.push_back(std::shared_ptr<B>(new B));
    vVec[0]->access().print(); 
}

如果您需要最少的开销并使用c ++ 11 std::unique_ptr是您的朋友,否则iirc boost有boost::unique_ptr。远离std::auto_ptr。除非您喜欢增加的错误复杂性/可能性或者您的代码对性能非常敏感(并且使用非常高效的自定义分配器),否则没有什么理由手动管理您的内存。

答案 4 :(得分:0)

B *vBptr;
vBptr->access().print(); //Segmentation fault!

嗯,当然;您创建了一个存储指向B对象的指针的变量,但实际上并没有指向B对象。 (提示:看看你为B类编写的代码,你在哪个地方写出了A* mP1点?)

std::vector<B*> vVec;
vVec.resize(1);
vVec[0]->access().print(); //Segmentation fault!

嗯,当然;你创建了一个指向B对象的指针向量,然后你调整了向量的大小以添加一个元素;该元素是默认构造的指向B的指针。对于指针类型,默认构造是空指针,因此您不再指向任何实际的B对象。

您的问题与任何A或B实施无关。

  

如何创建指向B对象的指针向量

为什么你认为你想这样做?制作B对象的向量有什么问题?

答案 5 :(得分:0)

B vB;
vB.access().print();

B *vBptr = &vB; // or B *vBptr = new B;
vBptr->access().print();

std::vector<B*> vVec;
vVec.push_back(&vB);  // vVec.push_back(new B);
vVec[0]->access().print(); 

但是,是否需要以这种方式写B?

class B {
 public:
  //   B() {}
  //   ~B() {}
  A& access() {return a;}

private:
   A a;};

答案 6 :(得分:0)

感谢所有贡献的人。以下是我现在采用的实现,即使用boost :: shared_ptr:

#include <iostream>
#include <vector>
#include "boost/smart_ptr.hpp"

class A {
public:
    A(const int pInt) {mInt = pInt;}
    void print() const {std::cout << mInt << std::endl;}
    void set(const int pInt) {mInt = pInt;}
private:
    int mInt; //some data
};

class B {
public:
    B() {}  //leave A pointer as null
    ~B() { } //delete handled by shared_ptr
    A& access() {return *mP1;} //return reference to the data
    boost::shared_ptr<A>& access_A_ptr() {return mP1;} //return the pointer for assignment

private:
    boost::shared_ptr<A> mP1;
};

int main() {

    std::vector< boost::shared_ptr<A> > vVecA; //data to be shared
    for (unsigned int i = 0; i < 5; i++) {
        boost::shared_ptr<A> vAptr(new A(i));
        vVecA.push_back(vAptr);
        vVecA[i]->print();
    }

    vVecA[2]->set(123);

    for (unsigned int i = 0; i < vVecA.size(); i++) {
        vVecA[i]->print(); //changes to the underlying objects are reflected
    }

    boost::shared_ptr<B> vBptr(new B); //make an empty B
    vBptr->access_A_ptr() = vVecA[2]; //assignment of shared pointer

    vBptr->access().print();

    std::cout << "use count of vVecA[1] = " << vVecA[1].use_count() << std::endl; // = 1
    std::cout << "use count of vVecA[2] = " << vVecA[2].use_count() << std::endl; // = 2

    std::vector< boost::shared_ptr<B> > vVecB;

    //vVecB.resize(vVecA.size(), boost::shared_ptr<B> (new B) ); //resize and init to a SINGLE B object, NO!!!

    for (unsigned int i = 0; i < vVecA.size(); i++) {
        vVecB.push_back(boost::shared_ptr<B> (new B)); //filling array with separate empty Bs
        vVecB[i]->access_A_ptr() = vVecA[i];
        vVecB[i]->access().print(); // = 0,1,123,3,4
    }

    vVecA[2]->set(2); //changes to A objects reflected in the B objects

    for (unsigned int i = 0; i < vVecB.size(); i++) {
        vVecB[i]->access().print(); // = 0,1,2,3,4
    }

    std::cout << "use count of vVecA[1] = " << vVecA[1].use_count() << std::endl; // = 2
    std::cout << "use count of vVecA[2] = " << vVecA[2].use_count() << std::endl; // = 3

}

这实际上已成为共享指针的练习。然而,评论在任何方面仍然非常受欢迎。

谢谢!