Question

Does anyone have advice for using the params in C# for method argument passing. I’m contemplating making overloads for the first 6 arguments and then a 7th using the params feature. My reasoning is to avoid the extra array allocation the params feature require. This is for some high performant utility methods. Any advice? Is it a waste of code to create all the overloads?

Answer 1

Honestly, I’m a little bothered by everyone shouting “premature optimization!” Here’s why.

1. What you say makes perfect sense, particularly as you have already indicated you are working on a high-performance library.
2. Even BCL classes follow this pattern. Consider all the overloads of string.Format or Console.WriteLine.
3. This is very easy to get right. The whole premise behind the movement against premature optimization is that when you do something tricky for the purposes of optimizing performance, you’re liable to break something by accident and make your code less maintainable. I don’t see how that’s a danger here; it should be very straightforward what you’re doing, to yourself as well as any future developer who may deal with your code.

Also, even if you profiled the results of both approaches and saw only a very small difference in speed, there’s still the issue of memory allocation. Creating a new array for every method call entails allocating more memory that will need to be garbage collected later. And in some scenarios where “nearly” real-time behavior is desired (such as algorithmic trading, the field I’m in), minimizing garbage collections is just as important as maximizing execution speed.

So, even if it earns me some downvotes: I say go for it.

(And to those who claim “the compiler surely already does something like this”–I wouldn’t be so sure. Firstly, if that were the case, I fail to see why BCL classes would follow this pattern, as I’ve already mentioned. But more importantly, there is a very big semantic difference between a method that accepts multiple arguments and one that accepts an array. Just because one can be used as a substitute for the other doesn’t mean the compiler would, or should, attempt such a substitution).