Feat: add protection against bad user input

Author: ariedel-cern

PWGCF/Femto/Core/collisionBuilder.h

@@ -245,7 +245,13 @@ class CollisionSelection : public baseselection::BaseSelection<float, o2::analys
 
     // event shape
     mQvecDetector = static_cast<modes::QvecDetector>(config.qvecDetector.value);
+    if (mQvecDetector >= modes::QvecDetector::kQvecDetectorLast) {
+      LOG(fatal) << "Qvector Detector is not supported";
+    }
     mQvecHarmonic = static_cast<modes::QvecHarmonic>(config.qvecHarmonic.value);
+    if (mQvecHarmonic < modes::QvecHarmonic::kN2 || mQvecHarmonic >= modes::QvecHarmonic::kQvecHarmonicLast) {
+      LOG(fatal) << "Qvector Harmonic is not supported";
+    }
 
     this->addSelection(kSel8, collisionSelectionNames.at(kSel8), config.sel8.value);
     this->addSelection(kNoSameBunchPileUp, collisionSelectionNames.at(kNoSameBunchPileUp), config.noSameBunchPileup.value);
@@ -371,9 +377,6 @@ class CollisionSelection : public baseselection::BaseSelection<float, o2::analys
   {
     auto harmonic = static_cast<float>(mQvecHarmonic);
     int index = static_cast<int>(mQvecHarmonic) - 2; // get index in the qvector vector
-    if (index >= 2) {
-      LOG(fatal) << "At the moment harmonics up to 3 are supported!";
-    }
     switch (mQvecDetector) {
       case modes::QvecDetector::kFT0C:
         mEventPlane = RecoDecay::constrainAngle((std::atan2(col.qvecFT0CImVec()[index], col.qvecFT0CReVec()[index])) / harmonic, 0, harmonic); // constrain between 0 and 2pi/harmonic

PWGCF/Femto/Core/modes.h

@@ -180,12 +180,14 @@ enum class CharmHadron : o2::analysis::femto::datatypes::CharmHadronType {
 
 enum class QvecDetector : o2::analysis::femto::datatypes::QvecDetectorType {
   kFT0C = 0,
-  kFT0A = 1
+  kFT0A = 1,
+  kQvecDetectorLast = 2
 };
 
 enum class QvecHarmonic : o2::analysis::femto::datatypes::QvecHarmonicType {
   kN2 = 2,
-  kN3 = 3
+  kN3 = 3,
+  kQvecHarmonicLast = 4
 };
 
 }; // namespace o2::analysis::femto::modes