0

我的家庭作业有问题。我们的作业由在线网站评分,我不断收到关于我在下面显示的代码的以下错误。

错误:

例外情况。为 null a.java.lang.NullPointerException 引发了不正确的异常
  public static int[] nearestK(int[] a, int val, int k) {

  int x = 0;

  if (k < x || a.length == 0 || a == null)
  {
     throw new IllegalArgumentException("k is not invalid");
  }

  if (k == 0 || k > a.length) 
  {
     int[] incorrect = new int[0];
     return incorrect ;
  }

  final int value = val; 
  Integer[] copy = new Integer[a.length]; 
  for (int i = 0; i < a.length; i++) {
     copy[i] = a[i];
  }

  Arrays.sort(copy, 
        new Comparator<Integer>() {                   
           @Override
           public int compare(Integer o1, Integer o2) {
              int distance1 = Math.abs(value - o1); 
              int distance2 = Math.abs(value - o2); 
              return Integer.compare(distance1, distance2);
           }
        });

  int[] answer = new int[k]; 
  for (int i = 0; i < answer.length; i++) {
     answer[i] = copy[i];
  }

  return answer;

}
4

2 回答 2

2

这条线失败了:

if (k < x || a.length == 0 || a == null)

因为a.length在它有机会检查之前会抛出一个 NullPointerException a == null

尝试将其更改为:

if (a == null || k < x || a.length == 0)

所以首先检查null。

于 2013-09-04T01:08:25.623 回答
0
 if (k < x || a.length == 0 || a == null)

这需要反过来,因为它是从左到右评估的,并且空检查需要保护长度检查。

实际上,a.length会产生空指针异常。

  if (k < x || a == null || a.length == 0)

使用这个序列,评估不会一直向右(这就是短路运算符||所做的)。

于 2013-09-04T01:07:14.490 回答