我试图计算一个arraylist的正平均值。它从一个arraylist获取值,但在测试时只计算“正”数字
Integer[] array = new Integer[]{3, 2, -4, -7};
ArrayList<Integer> arrayList = new ArrayList<Integer>(Arrays.asList(array));
System.out.printf(averagePositive(arrayList));
我尝试过使用abs但它没有用。
public static double averagePositive(ArrayList<Integer> values) {
if (values == null || values.isEmpty())
return 0.0;
int sum = 0;
int n = values.size();
for (int i = 0; i < n ; i++)
if (values.get(i) > 0.0) {
sum += values.get(i);
}
return ((double) sum) / n;
}
我想要的输出是2.50但是得到1.25
你的n
等于数组的大小。相反,它应该是总正数的计数
public static void main(String[] args) {
Integer[] array = new Integer[] { 3, 2, -4, -7 };
ArrayList arrayList = new ArrayList(Arrays.asList(array));
System.out.println(averagePositive(arrayList));
}
public static double averagePositive(ArrayList<Integer> values) {
if (values == null || values.isEmpty())
return 0.0;
int sum = 0;
int n = 0;
for (int i = 0; i < values.size(); i++)
if (values.get(i) > 0.0) {
sum += values.get(i);
n++;
}
return ((double) sum) / n;
}
你应该计算正数并除以那个数,而不是n
:
int count = 0;
for (int i = 0; i < n ; i++) {
if (values.get(i) > 0.0) {
sum += values.get(i);
count++;
}
}
if (count > 0) {
return ((double) sum) / count;
} else {
return 0;
}
你的代码几乎是正确的。您唯一缺少的部分是您想要采用平均积极因素,因此在您的情况下,元素总量应为2。你应该只计算积极因素,如下:
int n = 0;
for (int i = 0; i < n ; i++)
if (values.get(i) > 0.0) {
sum += values.get(i);
n++;
}
这应该可以解决问题。
您需要将总和除以正元素的数量,而不是总大小。
public static double averagePositive(ArrayList<Integer> values) {
if (values == null || values.isEmpty())
return 0.0;
int sum = 0;
int positive_n = 0;
for (int i = 0; i < n ; i++)
if (values.get(i) > 0.0) {
sum += values.get(i);
positive_n++;
}
return ((double) sum) / positive_n;
}
你应该计算你有多少正值除以列表大小:
int positiveCount = 0;
for (int i = 0; i < n ; i++)
if (values.get(i) > 0.0) {
sum += values.get(i);
positiveCount++;
}
return ((double) sum) / positiveCount;
或者您可以使用Streams:
public static double averagePositive(List<Integer> values) {
return values.stream()
.filter(d -> d > 0.0)
.mapToDouble(Function.identity())
.average().orElse(0.0);
}