2014-02-17 38 views
-2

我想用Java编写一个程序,它接受三个数组并返回删除了最小值的数组。我想我正在创建新的阵列是错误的。虽然它看起来编译罚款,我每次运行它的时候,我得到以下信息:线程“main” java.lang.ArrayIndexOutOfBoundsException从Java中的数组中删除最低值

例外:5

任何帮助将不胜感激!

这里是我的代码:

import java.util.Arrays; 

class LowestGrade 
{ 
    public static void main (String [] args) 
    { 
     int [] a = removeLowest (23, 90, 47, 55, 88); 
     int [] b = removeLowest (85, 93, 42); 
     int [] c = removeLowest (59, 92, 93, 47, 88, 47); 

     System.out.println ("a = " + Arrays.toString(a)); 
     System.out.println ("b = " + Arrays.toString(b)); 
     System.out.println ("c = " + Arrays.toString(c)); 
    } 

    public static int[] removeLowest (int...grades) 
    { 
     if (grades.length <= 1) 
     { 
     return grades; 
     } 

     else 
     { 
      int [] newArray = new int [grades.length - 1]; 
      int lowest = grades [0]; 

      for (int i = 0; i < grades.length; i++) 
      { 
      for (int n = 0; n <= grades.length; n++) 
      { 
       if (grades[n] > lowest) 
       { 
       newArray[i] = grades[n]; 
       i++; 
       } 

       else 
       { 
       lowest = grades[n]; 
       } 
      } 
      } 

      return newArray; 
     } 
    } 
} 
+8

你觉得什么错误意味着什么?它没有得到更明显的为什么发生错误比这个。 –

+0

错误在这里'如果(等级[n]>最低)'你的n大于'等级'大小。 – Brian

+0

另外你为什么使用嵌套循环?你可以在一个简单的循环中做到这一点(如果你不关心结果数组的顺序)。 –

回答

-3

友情提示:检查第二for -loop的for -loop界限。

1

你或许应该张贴整个例外:

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 5 
    at LowestGrade.removeLowest(LowestGrade.java:32) 
    at LowestGrade.main(LowestGrade.java:7) 

这意味着您正在访问具有小于6个元素的数组的第6元。在你的情况下,你阵列只有5种元素,其有效的索引是0,1,2,3和4。

解决方法是,以除去嵌套循环:

int lowest = grades [0]; 
for (int i = 1; i < grades.length; i++) 
{ 
    if(lowest > grades[i]) 
    { 
     lowest = grades[i]; 
    } 
} 
return removeElement(grades, lowest); 

.. 

public static int[] removeElement(int[] original, int element) 
{ 
    int[] n = new int[original.length - 1]; 
    System.arraycopy(original, 0, n, 0, element); 
    System.arraycopy(original, element+1, n, element, original.length - element-1); 
    return n; 
} 

removeElementthis answer

-1

以下行是一般的编程错误:

  1. newArray[i] = grades[n];

作为newArray尺寸已经减少了一个,它可以保持值i = grades.length时。

  1. for (int n = 0; n <= grades.length; n++)

当n = grades.length,等级[n]的抛出ArrayIndexOutOfBoundsException异常异常。所以,应该是for (int n = 0; n < grades.length; n++)

在你的方式,你可以简化如下重新编写方法:

public static int[] removeLowest(int... grades) { 

     if (grades.length <= 1) { 
      return grades; 
     } 

     else { 

      // find lowest first 

      int lowest = grades[ 0 ]; 

      for (int i = 1; i < grades.length; i++) { 

       if (grades[ i ] < lowest) { 
        lowest = grades[ i ]; 
       } 
      } 

      // build the new array 
      int[] newArray = new int[ grades.length - 1 ]; 

      for (int i = 0, j = 0; i < grades.length; i++) { 

       if (grades[ i ] == lowest) { 
        continue; 
       } 

       newArray[ j++ ] = grades[ i ]; 
      } 

      return newArray; 
     } 
    }