]> SALOME platform Git repositories - modules/shaper.git/commitdiff
Salome HOME
Clean code
authorJérôme <jerome.lucas@cesgenslab.fr>
Sat, 21 Nov 2020 17:05:59 +0000 (18:05 +0100)
committerJérôme <jerome.lucas@cesgenslab.fr>
Thu, 3 Dec 2020 15:28:24 +0000 (16:28 +0100)
src/BuildAPI/BuildAPI_Interpolation.cpp
src/BuildPlugin/BuildPlugin_Interpolation.cpp
src/BuildPlugin/BuildPlugin_Interpolation.h
src/BuildPlugin/BuildPlugin_Plugin.cpp
src/BuildPlugin/BuildPlugin_Validators.cpp
src/BuildPlugin/Test/TestInterpolation.py
src/BuildPlugin/interpolation_widget.xml
src/InitializationPlugin/InitializationPlugin_EvalListener.cpp
src/ModelAPI/ModelAPI_Events.h

index 353eff358a0ae2d0067eadd94091ce7aaebd405f..f651255be0f2edb27bb7d9d26f5dcd7e6adc7dc4 100644 (file)
@@ -31,14 +31,14 @@ BuildAPI_Interpolation::BuildAPI_Interpolation(const std::shared_ptr<ModelAPI_Fe
 
 //==================================================================================================
 BuildAPI_Interpolation::BuildAPI_Interpolation(const FeaturePtr& theFeature,
-                    const std::list<ModelHighAPI_Selection>& theBaseObjects,
-                    const ModelHighAPI_Selection& theStartTangent,
-                    const ModelHighAPI_Selection& theEndTangent,
-                    const bool theIsClosed,
-                    const bool theIsToReorder)
+                        const std::list<ModelHighAPI_Selection>& theBaseObjects,
+                        const ModelHighAPI_Selection& theStartTangent,
+                        const ModelHighAPI_Selection& theEndTangent,
+                        const bool theIsClosed,
+                        const bool theIsToReorder)
 : ModelHighAPI_Interface(theFeature)
 {
-  if(initialize()) {
+  if (initialize()) {
     fillAttribute(BuildPlugin_Interpolation::CREATION_METHOD_BY_SELECTION_ID(),mycreationmethod);
     setUseTangents(true);
     setTangents(theStartTangent, theEndTangent);
@@ -49,11 +49,10 @@ BuildAPI_Interpolation::BuildAPI_Interpolation(const FeaturePtr& theFeature,
 }
 
 //==================================================================================================
-BuildAPI_Interpolation::BuildAPI_Interpolation(
-                                  const FeaturePtr& theFeature,
-                                  const std::list<ModelHighAPI_Selection>& theBaseObjects,
-                                  const bool theIsClosed,
-                                  const bool theIsToReorder)
+BuildAPI_Interpolation::BuildAPI_Interpolation(const FeaturePtr& theFeature,
+                        const std::list<ModelHighAPI_Selection>& theBaseObjects,
+                        const bool theIsClosed,
+                        const bool theIsToReorder)
   : ModelHighAPI_Interface(theFeature)
 {
   if (initialize()) {
@@ -137,7 +136,7 @@ void BuildAPI_Interpolation::dump(ModelHighAPI_Dumper& theDumper) const
   FeaturePtr aBase = feature();
   std::string aPartName = theDumper.name(aBase->document());
 
-  ifaBase->string(BuildPlugin_Interpolation::CREATION_METHOD_ID())->value() ==
+  if (aBase->string(BuildPlugin_Interpolation::CREATION_METHOD_ID())->value() ==
             BuildPlugin_Interpolation::CREATION_METHOD_BY_SELECTION_ID() )
   {
     AttributeSelectionListPtr anAttrBaseObjects =
@@ -158,7 +157,7 @@ void BuildAPI_Interpolation::dump(ModelHighAPI_Dumper& theDumper) const
     }
 
     theDumper << closed() << ", " << reorder() << ")" << std::endl;
-  }else{
+  } else {
 
     theDumper << aBase << " = model.addInterpolation(" << aPartName ;
     AttributeStringPtr XtAttr = xt();
index 59e91ca81b18e1f404e365f2f8c1116172144760..2397c83319efb435426c9843f9fb8729dc4480af 100644 (file)
@@ -31,7 +31,6 @@
 
 #include <ModelAPI_AttributeSelectionList.h>
 #include <ModelAPI_ResultBody.h>
-#include <ModelAPI_ResultConstruction.h>
 #include <ModelAPI_Events.h>
 #include <ModelAPI_ResultParameter.h>
 
 #include <GeomAPI_Edge.h>
 #include <GeomAPI_Lin.h>
 #include <GeomAPI_ShapeExplorer.h>
-#include <ModelAPI_Tools.h>
-
-#include <ModelAPI_Expression.h>
-
 #include <algorithm>
-
-#include <iostream>
 #include <sstream>
 
 
@@ -98,7 +91,7 @@ void BuildPlugin_Interpolation::initAttributes()
   data()->reflist(ARGUMENTS_ID())->setIsArgument(false);
   ModelAPI_Session::get()->validators()->registerNotObligatory(getKind(), ARGUMENTS_ID());
 
-  if string( XT_ID())->value() == ""
+  if ( string( XT_ID())->value() == ""
     && string( YT_ID())->value() == ""
     && string( ZT_ID())->value() == "")
     {
@@ -112,9 +105,10 @@ void BuildPlugin_Interpolation::initAttributes()
     }
 }
 
+//=================================================================================================
 void BuildPlugin_Interpolation::attributeChanged(const std::string& theID)
 {
-  if(theID == XT_ID()
+  if ((theID == XT_ID()
     || theID == YT_ID()
     || theID == ZT_ID()
     || theID == MINT_ID()
@@ -128,6 +122,7 @@ void BuildPlugin_Interpolation::attributeChanged(const std::string& theID)
   }
 }
 
+//=================================================================================================
 void BuildPlugin_Interpolation::updateCoordinates()
 {
     std::wstring exp;
@@ -135,7 +130,7 @@ void BuildPlugin_Interpolation::updateCoordinates()
     double aMaxt = real(MAXT_ID())->value();
     int aNbrStep = integer(NUMSTEP_ID())->value();
 
-    if ( aMaxt < aMint ) {
+    if (aMaxt < aMint) {
       setError("The minimum value of the parameter must be less than maximum value !!!" );
     }
 
@@ -143,7 +138,7 @@ void BuildPlugin_Interpolation::updateCoordinates()
     string(VARIABLE_ID())->setValue("t");
 
     tables(VALUE_ID())->setSize(aNbrStep+1,4);
-    forint step = 0; step <= aNbrStep; step++ ){
+    for (int step = 0; step <= aNbrStep; step++ ){
       ModelAPI_AttributeTables::Value aVal;
       aVal.myDouble = step * aScale + aMint;
       tables(VALUE_ID())->setValue(aVal,step,0);
@@ -183,7 +178,7 @@ static GeomDirPtr selectionToDir(const AttributeSelectionPtr& theSelection)
 //=================================================================================================
 void BuildPlugin_Interpolation::execute()
 {
-  if( string(CREATION_METHOD_ID())->value() == CREATION_METHOD_BY_SELECTION_ID() )
+  if (string(CREATION_METHOD_ID())->value() == CREATION_METHOD_BY_SELECTION_ID())
   {
     // Get closed flag value
     bool isClosed = boolean(CLOSED_ID())->value();
@@ -244,9 +239,8 @@ void BuildPlugin_Interpolation::execute()
 
     setResult(aResultBody);
 
-  }else
-  {
-    if(   string( XT_ID())->value() == ""
+  } else {
+    if (string( XT_ID())->value() == ""
         ||string( YT_ID())->value() == ""
         ||string( ZT_ID())->value() == ""
         ||tables(VALUE_ID())->rows()== 0  )
@@ -258,7 +252,7 @@ void BuildPlugin_Interpolation::execute()
 
     AttributeTablesPtr aTable = tables( VALUE_ID() );
     std::list<std::vector<double> > aCoordPoints;
-    for( int step = 0; step < aTable->rows() ; step++ ){
+    for (int step = 0; step < aTable->rows(); step++){
       std::vector<double> aCoordPoint;
       ModelAPI_AttributeTables::Value value;
       //x
@@ -278,7 +272,7 @@ void BuildPlugin_Interpolation::execute()
     std::list<GeomVertexPtr> aVertices;
     std::list<std::vector<double> >::const_iterator anItCoordPoints = aCoordPoints.begin();
 
-    for( ; anItCoordPoints != aCoordPoints.end(); ++anItCoordPoints ){
+    for (; anItCoordPoints != aCoordPoints.end(); ++anItCoordPoints){
 
       GeomVertexPtr aVertex =
           GeomAlgoAPI_PointBuilder::vertex( (*anItCoordPoints)[0],
@@ -309,7 +303,8 @@ void BuildPlugin_Interpolation::execute()
   }
 }
 
-void  BuildPlugin_Interpolation::evaluate(std::string& theError)
+//=================================================================================================
+void BuildPlugin_Interpolation::evaluate(std::string& theError)
 {
   FeaturePtr aMyPtr = std::dynamic_pointer_cast<ModelAPI_Feature>(data()->owner());
   std::shared_ptr<ModelAPI_BuildEvalMessage> aProcessMessage =
index 6697162bb6d940bf3a5aaafe50a9e7c8d62176bc..1eaa6116d702665902a2bf328427921492cfcfec 100644 (file)
@@ -186,15 +186,18 @@ public:
   /// Creates a new part document if needed.
   BUILDPLUGIN_EXPORT virtual void execute();
 
+  /// Called on change of any argument-attribute of this object.
+  /// \param[in] theID identifier of changed attribute.
   BUILDPLUGIN_EXPORT virtual void attributeChanged(const std::string& theID);
 
   protected:
   /// Evaluates the expression x(t), y(t),z(t) in value table.
   void evaluate(std::string& theError);
 
-  //Update coordinates x,y,z
+  ///Update coordinates x,y,z
   void updateCoordinates();
 
+  ///Output error for python interpreter
   std::string outErrorMessage;
 
 };
index 6614d06847328bbb26a8cf8c7abffc6c0e2b1f4f..8cfc1132ff3b78371b1b0085911fba2ca8c12280 100644 (file)
@@ -36,9 +36,6 @@
 #include <BuildPlugin_Filling.h>
 #include <BuildPlugin_Validators.h>
 
-
-
-
 // the only created instance of this plugin
 static BuildPlugin_Plugin* MY_INSTANCE = new BuildPlugin_Plugin();
 
index be26b119646cedc161b890f9dd7c17fdaa66a7fd..f0657537db0b466d621e47f6c656d7e6349fb575 100644 (file)
@@ -573,7 +573,7 @@ bool BuildPlugin_ValidatorBaseForVertex::isValid(const AttributePtr& theAttribut
   return true;
 }
 
-
+//=================================================================================================
 bool BuildPlugin_ValidatorExpressionInterpolation::isValid(const AttributePtr& theAttribute,
                                                    const std::list<std::string>& /*theArguments*/,
                                                    Events_InfoMessage& theError) const
index a06488579f7f556a70715a02fa490cb5e6af02ca..d2460699fcf4d8a42e4e1611999c79caec5d680f 100644 (file)
@@ -178,9 +178,9 @@ Part_5 = model.addPart(partSet)
 Part_5_doc = Part_5.document()
 
 Interpolation_11 = model.addInterpolation(Part_5_doc, "sin(t)","cos(t)","t", 0, 100, 10)
-
-model.checkResult(Interpolation_11, model, 1, [0], [0], [0], [1], [2])
-
+model.do()
+model.testNbResults(Interpolation_11, 1)
+model.end()
 # =============================================================================
 # Test 13. Check Python dump
 # =============================================================================
index 21ccd2c84e36ce17dd8c34ad6f6a2616fe5613a5..b5036905f1ab5dab7e8c399c875a801c2d722e18 100644 (file)
@@ -36,7 +36,6 @@
     </box>
     <box id="analytical"
          title="Curve analytical"
-         tooltip="???"
          icon="icons/Build/feature_interpolation_analytical.png">
       <groupbox title="Curves parameters">
         <stringvalue id="xt" label="X(t) equation">
index 4699cc1bd37030c6abd2c4c0021552572f35c560..56193ea495f4146b7912556ffc5674749cc940b5 100644 (file)
@@ -49,7 +49,7 @@
 #include <iostream>
 #include <algorithm>
 
-//------------------------------------------------------------------------------
+//=================================================================================================
 // Tools
 
 std::wstring toString(double theValue)
@@ -67,7 +67,7 @@ std::set<std::wstring> toSet(const std::list<std::wstring>& theContainer)
   return std::set<std::wstring>(theContainer.begin(), theContainer.end());
 }
 
-//------------------------------------------------------------------------------
+//=================================================================================================
 
 InitializationPlugin_EvalListener::InitializationPlugin_EvalListener()
 {
@@ -82,10 +82,12 @@ InitializationPlugin_EvalListener::InitializationPlugin_EvalListener()
   myInterp->initialize();
 }
 
+//=================================================================================================
 InitializationPlugin_EvalListener::~InitializationPlugin_EvalListener()
 {
 }
 
+//=================================================================================================
 void InitializationPlugin_EvalListener::processEvent(
     const std::shared_ptr<Events_Message>& theMessage)
 {
@@ -143,7 +145,7 @@ void InitializationPlugin_EvalListener::processEvent(
     Locale::Convert::toWString(anExprAttr->value());
     anExpZ.erase(std::remove(anExpZ.begin(),anExpZ.end(), ' '), anExpZ.end());
 
-    for(int step =0; step < anValueAttr->rows(); step++ ){
+    for (int step =0; step < anValueAttr->rows(); step++){
       aVal.myDouble = evaluate(anVar,
                                 anValueAttr->value(step,0).myDouble,
                                 aParam,
@@ -151,9 +153,8 @@ void InitializationPlugin_EvalListener::processEvent(
                                 anError,
                                 aParamsList,
                                 anIsFirstTime);
-      if(!anError.empty()) break;
+      if (!anError.empty()) break;
       anValueAttr->setValue(aVal,step,1);
-      
       aVal.myDouble = evaluate(anVar,
                               anValueAttr->value(step,0).myDouble,
                               aParam,
@@ -161,9 +162,8 @@ void InitializationPlugin_EvalListener::processEvent(
                               anError,
                               aParamsList,
                               anIsFirstTime);
-      if(!anError.empty()) break;
+      if (!anError.empty()) break;
       anValueAttr->setValue(aVal,step,2);
-      
       aVal.myDouble = evaluate(anVar,
                                 anValueAttr->value(step,0).myDouble,
                                 aParam,
@@ -171,16 +171,16 @@ void InitializationPlugin_EvalListener::processEvent(
                                 anError,
                                 aParamsList,
                                 anIsFirstTime);
-      if(!anError.empty()) break;
+      if (!anError.empty()) break;
       anValueAttr->setValue(aVal,step,3);
-      if ( anIsFirstTime )
+      if (anIsFirstTime)
           anIsFirstTime = false;
     }
-
     aMsg->setResults(aParamsList, anError);
   }
 }
 
+//=================================================================================================
 double InitializationPlugin_EvalListener::evaluate(
   std::wstring& theVariable,
   double theValueVariable,
@@ -197,9 +197,8 @@ double InitializationPlugin_EvalListener::evaluate(
 
   std::list<std::wstring> anExprParams = myInterp->compile(theExpression);
   // find expression's params in the model
-  
   std::list<std::wstring>::iterator it = anExprParams.begin();
-  for ( ; it != anExprParams.end(); it++) {
+  for (; it != anExprParams.end(); it++) {
     double aValue;
     ResultParameterPtr aParamRes;
     // If variable does not exist python interpreter will generate an error.
@@ -207,11 +206,11 @@ double InitializationPlugin_EvalListener::evaluate(
                            *it, aValue, aParamRes, theParameter->document()))
       continue;
 
-    if( theIsFirstTime )
+    if (theIsFirstTime)
     {
       std::list<ResultParameterPtr>::iterator anIter =
                           std::find(theParamsList.begin(),theParamsList.end(), aParamRes );
-      if(anIter == theParamsList.end())
+      if (anIter == theParamsList.end())
         theParamsList.push_back(aParamRes);
     }
 
@@ -223,7 +222,7 @@ double InitializationPlugin_EvalListener::evaluate(
   return result;
 }
 
-
+//=================================================================================================
 double InitializationPlugin_EvalListener::evaluate(FeaturePtr theParameter,
   const std::wstring& theExpression, std::string& theError,
   std::list<std::shared_ptr<ModelAPI_ResultParameter> >& theParamsList,
@@ -253,6 +252,7 @@ double InitializationPlugin_EvalListener::evaluate(FeaturePtr theParameter,
   return result;
 }
 
+//=================================================================================================
 void InitializationPlugin_EvalListener::processEvaluationEvent(
     const std::shared_ptr<Events_Message>& theMessage)
 {
@@ -341,6 +341,7 @@ void InitializationPlugin_EvalListener::processEvaluationEvent(
   }
 }
 
+//=================================================================================================
 void InitializationPlugin_EvalListener::initDataModel()
 {
   myInterp->runString("import salome_iapp;salome_iapp.register_module_in_study(\"Shaper\")");
index 06972aa1757175115d8923fac2bbb725e9d40775..e11ab4b068d11b4fab7f5d214c90ef7e16b929e3 100644 (file)
@@ -357,7 +357,7 @@ class ModelAPI_BuildEvalMessage : public Events_Message
   FeaturePtr myParam; ///< parameters that should be evaluated
   bool myIsProcessed; ///< true if results were set
   std::string myError; ///< error of processing, empty if there is no error
-    /// result of processing, list of parameters in expression found
+  /// result of processing, list of parameters in expression found
   std::list<std::shared_ptr<ModelAPI_ResultParameter> > myParamsList;
 
  public:
@@ -375,8 +375,8 @@ class ModelAPI_BuildEvalMessage : public Events_Message
     send(FeaturePtr theParameter, const void* theSender)
   {
     std::shared_ptr<ModelAPI_BuildEvalMessage> aMessage =
-      std::shared_ptr<ModelAPI_BuildEvalMessage>(
-      new ModelAPI_BuildEvalMessage(eventId(), theSender));
+          std::shared_ptr<ModelAPI_BuildEvalMessage>(
+                new ModelAPI_BuildEvalMessage(eventId(), theSender));
     aMessage->setParameter(theParameter);
     Events_Loop::loop()->send(aMessage);
     return aMessage;
@@ -393,9 +393,9 @@ class ModelAPI_BuildEvalMessage : public Events_Message
   MODELAPI_EXPORT void setParameter(FeaturePtr theParam);
   /// Sets the results of processing
   MODELAPI_EXPORT void setResults(
-            const std::list<std::shared_ptr<ModelAPI_ResultParameter> >& theParamsList,
-            const std::string& theError);
-    /// Returns the results of processing: list of parameters found in the expression
+                       const std::list<std::shared_ptr<ModelAPI_ResultParameter> >& theParamsList,
+                       const std::string& theError);
+  /// Returns the results of processing: list of parameters found in the expression
   MODELAPI_EXPORT const std::list<std::shared_ptr<ModelAPI_ResultParameter> >& params() const;
   /// Returns true if the expression is processed
   MODELAPI_EXPORT bool isProcessed();