如何避免在增强for循环外部声明变量，减少命名空间污染？

你提到的用代码块包裹的方案是完全有效的。Java中，一对独立的花括号会创建局部作用域，index变量的生命周期仅限于这个代码块内部，块外无法访问它，确实能避免命名空间污染，也符合你不想让无用变量留在外部的需求。

如果你觉得这种写法不够优雅，这里还有几种更简洁的替代方案：

方案1：使用Stream API简化填充逻辑 #

利用Java 8+的Stream API，可以直接把所有bucket的元素拼接成流，一次性填充到数组中，完全不需要手动维护index变量：

// 替换原有的index循环部分
int[] temp = buckets.stream()
                    .flatMap(List::stream)
                    .mapToInt(Integer::intValue)
                    .toArray();
System.arraycopy(temp, 0, a, 0, temp.length);

这种写法语义清晰，彻底消除了index变量，代码更简洁。

方案2：抽离填充逻辑到私有方法 #

把数组填充的逻辑单独封装成私有方法，index变量只在方法内部存在，外部代码完全看不到：

// 在BucketSort类中添加私有方法
private static void fillArray(int[] a, List<List<Integer>> buckets) {
    int index = 0;
    for (List<Integer> bucket : buckets) {
        for (int el : bucket) {
            a[index++] = el;
        }
    }
}

// 原sort方法中调用这个方法即可
fillArray(a, buckets);

这种方式既保持了代码可读性，又把index变量限制在方法内部，不会污染外部命名空间。

方案3：迭代器+普通for循环（不推荐，但可行） #

如果不想用Stream也不想抽方法，也可以用普通for循环遍历buckets的索引，但本质还是维护index，仅作用域稍紧凑：

int index = 0;
for (int i = 0; i < buckets.size(); i++) {
    List<Integer> bucket = buckets.get(i);
    for (int el : bucket) {
        a[index++] = el;
    }
}

不过这种写法和原有代码区别不大，没有解决根本的作用域问题，不如前两种方案优雅。

回到你最初的疑问，代码块包裹的写法是合理的，只是看起来有些“非主流”。如果团队没有代码风格限制，完全可以使用；如果追求更优雅的写法，优先考虑Stream API或者方法抽离的方案。

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