如何在Groovy中允许0作为方法参数并处理非空非负整数输入

先帮你梳理这段代码里的核心问题，再给出符合需求的解决方案：

原代码的关键问题 #

• 基本类型无法接收null：你用的int是Java/Groovy的基本数据类型，它不能存储null值。如果controller可能只传递a或b其中一个参数，未传递的参数会是null，这时候用int会直接触发异常，得换成包装类型Integer。
• 条件逻辑错误：if(a >= 0 || a != null) 这个判断逻辑完全反了——你需要的是「参数不为null 并且 值大于等于0」的时候才赋值，应该用&&而不是||。而且如果参数是null的话，先执行a >=0会直接抛出空指针异常，得先判断null。
• 日志语句不完整：最后log.info logger...的语法不完整，需要补全才能正常运行。

修正后的代码 #

// 类成员变量，假设是service类里的全局变量
int x = 10;
int y = 10;

boolean doSomething(Integer a, Integer b) {
    StringBuilder logger = new StringBuilder(); // 多次字符串拼接用StringBuilder更高效
    
    // 处理参数a：不为null且>=0时更新x
    if (a != null && a >= 0) {
        x = a;
        logger.append("Value of x is ${x}. ");
    }
    
    // 处理参数b：不为null且>=0时更新y
    if (b != null && b >= 0) {
        y = b;
        logger.append("Value of y is ${y}. ");
    }
    
    // 如果有日志内容，输出日志
    if (logger.length() > 0) {
        log.info(logger.toString());
    }
    
    // 方法需要返回boolean，这里可以根据业务需求调整，比如返回是否有参数被成功更新
    return logger.length() > 0;
}

关键说明 #

• 用Integer替代int作为参数类型，这样就能接收controller传递的null值（当只传一个参数时，另一个参数会是null）。
• 先判断a != null再判断a >=0，既避免空指针异常，又保证只有合法的非负整数才会更新x/y的值。
• 用StringBuilder代替直接字符串拼接，在多次拼接场景下性能更优，也更符合编码规范。
• 最后的返回值可以根据你的业务需求调整，比如固定返回true，或者返回是否有参数被成功更新。

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