返回后 std::map 值丢失

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:

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

Answer 3

您在哪里评论过“在此处打印 FOO”，您是如何编译它的？您不能通过 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 对象，你必须在某个地方使用delete 来清理它们，否则你的程序会内存泄漏。因此，人们通常不会将指针直接放入映射或列表中。相反，只需将 Foo 对象的副本放入（无指针），或者使用 shared_ptr 等包装器为您管理删除。