試圖找到這個數組的最小值、最大值和平均值

Question

function minMaxAvgArray(arr){
  var min = arr[0];
  var largest = arr[0];
  var sum = arr[0];
  
  for (var i=0;i<arr.length; i++){
    if (arr[i]<min){
    min=arr[0];
    }
    if (arr[i]>largest){
      largest=arr[0];
    }
    
    sum += arr[i];
      
    }
  return [min, largest, sum/=arr.length];
}
y = minMaxAvgArray([-2,3,4,5]);
console.log(y);

我試圖找到給定數組的最小值、最大值和平均值，每次運行此代碼時，output 總是“（-2，-2 2）有人可以向我解釋嗎？

Answer 1

像這樣試試。

 function minMaxAvgArray(arr){ var min = arr[0]; var largest = arr[0]; var sum = 0; for (var i=0;i<arr.length; i++){ if (arr[i]<min){ min=arr[i]; } if (arr[i]>largest){ largest=arr[i]; } sum += arr[i]; } return [min, largest, sum/=arr.length]; } y = minMaxAvgArray([-2,3,4,5]); console.log(y);

Answer 2

我會嘗試使用分離的函數來簡化您的代碼，例如：

 const data = [-2, 3, 4, 5]; const sum = (d) => d.reduce((a, b) => a + b, 0); const avg = (d) => (sum(d) / d.length) || 0; const max = (d) => Math.max(...d.map(v => v)); const min = (d) => Math.min(...d.map(v => v)); const result = [min(data), max(data), avg(data)]; console.log(result);

Answer 3

您需要將一些索引更改為1而不是零，並從索引一開始循環，因為您從索引零獲取值作為起始值。

 function minMaxAvgArray(arr) { var min = arr[0]; var largest = arr[0]; var sum = arr[0]; for (var i = 1; i < arr.length; i++) { // iterate from 1 bc 0 is start value if (arr[i] < min) min = arr[i]; // assign index i if (arr[i] > largest) largest = arr[i]; // assign index i sum += arr[i]; } return [min, largest, sum /= arr.length]; } const y = minMaxAvgArray([-2, 3, 4, 5]); // declare variable console.log(y);