Design smell
From Infogalactic: the planetary knowledge core
(Redirected from Design Smell)
In computer programming, design smells are "structures in the design that indicate violation of fundamental design principles and negatively impact design quality".[1] The origin of the term "design smell" can be traced to the term "code smell" which was featured in the book Refactoring: Improving the Design of Existing Code by Martin Fowler.[2]
Different authors have defined the word "smell" in different ways:
- N. Moha et al.: "Code and design smells are poor solutions to recurring implementation and design problems."[3]
- R. C. Martin: "Design smells are the odors of rotting software."[4]
- Fowler: "Smells are certain structures in the code that suggest (sometimes they scream for) the possibility of refactoring."[2]
Design smells indicate the accumulated design debt (one of the prominent dimensions of technical debt). Bugs or unimplemented features are not accounted as design smells. Design smells arise from the poor design decisions that make the design fragile and difficult to maintain. It is a good practice to identify design smells in a software system and apply appropriate refactoring to eliminate it to avoid accumulation of technical debt.
The context (characterized by various factors such as the problem at hand, design eco-system, and platform) plays an important role to decide whether a certain structure or decision should be considered as a design smell. Many a times, it is appropriate to live with design smells due to constraints imposed by the context.
Common design smells
- Missing abstraction[1] when clumps of data or encoded strings are used instead of creating an abstraction. Also known as "primitive obsession" [2] and "data clumps".[2]
- Multifaceted abstraction[1] when an abstraction has multiple responsibilities assigned to it. Also known as "conceptualization abuse".[5]
- Duplicate abstraction[1] when two or more abstractions have identical names or implementation or both. Also known as "alternative classes with different interfaces"[2] and "duplicate design artifacts".[6]
- Deficient encapsulation[1] when the declared accessibility of one or more members of an abstraction is more permissive than actually required. Also known as "hideable public attributes/methods".[7]
- Unexploited encapsulation[1] when client code uses explicit type checks (using chained if-else or switch statements that check for the type of the object) instead of exploiting the variation in types already encapsulated within a hierarchy. Also known as "simulated polymorphism".[8]
- Broken modularization[1] when data and/or methods that ideally should have been localized into a single abstraction are separated and spread across multiple abstractions. Also known as "data class"[2] and "data records".[9]
- Insufficient modularization[1] when an abstraction exists that has not been completely decomposed, and a further decomposition could reduce its size, implementation complexity, or both. Also known as "God class",[8] "fat interface",[4] and "blob class".[10]
- Cyclically-dependent modularization[1] when two or more abstractions depend on each other directly or indirectly (creating a tight coupling between the abstractions). Also known as "dependency cycles"[11] and "cyclic dependencies".[12]
- Unfactored hierarchy[1] when there is unnecessary duplication among types in a hierarchy. Also known as "significant sibling duplication"[10] and "orphan sibling method/attribute".[8]
- Broken hierarchy[1] when a supertype and its subtype conceptually do not share an “IS-A” relationship resulting in broken substitutability. Also known as "inappropriate use of inheritance"[13] and "misapplying IS A".[14]
- Cyclic hierarchy [1] when a supertype in a hierarchy depends on any of its subtypes. Also known as "knows of derived"[8] and "inheritance/reference cycles".[15]
Tools to detect design smells
See also
References
- ↑ 1.00 1.01 1.02 1.03 1.04 1.05 1.06 1.07 1.08 1.09 1.10 1.11 Girish Suryanarayana, Ganesh SG, Tushar Sharma (2014). "Refactoring for software design smells: Managing technical debt". Morgan Kaufmann. ISBN 978-0128013977
- ↑ 2.0 2.1 2.2 2.3 2.4 2.5 Fowler, Martin (1999). Refactoring. Improving the Design of Existing Code. Addison-Wesley. ISBN 0-201-48567-2.
- ↑ N. Moha, Y. Gueheneuc, L. Duchien, and A. Le Meur. "Decor: A method for the specification and detection of code and design smells". IEEE Trans. Softw. Eng., 36(1):20–36, January 2010.
- ↑ 4.0 4.1 R. C. Martin. Agile Software Development, Principles, Patterns, and Practices. Addison-Wesley, 2003.
- ↑ Trifu A. "Automated strategy based restructuring of object oriented code". In Proceedings of the 7th German workshop on software-reengineering (WSR); 2005.
- ↑ Stal M. "Software architecture refactoring". Tutorial in The international conference on object oriented programming, systems, languages and applications (OOPSLA); 2007.
- ↑ Sonargraph-quality: A tool for assessing and monitoring technical quality. Available at: https://www.hello2morrow.com/products/sonargraph/quality.
- ↑ 8.0 8.1 8.2 8.3 Structural Investigation of Software Systems (SISSy) Tool. Available at: http://sissy.fzi.de/SISSy/CMS/index.html.
- ↑ Structural Analysis for Java Tool (Stan4J). Available at: http://stan4j.com/.
- ↑ 10.0 10.1 10.2 InFusion - Design flaw detection tool. Available at: http://www.intooitus.com/products/infusion.
- ↑ SDMetrics - A UML design quality metrics tool; 2012. Available at: http://www.sdmetrics.com.
- ↑ Page-Jones M. "The practical guide to structured systems design". 2nd ed. Prentice Hall; 1988.
- ↑ Budd T. "An introduction to object-oriented programming". 3rd ed. Addison Wesley; 2001.
- ↑ Page-Jones M. "Fundamentals of object-oriented design in UML". Addison-Wesley Professional; 1999.
- ↑ Sefika M, Sane A, Campbell RH. "Monitoring compliance of a software system with its high-level design models". In Proceedings of the 18th international conference on software engineering, ICSE ‘96, Washington, DC, USA; 1996. p. 387–96.
- ↑ Designite - A Software Design Quality Assessment Tool. Available at: http://www.designite-tools.com
