Question

我的复制构造函数存在不稳定问题。

我有一个MyData类，如下所示：

class MyData
{
    private:
    std::vector<double> wavelength;
    std::vector<double> amplitude;

    public:
    MyData::MyData(void) {}

    MyData::MyData(const MyData &cSource)
      : wavelength(cSource.wavelength), amplitude(cSource.amplitude)
    {}
}

在我的主程序中，我将MyData对象插入环形缓冲区。这就是我在主要阅读它的方式：

MyData data;
data = removeq(&q);

问题是有时复制的数据缺少某些值。等等。如果波长的原始大小为1，则复制的数据显示为0.我已经调试了我的程序，并且环形缓冲区中的数据是正确的等等，它显示正确的大小为1.

任何人都知道我的复制构造函数是错误的还是我需要赋值运算符重载？

谢谢！

我用于插入/删除环形缓冲区的代码：

void insertq(struct queue *p, MyData v)
{
    int t;
    t = (p->rear+1)%MAX;
    if(t == p->front)
    {   }
    else
    {
            p->rear = t;        
            p->arr[p->rear] = v;
    }
}
MyData removeq(struct queue *p)
{
    MyData empty;   

    if(isempty(p))
    {               
        return empty;
    }
    else
    {       
        p->front = (p->front + 1)%MAX;
        empty = p->arr[p->front];
        return empty;
    }
 }

Answer 1

为了调用复制构造函数，您必须在同一行中声明和初始化对象（使用另一个对象）。您当前的代码实际上调用了overloaded =运算符。

Answer 2

从现有对象创建新对象时，将调用复制构造函数。在这里，您正在调用赋值运算符：

MyData data;
data = removeq(&q);

如果您使用过

 Data oldDataObject;
 Data newDataObject = oldDataObject;

将调用复制构造函数。

在代码的上下文中，您应该覆盖'='运算符来解决此问题。

MyData& operator = (const MyData& data);

Answer 3

感谢大家的建议。我已按照建议删除了复制构造函数和赋值重载方法。

问题在于我使用的循环缓冲结构。我在这里将循环缓冲区代码更改为此示例：

http://www.codeproject.com/Articles/43510/Lock-Free-Single-Producer-Single-Consumer-Circular

它有效。似乎没有更多的错误。我最初认为问题是由于复制或赋值操作符，因为错误是间歇性的，所以我没有检查是否是导致错误的循环缓冲区。

C ++类复制构造函数

3 个答案: