Do all the things like ++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatarSign Up
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple APILearn More
magicMirror10362205dMuch better now.
IHateForALiving1682205dI'm sorry. It's not 2 methods. It's 3.
Side note, the config passed as a parameter is pissing me off beyond reason.
This is correct.
Max 6 lines, though there is no limit on how many characters >:D
100+ lines switch case?!
@lambda123 hell, no! 😆
I like your sarcasm and I‘m with you about the misuse of design patterns and abstraction.
But I consider 100+ lines of switch case not to be "clearly in one spot".
It could be reasonable in some rare case but as I said: It smells.
IHateForALiving1682204d@Lensflare it's 130 lines with logging, plenty of carriage returns, comments and stuff like that.
The actual code is stuff like this; you receive a message from Line, those messages can have varying formats, you forward those messages to our own application. There's no black magic going on there.