Issue with loops

Why does the constructor take a name (mName) that it doesn't use?

		char*	details;

You still don't understand that you REALLY need to declare an array...