std ::映射值在返回後丟失

Question

如何正確處理內存，同時將地圖從一個函數傳遞到另一個函數。

我有一個函數返回它構建的地圖。值對象是一個foo類。 我在三個不同的位置打印foo，並且它們都給出不同的值。第一次，它給出了正確的值。第二和第三是垃圾。

我知道我必須在正確的地方製作Foo對象指針。

我想知道在哪裏？

std::map<int,Foo*> function_that_returns_the_map(){ 

    std::map<int,Foo*> myMap; 

    { 

    int v = 0; 
    Foo *b = new Foo(); 

    // PRINTING FOO FIRST 

    std::cout<<""<<*b<<endl; 

    myMap.insert(std::pair<int,Foo*>(v,b)) 

    } 

    // PRINTING FOO AGAIN 

    for(map<int,Foo*>::iterator it = myMap.begin(); 
    it != myMap.end(); ++it) 
    {  
    std::cout << " " << *(it->second) << "\n"; 
    } 


    return myMap; 
    } 

std::map<int, Foo*> myMap; 
myMap = function_that_returns_the_map(); 

//PRINTING FOO AGAIN. 

std::map<int, Board*>::iterator it = myMap.begin(); 
for (it=myMap.begin(); it!=myMap.end(); ++it) 
    cout<<" "<<*(it->second)<<endl; 

Click here看到我的實際代碼。

更新：Foo的成員變量沒有使用'new'運算符分配。因此，他們超出了範圍，並在垃圾價值超出範圍之後出現。

Answer 1

你的代碼有很多小錯誤（我認爲這只是拼寫錯誤）。我固定的，並提供了Foo類，並將其編譯和運行良好，在所有三個地方打印正確的值：

#include <iostream> 
#include <map> 

struct Foo 
{ 
    Foo() : someValue(5) {}; 
    int someValue; 
}; 

std::map<int,Foo*> function_that_returns_the_map() 
{ 
    std::map<int,Foo*> myMap; 
    { 
     int v = 0; 
     Foo *b = new Foo(); 
     std::cout << (*b).someValue << std::endl; // PRINTING FOO FIRST 
     myMap.insert(std::pair<int,Foo*>(v,b)); 
    } 

    // PRINTING FOO AGAIN 
    std::map<int, Foo*>::iterator it = myMap.begin(); 
    for(it; it != myMap.end(); ++it) 
    { 
     std::cout << it->second->someValue << "\n"; 
    } 
    return myMap; 
} 

int main() 
{ 
    std::map<int, Foo*> myMap; 
    myMap = function_that_returns_the_map(); 

    //PRINTING FOO AGAIN. 

    std::map<int, Foo*>::iterator it = myMap.begin(); 
    for (it; it!=myMap.end(); ++it) 
    std::cout << it->second->someValue << std::endl; 
    return 0; 
} 

Click here查看輸出。

因此，這個問題必須在你的問題中沒有提到的問題。爲了能夠進一步提供幫助，我們需要查看真實的代碼。

Answer 2

確保您實際創建std::pair了正確的價值觀：

myMap.insert(std::pair<int,Foo*>(v, b)); 
//        ^^^^ 

或利用std::make_pair：「這裏的印刷FOO」

myMap.insert(std::make_pair(v, b)); 

Answer 3

你到哪兒去評論你怎麼連編譯它？您無法在任何地方通過b局部變量打印Foo，因爲它超出了範圍。您而必須檢索對象縮小地圖

我做了以下修改你的代碼，而這一切工作正常：

class Foo 
{ 
public: 
    const int Value; 
    Foo(int value) : Value(value) // add some kind of identifier to Foo so we can check it's not garbage 
    { } 
}; 

std::map<int,Foo*> function_that_returns_the_map() 
{ 
    std::map<int,Foo*> myMap; 

    { // introducing a new scope 

     int v = 0; 
     Foo *b = new Foo(98765); 

     // PRINTING FOO HERE. 
     std::cout << b->Value << std::endl; 

     myMap.insert(std::pair<int,Foo*>(v,b)); 
    } // v, b go out of scope, are no longer accessible 

    // PRINTING FOO HERE. 
    std::cout << myMap[0]->Value << std::endl; // we can't use v, b anymore, so go fish in the map to find the Foo 

    return myMap; 
} 

void main() 
{ 
    std::map<int, Foo*> myMap; 
    myMap = function_that_returns_the_map(); 

    //PRINTING FOO HERE. 
    std::cout << myMap[0]->Value << std::endl; // we can't use v, b anymore, so go fish in the map to find the Foo 
} 

的關鍵是作用域。 C++中的大括號意味着範圍。在該範圍內聲明的任何局部變量不能從外部使用。希望這將有助於解釋它。如果沒有，請評論任何沒有意義的事情。

PS：記住，因爲你已經使用new創建您Foo對象，你MUST使用delete地方清除它們，否則你的程序將內存泄漏。因此，人們通常不會將指針直接放入地圖或列表中。相反，只需將Foo對象的副本放在（無指針）中，或者使用包裝器（如shared_ptr）來管理您的刪除。