错误C ++运算符重载

Question

我正在编写一个使用向量并对其进行修改的程序，但是它给了我错误： “变量'Vector'不是类型名称”

我是C ++重载运算符的新手，因此它们可能有很多语法错误

这是什么问题，是在函数声明还是在实现中

另一件事，何时将const与参数一起使用？

class Vector{
private:
    int n;
    int *vArray;
public:
    Vector();
    Vector(int);
    ~Vector();
    friend istream& operator>>(istream& is , Vector &v );
    friend ostream& operator>>(ostream& os  , const Vector &v);
    Vector& operator+=(const Vector& );
    Vector operator+(Vector& rhs);
    double operator*(Vector&);
    Vector operator-(Vector);

};

Vector::Vector(){
    n = 2;
    vArray = new int[n];

}


Vector::Vector(int num){
    n = num;
    vArray = new int[n];
}

Vector::~Vector(){
    delete [] vArray;


}


istream& operator>>(istream& is, Vector& v){
    for (int i = 0; i < v.n; i++)
        is >> v.vArray[i];
    return is;

}
ostream& operator>>(ostream& os, const Vector& v){
    for (int i = 0; i < v.n; i++)
        os >> v.vArray[i];
    return os;


}

Vector& :: Vector operator+=(const Vector& rhs){
    if(n == rhs.n){
        for (int i = 0; i < n; i++){
            this->vArray[i] += rhs.vArray[i];
        }
        return *this;
    }
    cout << "Error: size not match" << endl;
}

Vector:: Vector operator+( Vector& rhs){
    if (n == rhs.n){
        Vector z(n);
        for (int i = 0; i < n; i++)
            z.vArray[i] = this->vArray[i] + rhs.vArray[i];

       return z;
    }
    cout << "Error: size not match"<<endl;


}


double::Vector operator* (Vector& v){
    double count;
    if (n == v.n){
        for (int i = 0; i <n; i++){
            count += this->vArray[i] * v.vArray[i];
        }
        return count;

    }
    cout << "Error: size not match";

}


Vector::Vector operator-(Vector v){
    if (n == v.n){
        Vector z(n);
    for (int i = 0; i < n; i++){
        z.vArray[i]=this->vArray[i]-v.vArray[i];

    }
    return z;
}
    cout << "Error: size not match" << endl;

}

Answer 1

您在以下几行中有一些错误：

Vector& :: Vector operator+=(const Vector& rhs){
Vector& :: Vector operator+=(const Vector& rhs){
Vector:: Vector operator+( Vector& rhs){
double::Vector operator* (Vector& v){
Vector::Vector operator-(Vector v){

这些应该是：

Vector& Vector::operator+=(const Vector& rhs){
Vector& Vector::operator+=(const Vector& rhs){
Vector Vector::operator+( Vector& rhs){
double Vector::operator* (Vector& v){
Vector Vector::operator-(Vector v){

不是类型名称错误，可能是Vector::Vector错误，因为您应该返回类型并且不标识Vector的构造函数。

另外，您的向量类不会定义/删除复制构造函数或赋值运算符。 如果您复制此类的实例，则它将使用默认的复制/赋值，该复制/赋值将仅复制指针成员的值而不执行深层复制-您将得到两次删除操作，这会使程序崩溃。

对于const参数问题。每当函数不修改传入的值时，都应使用const引用。同样，如果函数不修改类本身的对象，则成员函数也应为const。 例如，operator +应该接受const参数，并且本身也应为const：

Vector Vector::operator+(Vector const& rhs) const

优良作法是在任何可能的地方使用const-放宽限制比回顾性地添加限制要容易得多。

看看这个： http : //www.parashift.com/c++-faq/const-correctness.html