Salome HOME
#18963 Minimize compiler warnings
[modules/smesh.git] / src / SMESHUtils / SMESH_OctreeNode.cxx
index 821812340bb57920f4a870b80b1ede124a241022..ed65a74b885e5aa26136102993180fae498547c9 100644 (file)
@@ -1,4 +1,4 @@
-// Copyright (C) 2007-2012  CEA/DEN, EDF R&D, OPEN CASCADE
+// Copyright (C) 2007-2020  CEA/DEN, EDF R&D, OPEN CASCADE
 //
 // Copyright (C) 2003-2007  OPEN CASCADE, EADS/CCR, LIP6, CEA/DEN,
 // CEDRAT, EDF R&D, LEG, PRINCIPIA R&D, BUREAU VERITAS
@@ -6,7 +6,7 @@
 // This library is free software; you can redistribute it and/or
 // modify it under the terms of the GNU Lesser General Public
 // License as published by the Free Software Foundation; either
-// version 2.1 of the License.
+// version 2.1 of the License, or (at your option) any later version.
 //
 // This library is distributed in the hope that it will be useful,
 // but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ -30,6 +30,9 @@
 #include "SMESH_OctreeNode.hxx"
 
 #include "SMDS_SetIterator.hxx"
+#include "SMESH_MeshAlgos.hxx"
+#include "SMESH_TypeDefs.hxx"
+
 #include <gp_Pnt.hxx>
 
 using namespace std;
@@ -47,7 +50,7 @@ using namespace std;
 SMESH_OctreeNode::SMESH_OctreeNode (const TIDSortedNodeSet & theNodes, const int maxLevel,
                                     const int maxNbNodes , const double minBoxSize )
   :SMESH_Octree( new Limit( maxLevel,minBoxSize,maxNbNodes)),
-   myNodes(theNodes)
+   myNodes( theNodes.begin(), theNodes.end() )
 {
   compute();
 }
@@ -95,13 +98,10 @@ SMESH_Octree* SMESH_OctreeNode::newChild() const
 Bnd_B3d* SMESH_OctreeNode::buildRootBox()
 {
   Bnd_B3d* box = new Bnd_B3d;
-  TIDSortedNodeSet::iterator it = myNodes.begin();
-  for (; it != myNodes.end(); it++) {
-    const SMDS_MeshNode* n1 = *it;
-    gp_XYZ p1( n1->X(), n1->Y(), n1->Z() );
-    box->Add(p1);
-  }
-  if ( myNodes.size() <= getMaxNbNodes() )
+  for ( size_t i = 0; i < myNodes.size(); ++i )
+    box->Add( SMESH_NodeXYZ( myNodes[ i ]));
+
+  if ((int) myNodes.size() <= getMaxNbNodes() )
     myIsLeaf = true;
 
   return box;
@@ -116,12 +116,13 @@ Bnd_B3d* SMESH_OctreeNode::buildRootBox()
  */
 //====================================================================================
 
-const bool SMESH_OctreeNode::isInside (const gp_XYZ& p, const double precision)
+bool SMESH_OctreeNode::isInside ( const gp_XYZ& p, const double precision )
 {
-  if (precision <= 0.)
-    return !(getBox()->IsOut(p));
+  if ( precision <= 0.)
+    return !( getBox()->IsOut(p) );
+
   Bnd_B3d BoxWithPrecision = *getBox();
-  BoxWithPrecision.Enlarge(precision);
+  BoxWithPrecision.Enlarge( precision );
   return ! BoxWithPrecision.IsOut(p);
 }
 
@@ -131,27 +132,37 @@ const bool SMESH_OctreeNode::isInside (const gp_XYZ& p, const double precision)
  * Shares the father's data with each of his child
  */
 //================================================
+
 void SMESH_OctreeNode::buildChildrenData()
 {
   gp_XYZ min = getBox()->CornerMin();
   gp_XYZ max = getBox()->CornerMax();
   gp_XYZ mid = (min + max)/2.;
 
-  TIDSortedNodeSet::iterator it = myNodes.begin();
-  while (it != myNodes.end())
+  for ( int i = 0; i < 8; i++ )
+  {
+    SMESH_OctreeNode* myChild = static_cast<SMESH_OctreeNode*>( myChildren[ i ]);
+    myChild->myNodes.reserve( myNodes.size() / 8 );
+  }
+
+  for ( size_t i = 0; i < myNodes.size(); ++i )
   {
-    const SMDS_MeshNode* n1 = *it;
-    int ChildBoxNum = getChildIndex( n1->X(), n1->Y(), n1->Z(), mid );
-    SMESH_OctreeNode* myChild = dynamic_cast<SMESH_OctreeNode*> (myChildren[ChildBoxNum]);
-    myChild->myNodes.insert(myChild->myNodes.end(),n1);
-    myNodes.erase( it );
-    it = myNodes.begin();
+    SMESH_NodeXYZ n = myNodes[ i ];
+    int ChildBoxNum = getChildIndex( n.X(), n.Y(), n.Z(), mid );
+    SMESH_OctreeNode* myChild = static_cast<SMESH_OctreeNode*>( myChildren[ ChildBoxNum ]);
+    myChild->myNodes.push_back( myNodes[ i ]);
   }
-  for (int i = 0; i < 8; i++)
+  SMESHUtils::FreeVector( myNodes );
+
+  for ( int i = 0; i < 8; i++ )
   {
-    SMESH_OctreeNode* myChild = dynamic_cast<SMESH_OctreeNode*> (myChildren[i]);
-    if ( myChild->myNodes.size() <= getMaxNbNodes() )
+    SMESH_OctreeNode* myChild = static_cast<SMESH_OctreeNode*>( myChildren[ i ]);
+    if ((int) myChild->myNodes.size() <= getMaxNbNodes() )
+    {
       myChild->myIsLeaf = true;
+      if ( myChild->myNodes.empty() )
+        SMESHUtils::FreeVector( myChild->myNodes );
+    }
   }
 }
 
@@ -163,23 +174,24 @@ void SMESH_OctreeNode::buildChildrenData()
  * \param Result - list of Nodes potentials to be near Node
  */
 //====================================================================
-void SMESH_OctreeNode::NodesAround (const SMDS_MeshNode * Node,
-                                    list<const SMDS_MeshNode*>* Result,
-                                    const double precision)
+
+void SMESH_OctreeNode::AllNodesAround (const SMDS_MeshNode *              Node,
+                                       std::vector<const SMDS_MeshNode*>* Result,
+                                       const double                       precision)
 {
-  gp_XYZ p(Node->X(), Node->Y(), Node->Z());
-  if (isInside(p, precision))
+  SMESH_NodeXYZ p = Node;
+  if ( isInside( p, precision ))
   {
-    if (isLeaf())
+    if ( isLeaf() )
     {
-      Result->insert(Result->end(), myNodes.begin(), myNodes.end());
+      Result->insert( Result->end(), myNodes.begin(), myNodes.end() );
     }
     else
     {
-      for (int i = 0; i < 8; i++)
+      for ( int i = 0; i < 8; i++ )
       {
-        SMESH_OctreeNode* myChild = dynamic_cast<SMESH_OctreeNode*> (myChildren[i]);
-        myChild->NodesAround(Node, Result, precision);
+        SMESH_OctreeNode* myChild = static_cast<SMESH_OctreeNode*> (myChildren[i]);
+        myChild->AllNodesAround( Node, Result, precision );
       }
     }
   }
@@ -188,14 +200,15 @@ void SMESH_OctreeNode::NodesAround (const SMDS_MeshNode * Node,
 //================================================================================
 /*!
  * \brief Return in dist2Nodes nodes mapped to their square distance from Node
+ *        Tries to find a closest node.
  *  \param node - node to find nodes closest to
  *  \param dist2Nodes - map of found nodes and their distances
  *  \param precision - radius of a sphere to check nodes inside
- *  \retval bool - true if an exact overlapping found
+ *  \retval bool - true if an exact overlapping found !!!
  */
 //================================================================================
 
-bool SMESH_OctreeNode::NodesAround(const gp_XYZ &node,
+bool SMESH_OctreeNode::NodesAround(const gp_XYZ &                     node,
                                    map<double, const SMDS_MeshNode*>& dist2Nodes,
                                    double                             precision)
 {
@@ -204,17 +217,16 @@ bool SMESH_OctreeNode::NodesAround(const gp_XYZ &node,
   else if ( precision == 0. )
     precision = maxSize() / 2;
 
-  //gp_XYZ p(node->X(), node->Y(), node->Z());
-  if (isInside(node, precision))
+  if ( isInside( node, precision ))
   {
-    if (!isLeaf())
+    if ( !isLeaf() )
     {
       // first check a child containing node
       gp_XYZ mid = (getBox()->CornerMin() + getBox()->CornerMax()) / 2.;
       int nodeChild  = getChildIndex( node.X(), node.Y(), node.Z(), mid );
       if ( ((SMESH_OctreeNode*) myChildren[nodeChild])->NodesAround(node, dist2Nodes, precision))
         return true;
-      
+
       for (int i = 0; i < 8; i++)
         if ( i != nodeChild )
           if (((SMESH_OctreeNode*) myChildren[i])->NodesAround(node, dist2Nodes, precision))
@@ -223,24 +235,60 @@ bool SMESH_OctreeNode::NodesAround(const gp_XYZ &node,
     else if ( NbNodes() > 0 )
     {
       double minDist = precision * precision;
-      gp_Pnt p1 ( node.X(), node.Y(), node.Z() );
-      TIDSortedNodeSet::iterator nIt = myNodes.begin();
-      for ( ; nIt != myNodes.end(); ++nIt )
+      for ( size_t i = 0; i < myNodes.size(); ++i )
       {
-        gp_Pnt p2 ( (*nIt)->X(), (*nIt)->Y(), (*nIt)->Z() );
-        double dist2 = p1.SquareDistance( p2 );
+        SMESH_NodeXYZ p2 = myNodes[ i ];
+        double     dist2 = ( node - p2 ).SquareModulus();
         if ( dist2 < minDist )
-          dist2Nodes.insert( make_pair( minDist = dist2, *nIt ));
+          dist2Nodes.insert( std::make_pair( minDist = dist2, myNodes[ i ] ));
       }
-//       if ( dist2Nodes.size() > 1 ) // leave only closest node in dist2Nodes
-//         dist2Nodes.erase( ++dist2Nodes.begin(), dist2Nodes.end());
+      // if ( dist2Nodes.size() > 1 ) // leave only closest node in dist2Nodes
+      //   dist2Nodes.erase( ++dist2Nodes.begin(), dist2Nodes.end());
 
-      return ( sqrt( minDist) <= precision * 1e-12 );
+      // true if an exact overlapping found
+      return ( sqrt( minDist ) <= precision * 1e-12 );
     }
   }
   return false;
 }
 
+//================================================================================
+/*!
+ * \brief Return a list of nodes close to a point
+ *  \param [in] point - point
+ *  \param [out] nodes - found nodes
+ *  \param [in] precision - allowed distance from \a point
+ */
+//================================================================================
+
+void SMESH_OctreeNode::NodesAround(const gp_XYZ&                      point,
+                                   std::vector<const SMDS_MeshNode*>& nodes,
+                                   double                             precision)
+{
+  if ( isInside( point, precision ))
+  {
+    if ( isLeaf() && NbNodes() )
+    {
+      double minDist2 = precision * precision;
+      for ( size_t i = 0; i < myNodes.size(); ++i )
+      {
+        SMESH_NodeXYZ p2 = myNodes[ i ];
+        double dist2 = ( point - p2 ).SquareModulus();
+        if ( dist2 <= minDist2 )
+          nodes.push_back( myNodes[ i ] );
+      }
+    }
+    else if ( myChildren )
+    {
+      for (int i = 0; i < 8; i++)
+      {
+        SMESH_OctreeNode* myChild = static_cast<SMESH_OctreeNode*>( myChildren[ i ]);
+        myChild->NodesAround( point, nodes, precision );
+      }
+    }
+  }
+}
+
 //=============================
 /*!
  * \brief  Return in theGroupsOfNodes a list of group of nodes close to each other within theTolerance
@@ -253,15 +301,19 @@ bool SMESH_OctreeNode::NodesAround(const gp_XYZ &node,
  * \param maxNbNodes - maximum Nodes in a Leaf of the SMESH_OctreeNode constructed, default value is 5
  */
 //=============================
+
 void SMESH_OctreeNode::FindCoincidentNodes (TIDSortedNodeSet& theSetOfNodes,
-                                            list< list< const SMDS_MeshNode*> >* theGroupsOfNodes,
-                                            const double theTolerance,
-                                            const int maxLevel,
-                                            const int maxNbNodes)
+                                            TListOfNodeLists* theGroupsOfNodes,
+                                            const double      theTolerance,
+                                            const int         maxLevel,
+                                            const int         maxNbNodes)
 {
-  // VSR 14/10/2011: limit max number of the levels in order to avoid endless recursing
+  // VSR 14/10/2011: limit max number of the levels in order to avoid endless recursion
   const int MAX_LEVEL = 10;
-  SMESH_OctreeNode theOctreeNode(theSetOfNodes, maxLevel < 0 ? MAX_LEVEL : maxLevel, maxNbNodes, theTolerance);
+  SMESH_OctreeNode theOctreeNode(theSetOfNodes,
+                                 maxLevel < 0 ? MAX_LEVEL : maxLevel,
+                                 maxNbNodes,
+                                 theTolerance);
   theOctreeNode.FindCoincidentNodes (&theSetOfNodes, theTolerance, theGroupsOfNodes);
 }
 
@@ -275,45 +327,40 @@ void SMESH_OctreeNode::FindCoincidentNodes (TIDSortedNodeSet& theSetOfNodes,
  * \param theGroupsOfNodes - list of nodes closed to each other returned
  */
 //=============================
+
 void SMESH_OctreeNode::FindCoincidentNodes ( TIDSortedNodeSet* theSetOfNodes,
-                                             const double               theTolerance,
-                                             list< list< const SMDS_MeshNode*> >* theGroupsOfNodes)
+                                             const double      theTolerance,
+                                             TListOfNodeLists* theGroupsOfNodes )
 {
-  TIDSortedNodeSet::iterator it1 = theSetOfNodes->begin();
-  list<const SMDS_MeshNode*>::iterator it2;
+  // un-mark all nodes; we mark nodes added to theGroupsOfNodes
+  SMESH_MeshAlgos::MarkElems( SMESHUtils::elemSetIterator( *theSetOfNodes ), false );
 
-  while (it1 != theSetOfNodes->end())
+  vector<const SMDS_MeshNode*> coincidentNodes;
+  TIDCompare idLess;
+
+  TIDSortedNodeSet::iterator it1 = theSetOfNodes->begin();
+  for ( ; it1 != theSetOfNodes->end(); ++it1 )
   {
     const SMDS_MeshNode * n1 = *it1;
+    if ( n1->isMarked() )
+      continue;
+    n1->setIsMarked( true );
 
-    list<const SMDS_MeshNode*> ListOfCoincidentNodes;// Initialize the lists via a declaration, it's enough
-
-    list<const SMDS_MeshNode*> * groupPtr = 0;
-
-    // Searching for Nodes around n1 and put them in ListofCoincidentNodes.
+    // Searching for Nodes around n1 and put them in coincidentNodes.
     // Found nodes are also erased from theSetOfNodes
-    FindCoincidentNodes(n1, theSetOfNodes, &ListOfCoincidentNodes, theTolerance);
+    coincidentNodes.clear();
+    findCoincidentNodes( n1, theSetOfNodes, &coincidentNodes, theTolerance );
 
-    // We build a list {n1 + his neigbours} and add this list in theGroupsOfNodes
-    for (it2 = ListOfCoincidentNodes.begin(); it2 != ListOfCoincidentNodes.end(); it2++)
+    if ( !coincidentNodes.empty() )
     {
-      const SMDS_MeshNode* n2 = *it2;
-      if ( !groupPtr )
-      {
-        theGroupsOfNodes->push_back( list<const SMDS_MeshNode*>() );
-        groupPtr = & theGroupsOfNodes->back();
-        groupPtr->push_back( n1 );
-      }
-      if (groupPtr->front() > n2)
-        groupPtr->push_front( n2 );
-      else
-        groupPtr->push_back( n2 );
-    }
-    if (groupPtr != 0)
-      groupPtr->sort();
+      // We build a list {n1 + his neighbors} and add this list in theGroupsOfNodes
+      std::sort( coincidentNodes.begin(), coincidentNodes.end(), idLess );
+      list<const SMDS_MeshNode*> newGroup;
+      newGroup.push_back( n1 );
+      newGroup.insert( newGroup.end(), coincidentNodes.begin(), coincidentNodes.end() );
 
-    theSetOfNodes->erase(it1);
-    it1 = theSetOfNodes->begin();
+      theGroupsOfNodes->emplace_back( newGroup );
+    }
   }
 }
 
@@ -327,60 +374,45 @@ void SMESH_OctreeNode::FindCoincidentNodes ( TIDSortedNodeSet* theSetOfNodes,
  * \param precision - Precision used
  */
 //======================================================================================
-void SMESH_OctreeNode::FindCoincidentNodes (const SMDS_MeshNode * Node,
-                                            TIDSortedNodeSet* SetOfNodes,
-                                            list<const SMDS_MeshNode*>* Result,
-                                            const double precision)
+
+void SMESH_OctreeNode::findCoincidentNodes (const SMDS_MeshNode *              Node,
+                                            TIDSortedNodeSet*                  SetOfNodes,
+                                            std::vector<const SMDS_MeshNode*>* Result,
+                                            const double                       precision)
 {
-  gp_XYZ p(Node->X(), Node->Y(), Node->Z());
-  bool isInsideBool = isInside(p, precision);
+  SMESH_NodeXYZ p1 = Node;
 
-  if (isInsideBool)
+  if ( isInside( p1, precision ))
   {
     // I'm only looking in the leaves, since all the nodes are stored there.
-    if (isLeaf())
+    if ( isLeaf() )
     {
-      gp_Pnt p1 (Node->X(), Node->Y(), Node->Z());
+      const double tol2 = precision * precision;
 
-      TIDSortedNodeSet myNodesCopy = myNodes;
-      TIDSortedNodeSet::iterator it = myNodesCopy.begin();
-      double tol2 = precision * precision;
-      bool squareBool;
-
-      while (it != myNodesCopy.end())
+      for ( size_t i = 0; i < myNodes.size(); ++i )
       {
-        const SMDS_MeshNode* n2 = *it;
-        // We're only looking at nodes with a superior Id.
-        // JFA: Why?
-        //if (Node->GetID() < n2->GetID())
-        if (Node->GetID() != n2->GetID()) // JFA: for bug 0020185
-        {
-          gp_Pnt p2 (n2->X(), n2->Y(), n2->Z());
-          // Distance optimized computation
-          squareBool = (p1.SquareDistance( p2 ) <= tol2);
+        if ( myNodes[ i ]->isMarked() ) // coincident node already found
+          continue;
 
-          // If n2 inside the SquareDistance, we add it in Result and remove it from SetOfNodes and myNodes
-          if (squareBool)
+        //if ( Node != myNodes[ i ]) // JFA: for bug 0020185
+        {
+          // If n2 inside the SquareDistance, we add it in Result
+          bool coincide = ( p1.SquareDistance( myNodes[ i ]) <= tol2 );
+          if ( coincide )
           {
-            Result->insert(Result->begin(), n2);
-            SetOfNodes->erase( n2 );
-            myNodes.erase( n2 );
+            Result->push_back ( myNodes[ i ]);
+            myNodes[ i ]->setIsMarked( true );
           }
         }
-        //myNodesCopy.erase( it );
-        //it = myNodesCopy.begin();
-        it++;
       }
-      if (Result->size() > 0)
-        myNodes.erase(Node); // JFA: for bug 0020185
     }
     else
     {
       // If I'm not a leaf, I'm going to see my children !
-      for (int i = 0; i < 8; i++)
+      for ( int i = 0; i < 8; i++ )
       {
-        SMESH_OctreeNode* myChild = dynamic_cast<SMESH_OctreeNode*> (myChildren[i]);
-        myChild->FindCoincidentNodes(Node, SetOfNodes, Result, precision);
+        SMESH_OctreeNode* myChild = static_cast<SMESH_OctreeNode*> (myChildren[i]);
+        myChild->findCoincidentNodes( Node, SetOfNodes, Result, precision );
       }
     }
   }
@@ -396,17 +428,18 @@ void SMESH_OctreeNode::UpdateByMoveNode( const SMDS_MeshNode* node, const gp_Pnt
 {
   if ( isLeaf() )
   {
-    TIDSortedNodeSet::iterator pNode = myNodes.find( node );
-    bool nodeInMe = ( pNode != myNodes.end() );
+    std::vector< const SMDS_MeshNode* >::iterator pNode =
+      std::find( myNodes.begin(), myNodes.end(), node );
 
+    bool  nodeInMe = ( pNode != myNodes.end() );
     bool pointInMe = isInside( toPnt.Coord(), 1e-10 );
 
     if ( pointInMe != nodeInMe )
     {
       if ( pointInMe )
-        myNodes.insert( node );
+        myNodes.push_back( node );
       else
-        myNodes.erase( node );
+        myNodes.erase( pNode );
     }
   }
   else if ( myChildren )
@@ -427,6 +460,7 @@ void SMESH_OctreeNode::UpdateByMoveNode( const SMDS_MeshNode* node, const gp_Pnt
  * \brief Return iterator over children
  */
 //================================================================================
+
 SMESH_OctreeNodeIteratorPtr SMESH_OctreeNode::GetChildrenIterator()
 {
   return SMESH_OctreeNodeIteratorPtr
@@ -439,9 +473,8 @@ SMESH_OctreeNodeIteratorPtr SMESH_OctreeNode::GetChildrenIterator()
  * \brief Return nodes iterator
  */
 //================================================================================
+
 SMDS_NodeIteratorPtr SMESH_OctreeNode::GetNodeIterator()
 {
-  return SMDS_NodeIteratorPtr
-    ( new SMDS_SetIterator< SMDS_pNode, TIDSortedNodeSet::const_iterator >
-      ( myNodes.begin(), myNodes.size() ? myNodes.end() : myNodes.begin()));
+  return boost::make_shared< SMDS_NodeVectorIterator >( myNodes.begin(), myNodes.end());
 }