一次线上错误引发的事故，主角： java.util.NoSuchElementException: No value present

待我称王封你为后i 2023-10-11 01:28 40阅读 0赞

### 背景 ###

> 今天无意间使用一个没什么太大用的功能点时，突然我的小美不停的给我发消息...

![aad965d68d359872c658a2b92afce661.png][]

别误会，小美不是“人”，是机器人，诺~就是下面这货！

![watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L3FxXzI0MzczNTMz_size_16_color_FFFFFF_t_70][]

根据小美的提示找到代码位置，一看，我傻了，这么取值，不是写BUG吗？

![6f46b544dd7da211387b66b0b1db0bb9.gif][]

![20201214150655838.png][]

VAllCustomer customer = vAllCustomerRepository.findById(consume.getCustomerId()).get();

我们可以看到IDEA都有提示：**SonarLint: Call "Optional\#isPresent()" before accessing the value.**

说人话就是：就是用get的**调用主体时查询不到、为空**，就会报No value present 异常问题，需要我们在get()前先调用isPresent()进行**非空判断**。

![021624ccad52020c4d15c5249fa51779.png][]

> 一说到非空...不得不想到臭名昭著的NPE异常（NullPointerException），空指针。这货也是我们java程序员职业生涯中遇到最多的一个异常了。简直深恶痛绝..

在我们平常使用中，我相信大多数的人非空判断无非如下：

ConsumeIntegral consumeIntegral = consumeIntegralRepository.findByConsumeId(consume.getId());
    if (consumeIntegral != null) {
           if(consumeIntegral.getIntegral()!=null){
                vo.getData().setCurrentScore(consumeIntegral.getIntegral());
            }
           if(consumeIntegral.getTotalScore()!=null){
                vo.getData().setTotalScore(consumeIntegral.getTotalScore());
            }
          ......//这要是有三五十个参数不得要命？
    }

言归正传，找一找No value present错误的来源。

首先我们先看源码，可以看到返回的是Optional，这个错误就是由Optional统一抛出来的。

VAllCustomer customer = vAllCustomerRepository.findById(consume.getCustomerId()).get();
    
    Optional<T> findById(ID var1);

#### 看源码 ####

/**
         * If a value is present in this {@code Optional}, returns the value,
         * otherwise throws {@code NoSuchElementException}.
         *
         * @return the non-null value held by this {@code Optional}
         * @throws NoSuchElementException if there is no value present
         *
         * @see Optional#isPresent()
         */
        public T get() {
            if (value == null) {
                throw new NoSuchElementException("No value present");
            }
            return value;
        }

可以清楚的看到源码中，如果获取的对象是null，就会统一抛出这个异常提示了。所以，在我们取值的时候一定要加上非空判断！

### 解决 ###

在get()方法的下面，官方提供了一个判断的方法，我们可以直接使用。

/**
         * Return {@code true} if there is a value present, otherwise {@code false}.
         *
         * @return {@code true} if there is a value present, otherwise {@code false}
         */
        public boolean isPresent() {
            return value != null;
        }

Optional<SystemUser> createUser = systemUserRepository.findById(consume.getGmdUser());
    if(createUser.isPresent()){
        vo.getData().setCreateUser(createUser.get().getUserName()); 
    }//感觉还是不够big

换一种方式

/**
         * If a value is present, invoke the specified consumer with the value,
         * otherwise do nothing.
         *
         * @param consumer block to be executed if a value is present
         * @throws NullPointerException if value is present and {@code consumer} is
         * null
         */
        public void ifPresent(Consumer<? super T> consumer) {
            if (value != null)
                consumer.accept(value);
        }

ifPresent()是官方为我们提供的一个方法，已经为我们判断了空值，如果为空就不会执行后面的取值！

Optional<SystemUser> createUser = systemUserRepository.findById(consume.getGmdUser());
    createUser.ifPresent(systemUser -> vo.getData().setCreateUser(systemUser.getUserName()));

![466ebb203d4836e3ab9d52dc2677ec8c.png][]

再短一点

systemUserRepository.findById(consume.getGmdUser()).ifPresent(systemUser -> vo.getData().setCreateUser(systemUser.getUserName()));

![c36c09f0ea7f44e4f4d84ea196e3b868.png][]

### 结 ###

> 这下小美再也不会打扰我写bug了，不对...是代码。

[aad965d68d359872c658a2b92afce661.png]: https://img-blog.csdnimg.cn/img_convert/aad965d68d359872c658a2b92afce661.png
[watermark_type_ZmFuZ3poZW5naGVpdGk_shadow_10_text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L3FxXzI0MzczNTMz_size_16_color_FFFFFF_t_70]: https://img-blog.csdnimg.cn/20201214150454543.png?x-oss-process=image/watermark,type_ZmFuZ3poZW5naGVpdGk,shadow_10,text_aHR0cHM6Ly9ibG9nLmNzZG4ubmV0L3FxXzI0MzczNTMz,size_16,color_FFFFFF,t_70
[6f46b544dd7da211387b66b0b1db0bb9.gif]: https://img-blog.csdnimg.cn/img_convert/6f46b544dd7da211387b66b0b1db0bb9.gif
[20201214150655838.png]: https://img-blog.csdnimg.cn/20201214150655838.png
[021624ccad52020c4d15c5249fa51779.png]: https://img-blog.csdnimg.cn/img_convert/021624ccad52020c4d15c5249fa51779.png
[466ebb203d4836e3ab9d52dc2677ec8c.png]: https://img-blog.csdnimg.cn/img_convert/466ebb203d4836e3ab9d52dc2677ec8c.png
[c36c09f0ea7f44e4f4d84ea196e3b868.png]: https://img-blog.csdnimg.cn/img_convert/c36c09f0ea7f44e4f4d84ea196e3b868.png