Scala:失敗した先物を無視してList [Future]からFuture [List]へ


116

任意の長さのFutureのリストをFuture of Listに変換する方法を探しています。私はPlayframeworkを使用しているので、結局のところ、私が本当に欲しいのはですが、Future[Result]物事をより簡単にするために、ただ言っておきましょう。Future[List[Int]]これを行う通常の方法は、使用することですFuture.sequence(...)が、ひねりがあります...約10〜20の先物で、これらの先物のうちの1つが失敗することは珍しくありません(彼らは外部のWebサービス要求を行っています)。それらの1つが失敗した場合にすべてを再試行する代わりに、成功したものに到達してそれらを返すことができるようにしたいと思います。

たとえば、次の操作は機能しません

import scala.concurrent._
import scala.concurrent.ExecutionContext.Implicits.global
import scala.util.Success
import scala.util.Failure

val listOfFutures = Future.successful(1) :: Future.failed(new Exception("Failure")) :: 
                    Future.successful(3) :: Nil

val futureOfList = Future.sequence(listOfFutures)

futureOfList onComplete {
  case Success(x) => println("Success!!! " + x)
  case Failure(ex) => println("Failed !!! " + ex)
}

scala> Failed !!! java.lang.Exception: Failure

唯一の例外を取得する代わりに、1と3をそこから引き出しられるようにしたいと思います。私はを使ってみましたFuture.foldが、明らかにそれはFuture.sequence裏で呼び出されているようです。

助けてくれてありがとう!

回答:


146

トリックは、先物が失敗していないことを最初に確認することです。 .recoverここであなたの友達です、あなたはそれを組み合わせて、mapすべてのFuture[T]結果をFuture[Try[T]]]インスタンスに変換することができます。

注:ここでもOptionまたはEitherを使用できますTryが、特に例外をトラップする場合は最もクリーンな方法です

def futureToFutureTry[T](f: Future[T]): Future[Try[T]] =
  f.map(Success(_)).recover { case x => Failure(x)}

val listOfFutures = ...
val listOfFutureTrys = listOfFutures.map(futureToFutureTry(_))

次にFuture.sequence、以前と同じように使用して、Future[List[Try[T]]]

val futureListOfTrys = Future.sequence(listOfFutureTrys)

次にフィルタリングします。

val futureListOfSuccesses = futureListOfTrys.map(_.filter(_.isSuccess))

必要に応じて、特定の障害を取り除くこともできます。

val futureListOfFailures = futureListOfTrys.map(_.filter(_.isFailure))

ありがとう!.recover確かに私にとって欠けている部分でした。
Joe

20
のタイプを取り除くためにの_.collect{ case Success(x) => x}代わりに使用できます。_.filter(_.isSuccess)TryfutureListOfSuccesses
セニア2014年

43
.recover(x => Failure(x)).recover({case e => Failure(e)})
Scala

将来のラッパーが不足していると思います:def futureToFutureOfTry [A](f:Future [A]):Future [Try [A]] = {val p = Promise [Try [A]]()f.map {a => p.success(scala.util.Success(a))} .recover {case x:Throwable => p.success(Failure(x))} p.future}
Dario

そうではありません。私は未来を別の未来にマッピングしています。介在する約束は必要ではなく、無駄になるでしょう
ケビン・ライト

12

Scala 2.12は、Future.transformコードが少ないアンサーで利用できるように改善されています。

val futures = Seq(Future{1},Future{throw new Exception})

// instead of `map` and `recover`, use `transform`
val seq = Future.sequence(futures.map(_.transform(Success(_)))) 

val successes = seq.map(_.collect{case Success(x)=>x})
successes
//res1: Future[Seq[Int]] = Future(Success(List(1)))

val failures = seq.map(_.collect{case Failure(x)=>x})
failures
//res2: Future[Seq[Throwable]] = Future(Success(List(java.lang.Exception)))

11

私はケビンの答えを試しました、そして私のバージョンのScala(2.11.5)でグリッチに遭遇しました...私はそれを修正し、誰かが興味があればいくつかの追加のテストを書きました...これが私のバージョンです>

implicit class FutureCompanionOps(val f: Future.type) extends AnyVal {

    /** Given a list of futures `fs`, returns the future holding the list of Try's of the futures from `fs`.
      * The returned future is completed only once all of the futures in `fs` have been completed.
      */
    def allAsTrys[T](fItems: /* future items */ List[Future[T]]): Future[List[Try[T]]] = {
      val listOfFutureTrys: List[Future[Try[T]]] = fItems.map(futureToFutureTry)
      Future.sequence(listOfFutureTrys)
    }

    def futureToFutureTry[T](f: Future[T]): Future[Try[T]] = {
      f.map(Success(_)) .recover({case x => Failure(x)})
    }

    def allFailedAsTrys[T](fItems: /* future items */ List[Future[T]]): Future[List[Try[T]]] = {
      allAsTrys(fItems).map(_.filter(_.isFailure))
    }

    def allSucceededAsTrys[T](fItems: /* future items */ List[Future[T]]): Future[List[Try[T]]] = {
      allAsTrys(fItems).map(_.filter(_.isSuccess))
    }
}


// Tests... 



  // allAsTrys tests
  //
  test("futureToFutureTry returns Success if no exception") {
    val future =  Future.futureToFutureTry(Future{"mouse"})
    Thread.sleep(0, 100)
    val futureValue = future.value
    assert(futureValue == Some(Success(Success("mouse"))))
  }
  test("futureToFutureTry returns Failure if exception thrown") {
    val future =  Future.futureToFutureTry(Future{throw new IllegalStateException("bad news")})
    Thread.sleep(5)            // need to sleep a LOT longer to get Exception from failure case... interesting.....
    val futureValue = future.value

    assertResult(true) {
      futureValue match {
        case Some(Success(Failure(error: IllegalStateException)))  => true
      }
    }
  }
  test("Future.allAsTrys returns Nil given Nil list as input") {
    val future =  Future.allAsTrys(Nil)
    assert ( Await.result(future, 100 nanosecond).isEmpty )
  }
  test("Future.allAsTrys returns successful item even if preceded by failing item") {
    val future1 =  Future{throw new IllegalStateException("bad news")}
    var future2 = Future{"dog"}

    val futureListOfTrys =  Future.allAsTrys(List(future1,future2))
    val listOfTrys =  Await.result(futureListOfTrys, 10 milli)
    System.out.println("successItem:" + listOfTrys);

    assert(listOfTrys(0).failed.get.getMessage.contains("bad news"))
    assert(listOfTrys(1) == Success("dog"))
  }
  test("Future.allAsTrys returns successful item even if followed by failing item") {
    var future1 = Future{"dog"}
    val future2 =  Future{throw new IllegalStateException("bad news")}

    val futureListOfTrys =  Future.allAsTrys(List(future1,future2))
    val listOfTrys =  Await.result(futureListOfTrys,  10 milli)
    System.out.println("successItem:" + listOfTrys);

    assert(listOfTrys(1).failed.get.getMessage.contains("bad news"))
    assert(listOfTrys(0) == Success("dog"))
  }
  test("Future.allFailedAsTrys returns the failed item and only that item") {
    var future1 = Future{"dog"}
    val future2 =  Future{throw new IllegalStateException("bad news")}

    val futureListOfTrys =  Future.allFailedAsTrys(List(future1,future2))
    val listOfTrys =  Await.result(futureListOfTrys,  10 milli)
    assert(listOfTrys(0).failed.get.getMessage.contains("bad news"))
    assert(listOfTrys.size == 1)
  }
  test("Future.allSucceededAsTrys returns the succeeded item and only that item") {
    var future1 = Future{"dog"}
    val future2 =  Future{throw new IllegalStateException("bad news")}

    val futureListOfTrys =  Future.allSucceededAsTrys(List(future1,future2))
    val listOfTrys =  Await.result(futureListOfTrys,  10 milli)
    assert(listOfTrys(0) == Success("dog"))
    assert(listOfTrys.size == 1)
  }

7

私はこの質問に出くわしました、そして提供する別の解決策があります:

def allSuccessful[A, M[X] <: TraversableOnce[X]](in: M[Future[A]])
                                                (implicit cbf: CanBuildFrom[M[Future[A]], A, M[A]], 
                                                 executor: ExecutionContext): Future[M[A]] = {
    in.foldLeft(Future.successful(cbf(in))) {
      (fr, fa)(for (r ← fr; a ← fa) yield r += a) fallbackTo fr
    } map (_.result())
}

ここでの考え方は、フォールド内で(for内包構文を使用して)リスト内の次の要素が完了するのを待っており、次の要素が失敗した場合は、すでに持っているものにフォールバックするだけです。


名前は嫌いですが、シーケンスの
実装

1

オプションで将来の結果を簡単にラップし、リストをフラット化できます。

def futureToFutureOption[T](f: Future[T]): Future[Option[T]] =
    f.map(Some(_)).recover {
      case e => None
    }
val listOfFutureOptions = listOfFutures.map(futureToFutureOption(_))

val futureListOfOptions = Future.sequence(listOfFutureOptions)

val futureListOfSuccesses = futureListOfOptions.flatten

他の誰かが最初の関数でSomeのエラーに遭遇した場合に備えて、最初の関数を次のように書き換えて、コンパイラエラーを防ぐことができます。def futureToFutureOption [T](f:Future [T]):Future [Option [T]] = f.map(Option(_))。recover {case e => None}
Zee

0

成功した結果と失敗した結果を異なるリストで収集することもできます。

def safeSequence[A](futures: List[Future[A]]): Future[(List[Throwable], List[A])] = {
  futures.foldLeft(Future.successful((List.empty[Throwable], List.empty[A]))) { (flist, future) =>
    flist.flatMap { case (elist, alist) =>
      future
        .map { success => (elist, alist :+ success) }
        .recover { case error: Throwable => (elist :+ error, alist) }
    }
  }
}
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.