DaniWeb Forum Index > Software Development > C++

C++ RSS

My class is not a modifiable lvalue?

Please support our C++ advertiser: Intel Parallel Studio Home

•

Join Date: Feb 2008

Posts: 630

Reputation: daviddoria is a jewel in the rough

Solved Threads: 46

daviddoria daviddoria is offline

Offline

Practically a Master Poster

My class is not a modifiable lvalue?

Aug 14th, 2008

I have a very simple class

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
class Point
{
 
public:
Point();
Point(double x, double y);
  double x_;
double y_;
}
class Point
{

public:
Point();
Point(double x, double y);
  double x_;
double y_;
}

the problem is, I can't assign my class to another instance, ie:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
Point A(2,4);
Point B();
B = A;
Point A(2,4);
Point B();
B = A;

It says the lvalue must be modifiable. Why is it not?

Thanks,

Dave

•

Join Date: Mar 2008

Posts: 1,431

Reputation: William Hemsworth has much to be proud of

Solved Threads: 118

William Hemsworth William Hemsworth is offline

Offline

Nearly a Posting Virtuoso

Re: My class is not a modifiable lvalue?

Aug 14th, 2008

Try this.

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
class Point {
public:
 
   double x;
   double y;
 
   Point() {
      x = 0;
      y = 0;
   }
 
   Point(double _x, double _y);
      x = _x;
      y = _y;
   }
 
   Point &operator =(const Point &p) {
      x = p.x;
      y = p.y;
      return *this;
   }
};
class Point {
public:

   double x;
   double y;

   Point() {
      x = 0;
      y = 0;
   }

   Point(double _x, double _y);
      x = _x;
      y = _y;
   }

   Point &operator =(const Point &p) {
      x = p.x;
      y = p.y;
      return *this;
   }
};

•

Join Date: Dec 2006

Posts: 1,089

Reputation: vijayan121 is a name known to all

Solved Threads: 164

vijayan121 vijayan121 is offline

Offline

Veteran Poster

Re: My class is not a modifiable lvalue?

Aug 14th, 2008

Point B(); is the declaration of a function called 'B' which takes no arguments and returns a Point.
modify to Point B ; and you have the definition of a variable called 'B' which is of type Point (initialized via the default constructor).

Last edited by vijayan121; Aug 14th, 2008 at 5:08 pm.

•

Join Date: Feb 2008

Posts: 630

Reputation: daviddoria is a jewel in the rough

Solved Threads: 46

daviddoria daviddoria is offline

Offline

Practically a Master Poster

Re: My class is not a modifiable lvalue?

Aug 14th, 2008

ahhh that was the key ! Needed to use B instead of B()

Thanks!

•

Join Date: Aug 2008

Posts: 11

Reputation: Narf!!! is an unknown quantity at this point

Solved Threads: 1

Narf!!!

Offline

Newbie Poster

Re: My class is not a modifiable lvalue?

Aug 15th, 2008

•

Originally Posted by vijayan121

Point B(); is the declaration of a function called 'B' which takes no arguments and returns a Point.
modify to Point B ; and you have the definition of a variable called 'B' which is of type Point (initialized via the default constructor).

Are you completely sure about this?
As far as i know if you have declared the Point class' constructor Point() and then you state Point B(); then B is an instance of Point instantiated via the default constructor.
I also thought that that if you declare the copy constructor the assignation B = A should propery work

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
 
class Point{
private:
  int x_;
  int y_;
public:
  Point(){/* whatever*/}
  Point(int x,int y){/* whatever*/}
  Point(const Point& p){ 
     x_ = p.x_;
     y_ = p.y_;
  }
};
class Point{
private:
  int x_;
  int y_;
public:
  Point(){/* whatever*/}
  Point(int x,int y){/* whatever*/}
  Point(const Point& p){ 
     x_ = p.x_;
     y_ = p.y_;
  }
};

I consider myself a begginer and I learn as I go. Then this you posted doesn't seems to be correct for me. Pleas correct me.
Kind regards

•

Join Date: Jan 2008

Posts: 3,819

Reputation: VernonDozier has a reputation beyond repute

Solved Threads: 501

VernonDozier VernonDozier is offline

Offline

Senior Poster

Re: My class is not a modifiable lvalue?

Aug 15th, 2008

•

Originally Posted by Narf!!!

Are you completely sure about this?
As far as i know if you have declared the Point class' constructor Point() and then you state Point B(); then B is an instance of Point instantiated via the default constructor.
I also thought that that if you declare the copy constructor the assignation B = A should propery work
Help with Code Tags

C++ Syntax (Toggle Plain Text)
 
class Point{
private:
  int x_;
  int y_;
public:
  Point(){/* whatever*/}
  Point(int x,int y){/* whatever*/}
  Point(const Point& p){ 
     x_ = p.x_;
     y_ = p.y_;
  }
};
class Point{ private: int x_; int y_; public: Point(){/* whatever*/} Point(int x,int y){/* whatever*/} Point(const Point& p){ x_ = p.x_; y_ = p.y_; } };
I consider myself a begginer and I learn as I go. Then this you posted doesn't seems to be correct for me. Pleas correct me.
Kind regards

Yes, vijayan121 is correct. Try this program below and see what happens:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include <iostream>
using namespace std;
 
class Point {
public:
 
   double x;
   double y;
 
   Point() 
   {
      cout << "I'm in the constructor\n";
      x = 0;
      y = 0;
   }
 
   Point(double _x, double _y)
   {
      x = _x;
      y = _y;
   }
 
   Point &operator =(const Point &p) 
   {
      x = p.x;
      y = p.y;
      return *this;
   }
};
 
 
 
int main ()
{
    Point B();  // nothing displayed
    Point C; // constructor call
    B();     // function call
    Point D (); // nothing displayed
    return 0;
}
 
 
Point B ()
{
      cout << "I'm in the function\n";
      Point* A = new Point ();  // constructor call
      return *A;
}
#include <iostream>
using namespace std;

class Point {
public:

   double x;
   double y;

   Point() 
   {
      cout << "I'm in the constructor\n";
      x = 0;
      y = 0;
   }

   Point(double _x, double _y)
   {
      x = _x;
      y = _y;
   }

   Point &operator =(const Point &p) 
   {
      x = p.x;
      y = p.y;
      return *this;
   }
};



int main ()
{
    Point B();  // nothing displayed
    Point C; // constructor call
    B();     // function call
    Point D (); // nothing displayed
    return 0;
}


Point B ()
{
      cout << "I'm in the function\n";
      Point* A = new Point ();  // constructor call
      return *A;
}