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;
}
函数的功能:
实现对指定的列表进行随机轮训,即每次从指定的列表中选出指定数量排行条目的评论信息进行轮训
但是该函数却改变了入参的值(而该入参的值是放置在缓存中),导致每次轮训都会减少指定轮训数量的评论条目
该函数本身的功能非常简单,但是结合了二级缓存(每级缓存都有不同的刷新时间)后,增加了复杂性
会出现在每级缓存的刷新时间交替时正常,其他情况下异常的问题。
该问题定位了一个礼拜,结果请一个资深的程序员看出了这个问题,并被通报批评了,从中得到的教训,尽量不要随意的去改变入参的值。