Fixed MSVC warning

[clitk.git] / segmentation / clitkRelativePositionList.txx

diff --git a/segmentation/clitkRelativePositionList.txx b/segmentation/clitkRelativePositionList.txx
index aff21e41618eb80a373a85bd66cadc0d33ed677b..254fa4c4581cd6895444789003d8b70138f8387f 100644 (file)
--- a/segmentation/clitkRelativePositionList.txx
+++ b/segmentation/clitkRelativePositionList.txx
@@ -77,7 +77,6 @@ Read(std::string filename) {
       if (!is) stop = true;
       else {
         std::getline(is, s);
-        // DD(s);
         if (s.find("object") != std::string::npos) stop=true;
         else ss << s << std::endl;
         if (!is) stop = true;
@@ -90,15 +89,20 @@ Read(std::string filename) {
       os << text;
       os.close();
 
-      // Create a struct to store options
+      // Create a struct to store options. I use two step to allow to
+      // fill the args values with de default and then check
+      // automatically the options.
       ArgsInfoType args_info;
+      std::vector<char> writable(tmpfilename.size() + 1);
+      std::copy(tmpfilename.begin(), tmpfilename.end(), writable.begin());
+      char ** argv = new char*[1];
+      argv[0] = new char[1];
+      cmdline_parser_clitkRelativePosition2(1, argv, &args_info, 1, 1, 0);
       args_info.input_given = 1;
       args_info.input_arg = new char[1];
       args_info.output_given = 1;
       args_info.output_arg = new char[1];
-      std::vector<char> writable(tmpfilename.size() + 1);
-      std::copy(tmpfilename.begin(), tmpfilename.end(), writable.begin());
-      cmdline_parser_clitkRelativePosition_configfile(&writable[0], &args_info, 1, 1, 0);
+      cmdline_parser_clitkRelativePosition_configfile(&writable[0], &args_info, 0, 0, 1);
       
       // Store the args
       mArgsInfoList.push_back(args_info);
@@ -142,10 +146,12 @@ GenerateOutputInformation() {
   // Loop on RelativePositionList of operations
   std::string s = GetInputName();
   for(uint i=0; i<mArgsInfoList.size(); i++) {
-    std::string text = "["+s+"] limits "+
-      mArgsInfoList[i].orientation_arg[0]+" "+
-      mArgsInfoList[i].object_arg+" "+
-      toString(mArgsInfoList[i].threshold_arg);
+    std::string text = "["+s+"] limits ";
+    if (mArgsInfoList[i].orientation_given) text += std::string(mArgsInfoList[i].orientation_arg[0])+" ";
+    else text = text+"("+toString(mArgsInfoList[i].angle1_arg)+" "+
+           toString(mArgsInfoList[i].angle2_arg)+" "+
+           (mArgsInfoList[i].inverse_flag?"true":"false")+") ";
+    text = text+mArgsInfoList[i].object_arg+" "+toString(mArgsInfoList[i].threshold_arg);
     if (mArgsInfoList[i].sliceBySlice_flag) {
       text += " slice by slice";
     }
@@ -154,20 +160,33 @@ GenerateOutputInformation() {
     StartNewStep(text);  
     typename RelPosFilterType::Pointer relPosFilter;
 
-    // DD(mArgsInfoList[i].sliceBySlice_flag);
+    // Is it slice by slice or 3D ?
     if (mArgsInfoList[i].sliceBySlice_flag) {
-      relPosFilter = SliceRelPosFilterType::New();
-      dynamic_cast<SliceRelPosFilterType*>(&*relPosFilter)->SetDirection(2);
+      typename SliceRelPosFilterType::Pointer f = SliceRelPosFilterType::New();
+      relPosFilter = f;
+      SetFilterOptions(relPosFilter, mArgsInfoList[i]);  
+      f->SetDirection(2);
+      // Set SbS specific options
+      f->SetUniqueConnectedComponentBySliceFlag(mArgsInfoList[i].uniqueCCL_flag);
+      f->SetObjectCCLSelectionFlag(mArgsInfoList[i].uniqueObjectCCL_flag);
+      f->IgnoreEmptySliceObjectFlagOn();
+      //f->SetObjectCCLSelectionDimension(0);
+      //f->SetObjectCCLSelectionDirection(-1);
+      //f->SetAutoCropFlag(false);
+      // Print if needed
+      if (mArgsInfoList[i].verboseOptions_flag) f->PrintOptions();
     }
     else {
       relPosFilter = clitk::AddRelativePositionConstraintToLabelImageFilter<ImageType>::New();
+      SetFilterOptions(relPosFilter, mArgsInfoList[i]);  
+      // Print if needed
+      if (mArgsInfoList[i].verboseOptions_flag) relPosFilter->PrintOptions();
     }
-    relPosFilter->VerboseStepFlagOff();
-    relPosFilter->WriteStepFlagOff();
-    relPosFilter->SetVerboseImageSizeFlag(GetVerboseImageSizeFlag());
-    relPosFilter->SetInput(m_working_input);
-    SetFilterOptions(relPosFilter, mArgsInfoList[i]);    
-    //relPosFilter->PrintOptions();
+
+    // Set input
+    relPosFilter->SetInput(m_working_input);  
+  
+    // Run the filter
     relPosFilter->Update();
     m_working_input = relPosFilter->GetOutput();  
     StopCurrentStep<ImageType>(m_working_input);
@@ -192,33 +211,39 @@ GenerateData()
 template <class TImageType>
 void
 clitk::RelativePositionList<TImageType>::
-SetFilterOptions(typename RelPosFilterType::Pointer filter, 
-                 ArgsInfoType & options) {
+SetFilterOptions(typename RelPosFilterType::Pointer filter, ArgsInfoType & options) {
 
-  if (options.orientation_given != 1) {
-    DD("ERRROR DEBUG TODO no more than 1 orientation yet");
-    exit(0);
+  if (options.orientation_given > 1) {
+    clitkExceptionMacro("Error in the RelPos options. I need a single --orientation (for the moment)."
+                        << " Current options are for obj = '" << options.object_arg
+                        << "', threshold = " << options.threshold_arg << std::endl);
   }
 
   ImagePointer object = GetAFDB()->template GetImage<ImageType>(options.object_arg);
   filter->SetInputObject(object);
+  filter->WriteStepFlagOff();
+  filter->SetVerboseImageSizeFlag(GetVerboseImageSizeFlag());
   filter->SetFuzzyThreshold(options.threshold_arg);
   filter->SetInverseOrientationFlag(options.inverse_flag); // MUST BE BEFORE AddOrientationTypeString
-  for(uint i=0; i<options.orientation_given; i++) 
-    filter->AddOrientationTypeString(options.orientation_arg[i]);
+
+  if (options.orientation_given == 1)  {
+    for(uint i=0; i<options.orientation_given; i++) 
+      filter->AddOrientationTypeString(options.orientation_arg[i]);
+  }
+  else {
+    if (options.angle1_given && options.angle2_given) {
+      filter->AddAnglesInDeg(options.angle1_arg, options.angle2_arg);
+    }
+    else {
+      clitkExceptionMacro("Error in the RelPos options. I need --orientation or (--angle1 and --angle2)."
+                          << " Current options are for obj = '" << options.object_arg
+                          << "', threshold = " << options.threshold_arg << std::endl);
+    }
+  }
   filter->SetIntermediateSpacing(options.spacing_arg);
   if (options.spacing_arg == -1) filter->IntermediateSpacingFlagOff();
-  filter->IntermediateSpacingFlagOn();
-  
-  if (options.sliceBySlice_flag) {
-    SliceRelPosFilterType * f = dynamic_cast<SliceRelPosFilterType*>(&*filter);
-    f->SetUniqueConnectedComponentBySliceFlag(options.uniqueCCL_flag);
-    f->SetObjectCCLSelectionFlag(options.uniqueObjectCCL_flag);
-    f->IgnoreEmptySliceObjectFlagOn();
-    //filter->SetObjectCCLSelectionDimension(0);
-    //filter->SetObjectCCLSelectionDirection(-1);
-    //filter->SetAutoCropFlag(false);
-  }
+  else filter->IntermediateSpacingFlagOn();
+  filter->SetVerboseStepFlag(options.verboseStep_flag);
   filter->SetAutoCropFlag(!options.noAutoCrop_flag); 
 }
 //--------------------------------------------------------------------