Конструктор шаг

Question

У меня есть код:

#include "stdafx.h" 
#include "memory" 
#include <gtest\gtest.h> 
class Money 
{ 
public: 
    explicit Money(int value) :value(value) {} ; 
    Money(Money&& m) :value(m.returnValue()) {}; 
    Money(const Money &m) = default; 
    Money operator-(const Money &m) ; 
    Money &operator=(Money &&m) { return Money(m.returnValue()); }; 
    Money &operator=(const Money &m)=default; 
    int returnValue() const { return value; }; 
    ~Money() = default; 
private: 
    int value; 
}; 
Money Money::operator-(const Money &m) 
{ 
    return Money(value - m.returnValue()); 
} 


class Bank { 
public: 
    Bank(Money m) :propertiesBank(std::make_unique<PropertiesBank>(std::move(m))) {}; 
    int returnMoney() const { return propertiesBank->money->returnValue(); } 
    ~Bank() = default; 
private: 
    struct PropertiesBank; 
    std::unique_ptr<PropertiesBank> propertiesBank; 
}; 

struct Bank::PropertiesBank 
{ 
    std::shared_ptr<Money> money; 
    PropertiesBank(Money&& m) :money(std::make_shared<Money>(m)) {}; 
}; 
int main() 
{ 
    Money k(1000); 
    Bank bank(k); 
    return 0; 
} 

Я хочу отображения (returnMoney)() деньги в банке, но я не могу. Я мог бы создать класс со структурой Impl и unique_ptr для обучения. Я знаю, что уникальный не может копировать. Как я могу сделать эту программу? Остальная часть моего кода хорошо?

Ошибка

C2027 Ошибка использование неопределенного типа Error 'Банк :: PropertiesBank'
C2039 'ReturnValue': не является членом 'STD :: unique_ptr>'

Answer 1

Its не проблема с std::unique_ptr, ее факт, что вы попытались обратиться к члену объекта, тип которого PropertiesBank, когда компилятор не видел его полного определения. Вы должны переместить определение функции члена вне класса и в точке, где компилятор видел полное определение PropertiesBank:

Посмотреть комментарий в этом фрагменте ниже:

class Bank { 
public: 
    Bank(Money m) :propertiesBank(std::make_unique<PropertiesBank>(std::move(m))) {}; 

    int returnMoney() const;{ return propertiesBank->money->returnValue(); } 
    // .......The compiler doesn't know that `money`^^^^^^ belongs to `PropertiesBank` 

    ~Bank() = default; 
private: 
    struct PropertiesBank; 
    std::unique_ptr<PropertiesBank> propertiesBank; 
}; 

struct Bank::PropertiesBank 
{ 
    std::shared_ptr<Money> money; 
    PropertiesBank(Money&& m) :money(std::make_shared<Money>(m)) {}; 
}; 

Вы должны переместить определение функции к после где компилятор видел определение типа из propertiesBank:

class Bank { 
public: 
    Bank(Money m) :propertiesBank(std::make_unique<PropertiesBank>(std::move(m))) {}; 
    int returnMoney() const; //member function declaration 
    ~Bank() = default; 
private: 
    struct PropertiesBank; 
    std::unique_ptr<PropertiesBank> propertiesBank; 
}; 

struct Bank::PropertiesBank 
{ 
    std::shared_ptr<Money> money; 
    PropertiesBank(Money&& m) :money(std::make_shared<Money>(m)) {}; 
}; 

//Member function's definition 
int Bank::returnMoney() const { return propertiesBank->money->returnValue(); } 

Answer 2

Единственная проблема, которую я вижу в том, что определение Bank::returnMoney пытается получить доступ к Bank::PropertiesBank, если он только был объявлен вперед, а не определен. Перемещение PropertiesBank будет определено наклон в пределах Bank.

Однако, как мычание Duck указывает в комментариях, если ваше намерение осуществить pImpl idiom, то оба Bank::PropertiesBank и Bank::returnMoney должны быть определены в файле .cpp, а не в определении класса.

#include <memory> 

class Money 
{ 
public: 
    explicit Money(int value) :value(value) {} ; 
    Money(Money&& m) :value(m.returnValue()) {}; 
    Money(const Money &m) = default; 
    Money operator-(const Money &m) ; 
    Money operator==(Money &&m) { return Money(m.returnValue()); }; 
    int returnValue() const { return value; }; 
    ~Money() = default; 
private: 
    int value; 
}; 

Money Money::operator-(const Money &m) 
{ 
    return Money(value - m.returnValue()); 
} 

class Bank { 
public: 
    Bank(Money m) :propertiesBank(std::make_unique<PropertiesBank>(std::move(m))) {}; 
    int returnMoney() const { return propertiesBank->money->returnValue(); } 
    ~Bank() = default; 
private: 
    struct PropertiesBank 
    { 
     std::shared_ptr<Money> money; 
     int returnMoney() const { return money->returnValue(); } 
     PropertiesBank(Money&& m) :money(std::make_shared<Money>(m)) {}; 
    }; 

    std::unique_ptr<PropertiesBank> propertiesBank; 
}; 

#include <iostream> 

int main() 
{ 
    Money m(10); 
    Bank b(m); 

    std::cout << b.returnMoney(); 
    return 0; 
}