CodeSOD: A Generic Comment

To my mind, code comments are important to explain why the code what it does, not so much what it does. Ideally, the what is clear enough from the code that you don't have to. Today, we have no code, but we have some comments.

Chris recently was reviewing some C# code from 2016, and found a little conversation in the comments, which may or may not explain whats or whys. Line numbers included for, ahem context.

4550: //A bit funky, but entirely understandable: Something that is a C# generic on the storage side gets4551: //represented on the client side as an array. Why? A C# generic is rather specific, i.e., Java4552: //doesn't have, for example, a Generic List class. So we're messing with arrays. No biggie.

Now, honestly, I'm more confused than I probably would have been just from the code. Presumably as we're sending things to a client, we're going to serialize it to an intermediate representation, so like, sure, arrays. The comment probably tells me why, but it's hardly a necessary explanation here. And what does Java have to do with anything? And also, Java absolutely does support generics, so even if the Java trivia were relevant, it's not accurate.

I'm not the only one who had some... questions. The comment continues:

4553: //4554: //WTF does that have to do with anything? Who wrote this inane, stupid comment, 4555: //but decided not to put comments on anything useful?

Not to sound judgmental, but if you're having flamewars in your code comments, you may not be on a healthy, well-functioning team.

Then again, if this is someplace in the middle of your file, and you're on line 4550, you probably have some other problems going on.

[Advertisement] Otter - Provision your servers automatically without ever needing to log-in to a command prompt. Get started today!