在Java流中用外部计数器排序实现反转是否为不良实践？

这绝对是严重的不良实践，隐患远大于它带来的“代码简洁”优势，具体问题如下：

• 彻底违反Comparator的契约：Java规定Comparator必须是无状态、一致且满足传递性的。你用counter[0]++作为排序key，每次调用都会修改外部状态，同一个元素在不同比较场景下会返回不同的key，完全打破了排序逻辑的基础。如果换成并行流，或者JDK内部实现调整了排序的比较逻辑，代码会直接出现错误结果甚至抛出异常。
• 依赖未定义的实现细节：当前这段代码能“正常工作”，只是因为串行流的遍历顺序刚好和排序逻辑的key生成顺序匹配，但Java Stream规范从未保证过这种顺序。一旦JDK更新了流的内部处理逻辑，这段代码立刻失效，而且问题极难排查。
• 可读性与维护性为零：其他开发者看到Comparator.comparing(n -> counter[0]++)这种写法，根本无法理解它的真实意图是反转顺序。后续维护时很容易被误改，出了问题也需要花费大量时间才能定位原因。

正确的替代实现 #

完全不需要这种投机取巧的写法，用标准API就能简洁、安全地实现需求：

方案1：收集后反转（兼容所有Java版本）

String[] strs = {"Sally", "Sold", "Seashells", "By", "The", "Seashore"};
return Arrays.stream(strs)
    .filter(n -> n.charAt(0) == 'S')
    .collect(Collectors.collectingAndThen(Collectors.toList(), list -> {
        Collections.reverse(list);
        return String.join(" ", list);
    }));

方案2：Java 16+ 简化写法

String[] strs = {"Sally", "Sold", "Seashells", "By", "The", "Seashore"};
return Arrays.stream(strs)
    .filter(n -> n.charAt(0) == 'S')
    .toList()
    .reversed()
    .stream()
    .collect(Collectors.joining(" "));

方案3：利用LinkedList的反向遍历

String[] strs = {"Sally", "Sold", "Seashells", "By", "The", "Seashore"};
return Arrays.stream(strs)
    .filter(n -> n.charAt(0) == 'S')
    .collect(Collectors.toCollection(LinkedList::new))
    .descendingIterator()
    .stream()
    .collect(Collectors.joining(" "));

这些写法既符合Stream的设计规范，又清晰易懂，完全不会有隐藏的风险。

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