4

我试图通过删除重复的代码来改进现有的 C++ 代码，但无法提出令人信服的方法。非常感谢更有经验的 C++ 同事的任何见解。

所以我有两个我无法控制的结构定义：

struct struct_1
{
...
other_s * s1;
other_s * s2;
other_s * s3;
... //other fields
};

struct struct_2
{
...
other_s * s1;
other_s * s2;
other_s * s3;
... //other fields, different than struct_1, but not that important
};

最后是我想要改进的代码。我有两个几乎相同的类，它们以相同的方式对同名的结构字段进行操作，但这些字段来自不同的结构。这些类不对仅存在于一个结构中的结构字段进行操作。在这里（简化）：

class class_1
{
    struct_1 * s;

    class_1(){
        s = new struct_1(); //the only practical difference
        ...
    }

    void method()
    {
        ...

        //this is simplification, in reality the code is more complex
        //however the same as in class_2
        inner_data += s->s1;
        inner_data += s->s2;
        inner_data += s->s3; 
        ...
    }
    //other methods
};

class class_2
{
    struct_2 * s;

    class_2(){
        s = new struct_2(); //the only practical difference
        ...
    }

    void method()
    {
        ...

        //this is simplification, in reality the code is more complex
        //however the same as in class_1
        inner_data += s->s1;
        inner_data += s->s2;
        inner_data += s->s3; 
        ...
    }
    //other methods
};

我花了一些时间试图重做它，但最终无处可去。我的方法是只使用一个类 class_1，但我无法避免在没有多个 if 分散的情况下访问 struct_1 和 struct_2 的问题。感谢您的帮助！

4

3 回答 3

5

C++ 有一个模板：

template<typename T>
class MyClass
{
    T* s;

    MyClass(){
        s = new T(); //the only practical difference
        ...
    }

    void method()
    {
        ...

        //this is simplification, in reality the code is more complex
        //however the same as in class_2
        inner_data += s->s1;
        inner_data += s->s2;
        inner_data += s->s3; 
        ...
    }
    //other methods
};

现在您可以将您的课程用作：

MyClass<struct_1> a;

和

MyClass<struct_2> b;

编译器将根据您的模板为这些类生成定义。

不要忘记在析构函数中释放内存！

于 2013-08-15T18:36:06.797 回答
2

您正在寻找一个模板：

template <typename S>
class C {
    S * s;

    C() s(new S()) {}

    void method() {
        // This is all fine as long as `S` has these members
        inner_data += s->s1;
        inner_data += s->s2;
        inner_data += s->s3;
    }
};

那么你的课程可以是这方面的专业：

typedef C<struct_1> class_1;
typedef C<struct_2> class_2;

（并且，希望您的真实代码遵循三规则，因为它与低级内存分配有关。）

于 2013-08-15T18:38:02.293 回答
0

模板可以解决这个问题，但如果结构大多相同（例如，如果 struct_2 只是向 struct_1 添加更多字段），您可以将 struct_1 转换为基类，将 struct_2 转换为从它派生的类（仅添加新的数据成员） .

或者，您也可以将所有数据成员合并到 struct_1 中（再次假设数据成员与 struct_s 基本相同，只是添加了一些）并添加一个标志或类型来指示哪些成员是有效的。

于 2013-08-15T18:46:32.240 回答