Hacker News new | ask | show | jobs
by BearfootCoder 1834 days ago
The inverse of this is also useful:

// Dear maintainer,

// The code which follows was hacked together under desperate pressure.

// It's not smart. // It's not doing anything really subtle. // It's the only thing we could get to work in the time.

// If you want to chuck it and replace it, you absolutely should.

2 comments

Hey, that one I actually can replace with self-documenting code! Oh joy!
It's useful, until you realize you have this on 90% of your code.
That's when a full rebuild can be justified.

I'm doing that atm, but I think my predecessor lacked the humility to admit that his code was bad, which is how it went up to 150K of code that basically concatenates XML, converts it to JSON and concatenates HTML on the front-end from the result.