Hey guys, I have been trying to learn how to do OOP by using classes in C++, and I made a program that compiles, but the output is "256" and i don't know why and I was just wondering what I did wrong. All comments are appreciated. Thanks

#INCLUDE

using namespace std;
class CRectangle
{
int x, y;
public:
Sorry, the program is:

#include

using namespace std;
class CRectangle
{
int x, y;
public:
void values(int, int);
int area() {return (x*y);}
void CRectangle::shapeformat() {
x = 3;
y = 4;
}
};

int main()
{
CRectangle rect_shape;

cout< cout<
return 0;
}I don't know c++
But I was wondering,
Is it normal to have..
{{{
and
}}}}

Why aren't they paired?
{{{, }}}}? Where do you see that at, cuz I wouldn't consider that normal in C++ unless if you have parameters within parameters, you know what i mean. but i dont see that anywhere in the source code.Is the output an error?
I found this:

256 error?

What is 'shapeformat'?Hello,

The C++ COMPILER built into my brain says that it might be a case of uninitialized variables.
Code: [Select]#include <iostream>

using namespace std;

class CRectangle
{
int x, y;
public:
void values(int, int);
int area() {return (x*y);}
void CRectangle::shapeformat() {
x = 3;
y = 4;
}
};

int main()
{
CRectangle rect_shape;

cout<<endl<<rect_shape.area();
cout<<endl;

return 0;
}
In Line No. 10, QUALIFIER `CRectangle::` for the function `shapeformat` isn't necessary. Delete it.
Make a call to `shapeformat` from main().

Make it LOOK something like:

Code: [Select]#include <iostream>

using namespace std;

class CRectangle
{
int x, y;

public:
void values(int, int);
int area()
{
return (x*y);
}

void shapeformat()
{
x = 3;
y = 4;
}
};

int main()
{
CRectangle rect_shape;

rect_shape.shapeformat();

cout<<endl<<rect_shape.area();
cout<<endl;

return 0;
}
NOTE: I have modified the code and did some cleanup to avoid confusions. Try to use proper and uniform indentation, alignment and linespacing so that the code is more `readable`.

Secondly, you've called `endl` before and after call to the class member `area()`. The former CAUSES an empty line which may not be desired. It could be something like:
Code: [Select] cout <<rect_shape.area() << endl;
Geek-9pm,

Maybe you missed this one on Line No. 8:

int area() {return (x*y);}
Ok, thanks for the help and everything, but it still prints out "256." What I wanted it to do is to print out the product of 3 and 4 which is '12'. I know other ways to do it, but I want to know why this code does what it does for educational purposes.Hello,

Quote from: timtim41 on February 12, 2010, 11:48:14 AM

prints out "256."

Hello,

The C++ Compiler built into my brain says that it might be a case of uninitialized variables.
Code: [Select]#include <iostream>

using namespace std;

class CRectangle
{
int x, y;
public:
void values(int, int);
int area() {return (x*y);}
void CRectangle::shapeformat() {
x = 3;
y = 4;
}
};

int main()
{
CRectangle rect_shape;

cout<<endl<<rect_shape.area();
cout<<endl;

return 0;
}
In Line No. 10, qualifier `CRectangle::` for the function `shapeformat` isn't necessary. Delete it.
Make a call to `shapeformat` from main().

Make it look something like:

Code: [Select]#include <iostream>

using namespace std;

class CRectangle
{
int x, y;

public:
void values(int, int);
int area()
{
return (x*y);
}

void shapeformat()
{
x = 3;
y = 4;
}
};

int main()
{
CRectangle rect_shape;

rect_shape.shapeformat();

cout<<endl<<rect_shape.area();
cout<<endl;

return 0;
}
NOTE: I have modified the code and did some cleanup to avoid confusions. Try to use proper and uniform indentation, alignment and linespacing so that the code is more `readable`.

Secondly, you've called `endl` before and after call to the class member `area()`. The former causes an empty line which may not be desired. It could be something like:
Code: [Select] cout <<rect_shape.area() << endl;