更新

Question

我非常喜欢Scala 2.10中的scala.util.Try，以及它如何处理for-comprehension使得处理可能容易出错的多个步骤。

例如，我们可以使用以下代码来确保我们只打印出这两个数字，当且仅当一切都在控制之下并且我们才能正确获得价值。

def tryA: Try[Int] = {....}
def tryB: Try[Int] = {....}

for {
  a <- tryA
  b <- tryB
} { 
  println (s"We got:${a+b}")
}

但我关注的一点是，这段代码实际上忽略了任何异常，这意味着它看起来像下面的try-cactch块：

try {
  // .....
} catch {
  case _: Exception => // Swallow any exception
}

据我所知，有一种说法认为这种代码是难闻的气味，因为没有人会注意到发生了异常。

我想要实现的是仍然使用for来确保println仅在一切正常时执行，但如果在任何步骤中有任何异常，它将会爆炸并抛出直接排除异常。

目前这是我如何做到这一点，但它似乎不那么优雅，因为它引入了一个新的Try[Unit]对象，所以我想知道如何才能使这段代码变得更好？

例如，是否可以删除result变量和result.get语句，但仍然会抛出异常？

def tryA: Try[Int] = {....}
def tryB: Try[Int] = {....}

val result = for {
  a <- tryA
  b <- tryB
} yield { 
  println (s"We got:${a+b}")
}
result.get

更新

为了使事情更清楚，这是Scala REPL对此问题中第一个代码的结果。

scala> def tryA: Try[Int] = Success(1)
tryA: scala.util.Try[Int]

scala> def tryB: Try[Int] = Failure(new Exception("error"))
tryB: scala.util.Try[Int]

scala> for {
     |   a <- tryA
     |   b <- tryB
     | } { 
     |   println (s"We got:${a+b}")
     | }

scala>

我们可以看到此处没有任何结果，即使tryB是Failure也有例外。我想得到的是抛出异常，如果没有使用Try[Unit]引入新的yield对象，这可能吗？

Answer 1

您可以使用recover：

import scala.util.Try

def tryEven = Try { val i = (math.random * 1000).toInt; if (i % 2 != 0) throw new Exception("odd") else i }

def tryEvenOrNeg1 = Try { val i = (math.random * 1000).toInt; if (i % 2 != 0) throw new Exception("odd") else i } recover { case exx: Exception => -1 }

scala> for (a <- tryEven; b <- tryEvenOrNeg1) yield println(s"Got $a, $b")
res1: scala.util.Try[Unit] = Failure(java.lang.Exception: odd)

scala> for (a <- tryEven; b <- tryEvenOrNeg1) yield println(s"Got $a, $b")
res2: scala.util.Try[Unit] = Failure(java.lang.Exception: odd)

scala> for (a <- tryEven; b <- tryEvenOrNeg1) yield println(s"Got $a, $b")
res3: scala.util.Try[Unit] = Failure(java.lang.Exception: odd)

scala> for (a <- tryEven; b <- tryEvenOrNeg1) yield println(s"Got $a, $b")
Got 542, -1

scala> for (a <- tryEven; b <- tryEvenOrNeg1) yield println(s"Got $a, $b")
res5: scala.util.Try[Unit] = Failure(java.lang.Exception: odd)

scala> for (a <- tryEven; b <- tryEvenOrNeg1) yield println(s"Got $a, $b")
res6: scala.util.Try[Unit] = Failure(java.lang.Exception: odd)

scala> for (a <- tryEven; b <- tryEvenOrNeg1) yield println(s"Got $a, $b")
Got 692, 750

我删除了反映resNN的{{1}}。

Answer 2

好的，我忘了我们在Scala中总是有隐式转换。 ; - ）

所以我们可以自己实现这种行为，它会创建比yield版本更多的对象，但我认为这段代码的意图要清晰得多。

implicit class BlowUpTry[T](current: Try[T])  {

  def throwIfFailed: Try[T] = current match {
    case Success(value)     => current
    case Failure(exception) => throw exception
  }

}

def tryA: Try[Int] = Success(1)
def tryB: Try[Int] = Failure(new Exception("error"))

for {
  a <- tryA.throwIfFailed
  b <- tryB.throwIfFailed
} {
  println(s"We got ${a + b}")
}

使用scala.util.Try进行for-understanding时的快速失败

更新

2 个答案: