Code Smell 130 - AddressImpl
Maxi Contieri
Posted on April 18, 2022
It is nice to see a class implementing Interfaces. It is nicer to understand what it does
TL;DR: Name your classes after real-world concepts.
Problems
Solutions
- Find the correct name using the MAPPER
Context
Some languages bring idioms and common usages against good model naming.
We should pick our names carefully.
Sample Code
Wrong
public interface Address extends ChangeAware, Serializable {
/**
* Gets the street name.
*
* @return the street name
*/
String getStreet();
//...
}
//Wrong Name - There is no concept 'AddressImpl' in real world
public class AddressImpl implements Address {
private String street;
private String houseNumber;
private City city;
//..
}
Right
//Simple
public class Address {
private String street;
private String houseNumber;
private City city;
//..
}
//OR
//Both are real-world names
public class Address implements ContactLocation {
private String street;
private String houseNumber;
private City city;
//..
}
Detection
[X] Automatic
Since this is a naming smell.
We can search using regular expressions and rename these concepts.
Tags
- Naming
Conclusion
We should pick class names according to essential bijection and not follow accidental implementation.
Do not call I to your interfaces.
Relations
Code Smell 65 - Variables Named After Types
Maxi Contieri ・ Apr 2 '21
More Info
Credits
Photo by Paula Hayes on Unsplash
Encoded names are seldom pronounceable and are easy to miss-type.
Robert C. Martin
Software Engineering Great Quotes
Maxi Contieri ・ Dec 28 '20
This article is part of the CodeSmell Series.
Posted on April 18, 2022
Join Our Newsletter. No Spam, Only the good stuff.
Sign up to receive the latest update from our blog.