Задачка по С++, Придумал тут намедни задачку, как вам? Опции

[ViGOur]

Есть классы:

Исходный код классов

#include <sstream>
#include <iostream>
#include <string>
#include <vector>

class CClassMessage
{
protected:
    std::vector<std::string> m_properties;

public:
    enum eProperties
    {
        PRPT_END
    };
};

class CMsg0: public CClassMessage
{
public:
    enum eProperties
    {
        PRPT_Prefix = 0,
        PRPT_Name,
        PRPT_END
    };
};

class CMsg1: public CClassMessage
{
public:
    enum eProperties
    {
        PRPT_Prefix = 0,
        PRPT_State,
        PRPT_Protocol,
        PRPT_END
    };
};

Сделайте так, что функция работала правильно:

bool parseLine( std::vector<std::string> line)
    {
        if( line.size() >= m_properties.size() )
        {
            for( int n = 0; n < PRPT_END; ++n )
            {
                m_properties[n] = line[n];
            }
            return true;
        }
        return false;
    }

[Алексей1153]

ViGOur, начнём с того, что означает "правильно" - а что должна делать функция ? ) Может, она уже правильно работает ?

Допустим, под "правильно" имеется в виду определение значения PRPT_END в СТАТИЧЕСКОЙ функции

bool parseLine(CClassMessage* context, std::vector<std::string> line)

для переданного контекста (объекта класса CClassMessage или его потомка)

Раскрывающийся текст

тут вариантов несколько
1) "enum eProperties" всех классов заменить на мембер std::map<int> CClassMessage::m_props; и заполнить в конструкторе потомков
2) виртуальная функция CClassMessage::GetPropsCount
3) можно удариться в шаблоны, но тут лишнее

[ViGOur]

В принципе вопрос правильный!

Заменить enum на map не стоит, хотябы потому, что в каждом классе свои свойства, которые описываются в enum, к тому же как быт, если нужно обращение вроде: m_properties[PRPT_Name] ? Хардкодить - не по феншую...

[Алексей1153]

ViGOur, так в конструкторе каждого класса мап заполнится своими свойствами

если нужно обращение "m_properties[PRPT_Name]", то напишут "m_properties[PRPT_Name]" ))

если нужно запретить изменение - вернуть константную ссылку на мап

[ViGOur]

а ты проверь!

[Алексей1153]

ViGOur, не понял вопрос )

что именно нужно проверить ?

[ViGOur]

Ну а ты сделай по тем классам что описаны в первом посте примерно так:

std::vector<std::string> vec0 = { "prefix", "name"  };
std::vector<std::string> vec1 = { "prefix", "state", "protocol" };

CClassMessage *pMsg0 = new CMsg0();
pMsg0->parseLine( vec0 );

CClassMessage *pMsg1 = new CMsg1();
pMsg1->parseLine( vec1 );

и посмотри какие свойства будет у каждого объекта!

[lanz]

Раскрывающийся текст

#include <sstream>
#include <algorithm>
#include <iostream>
#include <string>
#include <vector>
#include <iterator>



class CClassMessage
{
public:
    std::vector<std::string> m_properties;

    enum eProperties
    {
        ePRPT_END
    };

    virtual std::size_t PRPT_END() const {
        return CClassMessage::ePRPT_END;    
    };
public:
    bool parseLine(std::vector<std::string> line)
    {
        if( line.size() >= m_properties.size() )
        {
            m_properties.resize(PRPT_END());
            for( int n = 0; n < PRPT_END(); ++n )
            {
                m_properties[n] = line[n];
            }
            return true;
        }
        return false;
    }
};

class CMsg0: public CClassMessage
{
public:
    enum eProperties
    {
        PRPT_Prefix = 0,
        PRPT_Name,
        ePRPT_END
    };
    
       virtual std::size_t PRPT_END() const {
        return CMsg0::ePRPT_END;    
    };
};

class CMsg1: public CClassMessage
{
public:
    enum eProperties
    {
        PRPT_Prefix = 0,
        PRPT_State,
        PRPT_Protocol,
        ePRPT_END
    };
    
    virtual std::size_t PRPT_END() const {
        return CMsg1::ePRPT_END;    
    };
};

int main() {
    std::vector<std::string> vec0 = { "prefix", "name"  };
    std::vector<std::string> vec1 = { "prefix", "state", "protocol" };
    
    CClassMessage *pMsg0 = new CMsg0();
    pMsg0->parseLine( vec0 );
    
    CClassMessage *pMsg1 = new CMsg1();
    pMsg1->parseLine( vec1 );
    std::cout << pMsg0->m_properties.size() << std::endl;
    std::cout << pMsg1->m_properties.size() << std::endl;
    
    std::copy(
        std::begin(pMsg0->m_properties),
        std::end(pMsg0->m_properties),
        std::ostream_iterator<std::string>(std::cout, ",")
    );
    std::cout << std::endl;
    std::copy(
        std::begin(pMsg1->m_properties),
        std::end(pMsg1->m_properties),
        std::ostream_iterator<std::string>(std::cout, ",")
    );
    
    return 0;
};

[AD]

А зачем дважды слова public? Одного раза, на мой взгляд, достаточно! )))

[lanz]

Одного раза, на мой взгляд, достаточно!

Это паттерн "паблик морозов" Для отладки запилил, да и забыл поменять.
Ну и вообще иногда использую для логического разделения и чтобы не забываить что все еще паблик. Например конструкторы под своим пабликом потом интерфейс 1, потом второй и т.п.