Well, you should call the method setCourseName(name) rather than just using the code for it in the constructor, repeating it in the setCourseName() but never using the method setCourseName().