0

我有一个班级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);
  }
}

也就是说，我在迭代指针时尝试修改指针的地址。问题是，虽然创建了新节点，但 in 的指针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);
  }
}

该代码运行良好，并按预期工作。

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

4

1 回答 1

2

将代码更改为

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?

于 2013-04-12T09:17:16.400 回答