3 ms·

I saw a very similar thing with eval. There was an evaluation of a nested JSON object x["a"]["b"]["c"] And the developer decided that this was best evalu

by rsj_hn 5y ago

I saw a very similar thing with eval. There was an evaluation of a nested JSON object x["a"]["b"]["c"] And the developer decided that this was best evaluated by eval. During the code review phase I talked to them and asked why they were using eval, and they didn't know it could be evaluated directly as they were a little unclear if javascript supported that syntax.

jdc 5y ago
How much longer did said dev continue working there?
- rsj_hn 5y ago
  err, no comment
  - dylan604 5y ago
    that dev is now your manager? /s
    - RexM 5y ago
      He is the dev
      - rsj_hn 5y ago
        LOL, no to both!
- dylan604 5y ago
  Fire the dev, or educate the dev. Sure, eval is just a sure sign of a lot of lack of understanding, but I'd hope for something less vile as eval would be a little more understanding that everyone learns something sometime
  - iratewizard 5y ago
    It's gross negligence of the dev is only a Javascript dev, but maybe understandable if it's just one of the three languages he uses regularly. I probably wouldn't fire, but I also can't imagine any of my devs doing that.
  - deleted 5y ago
    [deleted]
  - deleted 5y ago
    [deleted]