3 reasons why I think this approach should not be encouraged and should have 0 on Best Practices:

1. Security risk: Using eval() can be dagerous, especially if the input comes from an untrusted source. It can execute arbitrary code, leading to potential security vulnerabilities.
2. Hard to Debug: Errors in dynamically evaluated expressions can be harder to trace compared to explicit conditional branches.
3. Performance: eval() can be slower than direct operations due to the overhead of parsing and interpreting the string.