I still think you should throw an exception if name is null