从Java中的数组中删除最低值
问题描述:
我想用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;
}
}
}
答
友情提示:检查第二for
-loop的for
-loop界限。
答
你或许应该张贴整个例外:
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;
}
removeElement
从this answer。
答
以下行是一般的编程错误:
newArray[i] = grades[n];
作为newArray尺寸已经减少了一个,它可以保持值i = grades.length
时。
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;
}
}
你觉得什么错误意味着什么?它没有得到更明显的为什么发生错误比这个。 –
错误在这里'如果(等级[n]>最低)'你的n大于'等级'大小。 – Brian
另外你为什么使用嵌套循环?你可以在一个简单的循环中做到这一点(如果你不关心结果数组的顺序)。 –