请验证基于Java.net的CIDR范围内IP地址列表生成代码有效性

你提供的这段代码仅能在有限场景下正确工作，下面从适用范围、存在问题和改进方向三个部分分析：

一、适用的正确场景 #

当输入是合法的IPv4 CIDR格式，且前缀长度prefix在17~32之间时，代码可以正常生成对应范围内的IP列表。比如常见的192.168.1.0/24、10.0.0.0/28这类小范围网段，测试下来结果是准确的。

二、存在的关键问题 #

1. 仅支持IPv4，完全不兼容IPv6 #

代码用int存储IP地址（32位），刚好匹配IPv4的长度，但IPv6是128位，这段代码无法处理任何IPv6的CIDR格式。

2. 输入合法性校验缺失 #

• 未检查CIDR格式是否正确：比如输入无/的纯IP、/前后为空、前缀不是数字等情况，会直接抛出ArrayIndexOutOfBoundsException或NumberFormatException。
• 未校验前缀范围：前缀prefix必须在0~32之间，输入/33或/-1会导致numAddresses计算错误，甚至出现负数。
• 未校验IP合法性：输入无效IP（如abc/24）时，仅打印UnknownHostException栈轨迹，返回空列表，没有明确的错误提示。

3. 整数溢出导致结果错误 #

• numAddresses计算溢出：当prefix ≤ 0时，32-prefix ≥32，Math.pow(2,32)的结果是4294967296，强制转int会溢出为0，导致循环不执行，返回空列表。
• currentIP溢出：当prefix较小（比如/16，对应65536个IP），ipAddress + i会超过int的最大值（2^31-1），触发整数溢出，生成错误的IP地址（比如本该是255.255.xxx.xxx的IP会变成0.0.xxx.xxx）。

4. 大规模网段的性能问题 #

当prefix很小（比如/8，对应1600多万个IP），生成大量字符串会占用极高内存，极易触发OOM，且循环执行效率极低。

三、改进建议 #

针对上述问题，给出优化后的代码示例：

import java.net.InetAddress;
import java.net.UnknownHostException;
import java.util.ArrayList;
import java.util.List;
import java.util.regex.Pattern;

public class CIDRUtils {
    private static final Pattern CIDR_PATTERN = Pattern.compile("^((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\\.){3}(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)/([0-9]|[1-2][0-9]|3[0-2])$");

    public static List<String> getIPsInCIDR(String cidrRange) throws IllegalArgumentException, UnknownHostException {
        List<String> ips = new ArrayList<>();
        
        // 校验CIDR格式
        if (!CIDR_PATTERN.matcher(cidrRange).matches()) {
            throw new IllegalArgumentException("Invalid CIDR format, should be like '192.168.1.0/24'");
        }

        String[] parts = cidrRange.split("/");
        String ipStr = parts[0];
        int prefix = Integer.parseInt(parts[1]);

        // 转换IP为long类型（避免溢出）
        byte[] ipBytes = InetAddress.getByName(ipStr).getAddress();
        long ipAddress = 0;
        for (byte b : ipBytes) {
            ipAddress = (ipAddress << 8) | (b & 0xFF);
        }

        // 计算网段内的IP数量，用long存储
        long numAddresses = 1L << (32 - prefix);

        // 生成所有IP
        for (long i = 0; i < numAddresses; i++) {
            long currentIP = ipAddress + i;
            byte[] currentIPBytes = new byte[4];
            for (int j = 3; j >= 0; j--) {
                currentIPBytes[j] = (byte) (currentIP & 0xFF);
                currentIP >>= 8;
            }
            InetAddress address = InetAddress.getByAddress(currentIPBytes);
            ips.add(address.getHostAddress());
        }

        return ips;
    }

    public static void main(String[] args) {
        try {
            List<String> ips = getIPsInCIDR("192.168.1.0/24");
            System.out.println("Generated " + ips.size() + " IPs");
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}

优化点说明：

• 增加了CIDR格式的正则校验，提前拦截无效输入
• 用long存储IP地址和地址数量，彻底解决整数溢出问题
• 抛出明确的异常而非默默打印栈轨迹，便于调用方处理错误
• 保留原有的核心逻辑，仅修复关键问题

内容的提问来源于stack exchange，提问作者Karthick A.S