jonpsmith / genericservices.statusgeneric Goto Github PK

Hi,

Currently, if someone would like to cut processing short and exit from a method if an operation errors out while still carrying information about the errors in the operation, it would be necessary to write code like this:

var status = new StatusGenericHandler<SomeEntity>();

// ... some processing

IStatusGeneric<AnotherEntity> anotherStatus = await myService.SomeOperationThatReturnsAnotherEntity();
if (anotherStatus.HasErrors)
{
    status.CombineStatuses(anotherStatus);
    return status;
}

// ... more processing

return status;

However, if there was an extra CombineStatuses overload that would match IStatusGeneric<T> instead of IStatusGeneric, one would be able to simplify it a little bit to something like this:

var status = new StatusGenericHandler<SomeEntity>();

// ... some processing

IStatusGeneric<AnotherEntity> anotherStatus = await myService.SomeOperationThatReturnsAnotherEntity();
if (anotherStatus.HasErrors)
    return status.CombineStatuses(anotherStatus);

// ... more processing

return status;

It looks like a tiny change but sometimes it's cumbersome to keep writing the first version if you have lots of operations to call :-) Do you think this would be a good addition to the library?

I think there are some typos in the doc for CombineStatuses.