2016-05-31 221 views
-4

我需要帮助我的代码...我不知道什么是错的或如何解决它。我认为它与声明我的变量和引用有关,但我不完全确定如何修复这些错误。请帮忙!在C++中计算平均值和方差

#include <iostream> 
#include <iomanip> 
#include <cmath> 
using namespace std; 

double calculate_average(int test_values[], int& size, int& sum, double& average) { 
    int i; 
    for (i = 0; i < size; i++) { 
     sum += test_values[i]; 
    }; 
    return average = sum/i; 
    }; 

double var(int test_values[], int size, double average, double& variance) { 

    for (int j = 0; j < size; j++) { 
     variance += pow((test_values[j] - average), 2); 
    }; 
    return variance; 
}; 

int main() { 

    int test_values[] = { 89, 95, 72, 83, 99, 54, 86, 75, 92, 73, 79, 75, 82, 53 }; 
    int size = sizeof(test_values); 
    int sum; 
    double average, variance; 

    int calculate_average(int test_values[], int size, int sum, double average); 
    int var(int test_values[], int size, double average, double variance); 

    cout << fixed << showpoint << setprecision(2); 

    cout << test_values << endl; 
    cout << average << endl; 
    cout << variance << endl; 

    return 0; 
} 
+1

那么,我能给出的最好建议是你应该用调试器逐行检查你的代码,并观察变量值是如何变化的。 –

+2

“我不知道什么是错的”不是一个有用的问题描述。至少,你应该能够解释为什么你认为这是错误的。 –

+0

期望输出?实际产出? – ggrr

回答

2

你的程序有很多的问题,从中我想指出一些:

  1. 从我如何理解你的代码,int size = sizeof(test_values);应该是在整数数量的test_values数组。但是,sizeof(x)以字节为单位返回x的大小。在你的情况下,size等于4 * 14 = 56,因为int是四个字节大。要得到你想要的,你需要除以int的大小:int size = sizeof(test_values)/sizeof(int);。这将size设置为14.

  2. 对函数calculate_average()var()有一些不合理的论点。唯一需要的是数据和大小。

  3. 您忘记将差异除以大小。

  4. 调用函数的语法应该如下:double average = calculate_average(test_values, size);(类似于var())。您只需将您传递的变量的名称作为参数写入,类型将自动确定。

这里与你相似的工作示例:

#include <iostream> 

    double average (int values[], int size) { 
     double sum = 0; 

     for (int i = 0; i < size; i++) { 
      sum += values[i]; 
     } 

     return sum/size; 
    } 

    double variance (int values[], int size) { 
     double avg = average(values, size); 
     double var = 0; 

     for (int i = 0; i < size; i++) { 
      var += (values[i] - avg) * (values[i] - avg); 
     } 

     return var/size; 
    } 

    int main(int argc, char **argv) { 
     int test_values[] = { 89, 95, 72, 83, 99, 54, 86, 75, 92, 73, 79, 75, 82, 53 }; 
     int size = sizeof(test_values)/sizeof(*test_values); 

     std::cout << "Average:\t" << average(test_values, size) << std::endl; 
     std::cout << "Variance:\t" << variance(test_values, size) << std::endl; 

     return 0; 
    } 

,因为它似乎没有对如何正确使用功能有点混乱,所以我强烈建议通过对教程工作主题(例如this one)。