-1

我需要计算读取的 15 个数字的方差和标准差。我不知道为什么,但该功能不会执行,控制台只是闪烁。有人能解释一下是什么问题吗,谢谢。

 #include <iostream>
 using namespace std;

void stats(int array[], double& var, double& sd);

int main ()
{
int array[15];
double var = 0, sd = 0;
cout << "Please enter 15 numbers to calculate var and SD\n";
for(int i = 0; i < 15; i++)
{
    cout << "Enter number: ";
    cin >> array[i];
}

stats(array,var,sd);

cout << "Numbers entered are: ";
for(int i = 0; i < 15; i++)
{
    cout << array[i] << " ";
}

cout << "Variance is: " << var << endl;
cout << "Standard deviation is: " << sd << endl;
}

函数定义

void stats(int array[],double& var, double& sd)
{
int sum = 0;
double sum2 = 0;
double mean = 0;

for(int i = 0; i < 15; i++)
{sum = sum + array[i];
}

mean = sum/15;

for(int i = 0; i < 15;)
{sum2 = pow((array[i]-mean),2) + sum2;
}

var = sum2/15;
sd = pow(var,0.5);}
4

2 回答 2

6
for(int i = 0; i < 15;)
{sum2 = pow((array[i]-mean),2) + sum2;
}

该循环永远不会增加i,因此它会永远运行。你错过了i++那里,所以它应该开始像

for(int i = 0; i < 15; i++)
于 2013-11-05T15:52:53.590 回答
1

一些可能有助于改善您的风格的评论:

  • 如果您想摆脱硬编码的 15,请使用向量而不是数组以及push_back来自的任何内容cin
  • 即使对于 ints 它没有任何区别,也++ii++(对于迭代器它可能会在性能方面有所不同)
  • 您可以使用对向量的所有元素求和std::accumulate
  • pow将某物提高到 2 的幂有点低效,最好将数字乘以自身
  • main应该返回 0
于 2013-11-05T16:20:18.620 回答