Java 8 forEach Stream()与旧版forEach Loop的区别

Question

我正在尝试使用Spring的示例代码。其中一部分代码如下所示：

private List<Point> points;
long timeTakeninMilis = System.currentTimeMillis();

public List<Point> getPoints() {
    return points;
}

public void setPoints(List<Point> points) {
    this.points = points;
}

public void drawJava8() {
    points.stream().forEachOrdered(
            point -> System.out.println("Point : (" + point.getX() + ", "
                    + point.getY() + ")"));
    System.out.println("Total Time Taken drawJava8(): "
            + (System.currentTimeMillis() - timeTakeninMilis)
            + " miliseconds");
}

public void draw() {
    for (Point point : points) {
        System.out.println("Point = (" + point.getX() + ", " + point.getY()
                + " )");

    }
    System.out.println("Total Time Taken draw(): "
            + (System.currentTimeMillis() - timeTakeninMilis)
            + " miliseconds");
}

输出（OUTPUT），

  Jun 30, 2015 11:30:53 AM org.springframework.context.support.ClassPathXmlApplicationContext prepareRefresh
  INFO: Refreshing org.springframework.context.support.ClassPathXmlApplicationContext@7daf6ecc: startup date [Tue Jun 30 11:30:53 IST 2015]; root of context hierarchy
  Jun 30, 2015 11:30:53 AM org.springframework.beans.factory.xml.XmlBeanDefinitionReader loadBeanDefinitions
  INFO: Loading XML bean definitions from class path resource [spring.xml]
  Point = (0, 0 )
  Point = (-50, 0 )
  Point = (0, 50 )
  Total Time Taken draw(): 70 miliseconds
  Point : (0, 0)
  Point : (-50, 0)
  Point : (0, 50)
  Total Time Taken drawJava8(): 124 miliseconds
  Jun 30, 2015 11:30:54 AM org.springframework.context.support.ClassPathXmlApplicationContext doClose
  INFO: Closing org.springframework.context.support.ClassPathXmlApplicationContext@7daf6ecc: startup date [Tue Jun 30 11:30:53 IST 2015]; root of context hierarchy

为什么它需要更长的时间？或者是我做错了什么吗？

我原本期望它能够更快或与之前的速度相似...请帮我理解Lambda表达式的好处是什么？

信息：我在两个不同的程序中使用过它。时间是从那些程序中记录下来的，我将它们合并在一起以缩短篇幅。

- Rajkishan Swami

---

通过查看您的输出，您首先调用drawJava()，然后再调用drawJava8()。在调用drawJava8()之前，您是否使用当前时间重置了变量timeTakeninMilis？ - Madhavi Balan

编写Java基准测试很困难，因为JVM会不断分析和优化您的代码。请参考https://dev59.com/hHRB5IYBdhLWcg3wz6UK。 - beresfordt

你试过使用 forEach 而不是 forEachOrdered 吗？但正如 @beresfordt 所说：基准测试并不容易完成。更接近有效结果的一步是使用更多的点并运行测试多次，这样你就有了一些统计价值。现在你能说的只是，在这个单一的时间内，Java 8 方法的执行速度要慢得多。但这可能有比代码本身更多的原因。 - Fildor

@beresfordt：哇...这个帖子有很多有用的信息。非常感谢。 - Rajkishan Swami

@Fildor：是的，我试过了。没有发现任何区别。时间总是在122毫秒左右。有时会多或少2-3毫秒。实际上，我在一个中等规模的项目中尝试过，发现stream().forEach()实际上需要多花费1.5-2秒钟。代码运行了很多次。我认为你关于基准测试的想法是正确的，因为我在Java方面没有太多经验。可能还有其他因素在起作用。 - Rajkishan Swami

Answer 1

根据原帖作者的要求，我将这个内容作为分析添加进来。

我们无法准确预测现代JIT编译器对运行代码所执行的复杂分析和转换。因此，在对此类项目进行基准测试时，您不应仅通过运行两个方法调用来得出结论。

相反，创建各种样本输入集（边界情况），并通过重复调用您的测试用例而不关闭JVM来检查性能。例如，在这种情况下：

for (int i=0;i<100;i++){draw(); drawJava8();}

在获得结果后，找出平均执行时间，你可以安全地忽略第一次执行结果，因为它可能没有进行优化。

因此，你从测试中得出的结论并不完全正确。

- ring bearer

---

这太棒了！ - phoenixSid