Как избежать исключения InvalidOperationException, когда ChannelReader.WaitToReadAsync? - PullRequest
Новая
       32

Как избежать исключения InvalidOperationException, когда ChannelReader.WaitToReadAsync?

1 голос
/ 25 апреля 2019

Я написал асинхронную очередь, используя System.Threading.Channels.но когда я запустил программу для тестирования, в случайное время было сгенерировано следующее исключение, и рабочий поток был остановлен. 

System.InvalidOperationException: The asynchronous operation has not completed.
   at System.Threading.Channels.AsyncOperation.ThrowIncompleteOperationException()
   at System.Threading.Channels.AsyncOperation`1.GetResult(Int16 token)
   at AsyncChannels.Worker() in g:\src\gitrepos\dotnet-sandbox\channelstest\AsyncChannelsTest.cs:line 26

Если исключение было обнаружено и проигнорировано, код работает.Но я хочу избавиться от ошибки, причина которой не ясна.

здесь мое окружение и наименьший код.

  • TargetFramework = netcoreapp2.1
  • Версия System.Threading.Channels = 4.5.0 
using System.Threading.Channels;
using System.Threading;
using System.Threading.Tasks;
using System;
using System.Linq;

class AsyncChannels : IDisposable
{
    Channel<TaskCompletionSource<bool>> _Channel;
    Thread _Thread;
    CancellationTokenSource _Cancellation;
    public AsyncChannels()
    {
        _Channel = Channel.CreateUnbounded<TaskCompletionSource<bool>>();
        _Thread = new Thread(Worker);
        _Thread.Start();
        _Cancellation = new CancellationTokenSource();
    }
    private void Worker()
    {
        while (!_Cancellation.IsCancellationRequested)
        {
            // System.InvalidOperationException is thrown
            if (!_Channel.Reader.WaitToReadAsync(_Cancellation.Token).Result)
            {
                break;
            }
            while (_Channel.Reader.TryRead(out var item))
            {
                item.TrySetResult(true);
            }
        }
    }
    public void Dispose()
    {
        _Cancellation.Cancel();
        _Channel.Writer.TryComplete();
        _Thread.Join();
    }
    public Task<bool> Enqueue()
    {
        var tcs = new TaskCompletionSource<bool>(TaskCreationOptions.RunContinuationsAsynchronously);
        _Channel.Writer.TryWrite(tcs);
        return tcs.Task;
    }
    public static async Task Test()
    {
        using (var queue = new AsyncChannels())
        {
            for (int i = 0; i < 100000; i++)
            {
                await queue.Enqueue().ConfigureAwait(false);
            }
        }
    }
}

1 Ответ

1 голос
/ 25 апреля 2019

Вы не можете напрямую заблокировать ValueTask<T> следующим образом:

_Channel.Reader.WaitToReadAsync (_Cancellation.Token) .Result

Вы можете сделать только две вещи с ValueTask<T>: await it (только один раз) или преобразовать его в Task<T>, вызвав AsTask().Если вам нужно сделать что-то сложное, например, await, сделать это несколько раз или заблокировать его, тогда вам нужно использовать AsTask().

Или, в этом случае, просто использовать await встандартная схема потребления каналов: 

class AsyncChannels : IDisposable
{
  Channel<TaskCompletionSource<bool>> _Channel;
  Task _Thread;
  CancellationTokenSource _Cancellation;
  public AsyncChannels()
  {
    _Channel = Channel.CreateUnbounded<TaskCompletionSource<bool>>();
    _Thread = Task.Run(() => WorkerAsync());
    _Cancellation = new CancellationTokenSource();
  }
  private async Task WorkerAsync()
  {
    try
    {
      while (await _Channel.Reader.WaitToReadAsync(_Cancellation.Token))
        while (_Channel.Reader.TryRead(out var item))
          item.TrySetResult(true);
    }
    catch (OperationCanceledException)
    {
    }
  }
  public void Dispose()
  {
    _Cancellation.Cancel();
    _Channel.Writer.TryComplete();
  }
  ...
}
...