Data class smell
From CSSEMediaWiki
(Difference between revisions)
m (category) |
m (→See also) |
||
| Line 12: | Line 12: | ||
* [[Code smells]] | * [[Code smells]] | ||
* [[Refactoring]] | * [[Refactoring]] | ||
| + | |||
| + | {{Template:CodeSmells}} | ||
[[Category:Code smells]] | [[Category:Code smells]] | ||
Revision as of 09:30, 14 October 2009
The data class smell is present where a class has only fields, getters, setters and nothing else. It lacks the behaviours that make it a proper, stand-alone class (and consequently, the instantiated object).
Refactoring techniques
- Encapsulate Field - if there are public fields
- Encapsulate Collection - if there is collections in the class. This is to ensure unmodifiable, read only collections is returned
- Remove Setting Method - for fields whose values should not be changed
- Move Method or Extract Method - move method or extract method to move the behavior from the clients (that use the accessors of the data class) to data class
See also
