- Clone this repository
git clone https://github.com/EuropcarMobilityGroup/finder-refactoring - Build the docker container which contains all the required dependencies with
make build - Run the tests with
make test - Start refactoring!
The primary goal is to refactor the code in src/Algorithm/Finder.php - as it stands the code is incomprehensible.
- Start with simple rename refactors, so you can better understand the abstractions you are working with. Rename any class or any variable.
- Move on to extract methods and making the code more modular.
- See if you can also eliminate switch statements and multiple exit points from methods.
Anything is fair game, create new classes, new methods, and rename tests. The only restriction is that the existing tests have to keep working. Lean on the tests and run them after every small change to make sure you are on the right path.
You can stop when you feel the code is good enough, something you can come back to in 6 months and understand.
- Refactoring.guru Code Smells catalog
- Refactoring.guru Refactorings catalog
- SourceMaking Refactorings catalog
- Martin Fowler Refactorings catalog
This kata is a Scala port of the original Incomprehensible Finder Refactoring Kata created by K. Scott Allen.