4 ms·
> The remaining part of the team tried picking it up, but it was too alien, either because of the FPness or just because the code was opaque. Sounds like they
by catiopatio 4y ago
> The remaining part of the team tried picking it up, but it was too alien, either because of the FPness or just because the code was opaque.
Sounds like they hired people unqualified to take over product maintenance.
If your engineers can’t wrap their head around ”FP-ish” JavaScript, they’re not exactly batting .300.
- seattle_spring 4y agoThere’s plenty of code that can be so complicated that it becomes easier to rewrite versus understand. I worked with a coworkers who made the most absolutely spaghetti-like JS modules, insisting on event emitters and queues and stuff like that for even the most basic tasks. The result was an incomprehensible mishmash of events shooting from file to file to file. Touching how these events were handled in some way might destroy how they were interpreted for completely different tasks, it was wild. It’s very possible that the code in question wasn’t just opaque because it was “FP-ish”, giving the parent the benefit of the doubt.
- snovv_crash 4y agoExactly. Code that is too asynchronous with lots of pubsub basically turns into GOTOs with data - there's no way to trace the execution step by step.
- Kamq 4y agoAt least gotos generally tell you where they're going.
- wpietri 4y agoDepends on what you mean by "FP-ish". I never saw this code, but the way this was described to me as high church FP, the kind of people who really wanted to be working in an ivory tower FP language and so had coded as if that's what they were using. I don't think it's unreasonable for average Javascript programmers to look at that and say, "WTF", the same way that they'd look at code from somebody who wrote C in Javascript.