Refused bequest smell

From CSSEMediaWiki
(Difference between revisions)
Jump to: navigation, search
(New page: Subclasses inherit all the methods and data from their superclass. If a subclass doesn't want to use the stuff it inherits, this can be an indication that the class hierarchy is wrong. It ...)
 
m (Reverted edits by Ebybymic (Talk); changed back to last version by LukeRobinson)
 
(4 intermediate revisions by 4 users not shown)
Line 4: Line 4:
  
 
Fowler however notes that refused bequest often isn't a strong smell and can be overlooked in some cases unless it is causing problems and confusion.
 
Fowler however notes that refused bequest often isn't a strong smell and can be overlooked in some cases unless it is causing problems and confusion.
 +
 +
This may be seen in the [[Composite]] pattern where the leaf has no-op methods.
  
 
==See also==
 
==See also==
Line 10: Line 12:
 
* [[Avoid no-op overrides]]
 
* [[Avoid no-op overrides]]
 
* [[BaseBean]]
 
* [[BaseBean]]
 +
* [[Code smells]]
 +
 +
{{Template:CodeSmells}}
 +
 +
[[Category:Code smells]]

Latest revision as of 03:18, 25 November 2010

Subclasses inherit all the methods and data from their superclass. If a subclass doesn't want to use the stuff it inherits, this can be an indication that the class hierarchy is wrong. It can also indicate that you shouldn't be subclassing. Inheritance for implementation is a classic case where the subclass doesn't really want to use the stuff it inherits.

In some cases, a class overrides methods with no-ops which can be an indication that it is refusing the bequest of its superclasses and that you have the inheritance hierarchy wrong.

Fowler however notes that refused bequest often isn't a strong smell and can be overlooked in some cases unless it is causing problems and confusion.

This may be seen in the Composite pattern where the leaf has no-op methods.

See also


Personal tools