DevForce Community Forum : Coroutine Error Handling

Coroutine Error Handling : DevForce does actually check for...

Fri, 18 Mar 2011 17:02:01 -700

Author: kimj
Subject: 2565
Posted: 18-Mar-2011 at 5:02pm

DevForce does actually check for errors after each async operation, and does not call subsequent operations in the iterator block if a prior one failed, unless you've marked the error as handled. If you set up a handler for the Coroutine completed you can see there if an error occurred, plus all the "Notifications" for each async operation up to and including the operation which failed. You don't need to include error checking within your iterator unless you want to, or if you will be marking the error as handled so that subsquent operations can run.

Coroutine Error Handling : This was more of a question of,...

Thu, 17 Mar 2011 12:50:57 -700

Author: ken.nelson
Subject: 2565
Posted: 17-Mar-2011 at 12:50pm

This was more of a question of, "what is DevForce doing under the hood automatically?". From the sound of it, my conclusion is that DevForce is not checking the error state of the each async operation while executing in a coroutine. And this is perfectly ok, I just wasn't sure if it was necessary to check for errors after each operation or if it was already being done under the hood.

Thanks!

Coroutine Error Handling : Ken, write yourself some extension...

Thu, 17 Mar 2011 09:38:29 -700

Author: mikewishart
Subject: 2565
Posted: 17-Mar-2011 at 9:38am

Ken, write yourself some extension methods such as:

///

/// Same as ExecuteAsync() except it tests for an error before returning the result
///

    public static EntityQueryOperation ExecuteAsyncResult(this IEntityQuery query,
      Action> userCallback)
    {
      return query.ExecuteAsync(op => CheckExecuteAsyncResult(op, userCallback), null);
    }

    ///

/// Same as ExecuteAsync() except it tests for an error before returning the result
///

    public static EntityQueryOperation ExecuteAsyncResult(this IEntityQuery query,
      Action, object> userCallback, object userState)
    {
      return query.ExecuteAsync(op => CheckExecuteAsyncResult(op, r => userCallback(r, op.UserState)), userState);
    }

    private static void CheckExecuteAsyncResult(EntityQueryOperation op,
      Action> userCallback)
    {
      if (op.HasError)
        throw new Exception(op.Error.Message);

      userCallback(op.Results);
    }

Coroutine Error Handling : Hi Ken, It is always a good...

Wed, 16 Mar 2011 12:13:10 -700

Author: sbelini
Subject: 2565
Posted: 16-Mar-2011 at 12:13pm

Hi Ken,

It is always a good idea, and the developers responsability, to check for errors.

On a coroutine operation you should check for errors before yielding to the Coroutine consumer:

////////////////////////////////////////////////////////////////////////////////////////
private IEnumerable LoadTopCustomersCoroutine() {

  var userId = CurrentUser.UserId;

  var allCustOperation = Manager.Customers.ExecuteAsync();

// event handler
var allCustOperation.Completed += (s1, args1) => {
    if (args1.HasError) HandleError(args1);
  };

yield return allCustOperation; // SUSPEND
// more code
}

////////////////////////////////////////////////////////////////////////////////////////

private IEnumerable LoadTopCustomersCoroutine() {

  var allCustOperation = Manager.Customers.ExecuteAsync(
     // Callback checks for error
     op => { if (op.HasError) HandleError(op); }
    );

yield return allCustOperation;
// more code
}

////////////////////////////////////////////////////////////////////////////////////////

If an error occurs, you can let the error bubble up to the outer Coroutine or you can take care of the exception and mark the error as handled:

op.MarkErrorAsHandled();

You will also find this information in our DevForce Resource Center.

Regards,

Silvio.

Edited by sbelini - 16-Mar-2011 at 12:16pm

Coroutine Error Handling : Is it necessary to error check...

Wed, 16 Mar 2011 08:05:16 -700

Author: ken.nelson
Subject: 2565
Posted: 16-Mar-2011 at 8:05am

Is it necessary to error check each coroutine operation's error state after each yield return or is it eoungh to handle errors in the Coroutine completed callback? (Is the red highlighted code below necessary?)

Without the use of a Coroutine, my code would look something like the following, with error checked after each async call has completed:

private void LoadData() {

     em.Customers.Where().ExecuteAsync((customerArgs) => 
     {
          if (customerArgs.HasError) { // handle error }
          else 
          {
               em.Orders.Where().ExecuteAsync((orderArgs) => 
               {
                    if (orderArgs.HasError) { // handle error }
                    else
                    {
 
                    }
               });
          }
     });
}

Translating to Coroutine, my code is:

private void LoadData()
{
     Coroutine.Start(CoroutineActions, (completed) =>
     {
          if (completed.HasError) { // handle error }
          else
          {
 
          }
     });
}
 
private IEnumerable CoroutineActions()
{
     var op1 = em.Customers.Where().ExecuteAsync();
     yield return op1;
 
     // Do I have to continue to error check after each async operation, or
     // is it ok to skip this?
     if (op1.HasError) { // handle error }
     else
     {
          var op2 = em.Orders.Where().ExecuteAsync();
          yield return op2;
     }
}