C ++类复制构造函数

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对象插入到环形缓冲区中。 这就是我在main中读回它的方式：

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

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

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

而且有效。 似乎没有更多的错误了。 最初我以为问题是由于复制或赋值运算符引起的，因为错误是断断续续的，所以我没有检查是否是导致错误的循环缓冲区。