【品悟c】疑问3

zhou_jingzhi

126页，第四章问题25。
疑问1：main函数中第三行的int i;
这一行是否是多余了？因为主函数中没有使用到。

疑问2：calculate函数要实现取一个数组所有元素的平均数
void calculate(...float * p_ave)
{
     for(i = 0; i < n; i++)
     {
            *p_ave += array[i];
     }
     p_ave /= n;
}
其中，p_ave是代表平均数，显示利用他临时作为p_sum,然后再自除n，加一个sum变量，是否更明确一些？大家看下谢谢

zhou_jingzhi

好的，谢谢，清楚了，继续拜读。
回复 2# pmerofc


   

bruceteen

zhou_jingzhi 发表于 2012-10-14 15:57
加一个sum变量，是否更明确一些？

都是很明确的，不相伯仲。
用个局部变量的理由是，运行效率高
不需要局部变量的理由是，如今的编译器都能用局部变量来优化
相关的还可以Google一下C的关键字restrict

anders0913

我喜欢解惑帖子

asdmonster

bruceteen 发表于 2012-10-14 18:27
都是很明确的，不相伯仲。
用个局部变量的理由是，运行效率高
不需要局部变量的理由是，如今的编译器都 ...

这个函数问题大了去了。调用这个函数的人需要遵守的契约是什么？
1,如果调用前*p_ave != 0 呢？
2,如果还有另外一个线程在异步修改*p_ave呢?
3，如果编译器不优化呢？每个加法都做一个  *操作？

这个函数写得不明所以，放在项目里面就是个将来产生bug的源头。

ethantsien

还有两个问题：

1 p_ave可能会溢出

2 int转float可能出现精度损失，有舍入的情况出现

代码的前提一定要写明确了

liuiang

我记得论坛里经常有人喷 i j k啊。