don't use instance variable use method