程序运行时removechild报错

array

没有，建议这类操作都使用回调来完成；至少在我这里不会因此受到什么限制

口口广大

这个问题我这几天也碰到，而且折腾了我好几天，同样在MFC下，自己建立个一个简单的类osgObject
//构造函数,一个group节点下，挂一个PositionAttitudeTransform节点
osgObject:sgObject(void)
{ osg::ref_ptr <osg::group> group = new osg::Group;
osg::ref_ptr <osg:ositionAttitudeTransfrom >AT= new osg::PositionAttitudeTransform;
group ->addChild(PAT.get()); }

//析构函数,group节点移除PositionAttitudeTransform节点 osgObject::~osgObject(void)
{ group ->removeChild(PAT.get()); }

在vs2005 debug模式下，osg渲染采用独立的线程，在frame()前不断的创建删除osgObject对象，没有问题。

osgObject *ob = new osgObject ();
delete ob;

但是当MFC别的线程不断地删除osgObject对象，而我的frame()不断创建新的osgObject对象时，就会发生如楼主所述的错误，引起一个断言，表示的意思是某个STL的迭代器指针错误。

我想可能是由10楼的朋友所述，线程之间产生了冲突，应该在添加删除节点线程和frame线程使用线程锁。

array

为啥我不会遇到……我们这里是我负责编写三维代码，然后再结合到MFC界面下，一直没出过问题……也从来没用过互斥锁

lvzhihan

33# array

新书啥时候出？把希望寄托于此了

array

大约12月份可能性比较大。此外书里也不会讲解这部分内容。

lvzhihan

35# array


能给点这问题相关的你用的代码吗？学习下

array

抱歉我没有用过什么特别的代码，就是按部就班地setDataVariance，然后适时地在回调里addChild/removeChild而已。也很难抽出一个单独的例子~~

qele

晒一个我写的一个CallBack添加删除的回调

1.       
   
2. class CAddRemoveCallBack : public osg::NodeCallback
   
3. {
   
4. public:
   
5. CAddRemoveCallBack(OpenThreads::Mutex* mutex);
   
6. virtual ~CAddRemoveCallBack(void);
   
7. virtual void operator()(osg::Node* node, osg::NodeVisitor* nv);
   
8. public:
   
9. void AddChild( osg::Node* node );
   
10. void RemoveChild( osg::Node* node );
    
11. protected:
    
12. OpenThreads::Mutex* _mutex;
    
13. volatile BOOL _paused;
    
14. protected:
    
15. std::vector<osg::Node*> _vtAdding;
    
16. std::vector<osg::Node*> _vtRemoving;
    
17. };

复制代码

qele

1.       
   
2. /////////////////////////////////////////////////////////////////////////////
   
3. // class CAddRemoveCallBack implement
   
4. 
   
5. CAddRemoveCallBack::CAddRemoveCallBack(OpenThreads::Mutex* mutex) : _mutex(mutex), _paused(FALSE)
   
6. {}
   
7. 
   
8. CAddRemoveCallBack::~CAddRemoveCallBack(void)
   
9. {}
   
10. 
    
11. void CAddRemoveCallBack:perator()(osg::Node* node, osg::NodeVisitor* nv)
    
12. {
    
13.    if ( _paused )
    
14.    {
    
15.       OpenThreads::ScopedLock<OpenThreads::Mutex> lock( *_mutex);
    
16.       osg::Group* root = dynamic_cast<osg::Group*>(node);
    
17.       if ( root != NULL )
    
18.       {
    
19.          //        remove first
    
20.          for ( size_t i = 0; i < _vtRemoving.size(); ++i )
    
21.          {
    
22.                root->removeChild( _vtRemoving[i] );
    
23.          }
    
24.          _vtRemoving.clear();
    
25. 
    
26.          //        add node
    
27.          for ( size_t i = 0; i < _vtAdding.size(); ++i )
    
28.          {
    
29.                root->addChild( _vtAdding[i] );
    
30.          }
    
31.         vtAdding.clear();
    
32.       }
    
33.          _paused = FALSE;
    
34.      }
    
35. }
    
36. 
    
37. void CAddRemoveCallBack::AddChild( osg::Node* node )
    
38. {
    
39.        OpenThreads::ScopedLock<OpenThreads::Mutex> lock( *_mutex);
    
40.        _vtAdding.push_back( node );
    
41.        _paused = TRUE;
    
42. }
    
43. 
    
44. void CAddRemoveCallBack::RemoveChild( osg::Node* node )
    
45. {
    
46.         OpenThreads::ScopedLock<OpenThreads::Mutex> lock( *_mutex);
    
47.         _vtRemoving.push_back( node );
    
48.        _paused = TRUE;
    
49. }

复制代码

lvzhihan

root->removeChild( _vtRemoving );
               root->addChild( _vtAdding );

这两行有问题，“osg::Group::addChild”: 不能将参数 1 从“std::vector<_Ty>”转换为“osg::Node *”

lvzhihan

39# qele
给个用法实例

qele

40# lvzhihan
编辑错误，已修改

qele

41# lvzhihan

osg::Group* root = new osg::Group;

osg::ref_ptr<CAddRemoveCallBack> addRemoveManage = new CAddRemoveCallBack(&s_Mutex);

root->addUpdateCallback( addRemoveManage.get() );

...

addRemoveManage->AddChild( node );

lvzhihan

&s_Mutex
这个锁为什么要作为参数？都怎么用这个？

qele

operator()是在仿真循环里运行。而add函数和仿真循环不在一个线程，加一个锁为了安全。
可以在frame所在的cpp中写一个全局OpenThreads::Mutex，初始化CAddRemoveCallBack将其加入。另外在仿真循环frame（）前也加一个lock（哈哈，是不是有损效率啊？）

array

损失了不少效率……或者说，多线程的渲染恐怕在您这里失效了

lvzhihan

46# array
如果线程模型本来就设置为single是不是不会影响效率了

lvzhihan

OpenThreads::Mutex* _mutex;
放到最外层循环了，可是运行的时候总会报错，_mutex里面全是问号。

array

47# lvzhihan

那本身就不是多线程渲染啊~~

lvzhihan

49# array
我记得你在mfc+osg解决方案中提到：为了不跟mfc的线程冲突，把osg的渲染线程模型改为单线程。

lvzhihan

45# qele
写一个全局的OpenThreads::Mutex，
就是OpenThreads::Mutex* _mutex;这样可以吗？好像osg example里面都这样用的，但是我这样写在程序里面_mutex总是乱的。

还有，好像CAddRemoveCallBack:perator()里面的代码一直没运行到。

qele

不是全局的指针。而是全局对象OpenThreads::Mutex  s_mutex;
思路是，把回调添加到场景节点，然后用此回调对象添加删除其对应节点的子节点。43楼有示例。
如果有更好的方法，欢迎大家指出~

lvzhihan

46# array
array把你的代码贴出来学习下吧

lvzhihan

52# qele

我的场景图是这样子的：rootnode->storagetree->effectgroup->graphicgroup->selection->openmodel

将rootnode装入场景：

m_Viewer->setSceneData(rootnode);
m_Viewer ->realize() ;


graphicsGroup里就是场景中所有的数据，给graphicsGroup加上回调：
graphicsGroup->addUpdateCallback(AddRemoveCallBack.get() );

现在要清空场景更换数据的时候，需要把graphicsGroup里的孩子都删除。
for(size_t i = 0; i < graphicsGroup->getNumChildren(); ++i)
{
  AddRemoveCallBack->RemoveChild(graphicsGroup->getChild(i));
}


这样子下来没有执行到CAddRemoveCallBack:perator()

我也尝试过直接给rootnode加上回调：
rootnode->addUpdateCallback(AddRemoveCallBack.get() );
但是这样子好像rootnode用RemoveChild删除不掉graphicsGroup，也就是RemoveChild不能删除孙子。

这种情况遇到过吗？

qele

节点更新回调不遍历？除非节点setNodeMask（0）停止遍历了；我这儿回调正常

lvzhihan

你给的例子是把场景图的根节点放到setSceneData中，并且从根节点删除孩子：
osg::Group* root = new osg::Group;
osg::ref_ptr<CAddRemoveCallBack> addRemoveManage = new CAddRemoveCallBack(&s_Mutex);
root->addUpdateCallback( addRemoveManage.get() );
m_Viewer->setSceneData(root);
m_Viewer ->realize() ;

...

addRemoveManage->AddChild( node );


现在场景图的根节点setSceneData，而对子节点添加回调，回调是否可以执行？

qele

每个节点只要添加了更新回调，应该都在updateTraversal被遍历执行。Drawable类回调有所不同，必须实现UpdateCallback::update()函数。
你查一下为什么节点回调未被运行啊

sunjs

小弟也遇到删除错误问题！受教了……

sunjs

56# lvzhihan
你的问题解决没有啊？我按照qele的方法，只有第一次能执行到回调里面以外，接下来再循环就是执行不到……很纳闷……