search

coldstar96 / cse403

Budget Manager Project
1 stars 2 forks source link

Split the AddBudgetActivity into two activities for adding and editing #145

Closed brucec5 closed 11 years ago

brucec5 commented 11 years ago

This class refactors the AddBudgetActivity into two classes: the AddBudgetActivity and the EditBudgetActivity. They derive from the AbstractBudgetEditorActivity, which holds the shared behaviour.

This is done because the AddBudgetActivity was getting rather large and had a bunch of noncohesive behavior (it did double-duty for adding and editing and had a tonne of special cases). Also, due to that double-duty nature, a bunch of the methods were really long.

This deals with that. It adds no new features and fixes no outstanding bugs, it just makes our code better.

CardboardAqueduct commented 11 years ago

Style and functionality look good, as well as achieving some of our refactoring goals.