Scala: List [Future] to Future [List] pomijając nieudane futures

116

Szukam sposobu na przekonwertowanie dowolnej długości listy kontraktów futures na przyszłość listy. Używam Playframework, więc ostatecznie naprawdę chcę Future[Result], ale żeby wszystko było prostsze, powiedzmy, że Future[List[Int]]Normalnym sposobem byłoby użycie, Future.sequence(...)ale jest pewien zwrot akcji ... Lista, którą otrzymuję, zwykle ma około 10-20 kontraktów futures i nierzadko zdarza się, że jedna z nich zawodzi (wykonują zewnętrzne żądania usług internetowych). Zamiast ponawiać próby wszystkich z nich w przypadku, gdy jeden z nich zawiedzie, chciałbym mieć możliwość uzyskania dostępu do tych, którym się powiodło, i zwrócenia tych.

Na przykład wykonanie poniższych czynności nie działa

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

Zamiast otrzymywać jedyny wyjątek, chciałbym móc wyciągnąć stamtąd 1 i 3. Próbowałem użyć Future.fold, ale to najwyraźniej tylko dzwoni Future.sequenceza kulisami.

Z góry dziękuję za pomoc!

Joe
źródło

Odpowiedzi:

147

Sztuczka polega na tym, aby najpierw upewnić się, że żadna z przyszłości nie zawiodła. .recoverjest twoim przyjacielem, możesz go połączyć, mapaby przekonwertować wszystkie Future[T]wyniki na Future[Try[T]]]instancje, z których wszystkie z pewnością będą pomyślnymi przyszłościami.

Uwaga: Możesz użyć Optionlub Eitherrównież tutaj, ale Tryjest to najczystszy sposób, jeśli chcesz przechwytywać wyjątki

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

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

Następnie użyj Future.sequencejak poprzednio, aby uzyskać plikFuture[List[Try[T]]]

val futureListOfTrys = Future.sequence(listOfFutureTrys)

Następnie filtruj:

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

Możesz nawet wyciągnąć określone awarie, jeśli ich potrzebujesz:

val futureListOfFailures = futureListOfTrys.map(_.filter(_.isFailure))
Kevin Wright
źródło
Dzięki! .recoverbył rzeczywiście brakującym elementem dla mnie.
Joe,
20
Możesz użyć _.collect{ case Success(x) => x}zamiast, _.filter(_.isSuccess)aby pozbyć się Trytypu futureListOfSuccesses.
senia
43
W scali 2010 .recover(x => Failure(x))nie jest ważna, użyj .recover({case e => Failure(e)})zamiast tego
FGRibreau
Myślę, że brakuje ci opakowania przyszłości: def futureToFutureOfTry [A] (f: Future [A]): ​​Future [Try [A]] = {val p = Promise [Spróbuj [A]] () f.map {a => p.success (scala.util.Success (a))} .recover {przypadek x: Throwable => p.success (Failure (x))} p.future}
Dario
skąd. Mapuję przyszłość na inną przyszłość, obietnica interweniująca nie jest potrzebna i byłaby marnotrawstwem
Kevin Wright
12

Scala 2.12 ma ulepszenie, Future.transformktóre nadaje się do odpowiedzi z mniejszą liczbą kodów.

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)))
WeiChing 林 煒 清
źródło
11

Wypróbowałem odpowiedź Kevina i napotkałem usterkę w mojej wersji Scali (2.11.5) ... Poprawiłem to i napisałem kilka dodatkowych testów, jeśli ktoś jest zainteresowany ... oto moja wersja>

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)
  }
Chris Bedford
źródło
7

Właśnie natknąłem się na to pytanie i mam inne rozwiązanie do zaoferowania:

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())
}

Pomysł polega na tym, że w folderze czekasz na zakończenie następnego elementu na liście (używając składni for-compension), a jeśli następny zawiedzie, po prostu wracasz do tego, co już masz.

Idan Waisman
źródło
Nie lubię nazwy ale podoba mi się sposób to zrobić, prosto z sekwencji IMPL
Crak
1

Możesz łatwo zawinąć przyszły wynik opcją, a następnie spłaszczyć listę:

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
Amir Hossein Javan
źródło
Na wypadek, gdyby ktoś inny napotkał błąd z Some w pierwszej funkcji, pierwszą funkcję można przepisać w ten sposób, aby zapobiec wystąpieniu błędu kompilatora: def futureToFutureOption [T] (f: Future [T]): Future [Option [T]] = f.map (Opcja (_)). recovery {przypadek e => Brak}
Zee
0

Możesz również zbierać udane i nieudane wyniki na różnych listach:

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) }
    }
  }
}
Evgeniy Lyutikov
źródło