java - 分配与多重返回的最佳实践或任何性能影响

Question

关闭。这个问题是 opinion-based 。目前不接受答案。

---

想要改进这个问题吗？更新问题，以便 editing this post 可以用事实和引文来回答它。 .

已关闭 4 年前。

Improve this question

在代码审查期间，我和另一位开发人员就处理多个返回的正确方法进行了一场小争论，如果将指针分配给空，然后设置它与简单地返回值相比会产生任何差异，之后返回 null。

private ServiceParam getRequestServiceParam(SlingHttpServletRequest request) {
    ServiceParam sp = null;
    try {
        sp = new ServiceParam(IOUtils.toString(request.getReader()));
    } catch (IOException e) {
        LOGGER.error("IOException", e);
    }
    return sp;
}

对比

private ServiceParam getRequestServiceParam(SlingHttpServletRequest request) {
    try {
        return new ServiceParam(IOUtils.toString(request.getReader()));
    } catch (IOException e) {
        LOGGER.error("IOException", e);
    }
    return null;

}

从功能上讲，它们看起来相同，但我们不知道其中一个是否比另一个更正确。尽管我担心这可能只是制表符与空格的争论。

Answer 1

一个方法多次返回是否可以是一个见仁见智的问题。但这里需要权衡一些解决方案比其他解决方案更好的地方。 Sonarqube 的投诉是一个很好的起点，但还有更大的问题。

有些人认为有多个返回是不好的，并且无论如何他们都想遵循结构化编程规则。结构化编程比使用 goto 更好，并且它确实生成外观一致的代码。

其他人指出，强制执行一次返回会使控制语句嵌套更深，导致代码更难阅读。在第一个示例中，我发现自己在方法中上下扫描，寻找局部变量在哪里获得分配的值，我认为这很烦人。

(顺便说一句，当您有多个返回值时，Sonarqube 无法正确计算圈复杂度。因此，如果它提醒您避免这种做法，则可能存在自身利益的因素；它告诉您，不要去编写代码我们很难弄清楚。)

此外，这对任何性能都没有影响。这只是一个尴尬的编码和不太理想的异常处理的问题。当您返回 null 时，您就为调用代码创建了一个无法检查 null 引用的机会，最好让异常被抛出。

我不会为此使用Optional，除非它看起来像我在单子(monad)上下文中使用的东西，在该上下文中我将东西链接在一起并使用flatMap为我处理Optional空情况。否则，必须包装和解开它会很烦人，它比返回 null 好一点，只是因为它强制调用代码考虑空情况。

在这种情况下，实际的尴尬是由读取 httprequest 引起的，这就是抛出 IOException 的原因。您可以将 throws IOException 添加到方法中并引发异常。但最好将该代码移至任何正在获取 httprequest 的 servlet 或 Web Controller ，并以与 servlet 或 Controller 处理其他任何内容相同的方式处理通过调用它抛出的任何 IOException。

一旦您移动了该请求读取代码，就根本没有令人信服的理由让该方法存在。删除不必要的代码是一件好事。