我正在尝试使用下例中的unique_ptr来make_pair
(该代码是Facade Design模式的C++实现)
#include <unordered_map>
#include <utility>
#include <functional>
using namespace std;
// Step 1 : Design the Interface
class IAccount {
protected:
int accountNo;
int cash;
public:
virtual void deposit(float amount);
virtual void withdraw(float amount);
virtual void transfer(float amount);
virtual int getAccountNumber() = 0;
};
// Step 2 : Implement the interface with one or more subtypes
class Current : public IAccount{
public:
Current(int amt) { cash = amt; }
int getAccountNumber() {
return accountNo;
}
};
class Savings : public IAccount{
public:
Savings(int amt) { cash = amt; }
int getAccountNumber() {
return accountNo;
}
};
// Step 3: Create the Facade class and wrap the classes that implement the Interface
class BankService {
unordered_map<int, unique_ptr<IAccount>> *bankAccounts;
int index;
public:
BankService(): index(0){
bankAccounts = new unordered_map<int, unique_ptr<IAccount>>();
}
int createNewAccount(string name, int amount, string type, int accountNo) {
unique_ptr<IAccount> newAccount;
if(type.compare("current")) {
newAccount.reset(new Current(amount));
}
// Add to the unordered list --- Error here ---
pair<int, unique_ptr<IAccount>> toBeInserted = make_pair(accountNo, newAccount);
bankAccounts->insert(toBeInserted);
return accountNo;
}
};
代码吐出此错误:
/Library/Developer/CommandLineTools/usr/include/c++/v1/vector:266:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/__bit_reference:15:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/algorithm:639:
/Library/Developer/CommandLineTools/usr/include/c++/v1/utility:634:12: error: no matching constructor for initialization of 'pair<typename
__make_pair_return<int &>::type, typename __make_pair_return<unique_ptr<IAccount, default_delete<IAccount> > &>::type>' (aka 'pair<int,
std::__1::unique_ptr<IAccount, std::__1::default_delete<IAccount> > >')
return pair<typename __make_pair_return<_T1>::type, typename __make_pair_return<_T2>::type>
^
如果我将其更改为:
// Add to the unordered list (note added const)
pair<const int, unique_ptr<IAccount>> toBeInserted = make_pair(accountNo, newAccount);
bankAccounts->insert(toBeInserted);
我受到以下欢迎:
In file included from ./DesignPatterns/Structural/./../../Include/Common.h:1:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/vector:266:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/__bit_reference:15:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/algorithm:640:
/Library/Developer/CommandLineTools/usr/include/c++/v1/memory:1783:31: error: call to implicitly-deleted copy constructor of 'std::__1::pair<const
int, std::__1::unique_ptr<IAccount, std::__1::default_delete<IAccount> > >'
::new((void*)__p) _Up(_VSTD::forward<_Args>(__args)...);
我尝试使用Clang++和G++进行编译
lang铛版本:clang-900.0.39.2
G++版本:4.2.1
平台:OS X-x86_64-apple-darwin16.6.0
这是由于Apple发行的旧版G++吗?
最佳答案
不。这是因为您试图通过将唯一指针传递给make_pair
和insert
来复制唯一指针。std::unique_ptr
不可复制。实际上,它是仅移动的。改成:
pair<int, unique_ptr<IAccount>> toBeInserted = make_pair(accountNo, std::move(newAccount));
bankAccounts->insert(std::move(toBeInserted));
关于c++ - 用unique_ptr配对的正确方法是什么?,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/51116065/