获取可选基于条件Java[重复]

Question

我试图根据优先级和条件从多个可选项中获取值，比如在下面两组可选项中，可以返回的活动是步行和游泳。如果在任意一个选项中有任何活动，如果有游泳，那么游泳应该优先于步行。如果没有活动，则返回空。我设法写了它，但有太多的条件，想看看是否有一个聪明的方式来避免这么多的条件

public Optional<Activity> getActivity(){
    Optional<Activity> activityWhenSunShines= getActivityWhenSunShiningForUser(u);
    Optional<Activity> activityWhenDayIsGood= getActivityWhenDayIsGoodForUser(u);
    if(activityWhenSunShines.isPresent() && Activity.SWIMMING == activityWhenSunShines.get()){
      return activityWhenSunShines;
    }else if(activityWhenDayIsGood.isPresent() && Activity.SWIMMING == activityWhenDayIsGood.get()){
      return activityWhenDayIsGood;
    }else if(activityWhenSunShines.isPresent()){
      return activityWhenSunShines;
    }else if(activityWhenDayIsGood.isPresent()){
      return activityWhenSunShines;
    }else{
      return Optional.empty();
    }
  }

Answer 1

病人：医生，医生！我把锤子砸在脸上的时候很疼！

医生：好的。那就停止这样做。

可选的（至少在java生态系统中）基本上是不好的。对于流终端返回值来说，这没什么大不了的。

更好的替代方案是只进行数据结构设置，其中null和Optional都不相关。下一个最佳选择是使用null（如果您更喜欢编译时检查，还可以添加一些null注释）。一个非常遥远的糟糕的第三个解决方案是可怕的可选方案。

例如，在这里，为什么不进行活动呢。NOTHING，specgAWSSFU永远不返回null？

如果这样做，您的代码会是什么样子：

Activity activityWhenSunShines = getActivityWhenSunShiningForUser(u);
Activity activityWhenDayIsGood = getActivityWhenDayIsGoodForUser(u);

return
  (activityWhenSunShines == Activity.SWIMMING || activityWhenDayIsGood) ? Activity.SWIMMING :
  activityWhenSunShines != Activity.NOTHING ? activityWhenSunShines :
  activityWhenDayIsGood;

如果您在此处更喜欢null而不是什么，则它是完全相同的代码，只需将active替换为null即可。

Optional不组合（为了组合，泛型有co/contra/in/legacy variance）。Optional引入的空维度不玩这个游戏；例如，你可以编写一个包含数字列表或其中某个子类的方法。你不能编写一个（类型安全的）方法来包含字符串或可选的列表

Answer 2

这个代码

activityWhenSunShines.isPresent() && Activity.SWIMMING == activityWhenSunShines.get()

可以转换为更实用的样式，无需isPresent后跟get

activityWhenSunShines.map(a -> a == Activity.SWIMMING).orElse(false)

最后3个案例可替换为可选。或（在Java 9中添加）。

这给了你：

Optional<Activity> activityWhenSunShines = getActivityWhenSunShiningForUser(u);
Optional<Activity> activityWhenDayIsGood = getActivityWhenDayIsGoodForUser(u);

if(activityWhenSunShines.map(a -> a == Activity.SWIMMING).orElse(false)){
  return activityWhenSunShines;
} else if(activityWhenDayIsGood.map(a -> a == Activity.SWIMMING).orElse(false)){
  return activityWhenDayIsGood;
}
return activityWhenSunShines.or(() -> activityWhenDayIsGood);