Small Naming Tip: Use "from" Instead of "to" in Function and Variable Names
Today, I'm tackling a piece of code involving several mappings. Like many others, I typically label such maps as x_to_y
, and that's also what I've done here:
let mut value_to_var: HashMap<Value, (String, usize)> = HashMap::new();
let mut var_to_num: HashMap<String, usize> = HashMap::new();
// This is not a hash table but I use the index as the key
let mut num_to_canonical_var = vec![];
When I need to use those maps, my code often looks like this:
let canonical_variable = num_to_canonical_var[var_to_num.get(var).unwrap()];
As you can observe, the sequence isn't particularly intuitive. The code executes a transformation from "variable -> number -> canonical variable," but understanding it requires reading from the inside out, akin to unraveling a spiral. Reading inside out can be challenging. For example, despite my nearly decade-long experience with C and C++, I still sometimes struggle to decipher the inside-out syntax of C declarations. Others share this sentiment as well.
What if I change all the to
to from
?
let mut var_from_value: HashMap<Value, (String, usize)> = HashMap::new();
let mut num_from_var: HashMap<String, usize> = HashMap::new();
let mut canonical_var_from_num = vec![];
Now, we can write code like the following:
let canonical_variable = canonical_var_from_num[num_from_var.get(var).unwrap()];
This code is much easier to comprehend, as related names share physical proximity. Its transformation is also readily apparent just by reading from right to left.
The same idea can also be applied to functions. For instance, let shader = shader_from_file(file)
reads better than let shader = file_to_shader(file)
. It isn't a coincidence, considering that both functions (at least the pure ones) and a map data structure aim to create a mapping between data.
Update 2024-02-10: Several users on Mastodon have noted that they prefer to name their map variables as x_by_y
, which shares similarity with the x_from_y
I present here.
Update 2024-02-11: Another user on Mastodon mentioned that Joel Spolsky wrote about the exact same suggestion. The article also provides an excellent description of the "Apps Hungarian" notation and how it differs from the more cumbersome "Systems Hungarian" notation that most people are familiar with. Personally, I question the relevance of even Apps Hungarian in modern languages with expressive type systems. If something possesses unique semantics and is utilized frequently enough to warrant a naming convention, we should create a strongly typed wrapper for it.