Java实现O(n)复杂度BigInteger排序单循环方案失效排查
bigSorting大整数字符串排序函数故障定位
故障表现
函数接收存储大整数的字符串列表作为入参,开发人员尝试用单循环实现O(n)时间复杂度排序,但实际运行后列表顺序和输入完全一致,未完成排序:
- 测试输入:
"31415926535897932384626433832795","1","3","10","3","5" - 错误输出:和输入顺序完全相同,无排序效果
故障代码
public static List<String> bigSorting(List<String> u) { String S = ""; for(int i = 0; i<u.size()-1; i++) { BigInteger a = new BigInteger(u.get(i)); BigInteger b = new BigInteger(u.get(i+1)); int x = a.compareTo(b); if(x>1) { S = u.get(i); u.set(i,u.get(i+1)); u.set(i+1,S); i = -1; } } return u; }
根因定位
- 直接故障点:
compareTo返回值判断条件完全错误BigInteger.compareTo()的返回值只有三种可能:a大于b返回1,a等于b返回0,a小于b返回-1,不存在返回值大于1的场景。代码中写的判断条件if(x>1)永远不会成立,交换元素的逻辑从头到尾没有执行过,这就是列表完全没有排序的直接原因。正确的交换触发条件应该是if(x > 0),即前一个元素数值大于后一个元素时触发交换。 - 设计逻辑缺陷:时间复杂度不满足O(n)要求
代码每次完成元素交换就把i重置为-1,让循环从头开始遍历,本质是效率极低的冒泡排序变体,最坏时间复杂度为O(n²),完全达不到O(n)的设计目标。同时循环中反复创建BigInteger对象做转换,也带来了不必要的性能损耗。 - 优化方向提示:这类大整数字符串排序不需要转BigInteger类型,可以先按字符串长度升序排序(长度越短数值越小),长度相同的字符串直接按字典序比较即可(等长纯数字字符串的字典序和数值序完全一致),排序效率更高。
内容的提问来源于stack exchange,提问作者AbhishekVinayak
相关产品推荐
相关产品推荐

