C ++查找数组中数字的平均值

Question

for( k = 0; k < n; k++ )
{
   total += total + temps[k];
}

avgTemp = total / n;

temps是包含n元素的数组。 avgTemp存储temps中所有值的平均值。 k只是一些整数使我的循环工作。 k，n和total已经在上面适当地声明了。 total跟踪数组中元素的总数。

我的运动能告诉我这是错的。我做错了什么？

Answer 1

此

for( k = 0; k < n; k++ )
{
    /// here's the error.
    /// You assign the new value to total as (total = total + total + temps[k])
    total += total + temps[k];
}

avgTemp = total / n;

应该是

for( k = 0; k < n; k++ ) { total += temps[k]; }

avgTemp = total / n;

或

for( k = 0; k < n; k++ ) { total = total + temps[k]; }

avgTemp = total / n;

使用迭代求和会更好。它允许避免舍入错误。

avgTemp = temps[0];

for(k = 1 ; k < n ; k++) { total = (temps[k] + (double)(k-1) * total)/ (double)k; }

bames53还在评论中提供了一个很好的基于STL的代码。

Answer 2

首先，total += temps[k]

+ =意味着total = total + temps[k]已经

顺便说一下， 总数是浮动还是双倍？否则你正在进行整数除法。

Answer 3

您的代码在每次迭代时都会向自身添加“total”，这不是您想要的。你需要改变：

total += total + temps[k];

到

total += temps[k];

Answer 4

获取数字的总和时，您要将总数添加到自身，然后添加下一个元素。

total += total + temps[k];

应该是：

total += temps[k];

Answer 5

bames53's comment中提到的accumulate实际上是解决此问题的最佳选择。给定n，它是容器的大小，it，它是指向容器第一个元素的指针或迭代器，您可以对数组，动态分配的数组或向量执行此操作：

const auto avgTemp = accumulate(it, next(it, n), typename iterator_traits<remove_const_t<decltype(foo)>>::value_type{}) / n;

Live Example