c++ - 使用 if() 根据属性修改类行为是否有代码味道？

Question

我有一个代表一些参数的类。参数可以是数字、数组、枚举或位域——这是参数类型。这些类型之间的行为略有不同，因此它们是 paramBase 类的子类。参数可以存储在 RAM 中或者是静态的(即以某种方式硬编码，当前保存在文件中)。

void read() 在paramBase 中实现，并使用模板方法模式实现对任何参数类型的读取，但这仅适用于 RAM 存储。如果参数是静态的，则 read() 必须完全不同(即从文件中读取)。

一个简单的解决方案可以是进一步子类化，如 paramArrayStatic、paramNumberStatic 等(这将是 8 个子类)。paramArray 和paramArrayStatic 的区别基本上只在read() 方法上，所以直接解决会导致代码重复。

我也可以将 if( m_storage==static ) 添加到 read() 方法并修改行为，但这也是代码味道(AFIK)。

class paramBase
    {
    public:
        virtual paramType_t type() = 0; 
        paramStorage_t storage();

        virtual someDefaultImplementedMethod()
        {
            //default implementation
        }

        void read()
        {
            //template method pattern
            m_prop1 = blablabla;
            someDefaultImplementedMethod();
        }

    protected:
        paramStorage_t m_storage;
        int m_prop1;
        int m_prop2;
    };

    class paramArray: public paramBase
    {
    public:
        virtual paramType_t type()
        {
            return PT_ARRAY;
        }
        virtual someDefaultImplementedMethod()
        {
            //overriding default implementation of base
            //i.e. modify templated read() method behavior
        }
    protected:
        int m_additional_prop1;
        int m_additional_prop2;
    };

最后，我有 4 个基类子类，我需要通过静态/非静态修饰符修改 read() 的行为。

如何在没有代码重复和代码异味的情况下解决这个问题？ read() 中的条件 if( m_storage==static ) 是否是代码异味？

Answer 1

您永远不必重复代码:只需重新实现单个方法read。如果您需要从指向基类的指针使用它，virtual 就可以做到这一点。如果您在这 8 个 read 方法(或其中一些方法之间)之间有公共(public)代码，请将其放在公共(public)中间层中。

如果您想明确表示该类可能不会在基类使用该方法，您可以将其抽象化，为 RAM 情况添加第九个子类。

在同一个类中有一个巨大的开关调用 9 个不同的 read 方法对我来说似乎更糟糕。