It might make more sense to check for null first, since that is the most likely case right? Since you have newTopic = null at the creation of the controller. Gotta save those microseconds.