the smells of bad design
TRANSCRIPT
![Page 1: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/1.jpg)
The Smells of Bad Design
Is our App rotting?
![Page 2: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/2.jpg)
The Smells
Rigidity Fragility Immobility Viscosity Needless Complexity Needless Repetition Opacity
http://davidhayden.com/blog/dave/archive/2005/08/09/2421.aspx
Robert C. Martin Agile Patterns Practices and Principles in C# 2007
![Page 3: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/3.jpg)
Rigidity
System is difficult to change.– What is more difficult than should be expected?
Unanticipated repercussions to simple changes
“It was more complicated than I thought!”– Was it really harder than apparent or were we just
unfamiliar with the technology? “We can’t do that because of how it’s written”
![Page 4: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/4.jpg)
Fragility
Breaks down when a single change is made. Often a break where there is little or no visible relation to the
change.– What defines a ‘visible relation?’
Areas that developers are afraid of or constantly say are dangerous.
– “The transfers page needs help”– “I’m not touching the Login page again”
Often times fixing the problems causes more problems. We could Test “harder” but what can we do proactively?
![Page 5: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/5.jpg)
Immobility
Parts could be useful to other systems but are unable to be reused.
“Well, our system has a component that does that, but it’s tightly tied to the SDK”– Reusable business logic getting repeated.– What business rules should we be responsible
for? If it’s reusable, it should be moved to web
services for the enterprise.
![Page 6: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/6.jpg)
Viscosity
Two forms– Software– Environment
![Page 7: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/7.jpg)
Software Viscosity
We could change this piece of code by making the ‘good design’ choice or the ‘hack’
The hack is easier. What is a hack? What is clean code?
![Page 8: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/8.jpg)
Environment Viscosity
Compiles take too long so the developers makes a change to ensure the least amount of compile time. (change 1 class instead of all the classes needed)
The build is so slow, it’s simpler to throw this into the UI than modify the Middle Tier
What about our build annoys you as a developer? Tester? Manager?
![Page 9: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/9.jpg)
Needless Complexity
Unused methods, attributes, variables, unreachable areas of code.
Preparing for a future that may never come. Are you allowed to introduce scope creep any more
than the business? JIT Time Weight of dead code (size, maintenance, memory,
documentation) How are you USUALLY able to tell what code is
doing?
![Page 10: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/10.jpg)
Needless Repetition
Fravle the Arvadent example. Copy/Paste coding Hard to maintain the infinite repeats of similar
code. Find 1 bug and have to search the code base for every implementation.
![Page 11: The Smells Of Bad Design](https://reader036.vdocuments.net/reader036/viewer/2022082503/55867012d8b42a7e148b4764/html5/thumbnails/11.jpg)
Opacity
Difficulty to understand One person’s simple is another person’s difficult.
– What is simple to you?
Intimacy with the code wears off, do you know what it does?
Code commenting, Peer review, Test Driven Design, Coding Standards… what else could help?