Inappropriate intimacy smell
From CSSEMediaWiki
(Difference between revisions)
m (category) |
m (→See also) |
||
| Line 11: | Line 11: | ||
* [[Keep related data and behavior in one place]] | * [[Keep related data and behavior in one place]] | ||
* [[Code smells]] | * [[Code smells]] | ||
| + | |||
| + | {{Template:CodeSmells}} | ||
[[Category:Code smells]] | [[Category:Code smells]] | ||
Revision as of 09:31, 14 October 2009
Sometimes classes become too intimate and spend too much time touching each other's private parts or fields. This is bad because it breaches encapsulation and Information hiding.
If a class is more interested in the internals of another class, this can indicate that related data and behavior is not put in one place. Therefore, intimate classes should be refactored by moving the methods and fields in such a way that related data and behavior is put together and there is no need for a class to look at another class' internals.
Subclasses in particular are often very intimate with their parent classes though proponents of object encapsulation would argue that that is appropriate.
See also
- Information hiding
- Encapsulation boundary
- Hide data within its class
- Keep related data and behavior in one place
- Code smells
