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.