Over a million developers have joined DZone.
{{announcement.body}}
{{announcement.title}}

Functional C#: Extracting a higher order function with generics

DZone's Guide to

Functional C#: Extracting a higher order function with generics

·
Free Resource

While working on some code with Toni we realised that we'd managed to create two functions that were almost exactly the same except they made different service calls and returned collections of a different type.

The similar functions were like this:

private IEnumerable<Foo> GetFoos(Guid id)
{
IEnumerable<Foo> foos = new List<Foo>();
try
{
foos = fooService.GetFoosFor(id);
}
catch (Exception e)
{
// do some logging of the exception
}
return foos;
}
private IEnumerable<Bar> GetBars(Guid id)
{
IEnumerable<Bar> bars = new List<Bar>();
try
{
bars = barService.GetBarsFor(id);
}
catch (Exception e)
{
// do some logging of the exception
}
return bars;
}

We're defining the empty lists so that if the service throws an exception we can make use of an empty list further on in the code. A failure of the service in this context doesn't mean that the application should stop functioning.

My thinking here was that we should be able to pull out the service calls into a function but the annoying thing is that they return different types of collections so I initially thought that we'd be unable to remove the duplication.

Thinking about the problem later on I realised we could just define the return value of the service call in the function to use generics.

We therefore end up with this solution:

private IEnumerable<Bar> GetBars(Guid id)
{
return GetValues(() => barService.GetBarsFor(id));
}
private IEnumerable<Foo> GetFoos(Guid id)
{
return GetValues(() => fooService.GetFoosFor(id));
}
private IEnumerable<T> GetValues<T>(Func<IEnumerable<T>> getValues)
{
IEnumerable<T> values = new List<T>();
try
{
values = getValues();
}
catch (Exception e)
{
// do some logging of the exception
}
return values;
}

I think the code is still quite readable and it's relatively obvious what it's supposed to be doing.

 

Topics:

Published at DZone with permission of Mark Needham, DZone MVB. See the original article here.

Opinions expressed by DZone contributors are their own.

THE DZONE NEWSLETTER

Dev Resources & Solutions Straight to Your Inbox

Thanks for subscribing!

Awesome! Check your inbox to verify your email so you can start receiving the latest in tech news and resources.

X

{{ parent.title || parent.header.title}}

{{ parent.tldr }}

{{ parent.urlSource.name }}