You have this `before_action` where you find a city, but you aren't assigning it to anything. From looking at it, I don't think this controller will work. I like your thought of seeing duplication and trying to reduce it, but there is such a thing as being too DRY. We prefer assigning all instance variables inside the controller action instead of hiding them in other methods so that it is very easy to see what instance variables are available in the views with one quick glance. Also, it's easy to miss `before_actions` so we try to use them infrequently.

TL;DR -  keep the `@city = find_city` statements inside the controllers.