This is good, but I wonder if the code setting instance variables would be better placed in corresponding controllers (mostly a readability issue).