Of course, "brevity is the soul of wit" ..... But I would like to emphasize 3 points here:

1. Safety: Using eval can be dangerous if the input is not fully controlled. This can lead to the execution of unwanted or malicious code, which can be a security risk.

2. Performance: eval can be slower than other code execution methods because it requires interpreting the string as code.

3. Debugging and readability: Code that uses eval can be more difficult to debug and maintain. This can make the code less predictable and harder to understand.

I'm not judging, just sharing my opinion. Alternatively, you can use switch-case or map.

Not declaring variables is bad practice

Without 'use strict' this will work