条件语句:死代码



我正在开发一个JavaEE应用程序,我有以下方法:

public String alterar_data_ato_med (int cod_ato, GregorianCalendar nova_data) {
    AtoMedico a=em.find(AtoMedico.class,cod_ato);
    Medico m=a.getmedico();
    Utente u=a.getutente();
    GregorianCalendar today=new GregorianCalendar();
    if(a==null){
        return "Ato Médico inexistente!";
    }else{
        if(m.getAgenda_atos().contains(nova_data)||m.getAgenda_consultas().contains(nova_data)){
            return "Médico indisponível";
        }else{
            if(u.getAgenda().contains(nova_data)||nova_data.before(today)){
                return "Data indisponível!";
            }else{
                GregorianCalendar antiga_data=a.getData_ato_med();
                a.setData_ato_med(nova_data);
                m.getAgenda_atos().remove(antiga_data);
                u.getAgenda().remove(antiga_data);
                return "Data do ato médico alterada!";
            }
        }
    }
}

第一个 if-else 语句似乎是"死代码"。请人帮我理解为什么吗?

a

第一个 if 语句中不能为空。如果是这样,您的代码将在Medico m=a.getmedico()行抛出NullPointerException

好吧,当a到达 if 语句时,它永远不会为空。因此,不需要null检查,并且 if 语句中的代码永远无法执行 - 因此它是死的

AtoMedico a=em.find(AtoMedico.class,cod_ato); // At this point a may be null...
Medico m=a.getmedico(); // If a is null here, there will be a NullPointerException
// And you will not reach this code down here...

因此,您应该做的是这样的:

AtoMedico a=em.find(AtoMedico.class,cod_ato);
if (a == null) {
    return "Ato Médico inexistente!";
}
// Now, it is safe to continue referencing a
Medico m=a.getmedico();
// and so on...

或者,如果您使用 Java 8,则可以改用 Optional 类。此示例演示如何引发异常而不是返回。

AtoMedico a = Optional.ofNullable(em.find(AtoMedico.class,cod_ato))
                      .orElseThrow(() -> new IllegalStateException("Ato Médico inexistente!"));
// Now a is safe to use

而且,如果您不想这样做,可以使用 ifPresent .处理空值的好方法!

Optional<AtoMedico> a = Optional.ofNullable(em.find(AtoMedico.class,cod_ato));
a.ifPresent(ato -> {
    // Do your stuff here...
    Medico m=ato.getmedico();
});

有关Optional的详细信息,请查看此 Oracle 教程。

相关内容

  • 没有找到相关文章

最新更新