Here’s a pattern that I’ve seen occur a number of times - a pair of method overloads, one accepting IEnumerable<T> and the other accepting params T[].

void Process(IEnumerable<string> commandline) { ... }
void Process(params string[] parameters) { ... }

Often, the params edition is provided as a convenience overload, making the API simpler for consumers. The implementation is usually a simple delegation call to the main method:

void Process(params string[] parameters)
{
    Process(parameters);
}

There’s a trap here though - as shown, you’ll get a Stack Overflow exception thrown in good order.

The reason is simple - but perhaps not obvious. When the compiler is matching method signatures to work out which version of Process() to invoke, there’s a better match with string[] than with IEnumerable<string>, so the generated code goes into a tight loop.

The solution is to cast parameters to the more general type so that there is only one viable match:

void Process(params string[] parameters)
{
    Process( (IEnumerable<string>) parameters );
}

The way that the compiler selects which overload to call is entirely predictable, but complex, and sometimes gives unexpected results. It’s worth learning enough about the process to be able to troubleshoot problems (like this one). That said, if your specific problem is too complex, the best answer might be to rename some of the overloads so that a future maintainer doesn’t have the same problem to face.

Comments

blog comments powered by Disqus
Next Post
Doing less with LINQ  30 Jun 2010
Prior Post
NAntGraph v2.2 Released  16 Jun 2010
Related Posts
Error Methods  25 Nov 2017
Pass implementations, not representations  14 Oct 2017
Avoiding the Singleton Pattern  22 Jul 2017
Implementing the Singleton Pattern  15 Jul 2017
Static Analysis tools for the Win  15 Apr 2017
Exception Logging  19 Jan 2016
On the Merits of Simple Code  28 Nov 2015
Semantic Types  27 Sep 2015
Command Line Processing  21 Sep 2014
Easy String Conversion  24 Aug 2014
More smart-code posts »
Related Pages
June 2010 archive