It's not clear from your post (and you don't have to make it clear, of course) whether you're getting fired from your current company or not. Merely failing one task won't get you fired and if you have the self-awareness demonstrated above to realize that you haven't yet learned what you need to become senior, gives you a clear path (and one that won't even be that hard if you've been selling and making software for…
Wait, what was wrong with the #define solution? Just the name of the macro? Didn't he do what you wanted?
Let's imagine that we later decide we want to make a change and have it be a little bigger. Do we change it to:
#define SEVENTEEN 23
Instead, if it was: #define MENU_SCALING_FACTOR 17
then such a change would be sensible (which was the intention). I admit to having some sliver of culpability here (though the verbal explanation of why we'd prefer to pull the hard-coded numbers out of being sprinkled all over the codebase I assumed would have sufficed).This seemed to border on /r/MaliciousCompliance, but I know he was actually trying his best (and was pre-reddit, of course).