axes classes: express operator[] through binCenter

Base/Axis/FixedBinAxis.cpp

@@ -32,7 +32,7 @@ FixedBinAxis* FixedBinAxis::clone() const
     return result;
 }
 
-double FixedBinAxis::operator[](size_t index) const
+double FixedBinAxis::binCenter(size_t index) const
 {
     if (index >= m_nbins)
         throw std::runtime_error("FixedBinAxis::operator[] -> Error. Wrong index.");

Base/Axis/FixedBinAxis.h

@@ -34,14 +34,12 @@ public:
 
     size_t size() const override { return m_nbins; }
 
-    double operator[](size_t index) const override;
-
     Bin1D bin(size_t index) const override;
 
     double min() const override { return m_start; }
     double max() const override { return m_end; }
 
-    double binCenter(size_t index) const override { return (*this)[index]; }
+    double binCenter(size_t index) const override;
 
     size_t findClosestIndex(double value) const override;
 

Base/Axis/IAxis.h

@@ -65,7 +65,7 @@ public:
     virtual std::vector<double> binBoundaries() const;
 
     //! indexed accessor retrieves a sample
-    virtual double operator[](size_t index) const = 0;
+    double operator[](size_t index) const { return binCenter(index); }
 
     //! retrieve a 1d bin for the given index
     virtual Bin1D bin(size_t index) const = 0;

Base/Axis/PointwiseAxis.h

@@ -46,9 +46,6 @@ public:
     //! retrieve the number of bins
     size_t size() const override { return m_coordinates.size(); }
 
-    //! indexed accessor retrieves a sample
-    double operator[](size_t index) const override { return binCenter(index); }
-
     //! retrieve a 1d bin for the given index
     Bin1D bin(size_t index) const override;
 

Base/Axis/VariableBinAxis.cpp

@@ -37,11 +37,6 @@ VariableBinAxis* VariableBinAxis::clone() const
     return result;
 }
 
-double VariableBinAxis::operator[](size_t index) const
-{
-    return bin(index).center();
-}
-
 Bin1D VariableBinAxis::bin(size_t index) const
 {
     if (index >= m_nbins)

Base/Axis/VariableBinAxis.h

@@ -28,8 +28,6 @@ public:
 
     size_t size() const override { return m_nbins; }
 
-    double operator[](size_t index) const override;
-
     Bin1D bin(size_t index) const override;
 
     double min() const override;