如何确保服务中doSomething方法调用时param1参数合法？

嘿，针对你遇到的服务方法因非法参数引发故障的问题，我来分享下最靠谱的解决思路，结合你后来的更新调整下方向：

首先得明确：用单元测试去验证所有调用doSomething的地方的参数合法性，并不是最优方案。单元测试的核心是验证方法自身的业务逻辑是否符合预期，而不是去遍历所有调用点做参数检查——这种方式不仅工作量巨大，后续新增调用点时也很容易遗漏测试，维护成本极高。

真正应该从源头预防错误参数的传入，这里有两种非常实用的方案：

1. 在方法内部添加参数前置校验 #

直接在doSomething方法的开头就做参数合法性检查，一旦发现非法参数立刻抛出异常，阻止错误继续扩散。你可以用Guava的Preconditions工具类，也可以自己实现简单的检查逻辑：

import com.google.common.base.Preconditions;
import java.util.Set;
import static java.util.Set.of;

public boolean doSomething(String param1, String param2) {
    // 定义param1的合法值集合
    Set<String> validParam1Values = of("String1", "String2", "String3");
    
    // 校验参数，不合法则抛出IllegalArgumentException
    Preconditions.checkArgument(validParam1Values.contains(param1), 
        "param1必须为String1、String2或String3中的一个，当前传入值：%s", param1);
    
    // 执行后续业务逻辑
    // ...
    return true;
}

这种方式能确保不管哪个地方调用该方法，只要参数不合法都会立刻报错，把问题扼杀在萌芽状态。

2. 用枚举替代字符串参数（更安全的方案） #

如果param1的合法值是固定不变的，直接将参数类型从String改成自定义枚举，这样编译期就能阻止非法值传入，完全避免了运行时参数错误的可能：

// 定义枚举类，限定param1的合法值
public enum ValidParam1 {
    STRING1, STRING2, STRING3
}

// 修改方法参数类型
public boolean doSomething(ValidParam1 param1, String param2) {
    // 执行业务逻辑
    // ...
    return true;
}

调用方只能传入ValidParam1的枚举实例，根本不可能传入非法字符串，这是最彻底的预防手段。

更新：根据之前采纳的思路补充下——这类参数合法性校验的场景，确实不适合用单元测试来覆盖。单元测试聚焦于方法的行为验证，而参数校验属于方法的前置保障逻辑，把校验逻辑内置于方法本身（或用枚举约束），才是从根源上预防错误调用的正确方式。

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