标签:调用 get NPE list simple null Optional strings
使用Optional解决长调用链NPE问题
Java NPE即Null Pointer Exception,如果一次的调用链很长,一旦中间出现了null就很难判断是哪一层为null了。(羡慕Kotlin有?.
这种调用)
举个例子,现在有一个简单的类:
public class Simple {
List<String[]> list;
public List<String[]> getList() {
return list;
}
public void setList(List<String[]> list) {
this.list = list;
}
}
我们的需求是:判断一个Simple对象的list里面的第一个String数组的第一个元素是否以“10”开头,你可能会这么写:
public boolean checkSimple(Simple simple) {
return simple.getList().get(0)[0].startsWith("10");
}
如果你直接这么写,那么恭喜你,不仅可能会出现空指针异常,还有可能出现数组越界异常,所以我们在调用的时候需要进行null判定:
public boolean checkSimple(Simple simple) {
if (simple != null) {
List<String[]> list = simple.getList();
if (list != null && list.size() > 0) {
String[] strings = list.get(0);
if (strings != null && strings.length > 0) {
String string = strings[0];
if (string != null) {
return string.startsWith("10");
}
}
}
}
return false;
}
我们需要去判断中间出现的每一个对象是否为null,然后才能进行下一步,非常的繁琐。
但是如果使用Optional类则比较简洁了:
public boolean checkSimple(Simple simple) {
return Optional.ofNullable(simple)
.map(simple1 -> simple.getList())
.map(list -> list.size() > 0 ? list.get(0) : null)
.map(strings -> strings.length > 0 ? strings[0] : null)
.map(s -> s.startsWith("10"))
.orElse(false);
}
不过有个缺点是写起来爽了,但是可读性还真没有if判断好。如果不熟悉这种写法的话,可能第一眼还真看不出来意图是什么。
另外,使用Optional的时候尽量不要这么用:
if (optional.isPresent()) {
Object obj = optional.get();
//do something
}
一是因为这么用和直接判空没什么区别,二是在调用get()
的时候,如果其中元素为null,会直接抛出NoSuchElementException,所以尽量不要使用get()
方法。
正确的应该是这么用:
optional.ifPresent(obj -> {
//do something
});
用上Optional之后代码的简洁性会有提高,也更容易写出清爽干净的代码。
标签:调用,get,NPE,list,simple,null,Optional,strings 来源: https://www.cnblogs.com/lixin-link/p/12805089.html
本站声明: 1. iCode9 技术分享网(下文简称本站)提供的所有内容,仅供技术学习、探讨和分享; 2. 关于本站的所有留言、评论、转载及引用,纯属内容发起人的个人观点,与本站观点和立场无关; 3. 关于本站的所有言论和文字,纯属内容发起人的个人观点,与本站观点和立场无关; 4. 本站文章均是网友提供,不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属;如您发现该文章侵犯了您的权益,可联系我们第一时间进行删除; 5. 本站为非盈利性的个人网站,所有内容不会用来进行牟利,也不会利用任何形式的广告来间接获益,纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。