是否应重复校验方法参数？探讨temporal coupling与方法可见性

咱们先从你给出的示例代码说起，再分场景聊清楚这个问题——核心差异其实在于方法是私有（private）还是公有（public），这直接决定了最佳方案：

一、私有方法场景（如示例代码） #

首先看你的代码，两个方法都是private的，当前只在main方法里按顺序调用。这种情况下：

• 重复校验确实属于冗余操作：因为私有方法的调用范围完全在当前类内部，你可以通过团队约定或者代码注释明确printFirstArgument必须在checkInputArguments之后调用。就算有人不小心搞反了顺序，运行时抛出的异常也能快速定位问题——毕竟调用方就在同一个类里，维护成本极低。
• 所谓的「时序耦合」在这里影响很小：类内部的方法调用逻辑是封闭的，只要你在main方法里固定好调用顺序，其他地方不会有调用入口。如果担心后续有人乱改，可以把两个方法的调用封装成一个新的私有方法，从根源上避免时序错误：

private static void processArguments(String[] args) {
    checkInputArguments(args);
    printFirstArgument(args);
}

之后main方法只需要调用这个封装方法就行，彻底消除时序依赖的隐患。

二、如果方法是public的 #

这情况就完全不同了！公有方法是对外暴露的API，你根本无法控制调用方的行为——调用者可能直接跳过checkInputArguments，直接调用printFirstArgument。这时候如果没有参数校验，轻则抛出ArrayIndexOutOfBoundsException这种语义模糊的异常，重则导致程序崩溃。

• 这时候必须在每个public方法里独立做参数校验：公有方法要保证自身的健壮性，不能依赖调用方是否执行了前置操作。每个public方法都应该是“自给自足”的，能处理所有合法/非法输入，抛出清晰的错误信息。
• 担心代码冗余？可以把校验逻辑抽成一个公共的私有方法，让两个public方法都调用它：

// 公共校验逻辑，私有方法只供内部调用
private static void validateArguments(final String[] args) {
    if (args.length == 0) {
        throw new IllegalArgumentException("Argument is missing. Proper usage is: ...");
    }
}

public static void checkInputArguments(final String[] args) {
    validateArguments(args);
    // 这里可以加其他逻辑，比如打印校验成功日志等
}

public static void printFirstArgument(final String[] args) {
    validateArguments(args);
    System.out.println(args[0]);
}

这样既避免了代码重复，又保证了每个public方法的独立性和安全性。

三、其他需要考虑的因素 #

• 异常类型的合理性：你示例里用了RuntimeException，其实对于参数校验，更推荐用IllegalArgumentException——这是Java标准库专门为非法参数设计的异常，语义更清晰，调用者一眼就能明白是输入出了问题。
• 未来的扩展性：如果私有方法后续可能被其他方法调用（比如新增了一个processSecondArgument方法也需要用到printFirstArgument），那提前加上校验（或者复用校验逻辑）能避免后续出现隐性bug。毕竟需求是变化的，今天的私有方法可能明天就有新的调用场景。
• 团队编码规范：有些团队会要求所有方法（哪怕是私有）都做参数校验，保证每个方法的独立性，避免因为时序依赖导致的不易察觉的bug。这种情况下，复用校验逻辑就是符合规范的选择。

总结一下 #

• 私有方法+调用范围可控：可以去掉重复校验，通过封装调用或注释避免时序问题，减少代码冗余。
• 公有方法：必须独立校验（可复用校验逻辑），保证API的健壮性，不能依赖调用方的前置操作。
• 折中方案：如果担心私有方法后续扩展，也可以复用校验逻辑，既不冗余，又保留方法的独立性。

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