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); // the * b將引用指向b的對象ÿb並把它入堆棧以準備調用add()
3. void DClass::add(BClass s){ //一個解除引用的對象的深拷貝被放入該函數的堆棧幀
4. list.push_back(&s); //的的那臨時拷貝的地址原始對象正被用於添加到您的列表中
5. } //這就是樂趣發生的地方 - 一旦函數完成，它將展開堆棧備份，之前由該臨時副本佔用的內存將被重新 - 用於其他目的。在你的情況下 - 它將用於傳遞參數到函數d->getNext()（總是有一個隱藏的這個參數給非靜態成員函數），後來到printf()函數。記住 - 你以前的指針指向臨時副本仍然指向到堆棧，但它現在由不同的數據佔用，導致你看到腐敗

一般的經驗法則 - 從未使用指針來臨時對象;-)

Answer 2

在DClass::add函數中，BClass s是一個局部變量。

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

當你調用d->add(*b);，你是按值傳遞一個BClass，這意味着你就創建了一個副本，該副本的地址是不是原來的地址相同。

s會盡快走出去的範圍爲函數返回，並指向它的指針將是無效的。所以存儲這個指針對你來說並不好，因爲解引用它將是未定義的行爲。