DEV Community

Discussion on: Code Smell 89 - Math Feature Envy

View post

Replies for: I suppose it depends on the implementation, for example: function rectangularArea(rectangle) { return rectangle.width * rectangle.height; } ...

Collapse
 
mcsee profile image
Maxi Contieri • • Edited

You are using an external function, stripping object accidental properties and performing calculations outside of it. this violates reuse, information hiding principle, class cohesion and encapsulation.

The area is what the rectangle points out. not an external function.
You are confusing data (accidental) with behavior (essential)

Collapse
 
jamesrweb profile image
James Robb •

Can you give an example of each case you mentioned and a suitable resolution in your opinion?

What about in functional languages, how should this be approached there in your opinion?

Thread Thread
 
mcsee profile image
Maxi Contieri •

Hi again

Thanks for the comment.
I am not an expert on Functional Programming.
I'll stick to OOP, my area of knowledge.
Calculating the area of a shape (and how it does) it is shape's reposibility.
Example is on the article above.
Doing external manipulations is a code smell, according to my opinion