• benjirenji@slrpnk.net
    link
    fedilink
    English
    arrow-up
    1
    ·
    1 day ago

    You see it with code too. Constant refactoring, adding states to enums when they aren’t necessary, “beautifying” the code by adding useless checks, tests etc. The worst is when you got concurrency: focusing extremely on keeping protected sections small it keeps unnecessarily adding temporary variables, shared pointers etc. It’s so afraid of locks and data races the boilerplate for concurrency grows and grows.

    Reviewing such code is a pain. I feel like just adding comments like “why are you doing this” “justify this” “what does this have to do with your objective” and let my coworkers figure it out.