CSC/ECE 517 Fall 2013/oss ans: Difference between revisions

From Expertiza_Wiki
Jump to navigation Jump to search
No edit summary
No edit summary
Line 7: Line 7:
* Reduce the method complexity in the response controller
* Reduce the method complexity in the response controller
* Remove duplicated code
* Remove duplicated code
* There were “custom_create” and “custom_update” methods for “custom” (multipart) rubrics. This is a very bad way of achieving this      functionality, because frequent tests are required as to what kind of a rubric is in use. 
* There were “custom_create” and “custom_update” methods for “custom” (multipart) rubrics. These methods were only there to handle the multi part rubrics. We already had create and update methods that were handling the normal rubrics. Our job was to use polymorphism in order to eliminate the custom methods
== Design Choices ==


==Future Work ==
==Future Work ==


== Appendix ==
== Appendix ==

Revision as of 03:00, 30 October 2013

E807 : Refactoring and testing response_controller.rb


Introduction

The response controller creates, edits, and displays responses, that is, rubrics to be filled out, or filled-out rubrics. Our project requirement entailed the following things to be done.

  • Reduce the method complexity in the response controller
  • Remove duplicated code
  • There were “custom_create” and “custom_update” methods for “custom” (multipart) rubrics. These methods were only there to handle the multi part rubrics. We already had create and update methods that were handling the normal rubrics. Our job was to use polymorphism in order to eliminate the custom methods

Future Work

Appendix