c++ - 递归添加子树以 boost 属性树

我想用C++写一个参数服务器，在其中我可以将参数树递归地转储到属性树中，然后将其写入JSON文件。
转储函数如下所示:

 void Params::dump(string filename) {
   // Create a root
   pt::ptree root;

   // Fill the root with parameters
   mapToPt(curParams, root);

   // Write to cout
   pt::write_json(cout, root);
 }

mapToPt应该递归地遍历参数服务器的层次结构，并同时填充属性树:

  void Params::mapToPt(boost::shared_ptr<Params> curParams, pt::ptree &root) {
   // Fill current root with parameters from curParams ParameterMap
   map<string, boost::shared_ptr<Param> >::iterator it;
   for ( it = curParams->getParamMap().begin(); it != curParams-getParamMap().end(); it++ ) {
     root.put(it->first, it->second->getValue());
     cout << "Add Parameter: \n";
     cout << "Parameter name: " << it->first << "\n";
     cout << "Parameter value: " << it->second->getValue() << "\n";
   }

   // Recursively go through all children to do the same to them
   if(curParams->hasChildren()) { //ERROR LINE
     map<string, boost::shared_ptr<Params> >::iterator it;
     for ( it = curParams->getChildren().begin(); it != curParams-getChildren().end(); it++ ) {
       pt::ptree new_tree;
       root.add_child(it->second->getName(), new_tree);
       cout << "Add Child: \n";
       cout << "Child name: " << it->second->getName() << "\n";
       mapToPt(it->second, new_tree);
     }
   }
 }

我的问题是，一旦进入递归，错误就会在随机行上发生，而这不会成为错误的原因。错误消息为“basic_string::_ M_construct null无效”。我相信我可能会访问已删除的内容，这可能是由于我遍历属性树子级的方式所致。
我的做法是错误的还是其他方法？

谢谢。

最佳答案

当mapToPt还需要一个指向Params实例的指针时，为什么将其作为成员？

哎呀，有很多困惑。

在设计级别，您的Params类型似乎无法确定它是否是叶节点。此外，它还受“准类”设计的困扰，在这种设计中， setter/getter 本质上保证了不存在类不变性的可能性。在这种情况下，最好只使用带有成员字段的结构。



在实现级别，这会导致您遇到问题:

        pt::ptree new_tree;
        root.add_child(it->second->getName(), new_tree);

add_child插入new_tree的副本。将来对new_tree的任何修改均无效。相反，写:

        pt::ptree& new_tree = root.add_child(it->second->getName(), {});

在这里，new_tree成为对实际添加的树的引用。

尝试修复

风格仍然低于我的期望。我个人将仔细检查这段代码中对shared_ptr的使用。

但这可能会帮助您:

Live On Coliru

#include <boost/make_shared.hpp>
#include <boost/property_tree/json_parser.hpp>
#include <iostream>
#include <map>

namespace pt = boost::property_tree;

struct Param {
    std::string getValue() const { return "42"; }
};

struct Params {
    using ParamMap = std::map<std::string, boost::shared_ptr<Param> >;
    using Children = std::map<std::string, boost::shared_ptr<Params> >;

    Params(std::string name = "") : _name(name) {}

    std::string getName() const         { return _name; }

    ParamMap& getParamMap()             { return _map; }
    ParamMap const& getParamMap() const { return _map; }

    bool hasChildren() const            { return !_children.empty(); }
    Children& getChildren()             { return _children; }
    Children const& getChildren() const { return _children; }

    static void mapToPt(boost::shared_ptr<Params> curParams, pt::ptree &root);

  private:
    std::string _name;
    ParamMap _map;
    Children _children;
};

void Params::mapToPt(boost::shared_ptr<Params> curParams, pt::ptree &root) {
    // Fill current root with parameters from curParams ParameterMap
    std::map<std::string, boost::shared_ptr<Param> >::iterator it;
    for (it = curParams->getParamMap().begin(); it != curParams->getParamMap().end(); it++) {
        root.put(it->first, it->second->getValue());
        //std::cout << "Add Parameter: \n";
        //std::cout << "Parameter name: " << it->first << "\n";
        //std::cout << "Parameter value: " << it->second->getValue() << "\n";
    }

    // Recursively go through all children to do the same to them
    if (curParams->hasChildren()) {
        for (auto it = curParams->getChildren().begin(); it != curParams->getChildren().end(); it++) {
            pt::ptree& new_tree = root.add_child(it->second->getName(), {});
            //std::cout << "Add Child: \n";
            //std::cout << "Child name: " << it->second->getName() << "\n";
            mapToPt(it->second, new_tree);
        }
    }
}

int main() {
    auto a = boost::make_shared<Params>("rootparams");

    a->getParamMap().emplace("one", boost::make_shared<Param>());
    a->getParamMap().emplace("two", boost::make_shared<Param>());
    a->getParamMap().emplace("three", boost::make_shared<Param>());

    a->getChildren().emplace("child1", boost::make_shared<Params>("child1-name"))
        .first->second->getParamMap().emplace("four", boost::make_shared<Param>());

    a->getChildren().emplace("child2", boost::make_shared<Params>("child2-name"))
        .first->second->getParamMap().emplace("five", boost::make_shared<Param>());

    pt::ptree root;
    a->mapToPt(a, root);

    write_json(std::cout, root);
}

版画

{
    "one": "42",
    "three": "42",
    "two": "42",
    "child1-name": {
        "four": "42"
    },
    "child2-name": {
        "five": "42"
    }
}

关于c++ - 递归添加子树以 boost 属性树，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/50836911/