





Over the years, I’ve participated in numerous code reviews, and I’ve seen a few function names that were worthy of true shame. However, the Hall of Shame demands only the best (or the worst, depend on your point of view), and so, I have whittled it down to a few of the most worthy. Consequently, I nominate the following for the Hall of Shame:

NOTE: Could this function be any more ambiguous?

private string GetTableData() NOTE: Our new programmer just might be dyslexic or suffer from something worse

public int ManageJoinUsingDelete(…) NOTE: Do we need to know every step involved in the name?

private int GetUpperRangeOffsetBySubtractingProductOfTwoSums() NOTE: I hope that this ‘body’ refers to text and not a dead hooker

public void EndStreamingNoAttachIfBody(…) NOTE: This function needs more ‘temp’

public int PopulateTempTemporaryTransfer(…) NOTE: This seems so wrong on so many levels

public bool IsFalse() NOTE: He’s not exactly creative, is he?

public void TransformData(…)

public void TransfromDataPlus(…)

public void TransformDataPlusExtra(…) NOTE: They do say that string mathematics is a dying art

private string CalculateNextLessThanOrEqualNumber(string) NOTE: ???

private string GenerateTempGetUpdateOrGetInsertWhereCriteria(int) NOTE: I like it…straightforward, single in use, and stupidly simplistic

private string SurroundWithParantheses(string)

Peter Bolton is the author of Blowing the Bridge: A Software Story and has also been known to be a grumpy bastard on occasion.