使用可选对象进行验证是否会遭到反对？

Question

我有一份肮脏的工作。

阅读-

所以我们将其包装在一个可选对象中并返回它。

创建-

更新-

删除-

这里有一个ID，指定要更新或删除的对象（数据库中的数据行）。

问题是我不确定DB中是否存在具有此id的对象。

我只想在数据库中有ID为的值时执行更新或删除。

如果没有，我使用了一种引发Throw异常的方法。

但这似乎有些不对劲。

>

仅在检查是否有值后销毁查询后返回的可选项。

但除此之外，我想不出一种方法来知道DB是否真的有值。

写if==null不是更好吗？

下面是我实现的代码。

请建议一个好方法。

public Optional<CallCounselEntity> getCallCounselByUserId(UUID userId, UUID counselId) {
        return return callCounselRepository.findByUserId(userId, counselId);
    }



public CallCounselEntity updateCallCounsel(CallCounselEntity callCounsel) {
        **getCallCounselByUserId(callCounsel.getUserId(),callCounsel.getCounselId()).orElseThrow(() -> new NoSuchElementException("No Search Data"));**
        callCounselRepository.save(callCounsel));

        return callCounsel;
}

Answer 1

你所说的“可选被销毁”是什么意思？

您需要CallCounconSelentity实体=GetCallCounconSelbyUserId（…）。orElseThrow（…） 。如果代码没有抛出，那么该方法将继续，现在将为实体分配一个非可选值，您可以使用该值进行更新。如果代码抛出，那么，好吧，方法中引用实体的任何剩余代码都将永远无法访问。这就是重点。

因此，总而言之，以下是适用于您的场景的完全有效的代码：

CallCounselEntity entity = getCallCounselByUserId(...).orElseThrow(...)
/* perform your update here */

或者，您可以使用：

getCallCounselByUserId(...).ifPresent(existingCounsel -> { 
    /* perform your update here */ 
})

或

getCallCounselByUserId(...).ifPresentOrElse(existingCounsel -> { 
    /* perform your update here */ 
}, () -> throw new RuntimeException(...))

如果仍要为不存在的实体引发异常。