The Ternary Metaprogramming Trap
TL;DR: Avoid using ternary operators for dynamic method calls
Ternary metaprogramming uses conditional operators to select and invoke methods dynamically.
It leads to code that's harder to understand, debug, and maintain.
You risk introducing subtle bugs and making your code obscure to other developers.
Clean Code is the opposite of Clever Code.
const method = success ? 'start' : 'stop';
obj[method]();
if (success) {
obj.start();
} else {
obj.stop();
}
Your linters can detect this smell by looking for ternary operators to select method names, especially when combined with bracket notation for method calls.
You can also watch for variables that store method names based on conditions.
AI code generators might introduce this smell since they prioritize code brevity over readability.
They could generate ternary metaprogramming patterns when trying to produce concise code.
AI detectors can identify this smell by recognizing patterns of ternary operators used for method selection.
They may need specific instructions about readability and maintainability.
Remember AI Assistants make lots of mistakes
ChatGPT Claude Perplexity Gemini
Ternary metaprogramming can seem clever and concise but creates more problems than it solves.
By favoring explicit conditionals and well-named methods, you can write easier-to-understand, debug, and maintain code.
Remember that code is read far more often than written, so prioritize clarity over brevity.
https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-ii-o96s3wl4
https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-xlii
https://hackernoon.com/how-to-find-the-stinky-parts-of-your-code-part-v-evj3zs9
Code Smells are my opinion.
Programs must be written for people to read, and only incidentally for machines to execute.
Harold Abelson
This article is part of the CodeSmell Series.