Which bit of code do you think is more maintainable by someone other than the original developer:
if(_speakerTypeCode == 543678){
//do something
}
or
enum SpeakerTypes {
HornLoaded = 543678,
Electrostatic = 12334
}
if(_speakerTypeCode.Equals( SpeakerTypes.HornLoaded)){
//do something
}
I'm still amazed at some of the things I find in code, sometimes even my own code. The above monstrosity is not mine, although I do occasionally find something abhorrent in my own code. I read a blog recently, I believe it was the Shade Tree Developer, who said that this is a great way to gage personal growth. Go back and look at code you wrote and see if you find things that you'd never do today.