I think yes and no on overthinking. Let’s say we had a writing prompt, “an adventurer breaks into a sealed tomb and finds the undead creatures inside have kept it immaculate, and indeed, the adventurer sees skeletons and zombies cleaning the walls and sarcophagi.” I think we’d all come up with very different stories—starting with the adventurer’s sex. Maybe one person would write about how boredom has taken hold of them, maybe another would write a story about a necromancer who hates dust, maybe another would write about how this was the burial chambers for a castle’s servants, maybe another would be about how they’ve found religion and think sacred places should be treated with respect.
And if you were to reflect on the writing choices you made, you might think, for this particular bit I was inspired by The Lord of the Rings, or Earthsea, or Frankenstein, or Gulliver’s Travels, or Cat’s Cradle, or whatever.
Half the stories would be in first-person, half would be third-person, and someone would inevitably write in second-person. Some stories would be past tense, some would be present tense, and maybe some cheeky writer would try for an imperfect tense. (And Ursula le Guin would use mixed past and present tense.)
I wouldn’t view any of this as overthinking, not least because reflecting on the choices you made is worthwhile. And it’s definitely worth talking about the choices everyone made. Those choices would, at first order, be instinctual, from all your prior experience reading and writing. Even high-minded stuff like symbolism is usually something that happens midstream, like you instinctively focus on certain things, tease out what drew you to those things, and then start being more explicit about the ways you’re using them.
I know this is a bit roundabout, but I guess my point is, we are mostly talking about style here and style is a subtle thing. I don’t think my style is “right,” or that other styles are “wrong.” I mostly think it’s interesting. People have different tastes, and I was just posting about mine.
And I did say, I don’t think performance was actually what caused my instinct to case-enumeration (or monolithic) style. More than anything else, I tend to find code with a lot of indirect evaluation hard to understand. Which is at the heart of the original question: the fact that x2_s3_onhitcast is invoked from a 2da file of spell definitions would’ve taken a lot of work to figure out on my own. Especially since that’s very different from the way On Use: Unique Power is handled.