Java调用Equipment.setCommand报空指针异常 原因是实例e为null

这是非常典型的NullPointerException，触发逻辑没有任何歧义：

• 业务方法中调用equipmentRepository.findEquipmentBySerialNumber(serialNumber)查询设备时，没有查到和传入序列号匹配的记录，JPA/MyBatis等ORM框架在查询无结果时会直接返回null
• 代码拿到返回值后没有做任何非空判断，直接对null对象调用setCommand()实例方法，直接抛出异常。

常见的导致查询无结果的场景：

• 前端传参错误，传入的设备序列号在数据库中不存在
• 传入的序列号前后带了空格、制表符等不可见字符，和数据库存储值匹配不上
• 运行环境配置错误，服务连接了错误的数据库实例，对应库中没有目标设备数据

1. 必加：查询结果非空校验 #

拿到Repository返回的对象后第一时间做非空判断，从根源避免空指针：

@Override
public boolean changeEquipmentCmd( String serialNumber, String cmd )
{
  Equipment e = equipmentRepository.findEquipmentBySerialNumber( serialNumber );
  // 新增非空判断
  if (e == null) {
    throw new RuntimeException("设备不存在，序列号：" + serialNumber);
  }
  e.setCommand( cmd );
  equipmentRepository.save( e );
  return true;
}

如果项目做了全局异常拦截，可以把这里抛出的异常统一封装成结构化的接口返回，给前端返回明确的错误提示，不要把原始空指针栈信息暴露给前端。

2. 建议补充：Controller层入参校验 #

在接口入口处提前拦截非法参数，减少无效的数据库查询：

@RequestMapping( value = "/equipment/changeCmd", method = RequestMethod.PUT )
public boolean changeEquipmentCmd( @RequestParam( "sn" ) String serialNumber, @RequestParam( "cmd" ) String cmd ) 
{
  // 基础非空校验
  if (serialNumber == null || serialNumber.isBlank()) {
    throw new RuntimeException("设备序列号不能为空");
  }
  if (cmd == null || cmd.isBlank()) {
    throw new RuntimeException("目标指令不能为空");
  }
  // 对入参做trim处理，规避前后不可见字符导致的匹配失败
  return iotMetier.changeEquipmentCmd( serialNumber.trim(), cmd.trim() );
}

3. 按需选择：空值初始化逻辑 #

如果你的业务场景允许设备不存在时自动创建设备记录（比如设备首次上报自动注册的场景），可以在查询为空时直接初始化设备对象再赋值保存，不需要抛异常：

@Override
public boolean changeEquipmentCmd( String serialNumber, String cmd )
{
  Equipment e = equipmentRepository.findEquipmentBySerialNumber( serialNumber );
  if (e == null) {
    e = new Equipment();
    e.setSerialNumber(serialNumber);
  }
  e.setCommand( cmd );
  equipmentRepository.save( e );
  return true;
}

注意：这个方案不要乱用，如果是纯后台设备信息修改的场景用这个逻辑，会在数据库里插入大量不存在的脏设备数据，一定要匹配自己的业务规则再用。

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