4

私は gameinsitute の C++ プログラミング コースを受講しており、演算子のオーバーロードの例があり、常に

main.cpp|20|エラー: 'v + w' の 'operator+' に一致しません</p>

問題がどこにあるのかわかりません。

main.cpp

// main.cpp

#include "Vector3.h"
#include <iostream>

using namespace std;

int main()
{
    float coords[3] = {1.0f, 2.0f, 3.0f};
    Vector3 u;
    Vector3 v(coords);
    Vector3 w(-5.0f, 2.0f, 0.0f);

    cout << "u = ";    u.print();
    cout << "v = ";    v.print();
    cout << "w = ";    w.print();
    cout << endl;

    u = v + w; // this gives the error
    cout << "v + w = ";
    u.print();
    cout << endl;

    v.normalize();
    cout << "unit v = ";
    v.print();
    cout << "v.length() = "<< v.length() << endl;
    cout << endl;

    float dotP = u * w; // this also gives error
    cout << "u * w = " << dotP << endl;

    float* vArray = v.toFloatArray();

    cout <<
          "[0] = " << vArray[0] << ", "
          "[1] = " << vArray[1] << ", "
          "[2] = " << vArray[2] << endl <<endl;

    cout << "Input vector..." << endl;
    Vector3 m;
    m.input();
    cout << "m = ";
    m.print();

    return 0;
}

Vector3.h

#ifndef VECTOR3_H
#define VECTOR3_H

#include <iostream>

class Vector3
{
public:

    // constructors
    Vector3();
    Vector3(float coords[3]);
    Vector3(float x, float y, float z);
    Vector3(const Vector3& vec);

    // methods
    float  length();
    void    normalize();
    float* toFloatArray();
    void    print();
    void    input();

    // operators
    Vector3 operator=(const Vector3& rhs);
    Vector3 operator+(const Vector3& rhs) const;
    Vector3 operator-(const Vector3& rhs) const;
    float  operator*(const Vector3& rhs) const;
    Vector3 operator*(float scalar) const;

    // fields
    float mX;
    float mY;
    float mZ;
};

#endif // VECTOR3_H

Vector3.cpp

#include "Vector3.h"
#include <cmath>
#include <iostream>

using std::cout;
using std::cin;

Vector3::Vector3()
{
    mX = 0.0f;
    mY = 0.0f;
    mZ = 0.0f;
}

Vector3::Vector3(float coords[3])
{
    mX = coords[0];
    mY = coords[1];
    mZ = coords[2];
}

Vector3::Vector3(float x, float y, float z)
{
    mX = x;
    mY = y;
    mZ = z;
}

Vector3::Vector3(const Vector3& vec)
{
    mX = vec.mX;
    mY = vec.mY;
    mZ = vec.mZ;
}

float Vector3::length()
{
    return sqrt(mX*mX + mY*mY + mZ*mZ);
}

void Vector3::normalize()
{
    float len = length();
    mX /= len;
    mY /= len;
    mZ /= len;
}

float* Vector3::toFloatArray()
{
    return &mX;
}

void Vector3::print()
{
    cout << "<" << mX << ", " << mY << ", " << mZ << "> \n";
}

void Vector3::input()
{
    cout << "Enter x: ";
    cin >> mX;
    cout << "Enter y: ";
    cin >> mY;
    cout << "Enter z: ";
    cin >> mZ;
}

//operators

Vector3 Vector3::operator=(const Vector3& rhs)
{
    Vector3 vTemp;
    vTemp.mX = rhs.mX;
    vTemp.mY = rhs.mY;
    vTemp.mZ = rhs.mZ;

    return vTemp;
}

Vector3 Vector3::operator+(const Vector3& rhs) const
{
    Vector3 sum;
    sum.mX = mX + rhs.mX;
    sum.mY = mY + rhs.mY;
    sum.mZ = mZ + rhs.mZ;

    return sum;
}

Vector3 Vector3::operator-(const Vector3& rhs) const
{
    Vector3 dif;
    dif.mX = mX - rhs.mX;
    dif.mY = mY - rhs.mY;
    dif.mZ = mZ - rhs.mZ;

    return dif;
}

float Vector3::operator*(const Vector3& rhs) const
{
    float dotP = mX*rhs.mX + mY*rhs.mY + mZ*rhs.mZ;

    return dotP;
}

Vector3 Vector3::operator*(float scalar) const
{
    Vector3 p;
    p.mX = mX * scalar;
    p.mY = mY * scalar;
    p.mZ = mZ * scalar;

    return p;
}

事前に助けてくれてありがとう！

4

2 に答える 2

1

あなたには何も悪いことはありませんoperator+- それはあなたが間違っていて、あなたがしていると思っていることをoperator=しないという結果につながるのです. operator+それを修正すると、問題が解決されます。

Vector3 Vector3::operator=(const Vector3& rhs)
{
    mX = rhs.mX;
    mY = rhs.mY;
    mZ = rhs.mZ;
    return *this;
}
于 2012-07-25T13:53:11.633 に答える
-1

余談ですが、これらのオーバーロードの一部には問題があり、ローカル参照を返しています。申し訳ありませんが、これは答えではなく、反対票を投じる可能性がありますが、それでも：

たとえば、operator= を考えてみましょう。そのはず

Vector3& Vector3::operator=(const Vector3& rhs)
{
  this->mX = rhs.mX;
  this->mY = rhs.mY;
  this->mZ = rhs.mZ;
  return (*this);
}

そうしないと、この関数が終了するとガベージになるローカル参照が返されます。演算子 +- にも同じ問題があります。operator* に関しては、内積を行っていますが、これはベクトルの乗算とは異なります。そのために、別のベクトルを取り、自己と 2 番目のベクトルの間の内積を実行する dot() という関数を作成することをお勧めします。

または、 Eigen ライブラリを使用する必要があります

コードのコンパイル方法を教えていただければ、お手伝いできるかもしれません

于 2012-07-25T13:53:03.320 に答える