C++编译报错：变量av、s未在当前作用域内声明问题排查

编译C++代码时抛出3处作用域相关错误：

• 第20行、29行错误提示：'av' was not declared in this scope（'av'未在当前作用域内声明）
• 第30行错误提示：'s' was not declared in this scope（'s'未在当前作用域内声明）

原始出错代码如下：

#include <iostream>
#include <cmath>
using namespace std;
       
const int SIZE= 6;
int nums[SIZE]= {1,5,6,43,7,9};
int average(double av){
    //av is to get the average of the array
    int sum=0;
    for (int i=0; i<SIZE; i++){
        sum+= nums[i];
        av= static_cast<double>(sum/SIZE);
    }
    return av;
}
        
        
int deviation( double s ){
        
    int function = nums[0]-average(av);
    int nominator = pow(function, 2);
    s = sqrt(nominator/SIZE);
    return s;
}
        
        
int main(){
    cout << "Numbers: "<< nums[SIZE]<< endl;
    cout << "Average: "<< average(av)<<endl;
    cout << " Standard Deviation: "<< deviation(s)<<endl;
        
}

1. 违反C++变量作用域规则
   函数的形参属于函数内部的局部变量，仅在定义它的函数体内可访问。av是average函数的形参，仅在average的函数块内有效；s是deviation函数的形参，仅在deviation的函数块内有效。在deviation函数里调用average(av)、在main函数里调用average(av)和deviation(s)时，当前作用域根本没有声明过名为av、s的变量，编译器无法识别，就会抛出未声明的错误。
2. 函数设计逻辑倒置
   average、deviation是用来计算返回结果的函数，不需要调用方传入存储结果的参数。现有写法要求调用时传入一个根本不存在的变量，完全不符合参数使用逻辑。
3. 代码还存在4个隐藏逻辑bug，就算解决了作用域错误也无法得到正确结果：
   • 数组越界访问：C++数组下标从0开始，长度为6的nums数组合法下标范围是0~5，nums[SIZE]访问的是数组外的内存，会输出随机垃圾值。
   • 整数除法丢精度：sum和SIZE都是int类型，sum/SIZE会直接截断小数部分得到整数结果，再强转double也无法恢复丢失的小数位。
   • 返回值类型不匹配：平均值、标准差都是浮点数值，函数返回值定义为int会直接截断小数部分，计算结果不准确。
   • 标准差计算逻辑错误：现有代码只拿数组第一个元素和平均值的差算方差，完全不符合标准差的计算规则。

• 删除两个计算函数多余的形参，计算逻辑全部放在函数内部完成，最终直接返回计算结果，调用时不需要传入不存在的变量。
• 调整类型转换顺序，计算平均值时先把sum转为double类型再做除法，避免整数除法精度丢失。
• 把两个计算函数的返回值类型改为double，匹配浮点计算结果。
• 修正数组输出逻辑，遍历输出所有数组元素，避免越界访问。
• 补全标准差的正确计算逻辑：先算所有元素和平均值的差的平方和，再除以元素个数开平方。

修复后的可运行代码：

#include <iostream>
#include <cmath>
using namespace std;
       
const int SIZE = 6;
int nums[SIZE] = {1,5,6,43,7,9};

double average(){
    int sum = 0;
    for (int i = 0; i < SIZE; i++){
        sum += nums[i];
    }
    // 先转double再做除法，保留小数
    return static_cast<double>(sum) / SIZE;
}
        
double deviation(){
    double av = average();
    double sum_square_diff = 0;
    for (int i = 0; i < SIZE; i++){
        double diff = nums[i] - av;
        sum_square_diff += diff * diff;
    }
    return sqrt(sum_square_diff / SIZE);
}
        
int main(){
    cout << "Numbers: ";
    for (int i = 0; i < SIZE; i++){
        cout << nums[i] << " ";
    }
    cout << endl;
    cout << "Average: " << average() << endl;
    cout << "Standard Deviation: " << deviation() << endl;
    return 0;
}

内容的提问来源于stack exchange，提问作者Mohammed Al-Khater