As I program, I always encounter naming issues. What should I call this, and that? Sometimes, I spend more time thinking of a name then implementing. To help me let go of my name paralysis, I've come up with some naming patterns that seem to satisfy my endless pedantic-ism. This way, I can simply refer to these rules, choose a name, and move on with my life.
You have re-factored a method, but you need to keep the old one around, what do you call the new method?
This always happens to me. I re-factor a method, but need to keep the old one around. It's possible my new one has changed the behavior slightly, and things still depend on the old behavior, but I want all new code to adapt and depend on the new one instead. Until you re-factor everything, you need to keep the old one around.
SOLUTION
Add V followed by a numerical number to the new method.
If you have method();
you simply call the new one methodV2
. If you had an even newer one, you'd name it methodV3()
. Don't rename the first one to methodV1()
, that's just more confusing and will force you to change the call sites.
Pros
- It does not require you to change the name of the original, saving you some re-factoring work on the callers side.
- It's clear which method is the newest one.
- It adds minimal amount of extra characters to the method names, keeping it short, sweet and to the point.
- The V clearly indicates that it does the same thing as the older ones, but is just the newer version.
Cons
- If the old method ever becomes unused, and you're able to get rid of it, you'll now be stuck with all your methods having version numbers. Which asks the question? Should I decrease the numbering of all the methods? I would say no, don't bother. And if you ever create an even newer one, start it at the end, even though there's room at the beginning.
You have a method that does more then one thing, how can the name reflect all the effects the method has?
I come upon this one occasionally. You've got a setter method for example, but it actually sets three fields from an Object you pass it. You could say it's just bad design, a method should not do more then one thing, but the truth is, these often are very convenient, and sometimes, you have to do it for performance.
SOLUTION
Name the method based on its primary effect, and append AndMore
to it. Then document what the AndMore
is with a comment.
So if you have setFirstNameAndLastNameAndAgeAndEthnicity(User user)
, just name it setFirstNameAndMore(User user)
. Don't forget to document the AndMore
in a comment!
Pros
- It keeps the name short.
- It makes it clear what the primary effect and intent is.
- It's very clear that the method has more effects, and gives a way for them to be further detailed if the caller cares.
Cons
- If you want to know what the other effects are, you need to read the doc.
- Maybe you shouldn't have a method with more then one intended effect, you should double check that there's no way for you to split this out.
You have a single implementation of an interface, what should you name the interface and its single implementation?
You've created an Interface and you feel good about it. You then create a first implementation of it. Slowly you realize, your use case doesn't need any other implementation quite yet. What should you call the Interface and its Implementation?
SOLUTION
IInterface
and Interface
or Interface
and DefaultInterface
There's two ways to go about it here. I prefer the first way, where you prefix the Interface with a capital I
and you simply call the implementation with the same name but without the I
. I like this because when you auto-complete, if you type I, you easily find all interfaces and generally, you want to type to the interface, not the implementation. Java conventions is to not put a I
on your interface. In that case, if you want to follow convention, use the second way. Call the interface as it is Interface
and prefix Default
to the implementation: DefaultInterface
.
If you have more then one implementation, but one implementation is more commonly used, I believe it should also then be called using the above naming scheme. While the other implementations should describe in their name what is different from the common one.
IDog
Dog
BigDog
SmallDog
HandicappedDog
Only if you have equally common implementations should you not have any of them be non descriptive of what sets them apart.
IAnimal
Dog
Cat
Horse
Pros
- Descriptive when it needs to be, undescriptive when it doesn't.
- Becomes a known convention for which implementation is the original canonical one.
Cons
- Some people find having an interface when you only have one implementation is unnecessary code complexity. In which case, you can argue you should only have an interface when you have two or more implementation, and then they should always be more descriptive then the interface name.
This is an always evolving article, as I find more and more patterns, or improve on the ones already existing, I'll update this article to reflect that.