Steve L's was feeling pretty excited to start a new job. The company was getting into some challenging new projects and the guys he met in the interviews seemed bright and knowledgeable about the latest and greatest. Heck, one of them even had a blog.

After working there for a few days, Steve's excitement had faded into bewilderment. He questioned his new coworkers about their uniquely incorrect practices, but they just assured him that it was just different; some shops just did things differently. For example, they felt the need to have one extra layer of confirmation before doing "serious" things. Such "serious" methods would only do their thing when True was passed as the value of the Confirm parameter ...

Public Sub DeleteAccount(AccountNum As String, Confirm As Boolean)

  If Confirm <> True Then Exit Sub

  Dim cmd As New ADODB.Command
  With cmd
    .ActiveConnection = getConn()
    .CommandText = "DELETE_PLAYER_ACCOUNT"
    .Parameters.Append _
        .CreateParameter("ACCOUNT_NUM", adChar, adParamInput, 8, AccountNum)
    .Parameters.Append _
        .CreateParameter("CONFIRM", adInteger, adParamInput, 4, CInt(Confirm))
  End With

End Sub
[Advertisement] BuildMaster allows you to create a self-service release management platform that allows different teams to manage their applications. Explore how!