Java编译优化和性能

Question

 public static void main(String[] args) {
     double [] boxes;
        boxes = new double[]  {20, 10, 5, 40, 20, 41, 41, 2, 6, 7, 3, 4, 5, 6, 23, 34, 7, 8, 2, 2};
        double heaviest = 0;
        double normal = 0;
        double heavy = 0;
        double totalCost;
        double a = 0;
        double b = 0;
        int repeatCount=0;

        for (int i = 1; i < boxes.length; i++) {
            if (boxes[i] > heaviest)
                heaviest = boxes[i];
        }

        for(double element: boxes) {
            if(element==heaviest) {
                repeatCount = repeatCount+1;
            }
        }

        System.out.println("Count :" +repeatCount);

        for (int j =0; j<boxes.length; j++) {
            if (boxes[j] < heaviest) {
                a = boxes[j] * 2;
                normal = normal+a;
            } else {
                b =  (boxes[j] * 3.5);
                heavy = heavy+b;
            }
        }
        totalCost = normal+heavy;
        System.out.println("total cost of the insuranse is  "+ totalCost);
    }

第 1 部分：我需要将最大的元素乘以 3.5，然后再乘以 2，然后将其相加得到总数。

第 2 部分：我还想要最大元素的出现次数。 或我们也可以将最大的元素存储在另一个数组中。

在我的代码中，我已经完成了第 1 部分，对于第 2 部分，我正在计算最大元素的出现次数。

我的问题：

如果有超过1000 个元素，是否有其他方法可以减少编译时间或优化代码？

我还尝试通过将数组转换为列表来使用Collections.frequency(myArray, largestElement)。

Answer 1

一如既往，正确性比性能更重要。

不知道为什么它在寻找最重的框时会忽略第一个框。

您可以减少代码，但最大的问题是您的代码运行时间不够长，无法被 JIT 编译。即便如此，与 Java 的启动成本相比，它还是微不足道的。

简而言之，除非您的程序持续几百毫秒，否则我不会担心性能。

顺便说一句，您可以使用单个循环来缩短代码，但就像我说的那样，在这种情况下不会有太大区别。

我可能会这样写。

double[] boxes = {20, 10, 5, 40, 20, 41, 41, 2, 6, 7, 3, 4, 5, 6, 23, 34, 7, 8, 2, 2};
double heaviest = -1;
int count = 0;
double sum = 0;
for (double box : boxes) {
    sum += box;
    if (box > heaviest) {
        count = 1;
        heaviest = box;
    } else if (box == heaviest) {
        count++;
    }
}

// double total = sum * 2 - heaviest * count * 2 + heaviest * count * 3.5;
double total = sum * 2 + heaviest * count * 1.5;
System.out.println("total: " + total);

注意：只有一个循环。您可以随时计算计数和总和。

Answer 2

public static void main(String[] args) {
    double [] boxes = new double[]  {20, 10, 5, 40, 20, 41, 41, 2, 6, 7, 3, 4, 5, 6, 23, 34, 7, 8, 2, 2};
    double heaviest = 0;
    double normal = 0;
    double heavy = 0;
    double totalCost;
    int repeatCount=0;

    for (double d : boxes) {
        heaviest = Math.max (boxes[i], heaviest);
    }

    for (double d : boxes) {
        if(d == heaviest) {
            repeatCount++;
        }
    }
    System.out.println("Count: " + repeatCount);

    for (double d : boxes) {
        if (d == heaviest) {
            heavy += d * 2.0;       
        } else {
            heavy += d * 3.5;
        }
    }
    totalCost = normal+heavy;

    System.out.println("Total cost of the insuranse is: " + totalCost);
}

根据您的知识，这是在遵循您的代码，但修正了一点。您永远不需要使用i，也不需要将结果存储在a 或b，而是可以使用+= 运算符。

话虽如此，double d1 == double d2 使用起来很危险。我不建议直接使用它们。阅读this 文章或this。实际上，我建议您同时阅读。