1

我应该在两行中都得到相同的结果..会发生什么我得到两个不同的值..就像它针对不同的位置一样..我认为错误在里面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();
}
4

2 回答 2

5

当您将类的实例传递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()总是隐藏thisprintf()参数），然后再传递给函数。请记住 - 您之前指向该临时副本的指针仍然指向堆栈，但它现在被不同的数据占用，导致您看到损坏

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

于 2012-12-11T19:56:44.767 回答
1

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

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

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

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

于 2012-12-11T20:00:34.950 回答