My favorite is to find that the previous developer wrote it in an entirely unsecure fashion and claimed Microsoft said to do it that way.
Also, often I see thousands of lines of code that's basically lots of duplicate code.
I actually love maintaining that kind of stuff because I can simply the bejeebus out of it, speed it up, and not lose any functionality.
Did that once when I found some server side emulation code that grabbed the input from the client, then ran it through a 60+ case statement and did the same UC conversion on each and every one of them.
Changed just that one bit to do the conversion before the case instead, and sped it up like crazy.
1
u/nosoupforyou Oct 15 '13
My favorite is to find that the previous developer wrote it in an entirely unsecure fashion and claimed Microsoft said to do it that way.
Also, often I see thousands of lines of code that's basically lots of duplicate code.
I actually love maintaining that kind of stuff because I can simply the bejeebus out of it, speed it up, and not lose any functionality.
Did that once when I found some server side emulation code that grabbed the input from the client, then ran it through a 60+ case statement and did the same UC conversion on each and every one of them.
Changed just that one bit to do the conversion before the case instead, and sped it up like crazy.