View Issue Details

IDProjectCategoryView StatusLast Update
0000140Rabbit Ranchers[All Projects] Generalpublic2017-11-12 06:16
Assigned TorainChu 
Status closedResolutionsuspended 
Product Version 
Target VersionFutureFixed in Version 
Summary0000140: Refactor Task class
DescriptionCurrently, there is a ton of similar code across all classes that derive from Task. Some tasks inherit MoveTo, and do things a bit weirdly.

Refactor all classes that derive from Task to be sealed.

Remove the common m_subtask variable in derived classes of Task.

Add a List<Task> m_subtasks to the Task class, to be populated with all subtasks.
Add a currentSubtask property to Task class, returning the first task in the m_subtasks.
Remove the common State enum in all derived classes of Task, and track the state instead by popping subtasks off the front of m_subtasks.

Have Task be responsible for calling OnBegin et. of the subtasks, rather than derived classes.
Change failed property to return true if any subtasks have failed.
TagsNo tags attached.


child of 0000142 resolvedrainChu Simplify cooking 




2017-11-12 06:16

administrator   ~0000069

I'll do this later; it works for now, so there's no immediate need to redo it

Issue History

Date Modified Username Field Change
2017-05-13 19:13 rainChu New Issue
2017-05-13 20:10 rainChu Status new => confirmed
2017-05-13 20:10 rainChu Target Version Future => 0.0.6
2017-05-13 20:22 rainChu Product Version 0.0.6 => Future
2017-05-13 20:22 rainChu Product Version Future =>
2017-05-13 20:22 rainChu Target Version 0.0.6 => Future
2017-05-13 20:22 rainChu Product Version => 0.0.6
2017-05-13 20:27 rainChu Relationship added child of 0000142
2017-11-12 06:16 rainChu Assigned To => rainChu
2017-11-12 06:16 rainChu Status confirmed => closed
2017-11-12 06:16 rainChu Resolution open => suspended
2017-11-12 06:16 rainChu Note Added: 0000069