Monday, March 14, 2011

Feedback from Assessement 1

  1. My default constructor for Team created a new Employee for the team leader. Do I really want a new Employee each time I create a new Team with no details (answer: unlikely). Therefore, I should change the default constructor to set the new teamLeader to null
  2. If I change the default constructor, my toString method will fail anytime there is a null teamLeader.
  3. I need to modify my getTeamLeader method to check if the teamLeader is null, and if so, return a String "no team leader assigned" (or equiv)
  4. I also need to double-check my code for typos and accidential capitilisations which will just cause confusion!
All in all, it was a good first assessment, especially because I received full marks.

:)

No comments:

Post a Comment