C++：复制构造函数

Question

我的 C++ 程序有问题。我尝试编写复制构造函数。一切都很好，直到我开始从用作引用的对象中复制数组中的元素。 我可以打印应用数组的每个元素，但我不能将它复制到复制构造函数将创建的对象中的数组。

这是代码（我只显示头文件和一些需要复制构造函数的方法，因为我认为所有代码都太长了）

头文件.h

#pragma once
# include <iostream> 
# include <fstream> 
# include <cmath>
using namespace std;

class Polynomial {
private: 
    int degree; 
    double* coefficients; 


public: 
    static const string errors[5];
    Polynomial(const Polynomial& wzor); // konstruktor kopiujacy
    Polynomial(float n);
    ~Polynomial();
    void setCoefficient(unsigned int i, double value); 
    double getCoefficient(unsigned int i); 
    double value(double x); 
    friend ostream & operator <<(ostream & out, Polynomial & p);
    double operator[](const int index);
    double operator()(double x);
    void operator=(Polynomial &obiekt);
    friend Polynomial operator+(Polynomial &obiekt,Polynomial &obiekt1);
    friend Polynomial operator-(Polynomial &obiekt,Polynomial &obiekt1);
    friend Polynomial operator~(Polynomial &obiekt);
    friend Polynomial operator*(Polynomial &obiekt,Polynomial &obiekt1);
    friend Polynomial operator*(Polynomial &obiekt,double x);
};

Polynomial::Polynomial(float n) {

    if(n<0){
        throw 0;
    }
    else if(floor(n)-n != 0){
        throw 2;
    }

    degree=(int)n;
    coefficients=new double [(int)n+1]; 
    for(unsigned int i=0; i<n; i++) { 
        coefficients[i]=0.0; 
    } 
    coefficients[(int)n]=1.0; 
}

Polynomial::Polynomial(const Polynomial &wzor){
    degree=wzor.degree; // it's allright 
    for(int i=0; i<=wzor.degree; i++){
        coefficients[i]=wzor.coefficients[i]; // compilator says that this line is wrong 
    }
}

Polynomial::~Polynomial(){

    delete coefficients;

}

Polynomial operator+(Polynomial &obiekt,Polynomial &obiekt1){
    Polynomial nowy(1);
    if(obiekt1.degree > obiekt.degree){
        nowy=obiekt1;
        for(int i=0; i<=obiekt.degree; i++){
            nowy.coefficients[i]=nowy.coefficients[i]+obiekt.coefficients[i];
        }
    }
    else{
        nowy=obiekt;
        for(int i=0; i<=obiekt1.degree; i++){
            nowy.coefficients[i]=nowy.coefficients[i]+obiekt1.coefficients[i];
        }
    }
    return nowy;
}

Answer 1

class Polynomial {
private: 
    int degree; 
    double* coefficients; // Pointer to double

    /* ... */
}

您已将 coefficients 声明为 指向双精度的指针，而不是数组。

for(int i = 0; i <= wzor.degree; i++) {
    coefficients[i] = wzor.coefficients[i]; // Error! No memory allocated.
}

在这里，您尝试将值分配给不存在的内存。您需要先分配数组，然后才能将元素数据分配给它。此外，您应该在 initialization list 而不是 ctor 主体中初始化成员。试试这样的：

Polynomial::Polynomial(const Polynomial& wzor)
    : degree(wzor.degree), coefficients(new double[wzor.degree + 1]) { // Init
    for (int i=0;i<=wzor.degree;i++) {
        coefficients[i]=wzor.coefficients[i];
    }
}

请记住，复制构造函数是一个构造函数。它将构造对象的新实例，您需要为新副本分配新内存。

在你的析构函数中你必须使用delete[] 而不是delete，因为你正在删除一个数组。

赋值运算符应声明为：

Polynomial& operator= (const Polynomial& obiekt);

对于存储数组索引的变量，您应该使用类型 std::size_t 而不是 int。

最后，我建议使用标准库中的容器而不是内置数组，例如std::vector.

Answer 2

#include <vector>

std::vector<double> coefficients; 

您可以使用向量的副本，这意味着您不必自己实现它。 而且您不必执行删除操作。

使用 resize & [] 或 push_back 方法处理字符串。你的学位将是 coefficients.size()。

Answer 3

你的赋值运算符是错误的。应该是

Polynomial & operator=(const Polynomial &obiekt);