尝试块返回是好的做法吗？

Question

我很好奇在try块中使用return语句的最佳实践。

我有一个调用服务方法的方法，该方法返回一个整数，并可能抛出一个IllegalArgumentException。有两种方法可以做到这一点。

第一：

public int getLookupStatus(String lookupType)
{
    try
    {
        return this.lookupService.getCountOfLookupRecords(lookupType);
    }
    catch(IllegalArgumentException ex)
    {
        throw new RestException();
    }
}

第二：

public int getLookupStatus(String lookupType)
{
    int count;

    try
    {
        count = this.lookupService.getCountOfLookupRecords(lookupType);
    }
    catch(IllegalArgumentException ex)
    {
        throw new RestException();
    }

    return count;
}

在第二种方法中，计数变量似乎是不必要的，但出于某种原因，第一种方法在我看来是错误的。有什么特别的理由偏袒一方而不是另一方吗？

Answer 1

这两种方法产生相同的结果，在这两种情况下，如果没有错误或异常将被抛出，将返回一个值。

但就个人而言，我更喜欢第一个，更容易阅读。

Answer 2

我喜欢第一个版本，因为它看起来更可读。此外，如果引发了异常，那么这无关紧要，因为控件无论如何都会退出该方法。

我使用第二种模式的唯一原因是，如果我想在返回返回值之前对其进行处理。

另一方面，如果不处理IllegalArgumentException，代码可能会更好。在调用方法之前，应该有一种方法让您验证参数。

Answer 3

在try块中返回值没有问题，如果引发了illegargumentexception（或任何其他RuntimeException），您甚至不会从该方法返回任何内容，程序的正常执行流将因该异常而改变。

第二个例子中的模式用于catch块中使用/实例化的资源需要关闭的情况，然后在finally子句中正确处理这个问题。