CSC/ECE 517 Fall 2019- Project E1946 Refactor Questionnaire Controller

From Expertiza_Wiki
Revision as of 18:35, 29 October 2019 by Ggarima (talk | contribs)
Jump to navigation Jump to search

E1946. Refactoring the Questionnaire Controller

This page provides a description of the Expertiza based OSS project.



About Expertiza

Expertiza is an open source project based on Ruby on Rails framework. Expertiza allows the instructor to create new assignments and customize new or existing assignments. It also allows the instructor to create a list of topics the students can sign up for. Students can form teams in Expertiza to work on various projects and assignments. Students can also peer review other students' submissions. Expertiza supports submission across various document types, including the URLs and wiki pages.

About Questionnaire Controller

Questionnaire is the superclass for all kinds of questionnaires and rubrics, rubrics is used for evaluating submissions and teammate contributions, and taking quizzes and surveys. All of these are subclasses of Questionnaire. This controller is used for creating, displaying, and managing Questionnaires. Different type of questionnaire can be created in Expertiza like Review, Metareview, Teammate review, Quiz Questionnaire, Global Survey, Course Survey and many more. This Controller is widely used and malfunctions can cause many issues in different parts of the system.

Why Refactor Questionnaire Controller

In Questionnaire controller below are some of the problems which needs refactoring:

  • Renaming the methods breaking Rubys naming conventions.
  • Refactoring the methods doing multiple tasks and breaking them in different modules.
  • Reducing the length of methods longer than 50 lines of code.
  • Removing the string literal values used in controller.
  • Separating the business logic from controller and putting it in method.
  • Remove variables behaving same as ruby's given functionality.
  • Commenting the code properly for easy understanding.

Files Modified

Questionnaire Controller and Questionnaire model , Questionnaire Spec, Quiz Questionnaire was modified during this project.

  • Questionnaire Controller - This is the controller which handles all the CRUD operation performed on any kind of questionnaire - review or surveys, add weightage to all questions, selecting type of questions. This controller holds the major part of work done for creating questions. Creating of questionnaire was one method which was extremely complicated and not well commented about what is going on. Multiple things were done in just one method which we broke in modules doing that specific task. This controller was holding the business logic as well, which we can move to model. Variables were created separate to get value from UI. At many places string literals were being hardcoded, which were made as constants.
  • Questionnaire model - We have added self method in model which will create questionnaire object for us and we will get that object in controller and process it in controller.
  • Quiz Questionnaire Controller - Quiz questionnaire was calling multiple methods of quiz questionnaire by passing questionnaire id as parameter which we have removed.
  • Questionnaire SPEC - We have changed this file in order to test the changes we have made in Questionnaire controller.

Refactoring

We have used Github project board to keep track of work that needs to be refactored. Below were the user stories that we have created:

  • US1 - As a developer i want to analyze create questionnaire so that i can refactor and break it into modules.
  • US2 - As a developer i want to refactor the create_questionnaire method by giving it meaningful name and improving code so that it does not conflict with ruby naming standard.
  • US3 - As a developer i want to remove questionnaire_id with params[:id] so that we can remove unnecessary variables.
  • US4 - As a developer i want to remove all hardcoded values of alternatives, min label and max label and put them in literals, so that code can be aligned to coding standards.
  • US5 - As a developer i want to analyse why Questionnaire is being checked as QuizQuestionnaire multiple times and functionality is being changed, so that we can properly distinguish between quiz questionnaire and questionnaire.