数组都clone问题，求解释

Question

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

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

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

Answer 1

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

这是没效率的方法，不是反面教材？