c++向量丢失指针引用

c++ vector losing pointer reference

本文关键字：指针 引用 向量 c++      更新时间：2023-10-16

我应该在两行中得到相同的结果。。我得到了两个不同的值。。就像它瞄准了不同的位置。。我认为错误在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();
}

当您将类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()(非静态成员函数总是有一个隐藏的this参数)，然后传递给printf()函数。请记住，您以前指向该临时副本的指针仍然指向堆栈，但它现在被不同的数据占用，导致您看到损坏

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

在DClass::add函数中，BClass s是一个局部变量。

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

当您调用d->add(*b);时，您通过值传递BClass，这意味着您正在创建它的副本，而该副本的地址与原始地址不同。

一旦函数返回，s就会超出作用域，并且指向它的指针将无效。因此，存储该指针对您没有好处，因为取消引用它将是未定义的行为。

相关文章：