等待Rx观察者完成而不使用锁定

时间:2014-12-27 20:09:31

标签: c# multithreading system.reactive

我有一个应用程序运行带有多个观察者的可观察区间。每隔0.5秒,间隔从Web服务器加载一些XML数据,然后观察者在后台线程上执行一些特定于应用程序的处理。一旦不再需要数据,订阅和间隔可观察就会被处理掉,因此观察者的OnNext / OnCompleted / OnError不再被调用。到目前为止一切都很好。

我的问题:在极少数情况下,有可能在调用Dispose之后我的观察者的OnNext方法仍在运行!在处理后继续进行进一步操作之前,我想确保OnNext已经完成。

我目前的解决方案:我在观察者类中引入了一个锁定字段(参见代码)。处理后我尝试获取锁定并仅在获取锁定后继续。虽然这个解决方案有效(?),但它对我来说感觉不对。

问题:是否有更优雅,更多" Rx Way"解决这个问题?

using System;
using System.Collections.Generic;
using System.Linq;
using System.Reactive.Concurrency;
using System.Reactive.Linq;
using System.Text;
using System.Threading;
using System.Threading.Tasks;

namespace RxExperimental
{
    internal sealed class MyXmlDataFromWeb
    {
        public string SomeXmlDataFromWeb { get; set; }
    }

    internal sealed class MyObserver : IObserver<MyXmlDataFromWeb>
    {
        private readonly object _locker = new object();
        private readonly string _observerName;

        public MyObserver(string observerName) {
            this._observerName = observerName;
        }

        public object Locker {
            get { return this._locker; }
        }

        public void OnCompleted() {
            lock (this._locker) {
                Console.WriteLine("{0}: Completed.", this._observerName);
            }
        }

        public void OnError(Exception error) {
            lock (this._locker) {
                Console.WriteLine("{0}: An error occured: {1}", this._observerName, error.Message);
            }
        }

        public void OnNext(MyXmlDataFromWeb value) {
            lock (this._locker) {
                Console.WriteLine("  {0}: OnNext running on thread {1}... ", this._observerName, Thread.CurrentThread.ManagedThreadId);
                Console.WriteLine("  {0}: XML received: {1}", this._observerName, value.SomeXmlDataFromWeb);
                Thread.Sleep(5000); // simulate some long running operation
                Console.WriteLine("  {0}: OnNext running on thread {1}... Done.", this._observerName, Thread.CurrentThread.ManagedThreadId);
            }
        }
    }

    internal sealed class Program
    {
        private static void Main() {
            const int interval = 500;
            //
            var dataSource = Observable.Interval(TimeSpan.FromMilliseconds(interval), NewThreadScheduler.Default).Select(_ => {
                var data = new MyXmlDataFromWeb {
                    SomeXmlDataFromWeb = String.Format("<timestamp>{0:yyyy.MM.dd HH:mm:ss:fff}</timestamp>", DateTime.Now)
                };
                return data;
            }).Publish();
            //
            var observer1 = new MyObserver("Observer 1");
            var observer2 = new MyObserver("Observer 2");
            //
            var subscription1 = dataSource.ObserveOn(NewThreadScheduler.Default).Subscribe(observer1);
            var subscription2 = dataSource.ObserveOn(NewThreadScheduler.Default).Subscribe(observer2);
            //
            var connection = dataSource.Connect();
            //
            Console.WriteLine("Press any key to cancel ...");
            Console.ReadLine();
            //
            subscription1.Dispose();
            subscription2.Dispose();
            connection.Dispose();
            //
            lock (observer1.Locker) {
                Console.WriteLine("Observer 1 completed.");
            }
            lock (observer2.Locker) {
                Console.WriteLine("Observer 2 completed.");
            }
            //
            Console.WriteLine("Can only be executed, after all observers completed.");
        }
    }
}

1 个答案:

答案 0 :(得分:3)

是的,还有更多的Rx方法。

第一个观察结果是,从可观察流中取消订阅基本上与观察者当前发生的内容无关。没有任何反馈意见。由于您需要在观察结束时明确知道,因此需要将其建模到可观察流中。换句话说,您应该完成流,而不是从流中取消订阅,这样您就可以观察OnComplete事件。在您的情况下,您可以使用TakeUntil来结束observable而不是取消订阅。

第二个观察是你的主程序需要观察你的“观察者”何时完成它的工作。但是既然你让你的“观察者”成为一个真实的IObservable,你就没有办法做到这一点。当人们第一次开始使用Rx时,这是我看到的混淆的常见原因。如果您将“观察者”建模为可观察链中的另一个链接,那么您的主程序可以观察。具体来说,您的“观察者”只不过是一个映射操作(带有副作用),它将传入的Xml数据映射为“完成”消息。

因此,如果你重构代码,你可以得到你想要的......

public class MyObserver
{
    private readonly string _name;

    public MyObserver(string name) { _name = name; }

    public IObservable<Unit> Handle(IObservable<MyXmlDataFromWeb source)
    {
        return source.Select(value =>
        {
            Thread.Sleep(5000); // simulate work
            return Unit.Default;
        });
    }
}

// main
var endSignal = new Subject<Unit>();
var dataSource = Observable
    .Interval(...)
    .Select(...)
    .TakeUntil(endSignal)
    .Publish();
var observer1 = new MyObserver("Observer 1");
var observer2 = new MyObserver("Observer 2");
var results1 = observer1.Handle(dataSource.ObserveOn(...));
var results2 = observer2.Handle(dataSource.ObserveOn(...));
// since you just want to know when they are all done
// just merge them.
// use ToTask() to subscribe them and collect the results
// as a Task
var processingDone = results1.Merge(results2).Count().ToTask();

dataSource.Connect();

Console.WriteLine("Press any key to cancel ...");
Console.ReadLine();

// end the stream
endSignal.OnNext(Unit.Default);

// wait for the processing to complete.
// use await, or Task.Result
var numProcessed = await processingDone;