ошибка компиляции

[Cfon]

есть след код, но не компилируется в чем проблема?

class VectorStrPtr;
 
class VectorStr
{  
    typedef std::vector<std::string> vecstr_t;
    friend class VectorStrPtr;
    std::shared_ptr<vecstr_t> sp_;      
public:
    VectorStr() : sp_(std::make_shared<vecstr_t>()) {}  
    VectorStrPtr begin() { return VectorStrPtr(*this); } //<--- тут ошибка компиляции
    //VectorStrPtr end() { return VectorStrPtr(*this, sp_->size()); }  
};
 
class VectorStrPtr
{
    typedef std::vector<std::string> vecstr_t;
public:
    VectorStrPtr() {}
    VectorStrPtr(VectorStr& vec, size_t i=0) : wp_(vec.sp_), idx_(i) {}
private:
    std::weak_ptr<vecstr_t> wp_;
    size_t idx_;
};

ошибка след: error: return type 'class VectorStrPtr' is incomplete
VectorStrPtr begin() { return VectorStrPtr(*this); }

Сообщение отредактировано: Cfon - 23.01.17, 10:56

[Dushevny]

Так вроде же все очевидно? Компилятор в этот момент ничего не знает о VectorStrPtr, который будет полностью объявлен позже.
Унесите определение этой функции вниз, после определения VectorStrPtr.

[Cfon]

Цитата Dushevny @ 23.01.17, 13:04

Так вроде же все очевидно? Компилятор в этот момент ничего не знает о VectorStrPtr, который будет полностью объявлен позже.
Унесите определение этой функции вниз, после определения VectorStrPtr.

если вниз перенести то идёт обратная ошибка: error: 'class VectorStr' is incomplete

Добавлено 23.01.17, 13:25
точнее: error: invalid use of incomplete type 'class VectorStr'

[shm]

Кривая архитектура классов.

Добавлено 23.01.17, 13:41
Как вариант (безе переделывания архитектуры):

#include <vector>
#include <string>
#include <memory>
 
class VectorStr;
 
class VectorStrPtr
{
    typedef std::vector<std::string> vecstr_t;
public:
    VectorStrPtr() {}
    VectorStrPtr(VectorStr& vec, size_t i=0);
private:
    std::weak_ptr<vecstr_t> wp_;
    size_t idx_;
};
 
class VectorStr
{  
    typedef std::vector<std::string> vecstr_t;
    friend class VectorStrPtr;
    std::shared_ptr<vecstr_t> sp_;      
public:
    VectorStr() : sp_(std::make_shared<vecstr_t>()) {}  
    VectorStrPtr begin() { return VectorStrPtr(*this); } //<--- тут ошибка компиляции
    //VectorStrPtr end() { return VectorStrPtr(*this, sp_->size()); }  
};
 
VectorStrPtr::VectorStrPtr(VectorStr& vec, size_t i) :
    wp_(vec.sp_), idx_(i)
{
    
}

[Cfon]

Цитата shm @ 23.01.17, 13:38

Кривая архитектура классов.

Как вариант (безе переделывания архитектуры):

спс коллега
вот теперь все пучком!

а в чем кривая?

[shm]

Цитата Cfon @ 23.01.17, 14:23

а в чем кривая?

Классы сильно связаны.

[Cfon]

это я переживу, как и старина Стэн Липман

[Dushevny]

Цитата Cfon @ 23.01.17, 13:22

если вниз перенести то идёт обратная ошибка: error: 'class VectorStr' is incomplete

Так не весь класс VectorStr переностите, а только определение функции-члена:

inline VectorStrPtr VectorStr::begin() { return VectorStrPtr(*this); }

Добавлено: или как shm предложил, но он inline забыл.

Сообщение отредактировано: Dushevny - 24.01.17, 06:58

[shm]

Я ничего не забыл, но предполагал, что ТС заюзает раздельную компиляцию.

[Cfon]

Цитата Dushevny @ 24.01.17, 06:57

Так не весь класс VectorStr переностите, а только определение функции-члена:

inline VectorStrPtr VectorStr::begin() { return VectorStrPtr(*this); }

о пардон моя не внимательность