应该……接住环内还是环外?

我有一个这样的循环:

for (int i = 0; i < max; i++) {
    String myString = ...;
    float myNum = Float.parseFloat(myString);
    myFloats[i] = myNum;
}

这是一个方法的主要内容，该方法的唯一目的是返回浮点数数组。我想让这个方法在出现错误时返回null，所以我把循环放在try…Catch block，像这样:

try {
    for (int i = 0; i < max; i++) {
        String myString = ...;
        float myNum = Float.parseFloat(myString);
        myFloats[i] = myNum;
    }
} catch (NumberFormatException ex) {
    return null;
}

但后来我也想到试一试……Catch块在循环中，像这样:

for (int i = 0; i < max; i++) {
    String myString = ...;
    try {
        float myNum = Float.parseFloat(myString);
    } catch (NumberFormatException ex) {
        return null;
    }
    myFloats[i] = myNum;
}

是否有任何理由，性能或其他方面，更喜欢其中一个?

编辑:共识似乎是，将循环放在try/catch中更干净，可能是在它自己的方法中。然而，关于哪个速度更快仍存在争议。有人能测试一下并给出一个统一的答案吗?

当前回答

你应该喜欢外面的版本而不是里面的版本。这只是规则的一个特定版本，将任何可以移动到循环外的东西移动到循环外。根据IL编译器和JIT编译器的不同，您的两个版本最终可能具有不同的性能特征，也可能没有。

另一方面，你可能应该看看float。TryParse或Convert.ToFloat。

2008-09-26 19:57:18

其他回答

为try/catch设置一个特殊的堆栈框架会增加额外的开销，但是JVM可能能够检测到您正在返回并优化它。

根据迭代次数的不同，性能差异可能可以忽略不计。

然而，我同意其他人的观点，把它放在循环之外会让循环体看起来更干净。

如果您希望继续处理，而不是在存在无效数字时退出，那么您将希望代码位于循环中。

2008-09-26 20:01:47