项目中曾经犯过的错

1：改变的入参的值。错误的代码：

public static List<Object> randomRank(List<Object> rankComments, int num)
    {
        if (null == rankComments|| 0 == rankComments.size())
        {
            return null;
        }
       
  
        // 如果轮转个数大于数组长度，则设置为数组长度
        if (num >= rankComments.size())
        {
            num = rankComments.size();
        }

       List<Object> reList = new ArrayList<Object>(comments.size());
       int index;
        java.util.Random r = new java.util.Random();
        for (int i = 0; i < num; i++)
        {
            index = r.nextInt(rankComments.size());
            RankComment s = rankComments.remove(index);
            reList.add(s);
        }
        // 把其他元素保存到新数组中
        reList.addAll(rankComments);
        return reList;
    }

  函数的功能：
    实现对指定的列表进行随机轮训，即每次从指定的列表中选出指定数量排行条目的评论信息进行轮训
但是该函数却改变了入参的值（而该入参的值是放置在缓存中），导致每次轮训都会减少指定轮训数量的评论条目
该函数本身的功能非常简单，但是结合了二级缓存（每级缓存都有不同的刷新时间）后，增加了复杂性
会出现在每级缓存的刷新时间交替时正常，其他情况下异常的问题。

该问题定位了一个礼拜，结果请一个资深的程序员看出了这个问题，并被通报批评了，从中得到的教训，尽量不要随意的去改变入参的值。