Your definitions of start_time, start_time= are duplicated code. The attr_accessor call does the exact same thing. Same for duration.