Java 使用优雅的方式将两个可选项组合为一个

基本上，我试图在一行中创建这段代码。检查是否存在要删除的公会，然后尝试创建新公会，如果不为空，则添加到我的当前地图

Optional.ofNullable(getGuild(event.getGuild().getIdLong())).ifPresent(Guild::deleteMe);
Optional.ofNullable(Guild.create(event.getGuild().getIdLong(), event.getGuild().getName())).ifPresent(s -> addGuild(s));```

---

您可以以处理这两个操作的方式实现第一个

ifPresent

使用者，并将第二个限制为方法引用

Optional<Guild> eventGuild = event.getGuild();

if(eventGuild.isPresent()){

    long eventGuildId = eventGuild.getLongId();
    
    Optional.ofNullable(getGuild(eventGuildId)).ifPresent(guild -> {
        guild.deleteMe();
        Guild.create(eventGuildId, eventGuild.getName()).ifPresent(this::addGuilds);
    });
}

Optional eventGuild=event.getGuild（）；
if（eventGuild.isPresent（））{
long eventGuildId=eventGuild.getLongId（）；
可选。不可用（getGuild（eventGuildId））.ifPresent（guild->{
guild.deleteMe（）；
create（eventGuildId，eventGuild.getName（））.ifPresent（this:：addGuilds）；
});
}

---

我不知道这在你看来是否更优雅，但它肯定更具可读性。

如果没有额外的处理，我们可以使用

或

功能：

Optional.ofNullable("oldGuild").or(()->Optional.ofNullable("newGuild"));

注意：为了演示，我使用的是

String

类而不是Guild。
我们可以尝试

flatMap

，只需进入一个可选选项。这是不可读的，可能看起来不优雅。但我们可以将所有内容塞进一个选项中。
把旧行业协会换成新的工会：

Optional.ofNullable("oldGuild").<String>flatMap(guild->{
        if(guild!= null) guild.length();/*delete*/
        return Optional.ofNullable("newGuild");
    }).ifPresent(System.out::println);

Optional.ofNullable（“旧帮会”）.flatMap（帮会->{
如果（guild！=null）guild.length（）；/*删除*/
返回可选。不可用（“新公会”）；
}).ifPresent（System.out:：println）；

谢谢你们两位的评论。当然，这两种方式看起来都比我的好，但还是一团糟。Java应该在可选或更多功能中添加else-if样式。