-- FINISH --
Updated TECHDEBT.md with ideas / wishes Added NOTES.md for the pomodoro's
This commit is contained in:
parent
af40a996fe
commit
64d2b2d70d
2 changed files with 23 additions and 11 deletions
9
NOTES.md
Normal file
9
NOTES.md
Normal file
|
|
@ -0,0 +1,9 @@
|
|||
# Pomodoro's:
|
||||
|
||||
1. Setup project & start of refactor readability (comments, dead code, magic string/numbers)
|
||||
2. Continue readability (renaming, etc)
|
||||
3. Refactor: Complexity (long method & using mikado technique for Heading enum)
|
||||
4. Continue refactor: Complexity duplicate code & mikado heading enum
|
||||
5. Refactor: Reorder responsibilities (data class & message chain are done using mikado) which triggered most of the
|
||||
other responsibilities as well.
|
||||
6. Refactor: Refine abstractions (mostly the data clumps & middle man)
|
||||
25
TECHDEBT.md
25
TECHDEBT.md
|
|
@ -1,14 +1,10 @@
|
|||
# TODO
|
||||
- [x] Refine Abstractions
|
||||
- [x] long parameter list
|
||||
- [x] data clump
|
||||
- [x] primitive obsession
|
||||
- [x] middle man
|
||||
|
||||
# Mikado
|
||||
- [x] remove move() from heading
|
||||
- [x] use position.moveBy in RoverState.move
|
||||
- [x] add moveBy to Position class
|
||||
- [ ] Command Pattern for go()
|
||||
- [ ] Move command consts
|
||||
- [ ] Parser for Rover.constructor()
|
||||
- [ ] Move parser consts
|
||||
- [ ] magic string for ' ' char still in the split
|
||||
- [ ] Update ktlint check for multiline rule :')
|
||||
|
||||
# RPP
|
||||
- [ ] Design Patterns
|
||||
|
|
@ -32,6 +28,11 @@
|
|||
- [x] inappropriate intimacy
|
||||
- [x] data class
|
||||
- [x] message chain
|
||||
- [x] Refine Abstractions
|
||||
- [x] long parameter list
|
||||
- [x] data clump
|
||||
- [x] primitive obsession
|
||||
- [x] middle man
|
||||
|
||||
# Mikado
|
||||
- [x] Clean up constants and private fields
|
||||
|
|
@ -66,4 +67,6 @@
|
|||
- [x] Replace x, y with Position in Heading.move()
|
||||
- [x] RoverState.move use position
|
||||
- [x] Return position in move() instead of Pair
|
||||
|
||||
- [x] remove move() from heading
|
||||
- [x] use position.moveBy in RoverState.move
|
||||
- [x] add moveBy to Position class
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue