Question

Pretty new to C++, I have been following the intermediate tutorials at 3DBuzz.com, and trying to experiment with their tasks.

Current tutorial is on classes: http://www.3dbuzz.com/vbforum/sv_showvideo.php?v=37

I am trying to overload the &operator << to output my ‘Point’ as a stream when I want. The relevant part of the video starts at 39:00.

As far as I can tell my code is syntactically identical (though I am new so I’m probably missing something) but I get the error:

1>c:\users\jack\documents\visual studio 2010\projects\myfirstgame\myfirstgame\main.cpp(88): error C2146: syntax error : missing ‘;’ before identifier ‘myPoint

I realise that I declare the instance Point &myPoint in the operator overload function.. but I don’t know where else I could do it so the compiler knows what it is.. if that makes sense.

Any help is appreciated! Thanks

#include <iostream>
#include <cmath>

using namespace std;

class Point
{
public:
Point(float f_x = 0.0, float f_y = 0.0, float f_z = 0.0);

~Point();

void SetXYZ(float X, float Y, float Z);
void SetX(float X);
void SetY(float Y);
void SetZ(float Z);

void GetXYZ(float &X, float &Y, float &Z);
float GetX();
float GetY();
float GetZ();
private:

float x, y, z;

protected:

};

Point::Point(float f_x, float f_y, float f_z)
{
cout << "Constructor with ARGUMENTS!" << endl;

x = f_x;
y = f_y;
z = f_z;
}

void Point::GetXYZ(float &X, float &Y, float &Z)
{
X = GetX();
Y = GetY();
Z = GetZ();
}
float Point::GetX()
{
return x;
}

float Point::GetY()
 {
return y;
 }

float Point::GetZ()
{
return z;
}

void Point::SetXYZ(float X,float Y, float Z)
{
SetX(X);
SetY(Y);
SetZ(Z);
}
void Point::SetX(float X)
{
x = X;
}

void Point::SetY(float Y)
{
y = Y;
}

void Point::SetZ(float Z)
{
z = Z;
}

Point::~Point()
{
cout << "We're in the destructor" << endl;
}

 ostream &operator <<(ostream &stream, Point &myPoint)
{
stream << myPoint.GetX() << " " << myPoint.GetY() << " " myPoint.GetZ();
return stream;
}

void main()
{
float x, y, z; //Declaring floats for use in GetXYZ()

Point myLocation (1,2,-1); //Creating instance and using Point(...) function
cout << myLocation.GetX() << myLocation.GetY() << myLocation.GetZ() <<endl; // Getting xyz values and printing

myLocation.SetXYZ(2,3,-4); //Testing SetXYZ function
cout << myLocation.GetX() << myLocation.GetY() << myLocation.GetZ() <<endl; // Getting xyz values and printing


myLocation.GetXYZ(x, y, z);
cout << x << " " << y << " " << z << endl;

cout << myLocation;


    system("PAUSE");
}

EDIT: Unbelievable response! Love this site already. Thanks everyone who spotted this ^^

Answer 1

You are missing << in :

stream << myPoint.GetX() << " " << myPoint.GetY() << " " myPoint.GetZ();
                                                       ^^