在Akka Stream中抛出异常的最佳做法是什么 - Scala?

时间:2017-01-09 07:15:31

标签: scala akka akka-stream

我是Akka Stream的新手,并试图找出处理流中意外行为的最佳做法。

我正在构建一个使用Account Kit API将短期代码交换为长期访问令牌的流。

以下是构建流的相关代码:

override def ExchangeCode: Flow[String, AccessTokenInfo, NotUsed] =
Flow[String].mapAsync(1) { code =>
  ws.url(buildUrl("access_token"))
    .withQueryString(
      "grant_type" -> "authorization_code",
      "code" -> code,
      "access_token" -> appToken
    )
    .withRequestTimeout(timeout)
    .get
    .map { response =>
      if (response.status != 200) throw new RuntimeException("Unexpected response")
      else response.json
    }
    .map { json =>
      AccessTokenInfo("123456", 123, "123456")
    }
}

我想知道如果状态代码不是200就抛出异常是处理它的正确方法,但这是我知道它提前终止流的唯一方法。 (目前返回值是虚拟的)

1 个答案:

答案 0 :(得分:1)

如果状态不是200,为什么流终止是非常必要的?通常,您希望向下游发送故障,以便Flow的任何用户都能得到通知并采取相应措施。

处理此类故障的典型方法是使用Try。略微修改你的流程:

//                                  Add Try Output
//                                       |
//                                       v
override def ExchangeCode: Flow[String, Try[AccessTokenInfo], _] = 
  Flow[String] 
    ...
    .map { response => response.status match {
        case 200 => Try { response.json }
        case _   => Try { throw new RuntimeException("Unexpected Response") }
      }
    }
    .map( _ map (json => AccessTokenInfo("123456", 123, "123456")))

现在,如果可以检索或获取异常并且能够处理故障情况,那么流的任何用户都可以获得有效的访问令牌。

只有1种失败类型的选项

在问题的特定用例中,Try似乎甚至没有必要。由于只有1个Exception可以只生成1,因此Option似乎更好:

override def ExchangeCode: Flow[String, Option[AccessTokenInfo], _] = 
  ...
        case 200 => Some(response.json)
        case _   => None
  ...