MVC Refactor, Take 2

If you ask me, the template selection excerpt of Fowler’s code belongs in the view, in the main “Album” template itself. So, depending on what data and state the model holds, the proper representation would be selected. This controller class is also clearly breaking SRP, IMHO. If the template decision needs to change, like a third album type is added, this controller would need to change. If the model source needs to change, I’d need to change this controller too. Two reasons for change at two different times = broken SRP. Granted, the model source probably won’t change at all. But, that is beside the point.

Scott