首页 > 解决方案 > 在 Scala 中处理和抛出异常

问题描述

我有以下实现:

  val dateFormats = Seq("dd/MM/yyyy", "dd.MM.yyyy")
  implicit def dateTimeCSVConverter: CsvFieldReader[DateTime] = (s: String) => Try {
    val elem = dateFormats.map {
      format =>
        try {
          Some(DateTimeFormat.forPattern(format).parseDateTime(s))
        } catch {
          case _: IllegalArgumentException =>
            None
        }
    }.collectFirst {
      case e if e.isDefined => e.get
    }
    if (elem.isDefined)
      elem.get
    else
      throw new IllegalArgumentException(s"Unable to parse DateTime $s")
  }

所以基本上我正在做的是,我正在运行我的 Seq 并尝试用不同的格式解析 DateTime。然后我收集第一个成功的,如果没有,我把异常扔回去。

我对代码并不完全满意。有没有更好的方法让它变得更简单?我需要将异常消息传递给调用者。

标签: scala

解决方案


您的代码的一个问题是,无论日期是否已被解析,它都会尝试所有模式。你可以使用惰性收集,比如 Stream 来解决这个问题:

 def dateTimeCSVConverter(s: String) = Stream("dd/MM/yyyy", "dd.MM.yyyy")
       .map(f => Try(DateTimeFormat.forPattern(format).parseDateTime(s))
       .dropWhile(_.isFailure)
       .headOption

更好的是 jwvh 提出的带有 find 的解决方案(您不必调用headOption):

 def dateTimeCSVConverter(s: String) = Stream("dd/MM/yyyy", "dd.MM.yyyy")
       .map(f => Try(DateTimeFormat.forPattern(format).parseDateTime(s))
       .find(_.isSuccess)

None如果没有匹配的模式,则返回。如果你想在这种情况下抛出异常,你可以使用 uwrap 选项getOrElse

 ...
 .dropWhile(_.isFailure)
 .headOption
 .getOrElse(throw new IllegalArgumentException(s"Unable to parse DateTime $s"))

重要的是,当任何验证成功时,它不会更进一步,而是会立即返回解析日期。


推荐阅读