平均値が0になってしまう

Question

###実現したいこと
整数データを入力し、その平均値に一番近いデータをすべて出力したい。
###問題点
デバック用に求めた平均値がすべて0になってしまう。

データ数を入力してください
3
数字を入力してください
2 15 1
平均値0
平均値に最も近い値は
平均値0

#試したこと
平均値を計算する関数のところでdoubleやintに変えたりしたがどれも当てはまらなかった。
###コード

//平均値に一番近い値
#include<iostream>
#include<vector>
#include<cmath>

//平均値を計算する関数
double average(std::vector<int>& values)
{
    double sum=0.0;
    double avg=0.0;
    for(int x : values){
        sum+=x; //合計
    }
    avg=sum/values.size(); //平均
    return avg;
}

//平均値に一番近いデータをすべて求める関数
std::vector<int> closet_to_average(std::vector<int>& values)
{
    double avg=average(values); //平均値
    std::cout<<"平均値"<<avg; //デバック用
    double sa=0.0; //差
    int min=0;
    for(int v : values){ //平均値との差の絶対値
    sa=std::fabs(avg-(double)v); //絶対値 //valuesに保存される？
    }
    //最小値を求める
    min=values[0];
    for(int m : values){
        if(min>m){
            min=m;
        }
    }
    std::vector<int> mini={}; //空のコンテナを作成
    for(int again : values){ //もう一度平均値との差の絶対値を求める
    sa=std::fabs(avg-(double)again);
    if(sa==min){ //saが最小値と一致するデータをコンテナに保存する
    mini=values;
    }
    }
    return mini; //コンテナを返す
}
int main()
{
    int n; //データ数
    std::cout<<"データ数を入力してください\n";
    std::cin>>n;

    std::vector<int> pts={}; //配列
    pts.resize(n);
    std::cout<<"数字を入力してください\n";
    for(int num : pts){
        std::cin>>num;
    }

    //std::cout<<"平均値"<<average(pts)<<"\n"; //デバック用

    std::cout<<"平均値に最も近い値は\n";
    closet_to_average(pts);

    return 0;
}

Answer 1

    std::cout<<"数字を入力してください\n";
    for(int num : pts){
        std::cin>>num;
    }

ここのfor文のnumの宣言が値渡しとなっていて、都度numの中身だけを書き換えていて、ptsの中身は全く変更が行われていません。そのためptsの中身は初期化された0の配列となります。
以下のように参照渡しに変更してあげれば動くかと思います。

    std::cout<<"数字を入力してください\n";
    for(int &num : pts){
        std::cin>>num;
    }

Comments

[langhtorn]

ご指摘ありがとうございます。勉強になりました。