c++ - Visual Studio 2010 列表复制构造抛出 C2664 错误

Question

已解决:重新启动 Visual Studio

我正在为学校开展一个涉及 STL 列表的项目。并通过 xmemory 获取此错误。此时我正试图构建解决方案，但xmemory 正在杀死我

Error 1 error C2664: 'GroceryStoreItem::GroceryStoreItem(GroceryStoreItem &)' : cannot convert parameter 1 from 'std::string' to 'GroceryStoreItem &' d:\microsoft visual studio 10.0\vc\include\xmemory 208

这是我的标题

#include <string>
#include <sstream>
#include<iostream>
#include <iterator>
#include <list>
using namespace std;
//
//*****************************************************************
//                     USER DEFINED DATA TYPES
//

class GroceryStoreItem
{
friend ostream & operator<< (ostream &out, const GroceryStoreItem &RHS);

public:
GroceryStoreItem();
GroceryStoreItem(string Name, double cost, string location);
GroceryStoreItem(GroceryStoreItem & GroceryStoreItemCCIn);
GroceryStoreItem & operator= (const GroceryStoreItem &RHS);
string ReturnItemName();
string ReturnLocation();
double ReturnCost();

private:
string ItemName;
string Location;
double Cost;
};

和实现

#include "Grocery_Item.h"

using namespace std;
//*****************************************************************
//                  Grocery Item Constructors
//*****************************************************************
GroceryStoreItem::GroceryStoreItem()
{
ItemName = "default";
Location = "aisle 1";
Cost = 0.0;
}

GroceryStoreItem::GroceryStoreItem(string InName, double InCost, string InLocation)
{
ItemName = InName;
Location = InLocation;
if(InCost >= 0.0f)
{
Cost = InCost;
}
else
{
Cost = 0.0f;
}

 }

 GroceryStoreItem::GroceryStoreItem(GroceryStoreItem & GroceryStoreItemCCIn)       //Copy Constructor
{
ItemName=GroceryStoreItemCCIn.ItemName;
Location=GroceryStoreItemCCIn.Location;
Cost=GroceryStoreItemCCIn.Cost;
}

编辑 xmemory最后一行错误

    template<class _Other>
    void construct(pointer _Ptr, _Other&& _Val)
    {   // construct object at _Ptr with value _Val
    ::new ((void _FARQ *)_Ptr) _Ty(_STD forward<_Other>(_Val));

Answer 1

你需要让你的复制构造函数参数为const

GroceryStoreItem::GroceryStoreItem(const GroceryStoreItem& GroceryStoreItemCCIn)       

此外，在复制构造函数中使用初始化而不是赋值通常更好

GroceryStoreItem::GroceryStoreItem(const GroceryStoreItem& rhs) :
    ItemName(rhs.ItemName),
    Location(rhs.Location),
    Cost(rhs.Cost)
{
}

最后(这是最重要的一课)因为你做了正确的事情并且在你的类内部使用了 std::string 你实际上根本不需要复制构造函数.编译器生成的默认值无论如何都会做正确的事情。所以我实际上会删除你的复制构造函数，这也会修复错误。

赋值运算符的参数相同，也将其删除。