Question

我有一个班级node并宣布为std::vector<node*>。我想使用BOOST_FOREACH来迭代向量，但我遇到了一些问题。

我的代码是这样的：

data_flow_graph* dfg;
node* other_node;

[...]

BOOST_FOREACH(node* n, vector_of_nodes){
  if (n->get_id() < 100)){
    n = new node(last_id++);
    n->add_related_node(other_node);
    dfg->add_node(n);
  }
}

这是，我尝试在迭代时修改指针的地址。问题是，虽然创建了新节点，但vector_of_nodes中的指针保持不变。我想这是Boost如何管理对指针的访问的问题，因为如果我使用常规迭代器进行等效处理：

std::vector<node*>::iterator it;
std::vector<node*>::iterator et = vector_of_nodes.end();

for (it = vector_of_nodes.begin(); it != et; ++it){
  if ((*it)->get_id() < 100)){
    (*it) = new node(last_id++);
    (*it)->add_related_node(other_node);
    dfg->add_node(*it);
  }
}

代码运行良好，按预期运行。

我可以使用该版本，但我很好奇......为什么第一个版本不起作用？

Answer 1

将代码更改为

BOOST_FOREACH(node*& n, vector_of_nodes){
  if (n->get_id() < 100)){
    n = new node(last_id++);
    n->add_related_node(other_node);
    dfg->add_node(n);
  }
}

因为在您的版本中您更改了本地指针的地址。并且，在使用n = new node之前，为什么不删除旧的？由于您在手动管理资源时遇到问题，为什么不使用smart pointers？

使用BOOST_FOREACH修改std :: vector中的指针

1 个答案: