数组都clone问题，求解释

Question

最近看一本书叫《改善java程序的151个建议》，发现这么一个方法。

 public static int max(int[] data) {
        Arrays.sort(data.clone()); //秒处
        return data[data.length - 1]
 }

这个方法的意图是找到数组中最大的一个数，为什么排序的是克隆对象呢，
因为这个方法的目的是取最大数，通过克隆对象去取的话，不改变原有数组的结构，
可是读者发现这个方法有问题，排序的是克隆对象，取值却还是从原来的数组中取.
请教大家，这个方法应该到底怎么写。

Answer 1

首先我要解释一点，就是，保持程序的正确性，维持程序的简洁性，是作为程序员的第一要务Don’t Cut Yourself: Code Optimization as a Double-Edged Sword。中文翻译：过早优化是万恶之源。 代码优化的好处多多，但是这并不意味着所有的代码都需要进行优化，有时过度的优化反而适得其反——费时、费力、不讨好。 “现代计算机科学的鼻祖”Donald Knuth曾说过“过早的优化是万恶之源”，因为：让正确的程序更快，要比让快速的程序正确容易得多。文中讲了7个原则，简单罗列如下： 1. 究竟要优化什么？ 2. 选择一个正确的优化指标 3. 优化在刀刃上 4. 优化层次越高越好 5. 不要过早优化 6. 依赖性能分析，而不是直觉 7. 优化不是万金油因此脱离了上下文，不要去讨论代码的效率。以你的程序为例，我们首先要搞清楚的是，这个寻找最大数的代码，在整个程序中是否是在热区上。所谓热区，就是说，这个函数是频繁被调用么？或者是程序中耗时最多的代码么？大多数情况下，你写了一个程序（我说的是有一定规模的商业软件），90%的代码，其实都是几乎不被调用的，而只有10%的代码是反复调用，耗时巨大的。在不是热区的代码上“优化”，哪怕程序性能提高了1亿倍，但是程序的运行时间也只能压缩10%。然而这么做付出的代价是，程序变得面目全非，到处都是各种奇怪的技巧，难以修改。项目开发进度大大增加。甚至为了性能，引入bug，连正确性都没有办法保证。我没有你说的那本书，也没有看到你代码的上下文。但是如果这个代码不位于热区，并不需要苛求性能的话，我觉得这么写很清晰，体现了作者明白的意图，就是寻找最大数，并且通过克隆避免对原始数据的顺序的修改嘛。当然，我们有更快的做法，问题是一个会武功的人，需要在吃饭的时候也把筷子当作武器耍弄一番么？