正确处理RxJava中的空Observable

时间:2015-03-16 20:35:25

标签: java rx-java

我遇到的情况是我正在创建一个包含数据库结果的Observable。然后我将一系列过滤器应用于它们。然后我有一个订阅者正在记录结果。情况可能是没有元素通过过滤器。我的业务逻辑表明这不是错误。但是,当发生这种情况时,我的onError被调用并包含以下异常:java.util.NoSuchElementException: Sequence contains no elements

接受的做法是仅检测该类型的异常并忽略它吗?或者有更好的方法来解决这个问题吗?

版本是1.0.0。

这是一个简单的测试用例,揭示了我所看到的内容。它似乎与在到达地图之前过滤所有事件并减少相关。

  @Test
public void test()
{

    Integer values[] = new Integer[]{1, 2, 3, 4, 5};

    Observable.from(values).filter(new Func1<Integer, Boolean>()
    {
        @Override
        public Boolean call(Integer integer)
        {
            if (integer < 0)
                return true;
            else
                return false;
        }
    }).map(new Func1<Integer, String>()
    {
        @Override
        public String call(Integer integer)
        {
            return String.valueOf(integer);
        }
    }).reduce(new Func2<String, String, String>()
    {
        @Override
        public String call(String s, String s2)
        {
            return s + "," + s2;
        }
    })

            .subscribe(new Action1<String>()
            {
                @Override
                public void call(String s)
                {
                    System.out.println(s);
                }
            });
}

因为我使用的是安全订阅者,所以它最初会抛出一个包含以下异常的OnErrorNotImplementedException:

java.util.NoSuchElementException: Sequence contains no elements
    at rx.internal.operators.OperatorSingle$1.onCompleted(OperatorSingle.java:82)
    at rx.internal.operators.NotificationLite.accept(NotificationLite.java:140)
    at rx.internal.operators.TakeLastQueueProducer.emit(TakeLastQueueProducer.java:73)
    at rx.internal.operators.TakeLastQueueProducer.startEmitting(TakeLastQueueProducer.java:45)
    at rx.internal.operators.OperatorTakeLast$1.onCompleted(OperatorTakeLast.java:59)
    at rx.internal.operators.OperatorScan$2.onCompleted(OperatorScan.java:121)
    at rx.internal.operators.OperatorMap$1.onCompleted(OperatorMap.java:43)
    at rx.internal.operators.OperatorFilter$1.onCompleted(OperatorFilter.java:42)
    at rx.internal.operators.OnSubscribeFromIterable$IterableProducer.request(OnSubscribeFromIterable.java:79)
    at rx.internal.operators.OperatorScan$2$1.request(OperatorScan.java:147)
    at rx.Subscriber.setProducer(Subscriber.java:139)
    at rx.internal.operators.OperatorScan$2.setProducer(OperatorScan.java:139)
    at rx.Subscriber.setProducer(Subscriber.java:133)
    at rx.Subscriber.setProducer(Subscriber.java:133)
    at rx.internal.operators.OnSubscribeFromIterable.call(OnSubscribeFromIterable.java:47)
    at rx.internal.operators.OnSubscribeFromIterable.call(OnSubscribeFromIterable.java:33)
    at rx.Observable$1.call(Observable.java:144)
    at rx.Observable$1.call(Observable.java:136)
    at rx.Observable$1.call(Observable.java:144)
    at rx.Observable$1.call(Observable.java:136)
    at rx.Observable$1.call(Observable.java:144)
    at rx.Observable$1.call(Observable.java:136)
    at rx.Observable$1.call(Observable.java:144)
    at rx.Observable$1.call(Observable.java:136)
    at rx.Observable$1.call(Observable.java:144)
    at rx.Observable$1.call(Observable.java:136)
    at rx.Observable.subscribe(Observable.java:7284)

根据下面@davem的回答,我创建了一个新的测试用例:

@Test
public void testFromBlockingAndSingle()
{

    Integer values[] = new Integer[]{-2, -1, 0, 1, 2, 3, 4, 5};

    List<String> results = Observable.from(values).filter(new Func1<Integer, Boolean>()
    {
        @Override
        public Boolean call(Integer integer)
        {
            if (integer < 0)
                return true;
            else
                return false;
        }
    }).map(new Func1<Integer, String>()
    {
        @Override
        public String call(Integer integer)
        {
            return String.valueOf(integer);
        }
    }).reduce(new Func2<String, String, String>()
    {
        @Override
        public String call(String s, String s2)
        {
            return s + "," + s2;
        }
    }).toList().toBlocking().single();

    System.out.println("Test: " + results + " Size: " + results.size());

}

此测试会导致以下行为:

当输入为:

Integer values[] = new Integer[]{-2, -1, 0, 1, 2, 3, 4, 5};

然后结果(如预期的那样)是:

Test: [-2,-1] Size: 1

当输入为:

Integer values[] = new Integer[]{0, 1, 2, 3, 4, 5};

然后结果是以下堆栈跟踪:

java.util.NoSuchElementException: Sequence contains no elements
at rx.internal.operators.OperatorSingle$1.onCompleted(OperatorSingle.java:82)
at rx.internal.operators.NotificationLite.accept(NotificationLite.java:140)
at rx.internal.operators.TakeLastQueueProducer.emit(TakeLastQueueProducer.java:73)
at rx.internal.operators.TakeLastQueueProducer.startEmitting(TakeLastQueueProducer.java:45)
at rx.internal.operators.OperatorTakeLast$1.onCompleted(OperatorTakeLast.java:59)
at rx.internal.operators.OperatorScan$2.onCompleted(OperatorScan.java:121)
at rx.internal.operators.OperatorMap$1.onCompleted(OperatorMap.java:43)
at rx.internal.operators.OperatorFilter$1.onCompleted(OperatorFilter.java:42)
at rx.internal.operators.OnSubscribeFromIterable$IterableProducer.request(OnSubscribeFromIterable.java:79)
at rx.internal.operators.OperatorScan$2$1.request(OperatorScan.java:147)
at rx.Subscriber.setProducer(Subscriber.java:139)
at rx.internal.operators.OperatorScan$2.setProducer(OperatorScan.java:139)
at rx.Subscriber.setProducer(Subscriber.java:133)
at rx.Subscriber.setProducer(Subscriber.java:133)
at rx.internal.operators.OnSubscribeFromIterable.call(OnSubscribeFromIterable.java:47)
at rx.internal.operators.OnSubscribeFromIterable.call(OnSubscribeFromIterable.java:33)
at rx.Observable$1.call(Observable.java:144)
at rx.Observable$1.call(Observable.java:136)
at rx.Observable$1.call(Observable.java:144)
at rx.Observable$1.call(Observable.java:136)
at rx.Observable$1.call(Observable.java:144)
at rx.Observable$1.call(Observable.java:136)
at rx.Observable$1.call(Observable.java:144)
at rx.Observable$1.call(Observable.java:136)
at rx.Observable$1.call(Observable.java:144)
at rx.Observable$1.call(Observable.java:136)
at rx.Observable$1.call(Observable.java:144)
at rx.Observable$1.call(Observable.java:136)
at rx.Observable$1.call(Observable.java:144)
at rx.Observable$1.call(Observable.java:136)
at rx.Observable.subscribe(Observable.java:7284)
at rx.observables.BlockingObservable.blockForSingle(BlockingObservable.java:441)
at rx.observables.BlockingObservable.single(BlockingObservable.java:340)
at EmptyTest2.test(EmptyTest2.java:19)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:70)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:238)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:63)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:236)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:53)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:229)
at org.junit.runners.ParentRunner.run(ParentRunner.java:309)
at org.junit.runner.JUnitCore.run(JUnitCore.java:160)
at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:74)
at com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:211)
at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:67)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:134)

所以看来问题肯定是使用reduce函数。请参阅以下处理这两种情况的测试用例:

@Test
public void testNoReduce()
{

    Integer values[] = new Integer[]{-2, -1, 0, 1, 2, 3, 4, 5};

    List<String> results = Observable.from(values).filter(new Func1<Integer, Boolean>()
    {
        @Override
        public Boolean call(Integer integer)
        {
            if (integer < 0)
                return true;
            else
                return false;
        }
    }).map(new Func1<Integer, String>()
    {
        @Override
        public String call(Integer integer)
        {
            return String.valueOf(integer);
        }
    }).toList().toBlocking().first();

    Iterator<String> itr = results.iterator();
    StringBuilder b = new StringBuilder();

    while (itr.hasNext())
    {
        b.append(itr.next());

        if (itr.hasNext())
            b.append(",");
    }

    System.out.println("Test NoReduce: " + b);

}

使用以下输入:

Integer values[] = new Integer[]{-2, -1, 0, 1, 2, 3, 4, 5};

我得到了以下预期结果:

Test NoReduce: -2,-1

并输入以下内容:

Integer values[] = new Integer[]{0, 1, 2, 3, 4, 5};

我得到了以下预期的输出:

Test NoReduce: 

所以,除非我完全误解了某些东西,否则真正处理零元素Observable的唯一方法就是在一个map和reduce之后实现reduce元素。你们都同意这个说法吗?


最终解决方案

这是我实施TomášDvořák和David Motten建议的最终解决方案。我认为这个解决方案是合理的。

@Test
public void testWithToList()
{

    Integer values[] = new Integer[]{-2, -1, 0, 1, 2, 3, 4, 5};

     Observable.from(values).filter(new Func1<Integer, Boolean>()
     {
         @Override
         public Boolean call(Integer integer)
         {
             if (integer < 0)
                 return true;
             else
                 return false;
         }
     }).toList().map(new Func1<List<Integer>, String>()
     {
         @Override
         public String call(List<Integer> integers)
         {
             Iterator<Integer> intItr = integers.iterator();
             StringBuilder b = new StringBuilder();

             while (intItr.hasNext())
             {
                 b.append(intItr.next());

                 if (intItr.hasNext())
                 {
                     b.append(",");
                 }
             }

             return b.toString();
         }
     }).subscribe(new Action1<String>()
     {
         @Override
         public void call(String s)
         {
             System.out.println("With a toList: " + s);
         }
     });

}

以下是给定以下输入时此测试的行为方式。

当给定一个具有某些值的流通过过滤器时:

Integer values[] = new Integer[]{-2, -1, 0, 1, 2, 3, 4, 5};

结果是:

With a toList: -2,-1

当给定一个没有任何值的流通过过滤器时:

Integer values[] = new Integer[]{0, 1, 2, 3, 4, 5};

结果是:

With a toList: <empty string>

3 个答案:

答案 0 :(得分:15)

现在更新后,错误非常明显。 RxJava中的Reduce将失败并显示IllegalArgumentException,如果它正在减少的可观察值为空,则完全符合规范(http://reactivex.io/documentation/operators/reduce.html)。

在函数式编程中,通常有两个泛型运算符将集合聚合为单个值foldreduce。在公认的术语中,fold采用初始累加器值,以及从集合中获取正在运行的累加器和值并生成另一个累加器值的函数。伪代码中的一个例子:

[1, 2, 3, 4].fold(0, (accumulator, value) => accumulator + value)

将从0开始,最终将1,2,3,4添加到正在运行的累加器中,最后得到10,即值的总和。

Reduce非常相似,只是它没有显式获取初始累加器值,它使用第一个值作为初始累加器,然后累加所有剩余值。如果您是例如,这是有道理的。寻找最低或最高价值。

[1, 2, 3, 4].reduce((accumulator, value) => min(accumulator, value))

看看fold并减少不同的方式,只要聚合值在空集合中有意义就可以使用fold(例如,在sum中,0有意义),{{1否则(reduce对空集合没有意义,minimum将无法对此类集合进行操作,在您的情况下会抛出异常。)

您正在进行类似的聚合,使用逗号散布字符串集合以生成单个字符串。那是一个有点困难的情况。它可能对空集合有意义(你可能期望一个空字符串),但另一方面,如果你从一个空的累加器开始,你将在结果中再多一个逗号而不是你期望的。对此的正确解决方案是首先检查集合是否为空,并为空集合返回回退字符串,或对非空集合执行reduce。您可能会注意到,在空集合案例中,您实际上并不想要一个空字符串,但是像#34;集合是空的&#34;可能更合适,从而进一步向您保证这个解决方案是干净的。

顺便说一下,我在这里使用 collection 这个词而不是 observable ,只是出于教育目的。此外,在RxJava中,reducefold被称为相同的reduce,只有该方法有两个版本,一个只带一个参数,另外两个参数。

至于你的最后一个问题:你不必离开Observable连锁店。只需使用toList(),就像David Motten建议的那样。

reduce

其中.filter(...) .toList() .map(listOfValues => listOfValues.intersperse(", ")) 可以用intersperse实现,如果还没有库函数(这很常见)。

reduce

答案 1 :(得分:9)

发生这种情况的原因是您在空流中使用toBlocking().single()。如果您希望流中的0或1值可以toList().toBlocking().single(),那么请检查列表中的值(可能为空,但不会引发您获得的异常)。

答案 2 :(得分:4)

你可以使用reduce和初始值版本 Sub Macro7() ' ' Macro7 Macro ' ' Keyboard Shortcut: Ctrl+c Dim ws As Worksheet Dim lastRow As Integer Dim summaryRow As Integer summaryRow = 1 For Each ws In ActiveWorkbook.Worksheets 'Copy item number and paste on Summary Page' Range("A2").Select Selection.Copy Sheets("Summary").Select Range("A" & summaryRow).Select Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _ :=False, Transpose:=False 'Copy corresponding BOM item # and paste on Summary Page' ws.Select lastRow = .Cells(.Rows.Count, "L").End(xlUp).Row Range("L" & lastRow).Select Application.CutCopyMode = False Selection.Copy Sheets("Summary").Select Range("B" & summaryRow).Select Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _ :=False, Transpose:=False summaryRow = summaryRow + 1 Next ws End Sub 它可以作为由TomášDvořák解释的折叠算子