Click here to Skip to main content
15,860,859 members
Articles / Mobile Apps / Xamarin

Fixing TPL Using Responsive Tasks (Core Library)

Rate me:
Please Sign up or sign in to vote.
0.00/5 (No votes)
16 Apr 2021MIT2 min read 4.9K   2  
Replaces events and any other async void cases with awaitable Task signatures.
Here we look at: what's wrong with Task Parallel Library, and then do a brief overview of my code, including a digested Debug output from the ResponsiveAppDemo, and discussing how the Responsive Tasks library handles a variety of issues using a thread-safe "wait" strategy.

IMPORTANT

TPL: The Promise

When Microsoft announced the Task Parallel Library, C# programmers everywhere rejoiced. Simplified threads? Error handling within the library itself? What could possibly go wrong?

Just about everything, as it turns out.

A task requires a root in order to be properly awaited. For instance:

C#
// An awaitable task
public Task YouCanAwaitMe() { }

// A root where you an await a task
public async Task IWillAwait()
{
   await YouCanAwaitMe().WithoutChangingContext();
}

TPL: The Reality

Unfortunately, a Xamarin app doesn't have any valid roots. For instance, not at:

  • Constructors
  • Content changed
  • Binding context changed
  • Event handlers
  • Global messages
  • Overrides
  • Property Setters

Any location that fails to provide a Task signature is a false root. This causes unsafe results:

C#
public class FalselyRootedView : ContentView
{
   protected override async void OnBindingContextChanged()
   {
      base.OnBindingContextChanged();
      
      // Mega hack -- called from a void method (illegal!)
      await StartUpViewModel().WithoutChangingContext();
   }
   
   public virtual Task StartUpViewModel()
   {
      return Task.CompletedTask;
   }
}

// Derive and consume the falsely rooted view as if it were valid
public class FalseConsumer : FalselyRootedView
{
   pubic override async Task StartUpViewModel()
   {
      // Everything seems OK from this perspective, but this task can proceed at any time and 
      //    without our control; it was never properly awaited. Anything relying on it will 
      //    accelerate into a race condition; variables will not be set on time; nothing can 
      //    be relied upon in a predictable order.
      await SomeOtherTask().WithoutChangingContext();
   }
}

Until Microsoft converts all current code signatures to Tasks, programmers are stuck using these sorts of risky mechanisms.

The Proof is in the Output

This is a digested Debug output from the ResponsiveAppDemo when I originally created it. The demo called Tasks from all of the forbidden areas, including constructors. It was otherwise well-behaved -- at least according to Microsoft's guidance. So it resembles code that anyone would produce who has done the reading:

Location Task Type First/Last
Views.Subviews.DashboardView RunPostConstructionTasks FIRST
ViewModels.DashboardViewModel RunPostConstructionTasks FIRST
ViewModels.DashboardViewModel RunPostBindingTasks FIRST
Views.Subviews.DashboardView RunPostBindingTasks FIRST
------------------------------- -------------------------- ------------
Views.Subviews.DashboardView RunPostConstructionTasks LAST
ViewModels.DashboardViewModel RunPostConstructionTasks LAST
ViewModels.DashboardViewModel RunPostBindingTasks LAST
Views.Subviews.DashboardView RunPostBindingTasks LAST

Everything runs immediately and on top of each other. Nothing ever forms properly before something else piles on top and tries to rely on some imagined statefulness. This is what causes programs to hang and to crash.

The Most Obvious Solution is Also the Worst

So how do we achieve atomic completeness for each Task with no overlaps? How about this:

C#
public async void IncorrectlyRaiseATaskWithABlockingCall()
{
   await SomeTask.Wait().WithoutChangingContext();
}

Ironically, this solves concurrency issues because it only proceeds after completing a task. But that comes at an enormous cost: 100% of the UI thread. The user immediately senses their keyboard has died. Wait is a rusty razor blade in the bottom of your tool-belt.

The Right Solution: Responsive Tasks

The Responsive Tasks library handles all of the dilemmas mentioned here using a thread-safe "wait" strategy, plus base classes that support Tasks everywhere. You can easily copy the code samples into your own base views or view models, so this approach is not dogmatic.

Here is the output in the final ResponsiveTasksDemo. Everything is orderly now. Every process is stateful and predictable:

Location Task Type First/Last
Views.Subviews.DashboardView RunPostConstructionTasks FIRST
Views.Subviews.DashboardView RunPostConstructionTasks LAST
------------------------------- -------------------------- ------------
ViewModels.DashboardViewModel RunPostConstructionTasks FIRST
ViewModels.DashboardViewModel RunPostConstructionTasks LAST
------------------------------- -------------------------- ------------
ViewModels.DashboardViewModel RunPostBindingTasks FIRST
ViewModels.DashboardViewModel RunPostBindingTasks LAST
------------------------------- -------------------------- ------------
Views.Subviews.DashboardView RunPostBindingTasks FIRST
Views.Subviews.DashboardView RunPostBindingTasks LAST

Index

Each page describes a problem and its Responsive solution:

This article was originally posted at https://github.com/marcusts/Com.MarcusTS.ResponsiveTasks

License

This article, along with any associated source code and files, is licensed under The MIT License


Written By
United States United States
This member has not yet provided a Biography. Assume it's interesting and varied, and probably something to do with programming.

Comments and Discussions

 
-- There are no messages in this forum --