Question

JVM的新手，使用Scala和Play 2.0

我将遗留应用程序转换为Play，需要通过Authorize.net进行付款处理。通过java.net.URL源查看，有许多潜在的失败点。鉴于我在下面写的界面，你会在哪里实现try / catch块？我需要相应地调整方法签名，可能会将[错误，成功]返回到调用客户端代码

import java.net.{URL, URLEncoder}
import java.io.{BufferedReader, DataOutputStream, InputStreamReader}
import javax.net.ssl._

trait Authnet {
  private val prodUrl = "https://secure.authorize.net/gateway/transact.dll"
  private val testUrl = "https://test.authorize.net/gateway/transact.dll"

  protected def authNetProcess(params: Map[String,String]) = {
    val(conn, urlParams) = connect(params)
    val request = new DataOutputStream( conn.getOutputStream )
    request.write(urlParams.getBytes)
    request.flush()
    request.close()
    val response = new BufferedReader(new InputStreamReader(conn.getInputStream))
    val results = response.readLine().split("\\|")
    response.close()
    results.toList
  }  

  private def connect(params: Map[String,String]) = {
    val urlParams = (config ++ params) map { case(k,v) =>
        URLEncoder.encode(k, "UTF-8") + "=" + URLEncoder.encode(v, "UTF-8")
    } mkString("&")

    lazy val url = if (isDev) new URL(testUrl) else new URL(prodUrl)
    val conn = url.openConnection
    conn.setDoOutput(true)
    conn.setUseCaches(false)
    (conn, urlParams)
  }

  private val config = Map(
    'x_login        -> "...",
    'x_tran_key     -> "...",
    ...
  )
}

Answer 1

坚持拇指规则：

如果必须处理异常，则只捕获异常。

“必须处理”没有明确的定义，但这意味着你应该抵制捕获异常的冲动，因为你可以抛出不同的异常。

“必须处理”主要由应用程序的工作方式或其他依赖关系定义。

如果应用程序需要向用户显示错误而不是异常中止，那么这是必须的。

在这种情况下，抓住重复也会增加一些有意义的处理。

如果API需要抛出不同的异常，那么这是必须的，但API定义可能不合理。

我总是在质疑用另一个例外替换异常的附加价值。

将此应用于您的示例：

是否会添加一些值以从authNetProcess（）中的connect（）中捕获异常？

没有！无法在connect（）内处理该异常。因此可以将该异常留给authNetProcess的调用者。在那里，您可以根据异常的类型提供不同的处理。

Answer 2

修改
好吧，如果连接/流进程的任何部分失败，那么事务就会被清除，因此只能在打开连接时捕获错误。我只是将整个事务包装在catching (operation) option块中，并将其保留在该块中;我不太关心re：错误的确切原因（无论是什么记录）因为它是瞬态的，所以抓住它，让用户再试一次;如果错误仍然存在，请与我们联系......

<强> ORIGINAL 好吧，鉴于迄今为止的投票和缺乏评论，我能得出的唯一结论是......在这里没有人知道他们在做什么！嘿，嘿，开玩笑; - ）

虽然我是JVM的新手，但是尝试/ catch / finally膨胀变得越来越快;通过Scala类型推断的奇迹，我将一般的错误处理抽象为简洁的实现：
catching ( operation ) option
catching ( operation ) either

除非我收到反馈，否则现在我只是通过捕获连接创建（我相信，在这种情况下，最可能的错误条件）。这是新的实现：

protected def authNetProcess(params: Map[String,String]) = { connect() match { case Some(conn) => val request = new DataOutputStream(conn.getOutputStream) request.write(getUrlParams(params).getBytes) request.flush() request.close() val response = new BufferedReader(new InputStreamReader(conn.getInputStream)) val results = response.readLine().split("\\|") response.close() results.toList case None => List[String]() } } private def connect() = { lazy val url = if (isDev) new URL(testUrl) else new URL(prodUrl) catching ( url.openConnection ) option match { case Some(conn) => conn.setDoOutput(true) conn.setUseCaches(false) //conn.setRequestProperty("Content-Type", "application/x-www-form-urlencoded") Some(conn) case None => None // connection failed } }

我认为更严格的方法是将所有潜在的错误条件提取到可能的工作选项操作中，然后将它们全部包装起来以便理解。这可能是正确/负责任的方法......但是一天只有这么多小时，将在稍后重新审视

反馈赞赏！

最佳实践：在java.net.URL中捕获失败点

2 个答案: