Shared_ptr 在 std::Vector 中重新迭代时为空

Question

我有下面的执行程序class，它填充如下所示的地图

map<string,map<string,vector<StructAbsTypeObject>>>

我在这里制作共享对象并分配它们，这些对象在第一次检查时有效，但在第二次检查时 shared_ptr returns 为空。我需要知道原因。代码看起来不错，但不知道哪里错了。

//Code begins
#include <iostream>
#include <vector>
#include <map>
#include <string>
#include <memory>

using namespace std;

class Test {
public:
  Test(int i):t(i) {
  }

private:
  int t;
};

class ConcTypeObject {
public:
  ConcTypeObject() {
  }

  ConcTypeObject(const ConcTypeObject& other) {
    m_ptr_Test = other.m_ptr_Test;
  }

  ConcTypeObject& operator=(const ConcTypeObject& other) {
    m_ptr_Test = other.m_ptr_Test;
  }

  void setTest(shared_ptr<Test> ptr) {
    cout << "setTest" << endl;
    m_ptr_Test = ptr;
  }

  shared_ptr<Test> getTest() {
    return m_ptr_Test;
  }

  bool isValid() {
    if(m_ptr_Test) {
      return true;
    } else {
      return false;
    }
  }

private:
  shared_ptr<Test> m_ptr_Test;
};


class AbsTypeObject {
public:
  explicit AbsTypeObject(const string str) {
    m_str = str;
  }

  AbsTypeObject(const AbsTypeObject& other) {
    m_str = other.m_str;
    m_ptr_ConcTypeObject = other.m_ptr_ConcTypeObject;
  }

  AbsTypeObject& operator=(const AbsTypeObject& other) {
    m_str = other.m_str;
    m_ptr_ConcTypeObject = other.m_ptr_ConcTypeObject;
  }

  bool operator==(const AbsTypeObject& other) {
    if(m_str == other.m_str)
      return true;
    else 
      return false;  
  }

  void setConcTypeObject(shared_ptr<ConcTypeObject> ptr) {
    m_ptr_ConcTypeObject = ptr;
  }

  shared_ptr<ConcTypeObject> getConcTypeObject() {
    return m_ptr_ConcTypeObject;
  }

  bool isValid() {
    if(m_ptr_ConcTypeObject) {
      cout << "AbsTypeObject 1 " << endl;
      return m_ptr_ConcTypeObject->isValid();
    } else {
      cout << "AbsTypeObject 2 " << endl;
      return false;
    }
  }

private:
  string m_str;
  shared_ptr<ConcTypeObject> m_ptr_ConcTypeObject;
};


class StructAbsTypeObject {
  public:
    StructAbsTypeObject(const string str):m_AbsTypeObject(str) {
    }

    void SetAbsTypeObject(AbsTypeObject& id) {
      m_AbsTypeObject = id;
    }

    AbsTypeObject& GetAbsTypeObject() {
      return m_AbsTypeObject;
    }

    private:
    AbsTypeObject m_AbsTypeObject;
};


class Executor {
public:
    static Executor m_Executor;
    static Executor& get() {
        return m_Executor;
    }

    Executor() {
      StructAbsTypeObject sid(std::string("ABCD"));
      vector<StructAbsTypeObject> a_vecstid;
      a_vecstid.push_back(sid);
      m_executormap["ExecutorInterface"]["ExecutorName"] = a_vecstid;
    }

    void check() {
      for(auto outermap : m_executormap) {
        for(auto innermap : outermap.second) {
          for(auto vec_element: innermap.second) {
            if(vec_element.GetAbsTypeObject().isValid()) {
              cout << "PTR VALID" << endl;
            } else {
              cout << "PTR NOT Valid" << endl;
            }
          }
        }
      }
    }

    void fillAbsTypeObject(AbsTypeObject &id) {
      shared_ptr<Test> ptr_test = make_shared<Test>(20);
      shared_ptr<ConcTypeObject> ptr_ConcTypeObject = make_shared<ConcTypeObject>();
      id.setConcTypeObject(ptr_ConcTypeObject);
      id.getConcTypeObject()->setTest(ptr_test);
    }

    void Init(AbsTypeObject id) {
      for(auto outermap : m_executormap) {
        for(auto innermap : outermap.second) {
          for(auto vec_element: innermap.second) {
            if(vec_element.GetAbsTypeObject() == id) {
              cout << "Id Equal" << endl;
              fillAbsTypeObject(id);
              vec_element.SetAbsTypeObject(id);
              if(vec_element.GetAbsTypeObject().isValid()) {
                cout << "PTR VALID" << endl;
              } else {
                cout << "PTR NOT Valid" << endl;
              }
            }
          }
        }
      check();
      }
    }

private:
  using executormap = map<string,map<string,vector<StructAbsTypeObject>>>;
  executormap m_executormap;
};

Executor Executor::m_Executor;

int main() 
{
  AbsTypeObject id(std::string("ABCD"));
  Executor::get().Init(id);

}
//Code Ends

以上代码是完全可编译可运行的。目前我得到以下输出

//Output Begins

Id Equal
setTest
AbsTypeObject 1 
PTR VALID
AbsTypeObject 2 
PTR NOT Valid

//Output Ends

执行校验函数时输出PTR NOT VALID。期望输出是 PTR 在这两种情况下均有效。

请告诉我上面的代码出了什么问题。我确实尝试了一些东西，但没有用。如果不行，请问是什么原因，正确的方法是什么。

提前致谢。

Answer 1

在你的 for 循环中：

  for(auto outermap : m_executormap) {
    for(auto innermap : outermap.second) {
      for(auto vec_element: innermap.second) {

您正在使用默认为非引用类型的 auto，因此您要复制 map/vector 中的每个元素。您的更改正在应用于这些临时副本，因此已丢失。

只需将这些更改为引用即可更新原始列表：

  for(auto& outermap : m_executormap) {
    for(auto& innermap : outermap.second) {
      for(auto& vec_element: innermap.second) {

Shared_ptr 在 std::Vector 中重新迭代时为空

Shared_ptr is null on re-iteration in std::Vector

c++

smart-pointers

shared-ptr

stdvector

c++11