質問編集履歴

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

2021/06/10 04:36

投稿

wagashi_157

スコア51

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -6,7 +6,7 @@
 ### 発生している問題・エラーメッセージ
-今,大相撲の勝敗が書かれたファイルを読み込み,勝敗を計算して成績の良い順にソートした結果のファイルを吐き出すプログラムを作成しているのですが,コンパイルできるのにも関わらず,結果を出力しようとしたらエラーが出てしまいました。どこに原因があるのか分からないので,教えてください。特にファイルの読み込み方に自信がないので宜しくお願いします。
+今,大相撲の勝敗が書かれたファイルを読み込み,勝敗を計算して成績の良い順にソートした結果のファイルを吐き出す(白星の数が同じならばそのうち黒星の数が少ない方を上に持って行く)プログラムを作成しているのですが,コンパイルできるのにも関わらず,結果を出力しようとしたらエラーが出てしまいました。以前ファイルの読み込みに関するご指摘をもとに修正したのですが,それでもSegmentation faultが表示されます。どこに原因があるのか分からないので,教えてください。特にファイルの読み込み方・クラスへの格納に自信がないので宜しくお願いします。
 ```
@@ -60,7 +60,7 @@
-```ここに言語名を入力
+```C++
 #include<iostream>
@@ -196,7 +196,7 @@
     /*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());
@@ -206,6 +206,8 @@
       v[i].set_a(r.get_a());
+      v.push_back(r);
     }
   }
@@ -222,21 +224,19 @@
   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;
       }
@@ -318,7 +318,7 @@
       /*finからrankを取り出しrecordに格納*/
-      fin>>rank;
+      getline(fin,rank);
       record.set_r(rank);
@@ -328,7 +328,7 @@
       /*finからnameを取り出しrecordに格納*/
-      fin>>name;
+      getline(fin,name);
       record.set_n(name);
@@ -338,7 +338,7 @@
       /*finからscoreを取り出しrecordに格納. その際split関数を用いる*/
-      fin>>score;
+      getline(fin,score);
       split(score,' ','\t').push_back(score);
@@ -352,7 +352,7 @@
       /*recordをtableに追加*/
-      fin>>aite;
+      getline(fin,aite);
       split(aite,' ','\t').push_back(aite);
@@ -404,7 +404,7 @@
 ### 試したこと
-クラス型のvectorの表し方に問題がないか見直しました。また,降順に並べる際にnがつくという条件だったので降順に並べられるようなfor文を作成しました。そして,ファイルの読み込みではそれぞれの行から出力されるようにvectorの値に注意しながらプログラムを作成しました。
+クラス型のvectorの表し方に問題がないか見直しました。選択ソートの際,size_t n=v.size()を利用せずに表しました。そして,ファイルの読み込みではそれぞれの行から出力されるようにvectorの値に注意しながらプログラムを作成しました。行単位でファイルを読み込む際にgetline()関数を利用してプログラムを作成しました。

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

2021/06/10 04:36

投稿

wagashi_157

スコア51

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,18 +1,12 @@
 ### 前提・実現したいこと
-ここに質問の内容を詳しく書いてください。
-（例）PHP(CakePHP)で●●なシステムを作っています。
-■■な機能を実装中に以下のエラーメッセージが発生しました。
+大相撲の勝敗が書かれたファイルを読み込んで計算をし,成績の良い順にソートした結果のファイルを吐き出すプログラムを作成したい。
 ### 発生している問題・エラーメッセージ
-今,大相撲の勝敗が書かれたファイルを読み込み,勝敗を計算して成績の良い順にソートした結果のファイルを吐き出すプログラムを作成しているのですが,選択ソートが上手くいっていないからか,コンパイルできるのにも関わらず,結果を出力しようとしたらエラーが出てしまいました。どこに原因があるのか分からないので,教えてください。また,もしも選択ソート以外におかしい部分がありましたらそちらも教えてください。宜しくお願いします。
+今,大相撲の勝敗が書かれたファイルを読み込み,勝敗を計算して成績の良い順にソートした結果のファイルを吐き出すプログラムを作成しているのですが,コンパイルできるのにも関わらず,結果を出力しようとしたらエラーが出てしまいました。どこに原因があるのか分からないので,教えてください。特にファイルの読み込み方に自信がないので宜しくお願いします。
 ```
@@ -410,7 +404,7 @@
 ### 試したこと
-クラス型のvectorの表し方に問題がないか見直しました。また,降順に並べる際にnがつくという条件だったので降順に並べられるようなfor文を作成しました。
+クラス型のvectorの表し方に問題がないか見直しました。また,降順に並べる際にnがつくという条件だったので降順に並べられるようなfor文を作成しました。そして,ファイルの読み込みではそれぞれの行から出力されるようにvectorの値に注意しながらプログラムを作成しました。