c ++向量丢失指针引用

Question

我应该在两个行中得到相同的...发生了什么我得到两个不同的值...就像它针对的是不同的位置..我认为错误在于 d->add(*b)

输出是

  thiago 14333804
  Ph¿?? 2816532

更好地描述它我把下面的代码

我有一个程序

int main(int argc, char **argv) {

    CClass* c = new CClass();

    BClass* b = c->getNext();

    printf("%s %d \n", b->getValue(), b->getValue());

    DClass* d = new DClass();
    d->add(*b);

    printf("%s %d \n", d->getNext(), d->getNext());

    cin.get();

    return 0;
}

接口如下

class BClass
{
private:
    char* value;
    bool stale;
public:
    BClass(char* value);
    ~BClass(void);
    char* getValue();
    bool isStale();
};


class CClass
{
private:
    vector<BClass*> list;
public:
    CClass(void);
    ~CClass(void);

    BClass* getNext();
};


class DClass
{
private:
    vector<BClass*> list;
    static bool isStale(BClass* b) { return b->isStale();};
public:
    DClass(void);
    ~DClass(void);
    void add(BClass s);
    char* getNext();
};

并且实施如下

//BClass

BClass::BClass(char* value)
{
    this->value = value;
    this->stale = false;
}

BClass::~BClass(void)
{
}

char* BClass::getValue()
{
    return value;
}

bool BClass::isStale()
{
    return stale;
}



//CClass

CClass::CClass(void)
{
    list.push_back(new BClass("thiago"));
    list.push_back(new BClass("bruno"));
    list.push_back(new BClass("carlos"));
}


CClass::~CClass(void)
{
}

BClass* CClass::getNext()
{
    return list.at(0);
}

//DClass

DClass::DClass(void)
{
}

DClass::~DClass(void)
{
}

void DClass::add( BClass s )
{
    list.push_back(&s);
}

char* DClass::getNext()
{
    BClass* b = list.at(0);

    return b->getValue();
}

Answer 1

将类的实例B传入D::add()函数时,您可以创建对象的深层副本,并将该副本放在堆栈上.稍后您使用该副本的地址将其推入列表.函数完成后,此自动变量超出范围,因此您用于放入列表的指针不再有效.

要修复您的界面以避免深层复制,如下所示:

void DClass::add( BClass * s )
{
    list.push_back(s);
}

逐步了解您的代码正在做什么

1. BClass* b = c->getNext(); //从列表中获取第一个元素的地址(在构造函数中创建)并将其分配给 b
2. d->add(*b);//*b将取消引用指向的对象b并将其放入堆栈以准备调用add()
3. void DClass::add( BClass s ){ //将解除引用对象的深层副本放入此函数的堆栈帧中
4. list.push_back(&s);// 正在使用原始对象的临时副本的地址添加到列表中
5. }//这就是乐趣发生的地方 - 一旦完成该功能,它将取消堆栈备份,之前被该临时副本占用的内存将被重新用于其他目的.在您的情况下 - 它将用于将参数传递给函数d->getNext()(对于非静态成员函数总是隐藏此参数),然后用于printf()函数.请记住 - 您之前指向该临时副本的指针仍然指向堆栈,但它现在被不同的数据占用,导致您看到损坏

一般经验法则 - 永远不要使用指向临时对象的指针;-)