寫類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中的最大，最小和平均

回答

相關問題