Amit Tiwari

14%
Flag icon
In Listing 3-7 we had no choice because the callers were already passing that flag in, and I wanted to limit the scope of refactoring to the function and below. Still, the method call render(true) is just plain confusing to a poor reader. Mousing over the call and seeing render(boolean isSuite) helps a little, but not that much. We should have split the function into two: renderForSuite() and renderForSingleTest().
Clean Code: A Handbook of Agile Software Craftsmanship (Robert C. Martin Series)
Rate this book
Clear rating
Open Preview