输入为空的corner case到底怎么写是对的

public ListNode mergeKLists(ListNode[] lists) {
     if(lists==null||lists.length==0)
         return null;

以前onsite有烙印找茬,说这句有bug啥空指针异常

那么这么写ok吗？
     if(lists==null)return null;
     if(lists.length==0)return null;
烙印又会不又找茬说代码不简洁，要1行搞定？

wilsonzyx 发表于 2019-10-18 06:54
我的话肯定会说 || && 均有短路特点
. .иlist == null 为true 后面就不会执行，直接进if了
换句话说，如果用 ...

懂了， 那这个逻辑确实有问题
但是我怎么看到一大堆的网上sample solution全部都是这么写的？

我的话肯定会说 || && 均有短路特点
list == null 为true 后面就不会执行，直接进if了
换句话说，如果用到了list.length，list 必定不为null，也就可以执行list.length 不会跑异常

（也许boolean operator的短路性 反而是考点？

我的话肯定会说 || && 均有短路特点
list == null 为true 后面就不会执行，直接进if了
换句话说，如果用到了list.length，list 必定不为null，也就可以执行list.length 不会跑异常

（也许boolean operator的短路性 反而是考点？

这两个方式是一毛一样的啊。。。

dc761 发表于 2019-10-18 04:48
这两个方式是一毛一样的啊。。。

以前被ebay的死烙印找过茬，这过几周又要ebay二进宫了。

不知道我的理解是不是有问题，面试官是否想表达存在输入中某List header为空指针的情况，而不是输入数组为空的情况...?

iejr 发表于 2019-10-18 06:07
不知道我的理解是不是有问题，面试官是否想表达存在输入中某List header为空指针的情况，而不是输入数组为 ...

不是这个，你这个corner case在后面会判断做处理，list header不为空才放到队列里

我一辈子赖美帝 发表于 2019-10-18 07:57
懂了， 那这个逻辑确实有问题
但是我怎么看到一大堆的网上sample solution全部都是这么写的？

不是，层主的解释就是说这个写法没问题呀。他说的是被面试官问到的时候的解释

一般这些我都是if(!list) return nullptr的。。。。