Use attr_reader to define trivial reader methods.
Do not use :: for method calls.
unused variable
removed space