コンストラクタは、私は、コードを持っている

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でクラスを作ることができました。 私は知っている、ユニークなコピーすることはできません。 どうすればこのプログラムを作成できますか？ 残りのコードは正常ですか？

未定義の型 '銀行:: PropertiesBank' エラー
C2039 'のreturnValue' の

エラーC2027を使用

エラー：のメンバーではありません 'のstd :: unique_ptrを>'

Answer 1

その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)) {}; 
}; 

あなたは移動する必要があります私が見ることができる唯一の問題は、012の定義ということです

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::PropertiesBankにアクセスしようとします。移動PropertiesBankはBankの範囲内で傾斜を定義してこれを修正します。 Mooingダックはコメントで指摘するように、あなたの意図が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; 
}