Ruby primitives are so powerful that we use them everywhere in our code, even when we shouldn't. How many times we express a path or a URL with
String instead of
path = "path/to/file.rb" url = "https://dnsimple.com" # vs require "uri" require "pathname" path = Pathname.new("path/to/file.rb") url = URI.parse("https://dnsimple.com")
Ruby literals are handy to use and we don't recognize this scheme anymore. Until these are just values to pass around the two versions of the code above are more or less equivalent. But we can run soon into problems when we need to perform operations on these values. Does the file exist? Is the URI wellformed? Does it use HTTPS? These are questions that
String can't answer.
This problem is known as Primitive Obsession.
In a recent refactoring, I spotted a constant in our code:
REGIONS = [ ['California, US', 'SV1'], ['Illinois, US', 'ORD'], ['Virginia, US', 'IAD'], ['Amsterdam, NL', 'AMS'], ['Tokyo, JP', 'TKO'], ]
We used an array of arrays to express a domain concept: a region is a DNSimple data center. The first value in the array is a name and the second one is a code. Again, this usage is handy, but then we soon payed the price of obscure code scattered across the application:
This "innocent" line above hides a few maintenance problems:
REGIONSstructure. What if we use a
To solve this problem, there are just a few steps to follow:
Region, in our case.
REGIONSinside the class.
Now we have:
This refactored code solves a few problems:
Region was extracted, I delegated to it the responsibility to validate user input (via
Region.filter(data)) and I wrote an extensive set of unit tests to cover common and edge cases.
When we use proper model domain types we improve readability, robustness (via extra tests) and new functionalities has a obvious place where to be implemented.
Former astronaut, soccer player, superhero. All at the age of 10. For some reason now I write code.
DNSimple is moving our DNSSEC out of beta and into general availability.
Configure DNSimple as your secondary DNS provider to improve your domain's availability and redundancy with AXFR zone transfers.