You can leave it as is since later assignments don't depend on this one. With that said, it'd be a good practice for you to change it to "good MVC" by breaking the big controller into several smaller ones so that each handles a single function.