CSC/ECE 517 Fall 2013/oss E814 vd: Difference between revisions
| Line 22: | Line 22: | ||
====update_pos_property==== | ====update_pos_property==== | ||
The next change in generate_graph | The next change in generate_graph | ||
---- | |||
---- | |||
[[File:Update_pos_property_0.png]] | |||
---- | |||
---- | |||
[[File:Update_pos_property_1.png]] | |||
---- | |||
---- | |||
[[File:Update_pos_property_2.png]] | |||
---- | |||
====pos_edge_processing==== | ====pos_edge_processing==== | ||
---- | |||
---- | |||
[[File:Pos_edge_processing_0.png]] | |||
---- | |||
---- | |||
[[File:Pos_edge_processing_1.png]] | |||
---- | |||
---- | |||
[[File:Pos_edge_processing_2.png]] | |||
---- | |||
---- | |||
====add_nonexisting_edge==== | ====add_nonexisting_edge==== | ||
==References== | ==References== | ||
Revision as of 20:42, 30 October 2013
Expertiza is a web application that supports peer-review for a variety of entities. As it is still evolving, expertiza has several areas that require work. Among these include areas that require refactoring existing code to improve code quality. The graph_generator.rb file is one such area. This write-up describes our refactoring decisions for this file.
Background
Resources
Changes by Method
generate_graph
tagged_token_check
The first change that can be seen within this method is the extracting of code that adds on to a previous vertex or creates a new one, all based upon the part of speech defined of the tokens encountered while parsing the review. We called our extracted method tagged_token_check. BY extracting this method, we were able to reduce duplication (since such similar code occurred four times in the method for tokens that are nouns, adjectives, adverbs, and verbs. Since the code contained nested if statements, we also were able to reduce the complexity of the method. Screenshots of the code before and after refactoring are included below. The top picture is the code before refactoring, the middle is the code after, and the bottom is the code within the extracted method.
update_pos_property
The next change in generate_graph
pos_edge_processing
