Question

我一直在做一个状态，以替换我一直在使用的更糟糕的代码。我希望尽可能的现代化，所以我在适当的地方使用STL。我决定使用unique_ptr和shared_ptr，这是我以前从未做过的。结果，我遇到了一个非常可怕的运行时错误：munmap_chunk(): invalid pointer: 0x00007fff7b3f08e0我所知道的是错误与free()和malloc()有关，但我没有使用任何错误在我的代码中，我认为它来自STL内部。为了让事情变得更奇怪，如果我在main的开头加std::cout << "Hello World!\n";，我只得到一个Segmentation fault我的代码相对较小：

state.hpp：

#pragma once
#include "state_manager.hpp"

class state{
 using manager_ptr = std::shared_ptr<state_manager>;
protected:
 manager_ptr man;
public:
 const manager_ptr manager() const;
 void manager(manager_ptr&& parent_manager);
};

state.cpp：

#include "state.hpp"

const state::manager_ptr state::manager() const{
 return man;
}

void state::manager(manager_ptr&& parent_manager){
 man = parent_manager;
}

state_manager.hpp：

#pragma once
#include <stack>
#include <memory>

class state;

class state_manager{
 using element = std::unique_ptr<state>;
 using container =  std::stack<element>;
protected:
 container states;
public:
 void push(element&& to_push);
 void pop();
 void change(element&& change_to);
};

state_manager.cpp：

#include "state_manager.hpp"
#include "state.hpp"

void state_manager::push(element&& to_push){
 states.push(std::forward<element>(to_push));
 states.top()->manager(std::shared_ptr<state_manager>(this));
}

void state_manager::pop(){
 states.pop();
}

void state_manager::change(element&& change_to){
 states.pop();
 push(std::forward<element>(change_to));
}

main.cpp中：

#include "state_manager.hpp"
#include "state.hpp"
#include <iostream>

int main(int argc, const char* argv[]){
 state_manager test;
 test.push(std::make_unique<state>());
 test.change(std::make_unique<state>());
 test.pop();
}

Answer 1

在这行代码中：

states.top()->manager(std::shared_ptr<state_manager>(this));

您构建的shared_ptr指向this。这意味着当shared_ptr的引用计数降至零时，它将delete this。这很糟糕，因为班级不是this的所有者。在您的情况下，this指针甚至没有被new分配，因此shared_ptr将无法删除它。

您可以在man类中仅使用state_manager的普通C ++指针，因为它当前不需要指针的所有权。

或者，您需要一些其他方式来获得指向shared_ptr的{{1}}或weak_ptr，例如将其传递给this（这很奇怪）。

智能指针 - 程序终止时的分段错误

1 个答案: