C++ доступ к значению вектора <int>, который находится внутри класса

Question

У меня есть следующий класс:

class clsUnitArrayIndexToUnitID : public CBaseStructure 
{ 
    private: 
     std::vector<int> m_content; 
     long m_size; 
    protected: 
     void ProcessTxtLine(string line); 
    public: 
     clsUnitArrayIndexToUnitID(); 
     std::vector<int>* Content; 
     long Size(); 
}; 

Я хотел бы получить доступ к значениям из вне класса, например, так:

int iUnitID = m_MyClass.Content()[12]; 

Однако C++ говорит мне, что мне нужно использовать тип точки-функции. Я не уверен, что это точно.

Кроме того, если кто-то видит какой-либо недостаток в моем коде, скажите, пожалуйста.

Answer 1

Изменить его функции (настроить const для ваших нужд)

public: 
    const std::vector<int>& Content() const { return m_content; } 

и использовать, как вы описали, или разыменования указателя (небезопасно?) В любом случае:

m_MyClass.Content->at(12); 
(*m_MyClass.Content).at(12); 
(*m_MyClass.Content)[12]; 

Answer 2

Изменить определение следующим образом:

const std::vector<int>* Content() const { 
     return &m_content; 
} 

Первый сопзЬ ключевое слово не является необходимым, но второй один указывает, что метод не должен и не модифицирует экземпляр.

Answer 3

Вместо того, выставляя вектор, вы можете предоставить операторам с индексированным доступом вашего класса:

class clsUnitArrayIndexToUnitID : public CBaseStructure 
{ 
public: 
    int& operator[](unsigned int i) {return m_content[i];} 
    const int& operator[](unsigned int i) const {return m_content[i];} 
.... 
}; 

th ан

int iUnitID = m_MyClass[12]; 

Answer 4

Обеспечить способ обертки для извлечения содержимого или оператора перегрузки [], как предложенное juanchopanza.

class clsUnitArrayIndexToUnitID : public CBaseStructure 
{ 
    private: 
     std::vector<int> m_content; 
     long m_size; 
    protected: 
     void ProcessTxtLine(string line); 
    public: 
     clsUnitArrayIndexToUnitID(); 
     //std::vector<int>* Content; 

     int GetContent(int index) { 
     int retValue = INVALID_CONTENT 
     if(index >=0 && index<m_content.size()) 
      retValue = m_content[index]; 
     return retValue; 
     }   

     long Size(); 
}; 

И использовать его как

int iUnitID = m_MyClass.GetContent(12); 

Answer 5

Чтобы ответить на конкретный вопрос, вы должны сделать это:

(*m_MyClass.Content)[12] 

Или это:

m_MyClass.Content->at(12); //note that the semantics is slightly different - at() checks boundaries, [] does not 

Однако дело вы должны действительно сделать, вероятно, подумайте, что вы r дизайн. Почему у вашего класса есть открытый элемент данных указателя типа на контейнер? Если это всегда будет указывать на m_content, вы можете также сделать m_content общественностью и сэкономить на хлопот.

Общая идея частных членов данных заключается в предоставлении доступа на основе семантики, так что ваша функция будет публиковать функции, такие как «вставить в вектор», «получить элемент из вектора» и т. Д., Поскольку его семантика диктует. Или, например, вы можете предоставить функцию доступа только для чтения, возвращающую const std::vector<int>&, и оставить доступ к модификации специальным функциям-членам.

Если вы предоставляете обертку, дающую прямой неконстантный доступ, это вряд ли обертка (она всегда должна быть подкреплена чем-то).

Конечно, приведенное выше обсуждение применяется, если Content на самом деле является указателем на m_content, а не только указателем на другой вектор. Но в таком случае класс будет еще более запутанным, я думаю.

Answer 6

Вы можете вернуть ссылку вместо указателя:

std::vector<int> m_content(UNITS_MAX_SIZE); 

const std::vector<int> &clsUnitsUnitIDToArrayIndex::Content() const 
{ 
    return m_content; 
}