Question

I’m writing a library and instead of returning a byte array from an EventArgs derivation, it says I should return something like IList or ReadOnlyCollection instead.

Normally I’d be all for this but most of the existing .NET Framework uses byte arrays as opposed to generic list interfaces.

So if I were to use IList then when accessing the eventargs, if a client wanted to call File.WriteAllBytes he or she would have to do using System.Linq; and call the ToArray extension method to get the IList in the form of an array of bytes. Of course there are other ways to do this but this is the most elegant and typical.

Clients of this library are always going to want things to be in terms of an array of bytes so that they interface nicely with the rest of the framework.

Also, optimization may come in to play here. There is potential for large amounts of bytes to be manipulated so having to recopy the entire list just to get it in the form of a byte array each time would likely slow things down.

Lastly, it’s just plain unpleasant. If clients are always going to want a byte array, then why not just give it to them? Do framework design guidelines not apply in this situation? What would you do?

Answer 1

Clients of this library are always going to want things to be in terms
of an array of bytes so that they interface nicely with the rest of
the framework.

There you have your answer – FxCop output is in most cases just helpful suggestions – not commands – if this particular one doesn’t apply to you you can even turn it off.