Over engineering code is a common plague that leads to a number of unwanted side effects in your software ranch. This post does not want to be an exhaustive guide on over engineering but with the help of many quotes from different sources, I’ll try to summarize the impacts, when you can have evidence of over engineered code and the causes (to try to avoid them).
- unneccessary complexity => less agility in software
- more code, more bugs
- more code, more unit tests, more development time
- performance impacts
- longer catch up time for newcomers
- difficoult maintenance, fixing bugs requires more time that normally
- small changes require big efforts and are prone to having many bugs
- unwanted features are present even though there is not reference in requirements/backlogs and so you have to maintain them even if they are not used
- false perception of complexity in the code that leads to over inflated estimates for fixes, changes.
- extending an over engineered project makes extensions more complex than necessary
- dismantling of over engineered code takes way longer than it took to over engineer it
- Need for complete rewrite over time to reduce unneeded complexity when maintenance times become higher than rewrite time This is an interesting topic itself and needs a separate post)
Concrete symptons of over engineered code
Really difficult here to point out some easy and simple guidelines to recognize over engineered code. Often it is an individual perception and this does not help. I’ll try :
- When a well experienced coder, which has been working on that piece of code for at least an year, still takes way more than necessary/affordable to figure out where/how a specific feature works. I’m trying to find a better explanation for this concept
- Use a factory only if it is making more than one type of objects
- Use an interface if it is actually going to be implemented by more than one class
- From PrematureGeneralization
“This is really going to be a clean framework. I’ll make an abstract class out of this part so that folks can subclass it later, and I’ll put in a bunch of well-commented overridable hooks in the concrete subclasses so that folks can use them as templates, and just in case somebody ever needs to build special debug subclasses, I’ll put in extra stubs over there (somebody will thank me for ’em one of these days). Yeah, this is really going to be neat.Thus is bloated software produced when our artistic sense gets the better of us. — DaveSmith “
Some other signs of over engineering taken from stackoverflow :
One very strong warning sign of over engineering is when everything goes through so much indirection that it’s hard to find the piece of code that actually implements some concrete, domain-level piece of functionality. If you find that most of your functions do very little concrete work and just call other virtual functions, you may have a problem.
[..] Make factories where the factories make more then one type of object. Use dependency injection, where it immediately shows benefits. Make interfaces that are actually going to be implemented by more then one class [..] What I see too often in “true OO” is that advanced techniques are used to solve really simple problems in an overly complex way [..]
Causes of over engineering :
Accidental vs essential complexity
Interesting article, some really good points :
Software should behave predictably and accomplish its goals without too many surprises (that is, outages in production). The number of surprises directly correlates with the amount of unnecessary complexity found in a project. It’s therefore crucial to think about accidental complexity and essential complexity:
Accidental complexity relates to problems which engineers create and can fix, [whereas] essential complexity is caused by the problem to be solved, and nothing can remove it
— Fred Brooks in his seminal “No Silver Bullet” essay
Another interesting point on when a coder start to write unnecessary code :
Boredom is good precursor to over-engineered code. I’ll admit, when I got my first job, I felt so underutilized. I was just bored. And when I got bored, I wrote code. Not just any code — CATHEDRALS OF CODE.
No seriously, I had a mental picture of my code and abstractions as large towers with golden jutting spires, flying buttresses of glassy onyx, a wonderful vault supporting by arched domes topped with beautiful geometrical tracery, etc etc etc.
It was really fascinating to see the patterns working together for myself, but in retrospect, I am completely ashamed of the ungodly mess I left behind.
If you’re writing your own frameworks and DSLs code to while away the less stimulating hours at work, just stop. Time is better spent reading Wards Wiki, or writing an open source book, or you may just want to ask management for more work.
Techniques to avoid over engineering
Beware of complicated solutions (that someone was paid to find) @ntaleb
I’m still working on this point : here are some references to interesting concepts
The KISS principle
YAGNI : Your Arent Gonna Need It
DRY : Don’t repeat yourself
Simplicity is a prerequisite for reliability