我有两个类似的功能。一个在成功时返回一个字符串,第二个在成功时返回一个OptionString。如何才能以更优雅的方式编写它,也许可以调用第二个函数中的第一个函数呢?谢谢!
private def readString(j: JsValue, key: String): Validation[String, String] = {
j \ key match {
case j: JsString =>
Success(j.value)
case j: JsUndefined =>
Failure(s"Missing field $key")
case j: JsValue =>
Failure(s"Field $key value is not a string")
}
}
private def readStringOpt(j: JsValue, key: String): Validation[String, Option[String]] = {
(j \ key).as[JsValue] match {
case j: JsString =>
j.value.some.success
case j: JsUndefined =>
None.success
case x =>
Failure(s"Field $key value is not a string")
}
}
发布于 2013-11-19 06:50:42
首先,您的第二个方法中的.as[JsValue]
并不是真正必要的--它所做的就是查找JsValue
的Reads
实例,它只是通过它的参数传递,并且从不失败。
如果您想按照上面建议的路线(根据其他版本定义Opt
-less版本),Scalaz提供了一些稍微简洁的语法:
def readString(j: JsValue, key: String): Validation[String, String] =
readStringOpt(j, key).flatMap(_.toSuccess(s"Missing field $key"))
然而,这将在最近版本的Scalaz中给您一个不推荐的警告,因为Validation
没有单一实例,而且它的flatMap
是一种谎言。这意味着您有两个选项(除了忽略弃用警告之外):您可以切换到\/
,它是一元的,或者您可以使用折叠:
def readString(j: JsValue, key: String): Validation[String, String] =
readStringOpt(j, key).fold(_.failure, _.toSuccess(s"Missing field $key"))
这有点冗长,但它让你站在验证的右边,而不是一个单一的神。
https://stackoverflow.com/questions/20061808
复制相似问题