it would favourable if you change the method names (sort_asc and sort_desc) to match the names in the class diagram.