如何在不使用Try / Catch块的情况下编写此代码？

Question

我希望重写此scala函数，但是我是该语言的新手，我知道可以使用try \\ catch块来替代。 你们将如何重写此功能？

  def updateStationPost = Action { implicit request =>
    StationForm.bindFromRequest.fold(
      errors => { //needs to be revised!!
        BadRequest(html.updateStation(errors,
             Station(
              request.body.asFormUrlEncoded.get("id")(0).toLong,
              request.body.asFormUrlEncoded.get("operator")(0).toLong,
              request.body.asFormUrlEncoded.get("name")(0),
              try {
                request.body.asFormUrlEncoded.get("number")(0).toInt
              } catch {
                case e:Exception => { 0 } //this exception happens when trying to convert the number when there is nothing in the flash scope to convert.
              },
              request.body.asFormUrlEncoded.get("timezone")(0)
            ),
            Operators.retrieveJustOperators() //ugh... needs to be revised..
          )
        )
      },
      { case(stationFormObj) =>
        Stations.update(stationFormObj)
        Redirect(routes.StationsController.index)
      }
    )
  }

Answer 1

管理此问题的一般方法是使用“ 尝试包装”可能引发异常的代码。 使用它的一些方法如下所示：

def unpredictable() = {
  Try(Console.readLine("Int please: ").toInt) getOrElse 0
}

如果控制台读取的内容不包含可解析的整数，则它将引发异常。 如果发生错误，此代码仅返回0，但是您可以在其中放置其他语句。 或者，您可以使用模式匹配来处理这种情况。

def unpredictable() =  {
  Try(Console.readLine("Int please: ").toInt) match {
    case Success(i) => i
    case Failure(e) => println(e.getMessage()) 
  }
}

您也可以只返回Try，让调用者决定如何处理失败。

Answer 2

怎么样：

import scala.util.control.Exception.handling

// Create a val like this as you reuse it over and over
val form: Option[Map[String, Seq[String]]] = request.body.asFormUrlEncoded

// Create some helper functions like this
val nfeHandler = handling(classOf[NumberFormatException]) by (_ => 0)
val intNFEHandler = (str: String) => nfeHandler apply str.toInt
val longNFEHandler = (str: String) => nfeHandler apply str.toLong

// You can use this instead of your try catch.. but this is just a sugar.. perhaps cleaner
intNFEHandler apply form.get("id")(0)

这里的形式是这样的： Option(Map("id" -> Seq.empty[String]))

form.get("id")(0)会因java.lang.IndexOutOfBoundsException而form.get("id")(0) 。

我建议再找一个助手：

// takes fieldNames and returns Option(fieldValue)
val fieldValueOpt = (fieldName: String) => form.flatMap(_.get(fieldName).flatMap(_.headOption))

然后创建一个validate方法，该方法对所有fieldValue可选参数执行模式匹配，提取值并创建Station对象。