如果指针为空只是为了获取其字段，则创建新对象是否好？

编程问答 2022-04-21

问题描述

这样写好不好？

UserInfo myMethod(User user) {
    user = Optional.ofNullable(user).orElse(new User())?

    String name = user.getName()
    String type = user.getType();

    return new UserInfo(name,type);
}

或者更好地使用标准方式：

UserInfo myMethod(User user) {
    if(user == null) { return null }
    
    String name = user.getName()
    String type = user.getType();

    return new UserInfo(name,type);
}

也有人可以帮助提供良好的空安全最佳实践链接吗？

解决方法

第一个版本看起来很像代码味道：您返回一个 UserInfo 对象，该对象对应于一个空 User。这似乎是一种违反直觉的方法，如果字段 name 和 type 未指定，可能会导致各种验证问题。第二个版本更好，因为如果用户为空，它只返回空。另一种方法是强制调用者使用以下方法传递非空参数：

UserInfo myMethod(User user) {
    Objects.requireNonNull(user);
    ...
}
javajavanull-safetynullableoptionaloptionaloptional