C++ 2つの関数は同じ行を除いて同じです

Question

私のコードを複製したくありませんが、1行を除いて同じことをする2つの関数があります。 プライベート関数を呼び出す別の関数が必要です。それはテンプレートであり、私の重複したコードの問題を解決します。

double EuclideanDistance::calculateDistance(const Point &p1, const Point &p2) const 
{ 
    if (p1.getDimension()!=p2.getDimension()) 
     return 0; 
    double sum = 0.0; 
    for(size_t i = 0; i < p1.getDimension(); i++) 
    { 
     sum += pow(p1[i] - p2[i], 2.0); 
    } 
    return sqrt(sum); 
} 

double EuclideanDistance::calculateWeightedDistance(const Point &p1, const Point &p2, const double *weights) const 
{ 
    if (p1.getDimension()!=p2.getDimension()) 
     return 0; 
    double sum = 0.0; 
    for(size_t i = 0; i < p1.getDimension(); i++) 
    { 
     sum += pow(((p1[i] - p2[i])*weights[i]), 2.0);//here is the difference 
    } 
    return sqrt(sum); 
} 

あなたは何をお勧めしますか？

Answer 1

calculateWeightedDistanceのみを実装します。 およびcalculateDistanceの場合calculateWeightedDistance、重量の場合1.0

Answer 2

次はどうなりますか？ 3番目のパラメータnullptrで関数を呼び出すと、最初のバージョンの動作が実行されます。

double EuclideanDistance::calculateWeightedDistance(const Point& p1, const Point& p2, const double* weights) const { 
    if (p1.getDimension()!=p2.getDimension()) 
     return 0; 

    double sum = 0.0; 
    for(size_t i = 0; i < p1.getDimension(); i++) 
    { 
     const auto distance_d = (p1[i] - p2[i]) * 
           (weights != nullptr ? weights[i] : 1.0); 
     sum += pow(distance_d, 2.0); 
    } 
    return sqrt(sum); 
} 

Answer 3

あなたはこの方法を因数分解可能性があります

template <typename F, typename Ts...> 
double calculateDistanceImpl(F&& f, const Point &p1, const Point &p2, Ts&&... args) 
{ 
    if (p1.getDimension()!=p2.getDimension()) 
     return 0; 

    double sum = 0.0; 
    for(size_t i = 0; i != p1.getDimension(); i++) 
    { 
     sum += f(p1[i], p2[i], args[i]...); 
    } 
    return sqrt(sum); 
} 


double EuclideanDistance::calculateDistance(const Point &p1, const Point &p2) const 
{ 
    return calculateDistanceImpl(
     [](auto&& p1, auto&& p2){ return pow(p1 - p2, 2.0);}, 
     p1, p2); 
} 

double EuclideanDistance::calculateWeightedDistance(const Point &p1, 
                const Point &p2, 
                const double *weights) const 
{ 
    return calculateDistanceImpl(
     [](auto&& p1, auto&& p2, double weight){ return pow((p1 - p2) * weight, 2);}, 
     p1, p2, weights); 
}