-
Notifications
You must be signed in to change notification settings - Fork 0
Second Iteration Proposal
Below are our prioritized list of refactorings. Our initial plan is to tackle these in this order, but we may need to revise that plan as we move into the refactorings.
Rename the stupid and useless names found throughout the code base
Our plan is to rename things.
The refactoring will be easy, but we have to make sure that all names are changed to the same new name throughout.
This helps readability.
This will address the arrays that start at 1 instead of 0 in the machineShopSimulator class.
Our plan is to locate the problem and change things as necessary. We will start by changing all instances to start at 0 then go from there.
The refactoring will be difficult. Other groups have had problems with this one. We hope that they will give advice if we get stuck.
It follows java coding practices.
This will address complicated for loops and methods that belong in other classes. (We may just look at for loops depending on complexity)
Our plan is to simplify methods and create new ones if necessary.
The refactoring will be medium, as long as we test. A lot.
This has impact on the readability of the code.
For the renaming variables, it was simple and straight forward. We changed some of the longer, more complicated names to easier to understand ones For extracting, we used the refactoring tool. Then we created get and set methods to compensate for the removal of them in the Machine Shop Simulator. The changing of arrays caused us great difficulty. We were not able to successfully get the arrays to work properly. First, we went through the code to determine where there were instances of an array. We tried to change the initial creation of the arrays, but we still got out of bounds exceptions. We tinkered with other things and sometimes the tests would not run. Then other times the tests would ask us to "Enter number of machines and jobs."