2

更新 - 已解决

最终的解决方案与 Brandon 的建议有些不同，但他的回答让我走上了正轨。

class State
{
  public int Offset { get; set; }
  public HashSet<string> UniqueImageUrls = new HashSet<string>();
}

public IObservable<TPicture> GetPictures(ref object _state)
{
  var localState = (State) _state ?? new State();
  _state = localState;

  return Observable.Defer(()=>
  {
    return Observable.Defer(() => Observable.Return(GetPage(localState.Offset)))
      .SubscribeOn(TaskPoolScheduler.Default)
      .Do(x=> localState.Offset += 20)
      .Repeat()
      .TakeWhile(x=> x.Count > 0)
      .SelectMany(x=> x)
      .Where(x=> !localState.UniqueImageUrls.Contains(x.ImageUrl))
      .Do(x=> localState.UniqueImageUrls.Add(x.ImageUrl));
  });
}

IList<TPicture> GetPage(int offset)
{
  ... 
  return result;
}

原始问题

我目前正在努力解决以下问题。下面显示的 PictureProvider 实现使用偏移变量，该变量用于提供实际数据的后端服务的分页结果。我想要实现的是一个优雅的解决方案，使当前偏移量可供可观察对象的消费者使用，以允许稍后以正确的偏移量恢复可观察序列。GetPictures()的intialState参数已经说明了恢复。

以更类似于 RX 的方式改进代码的建议也将受到欢迎。我实际上不太确定 Task.Run() 的东西在这里是否合适。

  public class PictureProvider :
    IPictureProvider<Picture>
  {
    #region IPictureProvider implementation

    public IObservable<Picture> GetPictures(object initialState)
    {
      return Observable.Create<Picture>((IObserver<Picture> observer) =>
      {
        var state = new ProducerState(initialState);
        ProducePictures(observer, state);
        return state;
      });
    }

    #endregion

    void ProducePictures(IObserver<Picture> observer, ProducerState state)
    {
      Task.Run(() =>
      {
        try
        {
          while(!state.Terminate.WaitOne(0))
          {
            var page = GetPage(state.Offset);

            if(page.Count == 0)
            {
              observer.OnCompleted();
              break;
            }

            else
            {
              foreach(var picture in page)
                observer.OnNext(picture);


              state.Offset += page.Count;
            }
          }
        }

        catch (Exception ex)
        {
          observer.OnError(ex);
        }

        state.TerminateAck.Set();
      });
    }

    IList<Picture> GetPage(int offset)
    {
      var result = new List<Picture>();

      ... boring web service call here

      return result;
    }

    public class ProducerState :
      IDisposable
    {
      public ProducerState(object initialState)
      {
        Terminate = new ManualResetEvent(false);
        TerminateAck = new ManualResetEvent(false);

        if(initialState != null)
          Offset = (int) initialState;
      }

      public ManualResetEvent Terminate { get; private set; }
      public ManualResetEvent TerminateAck { get; private set; }

      public int Offset { get; set; }

      #region IDisposable implementation

      public void Dispose()
      {
        Terminate.Set();
        TerminateAck.WaitOne();

        Terminate.Dispose();
        TerminateAck.Dispose();
      }

      #endregion
    }
  }
4

2 回答 2

1

我建议重构您的界面以将状态作为数据的一部分。现在客户有他们需要重新订阅他们离开的地方。

此外，一旦你开始使用 Rx，你会发现ManualResetEvent很少需要使用同步原语。如果您重构代码以便检索每个页面都是它自己的Task，那么您可以消除所有同步代码。

此外，如果您在 中调用“无聊的 Web 服务” GetPage，则只需将其设为异步即可。Task.Run除了其他好处之外，这消除了打电话的需要。

这是一个重构版本，使用 .NET 4.5 async/await 语法。它也可以在没有 async/await 的情况下完成。我还添加了一个GetPageAsync方法，Observable.Run以防万一您确实无法将 Web 服务调用转换为异步调用

/// <summary>A set of pictures</summary>
public struct PictureSet
{
    public int Offset { get; private set; }
    public IList<Picture> Pictures { get; private set; }

    /// <summary>Clients will use this property if they want to pick up where they left off</summary>
    public int NextOffset { get { return Offset + Pictures.Count; } }
    public PictureSet(int offset, IList<Picture> pictures)
        :this() { Offset = offset; Pictures = pictures; }
}

public class PictureProvider : IPictureProvider<PictureSet>
{
    public IObservable<PictureSet> GetPictures(int offset = 0)
    {
        // use Defer() so we can capture a copy of offset
        // for each observer that subscribes (so multiple
        // observers do not update each other's offset
        return Observable.Defer<PictureSet>(() =>
        {
            var localOffset = offset;
            // Use Defer so we re-execute GetPageAsync()
            // each time through the loop.
            // Update localOffset after each GetPageAsync()
            // completes so that the next call to GetPageAsync()
            // uses the next offset
            return Observable.Defer(() => GetPageAsync(localOffset))
                .Select(pictures =>
                    {
                        var s = new PictureSet(localOffset, pictures);
                        localOffset += pictures.Count;
                    })
                .Repeat()
                .TakeWhile(pictureSet => pictureSet.Pictures.Count > 0);
        });
    }

    private async Task<IList<Picture>> GetPageAsync(int offset)
    {
        var data = await BoringWebServiceCallAsync(offset);
        result = data.Pictures.ToList();
    }

    // this version uses Observable.Run() (which just uses Task.Run under the hood)
    // in case you cannot convert your
    // web service call to be asynchronous
    private IObservable<IList<Picture>> GetPageAsync(int offset)
    {
        return Observable.Run(() =>
        {
            var result = new List<Picture>();
            ... boring web service call here
            return result;
        });
    }
}

客户只需添加一个SelectMany调用即可获取他们的IObservable<Picture>. pictureSet.NextOffset如果他们愿意，他们可以选择存储。

pictureProvider
    .GetPictures()
    .SelectMany(pictureSet => pictureSet.Pictures)
    .Subscribe(picture => whatever);
于 2013-07-01T03:26:00.493 回答
0

我不会考虑如何保存订阅状态，而是考虑如何重播输入的状态（即，我会尝试创建一个可序列化的 ReplaySubject，在恢复时，它只会重新订阅并赶回当前状态）。

于 2013-06-30T18:44:50.730 回答