CSC/ECE 517 Fall 2013/oss E818 mra

From Expertiza_Wiki
Revision as of 01:03, 30 October 2013 by Avrohama (talk | contribs) (Created page with "<mainpage-leftcolumn-start /> =E818 Refactoring and testing -- analytics= ==Problem statement:== ===Classes:=== * analytic_controller.rb (163 lines) * assignment_analytic.rb (20...")
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to navigation Jump to search

<mainpage-leftcolumn-start />

E818 Refactoring and testing -- analytics

Problem statement:

Classes:

  • analytic_controller.rb (163 lines)
  • assignment_analytic.rb (206 lines)
  • assignment_team_analytic.rb (126 lines)
  • course_analytic.rb (142 lines)
  • question_analytic.rb (12 lines)
  • questionnaire_analytic.rb (55 lines)
  • response_analytic.rb (101 lines)
  • score_analytic.rb (12 lines)

What they do:

Display information about various system components in summary form, to aid in analysis.

What needs to be done:

At least analytic_controller.rb and course_analytic.rb have much duplicated code, which should be factored out. Additionally, analytic_controller.rb has methods that are too complicated, and should be separated into shorter, well-named methods, to make the operation of the class more transparent. Also, there are absolutely no tests for any of these methods.