Item 12: An operator’s meaning should be consistent with its function name
This is a chapter from the book Effective Kotlin. You can find it on LeanPub or Amazon.
Operator overloading is a powerful feature and, like most powerful features, it is dangerous as well. In programming, with great power comes great responsibility. As a trainer, I’ve often seen how people can get carried away when they first discover operator overloading. For example, one exercise involves making a function for calculating the factorial of a number:
As this function is defined as an extension function to Int
, its usage is convenient:
All mathematicians know that there is a special notation for factorials. It is an exclamation mark after a number:
There is no support in Kotlin for such an operator; however, as one of my workshop participants noticed, we can use operator overloading for not
instead:
We can do this, but should we? The simplest answer is NO. You only need to read the function declaration to notice that the name of this function is not
. As this name suggests, it should not be used this way. It represents a logical operation, not a numeric factorial. This usage would be confusing and misleading. In Kotlin, all operators are just syntactic sugar for functions with concrete names, as presented in the table below. Every operator can always be invoked as a function instead of using the operator syntax. What would the following look like?
The meaning of each operator in Kotlin always stays the same. This is a very important design decision. Some languages, like Scala, give you unlimited operator overloading capabilities. This amount of freedom is known to be highly misused by some developers. Reading code using an unfamiliar library for the first time might be difficult, even if it has meaningful names of functions and classes. Now imagine operators being used with another meaning that is known only to developers familiar with category theory. It would be way harder to understand. You would need to understand each operator separately, remember what it means in the specific context, and then keep all this in mind to connect the pieces in order to understand the whole statement. We don’t have such a problem in Kotlin because each operator has a concrete meaning. For instance, when you see the following expression:
x + y == z
You know that this is the same as:
x.plus(y).equal(z)
Or it can be the following code if plus
declares a nullable return type:
(x.plus(y))?.equal(z) ?: (z === null)
These functions have concrete names, and we expect all functions to do what their names indicate. This greatly restricts what each operator can be used for. Using not
to return factorial
is a clear breach of this convention and should never happen.
It is good to mention, that this rule was even breached by Kotlin stdlib, as it defines div
extension on Path
to allow the following use:
It is not so beautiful when explicit name is used. Such extension make Kotlin more "magical", and I suggest to avoid using them if we want to keep our code readable.
Unclear cases
The biggest problem is when it is unclear if some usage fulfills the conventions. For instance, what does it mean when we triple a function? For some people, it is clear that this means making another function that repeats this function 3 times:
For others, it might be clear that it means that we want to call this function 3 timesdiff:
When the meaning is unclear, it is better to favor descriptive extension functions. If we want to keep their usage operator-like, we can make them infix:
Sometimes it is better to use a top-level function instead. Repeating a function 3 times is already implemented and distributed in the stdlib:
When is it fine to break this rule?
There is one very important case in which it is fine to use operator overloading in a strange way: when we design a Domain Specific Language (DSL). Think of a classic HTML DSL example:
You can see that to add text to an element, we use String.unaryPlus
. This is acceptable because it is clearly part of the Domain Specific Language (DSL). In this specific context, it’s not surprising to readers that different rules apply.
Summary
Use operator overloading conscientiously. A function’s name should always be coherent with its behavior. Avoid cases where operator meaning is unclear. Clarify it by using a regular function with a descriptive name instead. If you wish to have a more operator-like syntax, then use the infix
modifier or a top-level function.
The difference is that the first one produces a function while the other one calls a function. In the first case, the result of the multiplication is ()->Unit
, but in the second case it is Unit
.