写类ArrayCalc返回Java中的最大，最小和平均

问题：写类ArrayCalc它实现了以下的方法写类ArrayCalc返回Java中的最大，最小和平均

字符串arrayCalc（INT []数组）：返回以下列格式的字符串： “高：低：平均：”。假设任何元素的数组都不会有空元素。通过返回一个空字符串来处理一个空数组。

考虑下面的代码块：

ArrayCalc a = new ArrayCalc(); 
System.out.println(a.arrayCalc(new int[]{1, 2, 3})); //prints "high: 3, low: 1, average: 2" 
System.out.println(a.arrayCalc(new int[]{0})); //prints "high: 0, low: 0, average: 0" 
System.out.println(a.arrayCalc(null)); //prints ""

我有什么：

public class ArrayCalc { 
    String arrayCalc(int[] array) { 
     int sum = 0; 
     int average = 0; 
     int i = 0; 

     int min = Integer.MIN_VALUE; 
     int max = Integer.MAX_VALUE; 

     for (i = 0; i < array.length; i++) { 
      sum += array[i]; 

      if (array[i] < min) { 
       min = array[i]; 
      } else if (array[i] > max) { 
       max = array[i]; 
      } else if (array == null) { 
       return ""; 
      } 
     } 
     average = sum/array.length; 

     return ""; 

    } 
    public static void main(String[] args) { 
     ArrayCalc a = new ArrayCalc(); 
     System.out.println(a.arrayCalc(new int[]{1, 2, 3})); 
     System.out.println(a.arrayCalc(new int[]{0})); 
     System.out.println(a.arrayCalc(null)); 
    } 
}

什么它打印出：

Exception in thread "main" java.lang.NullPointerException 

    at ArrayCalc.arrayCalc(ArrayCalc.java:13) 
    at ArrayCalc.main(ArrayCalc.java:33) 
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
    at java.lang.reflect.Method.invoke(Method.java:497) 
    at com.intellij.rt.execution.application.AppMain.main(AppMain.java:140) 

Process finished with exit code 1

来源

2015-09-22 Connor V

在处理数组之前，您应该将空检查作为您的方法的第一个任务。 – Flown

另外，你应该指定arrayCalc是否是'public'，'private'或'protected'。我建议'公共'。在这种情况下，你不能使用'private'或'protected'。此外，还有两个错误：您应该将'min'和'max'的声明修改为：int max = Integer.MIN_VALUE和int min = Integer.MAX_VALUE。你能解释为什么吗？最后，你只返回空字符串。因此你不能期待看到输出。 – TuanDT

而且，方法始终只返回一个值。所以你应该在'arrayCalc（）'本身打印'max'，'min'和'average'。 – Satya

您不能检查NULL的长度。

在你的代码 - >

System.out.println(a.arrayCalc(null));

你传入空值的数组。这是造成异常的for循环（array.length不过NULL.length你正在检查）

for (i = 0; i < array.length; i++)

增加对验证部分的一些边界条件..其余的代码看起来文件

来源

2015-09-22 05:10:28 Vishal

我会用String.format(String, Object...) ，Math.max(int, int)和Math.min(int, int)。我还希望for-each loop可以迭代array中的值。像

String arrayCalc(int[] array) { 
    if (array == null || array.length == 0) { 
     return ""; 
    } 
    int sum = 0; 
    int high = Integer.MIN_VALUE; // <-- start low. 
    int low = Integer.MAX_VALUE; // <-- start high. 
    for (int v : array) { 
     high = Math.max(v, high); 
     low = Math.min(v, low); 
     sum += v; 
    } 
    return String.format("high: %d, low: %d, average: %d", high, low, 
      sum/array.length); 
}

我用你的main方法运行。我得到了（请求）

high: 3, low: 1, average: 2 
high: 0, low: 0, average: 0

来源

2015-09-22 05:12:10

把空和空管检查您的阵列和打印消息循环之前，如果阵列中找到空。

来源

2015-09-22 05:19:38

写类ArrayCalc返回Java中的最大，最小和平均

回答

相关问题