I have a new position where I finally have the long sought after Sr. prefix included. Only time will tell if I truly deserve the prefix, but I believe that I can do the job. One of the things that is different about this position by comparison to my most recent projects / positions is a more formalized deployment / implementation process (THANK GOD FOR THIS!!! :-D). Part of this process is a code review with several other developers that are at least as good as I am...and in many cases better.
I had my first code review today, and the only real issue anyone had with it (aside from learning naming conventions) involved a bit of code that I have used for years on end:
Set @RecordCount =
The code goes on to split the logic path based on @RecordCount = 0 vs. @RecordCount > 0. This is a section of code that I could have said I had down as well as tying my own shoes (there is a Ted Talks reference there, and this entire post lines up with it).
The conversation went something like this:Other developer:
Why did you use this code here?Me:
Because I was needing to split the logic path.Other developer:
Yes, I see that. What I mean was why didn't you use:
Set @RecordCount = @@RowCount
Because I had never heard of it, thought of it, or seen it before you just typed it.
This post is meant to remind not just me but everyone who reads this to look at those parts of our code that seem all but written in stone that we will use the same approach each time. Is there a better way to do what you're doing?
A small advantage in any area of life can sometimes be the difference between adequate results and all star results.
“Any fool can know. The point is to understand.”
- Albert Einstein
- Homer Simpson