← Back to context

Comment by shermantanktop

1 day ago

In Java it can be a bad toString() implementation hiding behind a + used for string assembly.

Or another great one: new instances of ObjectMapper created inside a method for a single call and then thrown away.

To be clear this is often sloppy code that shouldn’t have been written. But in a legacy codebase this stuff can easily happen.

  • A huge chunk of a "legacy codebase" is "sloppy code that shouldn’t have been written"

    Unless you're inheriting code written by Bill Atkinson or something.