エラーno match for 'operator='の対処法がわからない。

Question

###実現したいこと
平均値に一番近い値を表示するプログラムを作りたい。
###エラー内容

 error: no match for 'operator=' (operand types are 'std::vector<int>' and 'int')
   41 |     mini=again;
      |          ^~~~~

###試したこと

if(sa==min){ //saが最小値と一致するデータをコンテナに保存する
    mini=values;
    }

これで実行してみると、実行結果は

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

になりました。結果が正常に表示されませんでした。
ちなみに、範囲for文を学習したばかりなので使い方がまだよくわかっていません。。。
###コード

//平均値に一番近い値
#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=again;
    }
    }
    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";
   std::vector<int> nearavg=closet_to_average(pts);
   for(int x : nearavg){ //表示
       std::cout<<x<<" ";
   }

    return 0;
}

Answer 1

試したことは単にエラーにならないためにしてみたって感じですよね…

 error: no match for 'operator=' (operand types are 'std::vector<int>' and 'int')
   41 |     mini=again;
      |          ^~~~~

の意味はminiはstd::vector<int>型でagainはint型なので代入できないってことです。

std::vector<int>型にint型を入れたい場合は
std::vector
を見てください

---

さくっと見た限り出ない理由は

    //最小値を求める
    min=values[0];
    for(int m : values){
        if(min>m){
            min=m;
        }
    }

minはvaluesの最小値

    sa=std::fabs(avg-(double)again);

saはvaluesの各値に関して平均値から引いた絶対値なので

    if(sa==min){ //saが最小値と一致するデータをコンテナに保存する

で比較するのはおかしいのでは？

Comments

[langhtorn]

vectorをもう一度復習したらできました。ありがとうございました。

Answer 2

 error: no match for 'operator=' (operand types are 'std::vector<int>' and 'int')
   41 |     mini=again;

std::vector<int>にはintを代入するoperator=は存在しないためエラーとなっています。
mini.push_back(again);
最も近い値と言いつつ、vectorにしているのは平均値との差の絶対値が同じ値が複数ある場合はすべて返したいということでよいでしょうか？

---

sa = std::fabs(avg - (double)v); //絶対値 //valuesに保存される？

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 = again;
        }
    }

平均値とvaluesの各要素との差と、valuesの最小値を比較しているので処理が間違っています。

---

動くように直すならこうですかね。

std::vector<int> closet_to_average(std::vector<int>& values)
{
    double avg = average(values); //平均値
    double sa = 0.0; //差
    double min = std::numeric_limits<double>::max();
    for (int v : values) { //平均値との差の絶対値
        sa = std::fabs(avg - v); //絶対値
        if (min > sa) {
            min = sa;
        }
    }
    std::vector<int> mini; //空のコンテナを作成
    for (int again : values) { //もう一度平均値との差の絶対値を求める
        sa = std::fabs(avg - again);
        if (sa == min) { //saが平均値との差の絶対値の最小値と一致するデータをコンテナに保存する
            mini.push_back(again);
        }
    }
    return mini; //コンテナを返す
}

---

その他

• データ数0の場合が考慮されていませんね。
• 合計値はstd::accumulateで求められます。範囲for文の練習として書いたのかもしれませんが…

Comments

[langhtorn]

参考にさせていただきました。std::accumulate使ってみます！ありがとうございました。