Оператор C ++ >> проблема с перегрузкой: изменение объекта только один раз - PullRequest
Новая
       39

Оператор C ++ >> проблема с перегрузкой: изменение объекта только один раз

0 голосов
/ 16 декабря 2018

У меня есть следующий код Vector3D, и по какой-то причине cin может изменить значения объекта только один раз.у тебя есть идеи почему?Я думаю, это связано с определением «const», но я не уверен.добавление как кода (частичного), так и теста 

// code:
class Vector3D
{
private:
    double _x, _y, _z;
public:
    Vector3D() : _x(0), _y(0), _z(0) {};
    Vector3D(double x, double y, double z) : _x(x), _y(y), _z(z) {};
    Vector3D(const double parm[DIMENSION]) : _x(parm[0]), _y(parm[1]), 
             _z(parm[2]) {};

    const Vector3D operator+(const Vector3D &other) const; 
    Vector3D &operator+=(const Vector3D &other);
    const double &operator[](int idx) const;
    double &operator[](int idx);
    friend ostream &operator<<(ostream &out, const Vector3D &c);
    friend istream &operator>>(istream &in, Vector3D &c);
};

const Vector3D Vector3D::operator+(const Vector3D &other) const {
    return Vector3D(*this) += other; 
}

Vector3D &Vector3D::operator+=(const Vector3D &other) {
    (*this)._x += other._x;
    (*this)._y += other._y;
    (*this)._z += other._z;
    return *this; 
}

const double &Vector3D::operator[](int idx) const {
    assert(idx >= 0 && idx < DIMENSION);
    switch (idx)
    {
        case 0:
            return _x;
        case 1:
            return _y;
        case 2:
            return _z;
        default:
            return _x; // not reachable
    }
}

double &Vector3D::operator[](int idx)
{
    assert(idx >= 0 && idx < DIMENSION);
    switch (idx)
    {
        case 0:
            return _x;
        case 1:
            return _y;
        case 2:
            return _z;
        default:
            return _x; // not reachable
    }
}

ostream &operator<<(ostream &out, const Vector3D &c)
{
    out << c._x << " " << c._y << " " << c._z;
    return out;
}

istream &operator>>(istream &in, Vector3D &c)
{
    in >> c._x >> c._y >> c._z;
    return in;
}



// test: 
void readFromStreamTest(int &tests) {
    std::stringstream os;
    os << 8.0 << " " << 3.0 << " " << 4.0;
    Vector3D a;
    os >> a;
    os << a; // "8 3 4"  as should be
    os.str(std::string());
    os << 2.1 << " " << 3.1 << " " << 4.1;
    os >> a;
    os.str(std::string());
    os << a;
    if (os.str()!=("2.1 3.1 4.1"))
    {
        std::cerr << "failed. should be 2.1 3.1 4.1 but was " << a;
    }
}

Результаты теста: «не удалось. должно быть 2,1 3,1 4,1, но было 8 3 4» (старые значения !!)

1 Ответ

0 голосов
/ 16 декабря 2018

Вы не можете использовать stringstream подобным образом, после извлечения содержимого throw os >> вы не сможете снова изменить его на os.str(value) или os <<, если вы ранее не вызывали os.clear(), дляэкземпляр 

void readFromStreamTest(int &tests) {
  std::stringstream os;
  os << 8.0 << " " << 3.0 << " " << 4.0;
  Vector3D a;
  os >> a;
  os.clear();
  os << a; // "8 3 4"  as should be
  os.str(std::string());
  os << 2.1 << " " << 3.1 << " " << 4.1;
  os >> a;
  os.str(std::string());
  os.clear();
  os << a;
  if (os.str()!=("2.1 3.1 4.1"))
  {
    std::cerr << "failed. should be 2.1 3.1 4.1 but was " << a;
  }
}

PS Я изменил ваш исходный код, чтобы сделать его компилируемым

...