#include <iostream>
#include <Windows.h>
#include <atldbcli.h>
#include <stdlib.h>




class DCMK
{
public: 
	DCMK() { int Aggression = 1; int Agility = 1; }
	DCMK(int tempA);
	~DCMK() {  std::cout << "Destructor Called <<\n"; }
	int GetAggression() { return(Aggression); }
	void SetAggression(int x) { int Aggression = x; }
	int GetAgility() { return(Agility); }
	void SetAgility(int x) { int Agility = x; }
private:
	int Aggression;
	int Agility;
};

DCMK::DCMK(int tempA) {
	int Aggression = tempA;
	int Agility = tempA;
	std::cout << "Constructor Called " << tempA << " \n";
}

class Data {
public:
	Data() { int pVal = rand();  int pStor = 1; } 
	Data(int val) { int pVal = val; }
	int GetValue(int wot); 
	int Beast(Data const &);
private:
	int pVal;
	int pStor;
};

Data::GetValue(int wot) {
	if(int wot = 1) { return pVal; }
	else { return wot; }
}

Data::Beast(Data const & other) {
	if(other.pStor < pStor)
		std::cout << "Worked" << this->pStor << "Sorted Geez!\n";
		std::cout << "Brup Brup!!!\n";
		return other.pStor;

}


int main()
{
	DCMK * pPoint = new DCMK; //pointer 1
	Data * D1; //pointer 2
	Data * D2(4) = new Data;  //pointer 3
	std::cout << "Hello, world!\n";
	std::cout << "Number is " << pPoint->GetAggression() << " for aggression\n";
	std::cout << "Number is " << pPoint->GetAgility() << " for aggression\n";
	std::cout << "Data is"  << D1.GetValue(1) << " Value\n";
std::cout << "Data is"  << D2.GetValue(1) << " Value\n";
	std::cout << "Data is"  << D2.Beast(2) << " Value\n";
	system ("pause");
	return 0;
};

Data::GetValue(int wot) {	if(int wot == 1)  		return pVal;	else		return wot;}Data::Beast(Data const & other) {	if(other.pStor < pStor)		std::cout << "Worked" << this->pStor << "Sorted Geez!\n";		std::cout << "Brup Brup!!!\n";		return other.pStor;	else 		std::cout << "Buggered\n"; 		return other.pStor;}

Data::GetValue(int wot) {	if(wot == 1)  		return pVal;	else		return wot;}Data::Beast(Data const & other) {	if(other.pStor < pStor)        {		std::cout << "Worked" << this->pStor << "Sorted Geez!\n";		std::cout << "Brup Brup!!!\n";		return other.pStor;        }	else        {		std::cout << "Buggered\n"; 		return other.pStor;        }}

Data::GetValue(int wot) {	if(int wot == 1)  

Data::GetValue(int wot) {	if(wot == 1)  

class DCMK{public: 	DCMK() { int Aggression = 1; int Agility = 1; }	DCMK(int tempA);	~DCMK() {  std::cout << "Destructor Called <<\n"; }	int GetAggression() { return(Aggression); }	void SetAggression(int x) { int Aggression = x; }	int GetAgility() { return(Agility); }	void SetAgility(int x) { int Agility = x; }private:	int Aggression;	int Agility;};

class DCMK{public: 	DCMK() {Aggression = 1; Agility = 1; }	DCMK(int tempA);	~DCMK() {  std::cout << "Destructor Called <<\n"; }	int GetAggression() { return(Aggression); }	void SetAggression(int x) {Aggression = x; }	int GetAgility() { return(Agility); }	void SetAgility(int x) {Agility = x; }private:	int Aggression;	int Agility;};

class DCMK{public: 	DCMK() { int Aggression = 1; int Agility = 1; }// These do not reference the member variables, but instead create locals with// the same names, assign to them, and promptly throw them away. The same// problem exists with the other constructor and the set methods.// ( Having the set and get methods points at a style problem too, but it won't// break things at least ;) )// What you want to do is leave off the type names. The member variables are// "in scope" here and don't need a declaration (which such a line is when// you include a type name). Just like you can write "return Agility;", so you// can just do "Agility = 1;".	class Data {public:	Data() { int pVal = rand();  int pStor = 1; }// Similarly in here. The names are awful, too. Whoever's telling you how to// name them so far, please stop. (Not only is this particular style of// Hungarian notation ugly and useless, you have the wrong prefixes here. ;) )Data::GetValue(int wot) {	if(int wot = 1) { return pVal; }	else { return wot; }}// This will always return pVal, because first of all you assign rather than// compare, and secondly, you again use a new local "wot" in your if statement// instead of the passed in one. But then, the intended semantics are totally// bizarre anyway.Data::Beast(Data const & other) {	if(other.pStor < pStor)		std::cout << "Worked" << this->pStor << "Sorted Geez!\n";		std::cout << "Brup Brup!!!\n";		return other.pStor;}// Only the first line is conditional; the second output will always happen.// So check your indentation, it's lying to you. :)// Also, you should declare the return type rather than relying on the implicit// 'int' return type (the same goes for GetValue()).int main(){	DCMK * pPoint = new DCMK; //pointer 1	Data * D1; //pointer 2// OK, but it doesn't point at anything yet.	Data * D2(4) = new Data;  //pointer 3// The (4) there is wrong. If you want to pass it to the constructor, it goes here://      Data * D2 = new Data(4);	std::cout << "Hello, world!\n";	std::cout << "Number is " << pPoint->GetAggression() << " for aggression\n";	std::cout << "Number is " << pPoint->GetAgility() << " for aggression\n";// You get garbage values here for the reasons noted above.	std::cout << "Data is"  << D1.GetValue(1) << " Value\n";// You get a compile error here because D1 is a pointer and you try to use it// like an actual object; you need -> instead of . the same as you did with// pPoint. After that, you get a runtime crash, because D1 doesn't actually// point at an object of that type.std::cout << "Data is"  << D2.GetValue(1) << " Value\n";	std::cout << "Data is"  << D2.Beast(2) << " Value\n";// After fixing the compile errors, these will "work" :\	system ("pause");// Please learn to run your programs from the command line instead of artificially// pausing them at the end.	return 0;// Not needed, but not particularly bad.};// No ; needed here, just at the ends of classes. Not particularly bad either though.