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.