应该尝试…捕捉到循环内还是循环外？

Question

我有一个看起来像这样的循环：

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

这是方法的主要内容，其唯一目的是返回浮点数数组。我希望此方法null在出现错误时返回，因此我将循环放在一个try...catch块中，如下所示：

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

但是后来我也想到将try...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中（可能在其自己的方法中）更干净。但是，仍然存在关于哪个更快的争论。有人可以测试一下并返回统一答案吗？

Answer 1

好的，在Jeffrey L Whitledge说没有性能差异（截至1997年）之后，我去测试了它。我运行了这个小基准：

public class Main {

    private static final int NUM_TESTS = 100;
    private static int ITERATIONS = 1000000;
    // time counters
    private static long inTime = 0L;
    private static long aroundTime = 0L;

    public static void main(String[] args) {
        for (int i = 0; i < NUM_TESTS; i++) {
            test();
            ITERATIONS += 1; // so the tests don't always return the same number
        }
        System.out.println("Inside loop: " + (inTime/1000000.0) + " ms.");
        System.out.println("Around loop: " + (aroundTime/1000000.0) + " ms.");
    }
    public static void test() {
        aroundTime += testAround();
        inTime += testIn();
    }
    public static long testIn() {
        long start = System.nanoTime();
        Integer i = tryInLoop();
        long ret = System.nanoTime() - start;
        System.out.println(i); // don't optimize it away
        return ret;
    }
    public static long testAround() {
        long start = System.nanoTime();
        Integer i = tryAroundLoop();
        long ret = System.nanoTime() - start;
        System.out.println(i); // don't optimize it away
        return ret;
    }
    public static Integer tryInLoop() {
        int count = 0;
        for (int i = 0; i < ITERATIONS; i++) {
            try {
                count = Integer.parseInt(Integer.toString(count)) + 1;
            } catch (NumberFormatException ex) {
                return null;
            }
        }
        return count;
    }
    public static Integer tryAroundLoop() {
        int count = 0;
        try {
            for (int i = 0; i < ITERATIONS; i++) {
                count = Integer.parseInt(Integer.toString(count)) + 1;
            }
            return count;
        } catch (NumberFormatException ex) {
            return null;
        }
    }
}

我使用javap检查了生成的字节码，以确保没有内联任何内容。

结果表明，假设微不足道的JIT优化， Jeffrey是正确的 。 Java 6，Sun客户端VM上 绝对 没有性能差异
（我无法访问其他版本）。在整个测试中，总的时间差约为几毫秒。

因此，唯一的考虑就是看起来最干净的东西。我发现第二种方法很难看，所以我会坚持第一种方法或Ray
Hayes的方法