I would much rather have it return a null object instead of nil. You could make it better by adding a #guest? method to your User class. if current_user.gu

I would much rather have it return a null object instead of nil. You could make it better by adding a #guest? method to your User class. if current_user.guest? Although I don't like how the example given returns an OpenStruct instead of a NullUser, for instance.