First off you need to switch around your assignment statements. Your constructors purpose is to set the values of the object to that of the parameters passed to it.