I thought we were only going to allow null args for lastName and secondPhone?  We shouldn't be able to construct a firstName or phoneNumber with null -- I think we should be throwing a RuntimeException() in those if statements instead of changing it to "".