Nothing is wrong in that of course. I've run into home-rolled code that should have never gone into production. IIRC it was an implementation for encoding a string, instead of using either the java.net implementation or Apache commons implementation. Well their implementation forgot to encode the all-important new-line character, so they were sending out incomplete emails, emails that, depending on the response, could generate thousands of dollars of revenue per month and therefore had already cost the company tens of thousands, before I figured out the problem. There's even a name for this anti-pattern: "not invented here" (or "re-inventing the wheel")
Some comments have been hidden by the post's author - find out more
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
Nothing is wrong in that of course. I've run into home-rolled code that should have never gone into production. IIRC it was an implementation for encoding a string, instead of using either the java.net implementation or Apache commons implementation. Well their implementation forgot to encode the all-important new-line character, so they were sending out incomplete emails, emails that, depending on the response, could generate thousands of dollars of revenue per month and therefore had already cost the company tens of thousands, before I figured out the problem. There's even a name for this anti-pattern: "not invented here" (or "re-inventing the wheel")