質問編集履歴

選択ソートはnを使わない方法にしました。ファイルの読み込みをgetline()関数を用いて作成しました。白星の数が多い順に表示させるのですが,同じ場合は黒星の数が少ない方を上にすることを追記しました。

2021/06/10 04:36

投稿

wagashi_157

スコア51

title CHANGED Viewed

File without changes

body CHANGED Viewed

@@ -2,7 +2,7 @@
 大相撲の勝敗が書かれたファイルを読み込んで計算をし,成績の良い順にソートした結果のファイルを吐き出すプログラムを作成したい。
 ### 発生している問題・エラーメッセージ
-今,大相撲の勝敗が書かれたファイルを読み込み,勝敗を計算して成績の良い順にソートした結果のファイルを吐き出すプログラムを作成しているのですが,コンパイルできるのにも関わらず,結果を出力しようとしたらエラーが出てしまいました。どこに原因があるのか分からないので,教えてください。特にファイルの読み込み方に自信がないので宜しくお願いします。
+今,大相撲の勝敗が書かれたファイルを読み込み,勝敗を計算して成績の良い順にソートした結果のファイルを吐き出す(白星の数が同じならばそのうち黒星の数が少ない方を上に持って行く)プログラムを作成しているのですが,コンパイルできるのにも関わらず,結果を出力しようとしたらエラーが出てしまいました。以前ファイルの読み込みに関するご指摘をもとに修正したのですが,それでもSegmentation faultが表示されます。どこに原因があるのか分からないので,教えてください。特にファイルの読み込み方・クラスへの格納に自信がないので宜しくお願いします。
 ```
 Segmentation fault
 ```
@@ -29,7 +29,7 @@
 ### 該当のソースコード
-```ここに言語名を入力
+```C++
 #include<iostream>
 #include<fstream>
 #include<vector>
@@ -97,11 +97,12 @@
   public:
   void add(Record r){
     /*vにrを追加*/
-    for (auto i=0; i<v.size(); ++i){
+    for (size_t i=0; i<v.size(); i++){
       v[i].set_r(r.get_r());
       v[i].set_n(r.get_n());
       v[i].set_s(r.get_s());
       v[i].set_a(r.get_a());
+      v.push_back(r);
     }
   }
   void print(){
@@ -110,14 +111,13 @@
     }
   }
   void sort(){
-    size_t n=v.size();
-    for(size_t i=0;i<v.size()-1;++i){
+    for(size_t i=0;i<v.size()-1;i++){
       size_t max{i};
-      for(size_t j=i+1;j<v.size();++j) {
+      for(size_t j=i+1;j<v.size();j++) {
       /*勝敗(白星，不戦勝，黒星，不戦敗)によってvをソートする(選択ソート) */
-        if (v[n-j-1].shiro()<v[n-max-1].shiro()) max=j;
+        if (v[j].shiro()>v[max].shiro()) max=j;
-        if (v[n-j-1].shiro()==v[n-max-1].shiro())
+        if (v[j].shiro()==v[max].shiro())
-          if (v[n-j-1].kuro()>v[n-max-1].kuro()) max=j;
+          if (v[j].kuro()<v[max].kuro()) max=j;
       }
       std::swap(v[i],v[max]);
     }
@@ -158,24 +158,24 @@
   while(fin){
     if(n%5==0){
       /*finからrankを取り出しrecordに格納*/
-      fin>>rank;
+      getline(fin,rank);
       record.set_r(rank);
     }
     else if(n%5==1){
       /*finからnameを取り出しrecordに格納*/
-      fin>>name;
+      getline(fin,name);
       record.set_n(name);
     }
     else if(n%5==2){
       /*finからscoreを取り出しrecordに格納. その際split関数を用いる*/
-      fin>>score;
+      getline(fin,score);
       split(score,' ','\t').push_back(score);
       record.set_s(split(score,' ','\t'));
     }
     else if(n%5==3){
       /*finからaiteを取り出しrecordに格納．その際split関数を用いる*/
       /*recordをtableに追加*/
-      fin>>aite;
+      getline(fin,aite);
       split(aite,' ','\t').push_back(aite);
       record.set_a(split(aite,' ','\t'));
       table.add(record);
@@ -201,7 +201,7 @@
 ```
 ### 試したこと
-クラス型のvectorの表し方に問題がないか見直しました。また,降順に並べる際にnがつくという条件だったので降順に並べられるようなfor文を作成しました。そして,ファイルの読み込みではそれぞれの行から出力されるようにvectorの値に注意しながらプログラムを作成しました。
+クラス型のvectorの表し方に問題がないか見直しました。選択ソートの際,size_t n=v.size()を利用せずに表しました。そして,ファイルの読み込みではそれぞれの行から出力されるようにvectorの値に注意しながらプログラムを作成しました。行単位でファイルを読み込む際にgetline()関数を利用してプログラムを作成しました。
 ### 補足情報（FW/ツールのバージョンなど）

選択ソートはnを使えば大丈夫そうだったので,ファイルの読み込みの心配について述べました。