より効率的なC++の構造化/パターン化方法がありますか？

Question

基本的な考え方は、すべて同じことをするクラスの「ファミリ」がありますが、やや異なる方法です。この「ファミリ」は「高性能」ループで使用されるため、速度は本質的です。さらに、特定のファミリはコンフィグレーションファイルで指定されています。

ここでの主な機能には驚異的なコードの繰り返しがあります。これを構造化するより良い方法があるので、HP<objx> testとtest.loop(bobloblaw)を3回書く必要はありませんか？

class obj1 { 
public: 
    double f(double x) const { return 1.; } 
}; 

class obj2 { 
public: 
    double f(double x) const { return x; } 
}; 

class obj3 { 
public: 
    double f(double x) const { return x*x; } 
}; 

template <class O> 
class HP { 
private: 
    O obj; 

public: 

    double loop(const vector<double>& x) { 
     double s = 0.; 
     for (auto i : x) s += obj.f(i); 

    return s; 
    } 
}; 

int main() { 
    string config = "bob"; 
    double result = 0; 
    vector<double> bobloblaw; 

    /* Read configuration file to determine which object to use. */ 
    if (config == "obj1") { 
     HP<obj1> test; 
     result = test.loop(bobloblaw); 
    } else if (config == "obj2") { 
     HP<obj2> test; 
     result = test.loop(bobloblaw); 
    } else if (config == "obj3") { 
     HP<obj3> test; 
     result = test.loop(bobloblaw); 
    } 

    return result; 
} 

Answer 1

次はテストされていないが、動作するはずです：

class obj1 
{ 
public: 
    double f(double x) const { return 1.; } 
}; 

class obj2 
{ 
public: 
    double f(double x) const { return x; } 
}; 

class obj3 
{ 
public: 
    double f(double x) const { return x*x; } 
}; 

class HPbase 
{ 
public: 
    virtual double loop(const vector<double>&) = 0; 
}; 

template <class O> class HP: 
    public HPbase 
{ 
public: 
    double loop(const vector<double>& x) 
    { 
    double s = 0.; 
    for (auto i : x) 
     s += obj.f(i); 
    return s; 
    } 
private: 
    O obj; 
}; 

std::unordered_map<std::string, std::unique_ptr<HPbase>> decode{ 
    {"obj1"}, new HP<obj1>()}, 
    {"obj2"}, new HP<obj2>()}, 
    {"obj3"}, new HP<obj3>()} }; 

int main() 
{ 
    string config = "bob"; 
    double result = 0; 
    vector<double> bobloblaw; 

    /* Read configuration file to determine which object to use. */ 
    result = decode[config].loop(bobloblaw); 
} 

注添加だけがHP<>の基底クラスと、コードのif/elseロジックを置き換えるマップであること。

Answer 2

あなたが同じ親/ wのこれらの3つのクラスのすべてのことのサブクラスを持っていませんでした（実際には、このコードの塊が... 2よりも多くの行ですか）？ mainは依然として正しいサブクラスを使用してテストに割り当てる必要がありますが、テスト自体は共通のスーパークラスであると宣言されます。

ここでの欠点は、バージョンが使用されるtest.loopのコードをコンパイラがいつ作成するかをコンパイラが知りません（実行時に決定する必要があるため）。

これを回避する別の方法として、共通コードをマクロとして書き込むことができます。これは、一度しか書き込まれず、3つの異なるコピーに展開されます。それぞれのコンパイラは、テストが使用されています。デバッグを邪魔にすることはできますが、パフォーマンスに重点を置いている場合は、ソースの冗長性を減らす（とオブジェクトにはあまりない）、それは良いトレードオフになるかもしれません。