Something wrong with my constructor/function?

[Austin J]

This program is supposed to accurately calculate the perimeter of two rectangles. It always prints out the number 858993466 for each however. I'm sure it's a simple syntax error.

#include<iostream>
using namespace std;

class Rect
{
	int a, b, c, d;
public:
	Rect (int, int, int, int);
	int perimeter()
	{
		return (a + b + c + d);
	}
};

Rect::Rect(int one, int two, int three, int four)
{
	one = a;
	two = b;
	three = c;
	four = d;
}

int main()
{
	Rect uno (1,1,1,1);
	Rect dos (4,11,4,6);
	cout << "Rect 1 P: " << uno.perimeter() << endl;
	cout << "Rect 2 P: " << dos.perimeter() << endl;
	return 0;
}

[maeriden]

You wrote the assignments in the constructor backwards. This way the arguments get the same value as the uninitialized members, and the members themselves remain uninitialized.

[Austin J]

You should have heard the derp slap I made lol. Thank you kind sir for showing me my simple error.

[Lowest0ne]

Rect dos (4,11,4,6);

That isnt a rectangle.

If a Rectangle has four arguments, then I expect the first two to be an x-y coordinate, and the others to be width and height,