为何PagingAndSortingRepository的findAll(Pageable)返回错误排序？

这问题的核心原因非常直接：你把排序好的结果放进了HashSet里，而HashSet是不保证元素顺序的！

咱们拆解下你的代码执行流程：

• repository.findAll(paging)返回的Page对象中，getContent()给出的List<City>其实已经是按照population字段正确排序后的结果，和你预期的SQL执行结果顺序完全一致。
• 但你紧接着把这个有序List转成了HashSet——HashSet基于哈希表实现，它的元素顺序由元素的哈希值决定，和插入顺序、排序顺序都无关，自然会把原本的排序彻底打乱，导致你遍历的时候看到的是无序结果。

解决办法 #

根据你的业务需求，有两种常见修正方式：

1. 改用LinkedHashSet：如果你确实需要Set类型（比如要做去重），可以用LinkedHashSet替代HashSet，它会维护元素的插入顺序，这样就能保留List原本的排序：

   return new LinkedHashSet<>(repository.findAll(paging).getContent());
   

2. 直接返回List：如果业务上不需要去重，完全可以直接返回List类型，排序会被完整保留：

   public List<City> findAllPageable(int page, int size, String sortBy) { 
       Pageable paging = PageRequest.of(page, size, Sort.by(sortBy)); 
       return repository.findAll(paging).getContent(); 
   }
   

内容的提问来源于stack exchange，提问作者Ilja Tarasovs