I continue to see signs of the apocalypse when I’m a work. Today it was
the implementation of a “dirty” flag (indicates when a form has been
edited and should trigger a save). Most people would create a
module/class level variable that would store this value. Not this
developer. Why do something so clean when you can add a textbox to the
control, make it invisible and push a zero or one into it?
Better yet was the function that I found today which had an ID value
being passed in as a parameter. Instead of making the parameter of an
Integer data type it was a String. Inside the function the string was
cast to an integer and passed into the stored procedure. The reason I
found this was because the function’s parameter was getting passed in
as a blank value in some instances. You cant cast a blank to an integer.