Ломать циклическую зависимость в C++

Question

У меня проблема следующей циклической зависимости я пытаюсь решить:

typedef std::map<int, my_class> my_map; 

class my_class { 
... 
private: 
    my_map::iterator iter; 
}; 

class otherclass{ 
public: 
    my_map::iterator getIter(); 
private: 
    my_map map; 
}; 

компилятору не нравится это, поскольку my_class не был объявлен до ЬурейеГо.

если я пытаюсь передать, объявить MyClass следующим образом:

class my_class; 

typedef std::map<int, my_class> my_map; 

class my_class { 
... 
private: 
    my_map::iterator iter; 
}; 

class otherclass{ 
public: 
    my_map::iterator getIter(); 
private: 
    my_map map; 
}; 

я получаю «ошибка: опережающее объявление о„my_class“».

Как я могу разбить этот порочный круг?

---

Извините, но я должен пересмотреть свой вопрос, поскольку я заметил, что мое представление немного ошибочно.

Ниже правильное представление моей проблемы:

class my_container; 

typedef std::map<int, my_container> my_map; 

class my_class { 
... 
private: 
    my_map::iterator iter; 
}; 

class my_container { 
public: 
    my_class a_method(); 
private: 
    vector<my_class> v; 
}; 

class otherclass{ 
public: 
    my_map::iterator a_method(); 
    my_class another_method(); 
    my_container yet_another_method(); 
private: 
    my_map map; 
}; 

К сожалению об этом

Answer 1

class my_class; 

typedef std::map < int, my_class* > my_map; 
         ~~~~~~~~~~ use pointer here! 

Answer 2

поставил my_map в качестве члена в my_class, как это:

class my_class { 
public:  typedef std::map<int, my_class> my_map; 
... 
private: 
     my_map::iterator iter; 
}; 

class otherclass{ 
public: 
     my_class::my_map::iterator getIter(); 
private: 
     my_class::my_map map; 
}; 

Если вы не» t хочу всегда использовать my_class::, затем сделать еще один typedef.

Answer 3

Вы можете поместить ЬурейеЕ внутри my_class:

class my_class { 
public: 
    typedef std::map<int, my_class> my_map; 
... 
private: 
    my_map::iterator iter; 
}; 

class otherclass{ 
public: 
    my_class::my_map::iterator getIter(); 
private: 
    my_class::my_map map; 
}; 

Answer 4

Круговой зависимость, как правило, плохо. Можете ли вы немного подумать о своем дизайне? Нужно ли my_class знать, что такое его контейнер?

Если это не практично, вместо итератора в my_map вы могли бы уйти всего лишь с my_container*? У компилятора не будет проблемы с использованием неполного типа для указателя.

Answer 5

Как насчет:

#include <map> 

class my_class;       // forward declare the type. 
typedef std::map<int, my_class> my_map; 

class my_class 
{ 
    private: 
    my_map::iterator iter; 
}; 

class otherclass 
{ 
    public: 
    my_map::iterator getIter(); 
    private: 
    my_map map; 
};