Java中前置条件检查前初始化实例变量是否合理？及代码问题排查

第一段代码的致命逻辑错误 #

你第一段代码的问题完全是检查对象搞反了——你在检查未赋值的实例变量this.healthProvider，而不是传入的参数healthProvider：

public Provider(String healthProvider) {
  // 此时this.healthProvider还没赋值，默认是null，和传入参数无关
  if (this.healthProvider == null) {
    throw new IllegalArgumentException(PROVIDER_NULL);
  }

  if (this.healthProvider.isBlank()) { // 这里直接抛NPE，因为this.healthProvider是null
    throw new IllegalArgumentException(PROVIDER_ISBLANK);
  }
  this.healthProvider = healthProvider;
  this.patients = new ArrayList<Patient>();
}

不管你传入的参数是null、空字符串还是合法值，第一个检查都会触发（因为this.healthProvider始终是null），如果强行跳过第一个检查，第二个isBlank()调用会直接抛出空指针异常——这就是你说“isBlank检查未通过，但null检查通过”的本质原因，代码逻辑完全颠倒。

第二段代码的潜在问题 #

第二段代码先赋值再检查，虽然测试能过，但违反了失败快速原则：如果参数不合法（比如null），你已经初始化了patients集合，这属于无效的资源分配。虽然ArrayList的开销很小，但在复杂场景下（比如初始化数据库连接、大内存对象），会造成不必要的资源浪费。

正确的实现方式 #

正确的做法是先直接验证传入参数的合法性，再初始化实例变量，全程不需要用this.xxx来检查参数：

public Provider(String healthProvider) {
  // 直接检查参数本身
  if (healthProvider == null) {
    throw new IllegalArgumentException(PROVIDER_NULL);
  }
  if (healthProvider.isBlank()) {
    throw new IllegalArgumentException(PROVIDER_ISBLANK);
  }
  // 参数合法后，再初始化实例变量
  this.healthProvider = healthProvider;
  this.patients = new ArrayList<Patient>();
}

这种写法既保证了参数不合法时立刻抛出异常，不做任何无效初始化，又彻底避免了第一段代码的逻辑错误，测试也能正常通过。

关于你的测试问题 #

第一段代码测试不通过的原因很简单：它的检查逻辑完全错误，永远在判断一个未赋值的null变量，导致所有测试用例都会触发null异常，根本走不到isBlank的检查逻辑，自然无法通过测试。

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