Skip to content
cbeckett edited this page Dec 3, 2012 · 11 revisions

#Refactoring ##FeatureEnvy ###LocalTaskController and SharedTaskController ####Move Method Method 'updateTask(Context context, Task task, int index)' was moved to parent class TaskController commit ##Long Method

AddTaskActivity

####Extract Method Method numRequiredItems() extracted from createTask(). commit ###LocalTaskIOAdapter ####Extract Method Method localTasksFromFile(Context context) extracted from readLocalLog. Improves modularity of tasks in method. commit ###Text Item ####Extract Method Private class date() extracted from toString(). Extraction helps clarify converting Calendar to a String. commit ###CompletedTaskBaseAdapter ####Extract Method Method holder(View convertView) extracted from getView(int position, View convertView, ViewGroup parent). Extracting method helps context of getView as it no longer needs to worry if the view is null, it can call the method to grab it. commit ###TaskItemAdapter ####Extract Method Method 'private Class<?> classType(JsonElement json)' extracted from 'deserialize'. Extraction makes sense as it seperates responsibilities out of 'deserialize'. commit

Clone this wiki locally