这是一个创建于 980 天前的主题，其中的信息可能已经有所发展或是发生改变。
这个我怎么想都觉得不可能出空指针，那不这么写有啥好方法
说 java 代码中“.”的使用不要 xx.xx.xx 这种写法，但这个地方用 stream 难道不是最好的写法
说 java 代码中“.”的使用不要 xx.xx.xx 这种写法，但这个地方用 stream 难道不是最好的写法
 |
1
ikas 2022-03-19 13:18:46 +08:00  2
如果真的非要说空指针,这里问题不是 xx.xx.xx 写法
而是,你的 list 包含可能为 null 的 item,然后 getStationId 有可能返回 null,然后再次返回包含 null 的 list |
 |
2
LxExExl 2022-03-19 13:19:56 +08:00 via iPhone
mylist?.getFirst()?.getAttribute()?.toString()
有语法糖的语言可以这样 java 楼主的截图确实会有空指针问题吧 那么怎么解决。一路嵌套 if 还是 catch 住异常？ |
 |
3
9c04C5dO01Sw5DNL 2022-03-19 13:27:10 +08:00 2
stationId 如果事实不可能为空的话，你让他给你造一个 null 出来试试。
可能为空的话就 filter 一下完事。 另外,判断 list 是否 empty 有点多余了(前提 list 不为 null ） |
 |
4
cxsz OP @ikas #1 这个确实可能会存在，返回的 list 在操作前也有判空处理，但就单独这行来说，我觉得用 stream 语法糖没啥问题，用循环加判断，一样会存在同样的问题
|
 |
5
kisick 2022-03-19 13:38:51 +08:00 2
userStationList.add(null);
这样是有可能空指针的，加个.filter(Objects::nonNull)就好了 |
|
6
cxsz OP |
 |
7
Hurriance 2022-03-19 13:43:22 +08:00
应该是 stream().map(null) 这个会 NPE 吧，最近有遇到过
|
 |
8
maojun 2022-03-19 13:48:49 +08:00 via iPhone
你列表里的元素如果有 null 的话，调用 getStationId 不就空指针了
|
 |
9
TWorldIsNButThis 2022-03-19 14:09:27 +08:00 via iPhone
可以包一层 optional
|
 |
10
wolfie 2022-03-19 14:09:49 +08:00
userStationList 为什么会包含空？
|
 |
12
Suddoo 2022-03-19 15:20:28 +08:00
集合里的对象全为 null ，遍历集合里每个对象取其中的属性，空指针出现了
|
 |
13
q1angch0u 2022-03-19 15:25:23 +08:00 via iPhone
optional 啊…
|
 |
14
cryboy007 2022-03-19 16:02:03 +08:00
请先 filter 过滤在 map
|
 |
15
night98 2022-03-19 17:48:48 +08:00
楼主写的没问题，这个 list 一看就知道是从数据库拿的，也判空 list 了，如果 list 里面还有元素能为空的话，先把数据提供方拉出去打一顿吧。
|
|
16
wolfie 2022-03-19 17:49:06 +08:00
|
 |
17
daimubai 2022-03-19 23:59:33 +08:00
@ikas 如果有包含为 null 的 item, 在 getStationId 时就已经抛出 NPE 了。 你说的应该是如果有 item 的 stationId 是 null ，才会返回包含 null 的 list
|
|
19
ikas 2022-03-20 00:21:26 +08:00 2
@cxsz 对的..主要原因是..List 这种结构我们无法判定是否包含 null,本身使用 stream 这种用法没有任何问题,如果别人拿这个说事情其实就是老顽固而已..
如同其他的回答一样,使用保守写法,加上 filter,然后 getid 里也根据业务加上是否需要剔除 null 其实这里问题就好多了... 返回 list 是否可能 null 如果 getid 如果为 null,是要抛出异常,还是要剔除,这些都是要根据业务的 |
 |
20
ubuntuGary 2022-03-20 22:39:53 +08:00 1
如果被怼是理由是你用 Stream 流式的写法，就很没有道理了，list 里有 null 元素，你换成外部迭代写法也一样有可能 NPE ，而如果被怼的原因就是你没考虑 null 元素，那你只能认，不能说 Stream 流式写法不好，Stream 流加个 filter 也就解决了，不过相比你这个问题，上游的 list 里有 null 元素更加不可原谅。
|
 |
21
chainsR 2022-03-21 09:19:51 +08:00 via iPhone
要过滤的，不要相信数据
|
 |
22
rehoni 2022-04-27 16:52:55 +08:00
肯定不是 stream 的问题，就算你 foreach 也会 npe 的，最好确保数据可靠性，给你找了个写法
[如果整个 list 可能为 null] 用 Optional 包装一下 [如果 list 中的某个 object 可能为 null] 用 .filter(Objects::nonNull) List<Person> personList = new ArrayList<>(); personList.add(new Person()); personList.add(null); personList.add(new Person("小明", 10)); personList.add(new Person("小红", 12)); Optional.ofNullable(personList).orElseGet(() -> { System.out.println("personList 为 null ！"); return new ArrayList<>(); }).stream().filter(Objects::nonNull).forEach(person -> { System.out.println(person.getName()); System.out.println(person.getAge()); }); |
Select Language