麻烦帮忙看一下代码

老师帮忙看一下，以下代码有什么问题：

public PageModel pagination(int category,int page,int rows) {

List<painting> list=XmlDateSource.getRawData();

List <painting>categoryList=XmlDateSource.getRawData();

for(painting p:list) {

if(p.getCategroy()==category) {

categoryList.add(p);

}

}

PageModel pageModel=new PageModel(list,page,rows);

return pageModel;

}

如果直接运行时，服务器会报：内部服务器错误，list集合内元素不是可以重复的吗，如果把以上代码的categoryList.add(p);注释掉，就不会报错，怎么回事