Использование класса генератора для генерации объектов

Question

Использование класса генератора для генерации объектов

0

У меня мало классов:

class Item;
class Component;
class BrokenItem;

Мне нужно создать Item разных конфигураций из разных комбинаций Component

Вот что я сделал:

class ItemGenerator
{
public:
    ItemGenerator(const std::vector<Component>& components_);
    ItemGenerator(const BrokenItem& brokenItem_);
    // and a few more ways to generate

    const std::vector<Item>& getGeneratedItems() const;

private:
    // vector to store generated objects
    // a vector is used because multiple combination of
    // configuration can be generated
    std::vector<Item> generatedItems_;
}

Я намерен использовать его так:

std::vector<Component> comps { 1000, 1020, 1040, 1080 };
ItemGenerator ig(comps);
std::vector<Item> vi = ig.getGeneratedItems();
for (auto i : vi)
    std::cout << "generated item: " << i << std::endl;

Вопрос: Я делаю это неправильно? Если да, то какой лучший подход? Если нет, что я могу сделать, чтобы сделать его лучше?

DanDan 17 июнь 2014, в 12:27

Источник

0

Это должен быть класс? Если вы не сохраняете состояние, это могут быть просто отдельные функции в пространстве имен.
DanDan 17 июнь 2014, в 10:23
0

@DanDan Я думал, что смогу сгруппировать похожие функции вместе. Это плохо?
user3235674 17 июнь 2014, в 10:25
1

@Xan Более естественное выражение «похожих функций» в C ++ - просто использовать пространство имен, а не форсировать иерархию классов, которая бесполезна (как было бы необходимо в Java).
Dennis 17 июнь 2014, в 10:30

Показать ещё 1 комментарий

Теги:

c++

class

design

1 ответ

Ещё вопросы

Это должен быть класс? Если вы не сохраняете состояние, это могут быть просто отдельные функции в пространстве имен.
@DanDan Я думал, что смогу сгруппировать похожие функции вместе. Это плохо?
@Xan Более естественное выражение «похожих функций» в C ++ - просто использовать пространство имен, а не форсировать иерархию классов, которая бесполезна (как было бы необходимо в Java).

DanDan · Accepted Answer · 2014-06-17T09-08-00.000Z

Вы можете группировать подобные функции с помощью пространства имен:

namespace ItemGenerator
{
    std::vector<Item> FromComponents(const std::vector<Component>& components);
    Item FromBrokenItem(const BrokenItem& brokenItem);
}