At a glance, I would say your original code, while not dry is actually better in terms of long term maintenance and easier for people to jump into.
The reason I say this is that we learn to make things dry and obsess to reduce our code but when you have to maintain a project for 5+ years and developers come and go you realize this is where the mantra "Clever vs Clear" steps into effect and you look at your code in a different way.
You're already a Pro 👍
Thanks for the comment! I think I’m going to disagree. Although I concede that at a glance, reading the hard coded version is easier to jump into, from a long term maintenance perspective it isn’t better. I’ve had to update the data for this interactive twice now, and both times something went wrong at first because I forgot to fully implement the new year. I’ve built a lot more interactives using the data driven approach that don’t have that same problem and can be updated just by uploading a new file. Yes, it takes longer for a new dev to read the code to understand how it works, but it’s straight forward enough that I think that extra mental effort pays for itself.
There are so many unknowns still unanswered so the best reply I could give was 'at a glance'.
We're a place where coders share, stay up-to-date and grow their careers.
We strive for transparency and don't collect excess data.