deiga/hw2_rottenpotatoes — comment on commit 54d5500379832cf449825b2bcd412f44909f8826
File: app/controllers/movies_controller.rb line 16
By kheldysh:

`selected_ratings = Movie.all_ratings if @ratings.nil? or @ratings.empty?` korvaa molemmat yllä. Myös hashilla on metodi `empty?`, ja luonnollisesti `hash.keys` on tyhjä jos ja vain jos hash itse on tyhjä.

`@all_ratings` vaikuttaa myös hieman turhalta, jos sen ainoa tehtävä on toimia placeholderina Movie.all_ratingsille tässä. Tämän idiomaattisuudesta mulla ei tosin ole niin vahvaa kuvaa suuntaan tai toiseen.