调用Element.getText()报空指针，用if-else规避是否为合理方案？

方案合理性判断 #

你提供的if-else方案是合理的基础解决方案，可以完全规避当前代码抛出的空指针异常：

• 原代码的问题是：无论element是否为null，都会执行element.getText()，当element为null时自然触发空指针
• 修改后的代码仅在element不为null时才调用getText()，从执行路径上完全避免了空引用调用的场景

但该方案属于最基础的判空实现，存在可优化空间：如果后续需要使用变量a，依然需要额外判空，代码分支多了之后会大幅降低可读性。

更优的处理方式 #

根据JDK版本和业务场景的不同，你可以选择以下更简洁的实现：

1. JDK8+ 使用Optional类（推荐） #

如果需要保留element为null时打印日志的逻辑，可以用ifPresentOrElse实现：

String a = null;
Element element = ...;
Optional.ofNullable(element)
        .ifPresentOrElse(
            e -> a = e.getText(),
            () -> System.out.println("...")
        );

如果不需要打印日志，只需要给a设置默认值，可以进一步简化：

// element为null时a赋值为默认字符串
String a = Optional.ofNullable(element)
                   .map(Element::getText)
                   .orElse("默认值");
// element为null时a赋值为null
String a = Optional.ofNullable(element)
                   .map(Element::getText)
                   .orElse(null);

2. 异常前置（适用空值属于非法场景的情况） #

如果业务逻辑上element不允许为null，空值属于参数错误/流程异常，建议直接抛出明确的业务异常，避免后续隐式问题：

Element element = ...;
if (element == null) {
    throw new IllegalArgumentException("获取Element实例为空，流程不允许继续");
}
String a = element.getText();

3. 第三方工具类（适用已有依赖的项目） #

如果项目已经引入Guava、Apache Commons等工具库，可以直接用封装好的判空工具简化代码，和Optional效果类似，适配低版本JDK。

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