Сокращение количества перегруженных функций и конструкторов - PullRequest
0 голосов
/ 18 февраля 2019

Рассмотрим следующее определение класса:

class Car : public Vehicle {
    private:
        TouchScreen a;
        RadioAntenna b;
        Blah c;
    public:
        Car(TouchScreen &a1) {
            a = a1; //copy ctor
        }
        Car(RadioAntenna &b1) {
            b = b1;
        }
        Car(Blah &c1) {
            c = c1;
        }
        void update(Touchscreen &a1) {
            a.update(a1);
        }
        void update(RadioAntenna &b1) {
            b.update(b1);
        }
        void update(Blah &c1) {
            c.update(c1);
        }
}

class Fleet {
    private:
        std::map<int, Car> _fleet; //vehicle number, Car
    public:
        template <typename T>
        void update(int vehicle_num, T &t) {
            auto it = _fleet.find(vehicle_num);
            if (it == _fleet.end()) {
                _fleet[vehicle_num] = Car(t);
            }
            else {
                it->second.update(t);
            }
        }
}

Fleet содержит коллекцию автомобилей.Если вы хотите обновить переменную-член определенной машины,

Fleet f;
f.update<TouchScreen>(4, a1);
f.update<Blah>(10, c1); 

В будущем в Car можно определить больше экземпляров класса.Есть ли способ уменьшить количество перегруженных функций Constructor и update ()?Использование шаблонов возможно?Я чувствую, что это выглядит некрасиво, мудрый дизайн, использующий так много перегруженных функций.

Ответы [ 2 ]

0 голосов
/ 18 февраля 2019

Моя идея - использовать std :: any для этого.Предполагается, что у вас есть только один экземпляр для каждого типа в классе автомобилей.

Вот мое решение:

class Car : public Vehicle {
private:
    std::unordered_map<std::string, std::any> components;
public:
    Car(std::any &obj) {
        std::string typeName(obj.type().name());
        components.insert(make_pair(typeName, obj));
    }

    template<typename T> 
    void update(T& found, T &a1)
    {
        (std::any_cast<T>(found)).update(std::any_cast<T>(a1));
    }

    void update(std::any  &a1) {
        std::string typeName(a1.type().name());
        std::any found  =  components.find(typeName);
        if (found != components.end())
        {
            if (typeName ==  std::string(typeid(TouchScreen).name())
            {
                update<TouchScreen>(found.second, a1);
            }
            else if (typeName ==  std::string(typeid(RadioAntenna ).name())
            {
                update<RadioAntenna>(found.second, a1);
            }
            else if (typeName ==  std::string(typeid(Blah  ).name())
            {
                update<Blah>(found.second, a1);
            }

        }
    }

}
0 голосов
/ 18 февраля 2019

Альтернативный код может быть:

void Car::update(Touchscreen* a1=nullptr, RadioAntenna* b1=nullptr, Blah* c1=nullptr ) {
        if(a1) a.update(*a1);
        if(b1) b.update(*b1);
        if(c1) c.update(*c1);                        
}
...