What was I thinking? — A case for code reviews
At my 8-5 job, I’m the only Windows developer we have. I have ported an existing old app from VB6 to .Net and then maintained that app for the past 18 months. Today we’re working on some major refactoring that requires me to pull out chunks and make libraries (instead of a large Winform application.)
Today I’m working on the part that loads some data from the database. I found this gem:
loadedQuote.ParentNumber = loadedQuote.ParentNumber;
I guess I just really wanted to make sure that the Parent Number was the Parent Number. Unfortunately, since I’m the sole person to touch this entire application, I have nobody to blame but myself.