重心座標の計算結果がおかしい。

Question

###実現したいこと
重心をクラスとコンテナfor文を使って求めたい。
###問題点
重心の値がおかしい。

点の個数を入力してください
3
3点の座標を入力してください
2 3
-1 0
2 6
入力された点の重心は
(1,5.59241e+06)

foreach文を学習したばかりなので使い方があっているか心配です。
どこが間違っているか教えてください。
###コード

//重心をクラスとコンテナfor文を使って求める
#include<iostream>
#include<vector>
//点の定義
struct Point
{
    double x; //x座標
    double y; //y座標

    void output()
    {
        std::cout<<"("<<x<<","<<y<<")"; //(x座標,y座標)
    }
};

//表示する関数
void output(std::vector<Point>& pts)
{
    for(Point p:pts){
        std::cout<<"("<<p.x<<","<<p.y<<")\n";
    }
}

//重心を求める関数
Point centroid(std::vector<Point>& points)
{
    int xsum,ysum; //合計
    Point cent; //重心
    for(Point x : points){
        xsum+=x.x;
    }
    cent.x=xsum/points.size();
    for(Point y : points){
        ysum+=y.y;
    }
    cent.y=ysum/points.size();
    return cent;
}
int main()
{
    int n;
    std::cout<<"点の個数を入力してください\n";
    std::cin>>n;

    std::vector<Point> pts={}; //配列
    pts.resize(n); //サイズ変更
    std::cout<<n<<"点の座標を入力してください\n";
    for(Point& p:pts){
        std::cin>>p.x>>p.y;
    }

    std::cout<<"入力された点の重心は\n";
    Point cent=centroid(pts); //重心を求める関数の呼び出し
    
    cent.output();

    return 0;
}

Comments

[episteme]

foreach文? C++にそんなのないよ。

[langhtorn]

ほんとですか？こちらにのっているのを引用したのですが。。。言い方が違うのかもしれません。間違っていたらすみません。
https://ja.wikipedia.org/wiki/Foreach%E6%96%87#C++

[episteme]

関数 for_each はありますが foreach 文はありません。

[langhtorn]

わかりました。ありがとうございます。

[episteme]

コンテナfor文 もありません。

Answer 1

//重心を求める関数
Point centroid(std::vector<Point>& points)
{
//  int xsum,ysum; //合計  ← なんでintなんよ。初期化も忘れてるし。
    double xsum = 0.0, ysum = 0.0; //合計
    Point cent; //重心
    ...