可选的应用是减少空值。
检查提供的可选项是否为null并不足以让编译器确定其值不为null。
要检查可选值，请使用optional.isEmpty();或optional.isPresent();。
在您的情况下：检查存储库的响应，如下所示：

Optional<Animal> animal = animalRepository.findById(animalId);
if (animal.isEmpty()) {
            throw new DeviceNotValidException("...");
}

或者更简单：

Animal animal = animalRepository.findById(animalId)
                      .orElseThrow(() -> new DeviceNotValidException("...");

最后一种方法确保animal不为空。

行if (animal == null)不符合您所需的行为。虽然Optional的内容可以为null，但Optional本身是一个 Package 器，不应为null。
将该行改为：if (!animal.isPresent()).

由于您使用的是Spring Data，因此您遇到的问题是检查Optional是否为null，其中所说的Optional是为了减少空检查。
Z-100建议的正确用法是使用orElseThrow：

Animal animal = animalRepository.findById(animalId)
                      .orElseThrow(() -> new DeviceNotValidException("...");

但由于您使用的是Spring Data，因此也可以使用getReferenceById：

Animal animal = animalRepository.getReferenceById(animalId);

它通常会抛出一个EntityNotFoundException。