回答編集履歴

10

先程の編集でコードが動かなくなったので編集

2020/07/16 02:54

投稿

退会済みユーザー
test CHANGED
@@ -108,7 +108,7 @@
108
108
 
109
109
  //
110
110
 
111
- // std::ifstreamからXY情報取得
111
+ // std::ifstreamからxy型への変換
112
112
 
113
113
  //
114
114
 
@@ -116,6 +116,8 @@
116
116
 
117
117
  {
118
118
 
119
+
120
+
119
121
  std::string one_line;
120
122
 
121
123
  std::vector<std::string> text;
@@ -160,7 +162,7 @@
160
162
 
161
163
  //
162
164
 
163
- // XY型のオブジェクトからstd::stringへの変換
165
+ // XY型からstd::stringへの変換
164
166
 
165
167
  //
166
168
 
@@ -202,7 +204,7 @@
202
204
 
203
205
  std::ofstream ofs(output_path);//出力ストリーム
204
206
 
205
- auto xy = ifstream2xy(ifs, start_line);//ifsからXY情報取得
207
+ auto xy = ifstream2xy(ifs, start_line);//ifsからxy型への変換
206
208
 
207
209
 
208
210
 
@@ -226,7 +228,7 @@
226
228
 
227
229
  auto str = xy2str(xy, start_line);//xyからstd::stringへ変換
228
230
 
229
- //std::cout << str << std::endl;//strを標準出力
231
+ std::cout << str << std::endl;//strを標準出力
230
232
 
231
233
 
232
234
 

9

単純化を狙いsscanfのエラーチェック除外

2020/07/16 02:53

投稿

退会済みユーザー
test CHANGED
@@ -140,11 +140,9 @@
140
140
 
141
141
  if(ct>=start_line){
142
142
 
143
- if(sscanf(line.c_str(), "%d,%d", &x, &y)!=EOF){
143
+ sscanf(line.c_str(), "%d,%d", &x, &y);
144
-
144
+
145
- result.push_back({x,y});
145
+ result.push_back({x,y});
146
-
147
- }
148
146
 
149
147
  }
150
148
 

8

不備を修正

2020/07/16 02:52

投稿

退会済みユーザー
test CHANGED
@@ -52,7 +52,7 @@
52
52
 
53
53
  したい...
54
54
 
55
- というのであればsscanfを使っているの
55
+ というのであればsscanfを使っていたりエラー処理省いていたり
56
56
 
57
57
  あまりいいコードとは言えませんが
58
58
 

7

sscanfが危険な関数であることを記載

2020/07/16 02:48

投稿

退会済みユーザー
test CHANGED
@@ -52,7 +52,9 @@
52
52
 
53
53
  したい...
54
54
 
55
- というのであればエラー処理など省いていますが
55
+ というのであればsscanfを使っているので
56
+
57
+ あまりいいコードとは言えませんが
56
58
 
57
59
  下記のコードを参考にしていただいたらどうでしょうか。
58
60
 

6

不要な部分削除

2020/07/16 01:41

投稿

退会済みユーザー
test CHANGED
@@ -142,10 +142,6 @@
142
142
 
143
143
  result.push_back({x,y});
144
144
 
145
- }else{
146
-
147
- //エラー処理
148
-
149
145
  }
150
146
 
151
147
  }

5

誤解を招きそうな部分修正

2020/07/16 00:54

投稿

退会済みユーザー
test CHANGED
@@ -164,7 +164,7 @@
164
164
 
165
165
  //
166
166
 
167
- // XY型からstd::stringへの変換
167
+ // XY型のオブジェクトからstd::stringへの変換
168
168
 
169
169
  //
170
170
 

4

無駄な部分修正

2020/07/16 00:35

投稿

退会済みユーザー
test CHANGED
@@ -114,13 +114,9 @@
114
114
 
115
115
  {
116
116
 
117
- using Text = std::vector<std::string>;
118
-
119
-
120
-
121
117
  std::string one_line;
122
118
 
123
- Text text;
119
+ std::vector<std::string> text;
124
120
 
125
121
 
126
122
 

3

誤記修正

2020/07/16 00:34

投稿

退会済みユーザー
test CHANGED
@@ -222,7 +222,7 @@
222
222
 
223
223
  //xy[2][1] = 12345;//二行目のyを12345に書き換える
224
224
 
225
- for(auto&& line:xy)for(auto&& axis:line)axis += 10;//全要素に10を足す
225
+ for(auto& line:xy)for(auto& axis:line)axis += 10;//全要素に10を足す
226
226
 
227
227
 
228
228
 

2

誤表現修正

2020/07/16 00:30

投稿

退会済みユーザー
test CHANGED
@@ -106,7 +106,7 @@
106
106
 
107
107
  //
108
108
 
109
- // std::ifstreamからXY型への取得
109
+ // std::ifstreamからXY情報取得
110
110
 
111
111
  //
112
112
 
@@ -210,7 +210,7 @@
210
210
 
211
211
  std::ofstream ofs(output_path);//出力ストリーム
212
212
 
213
- auto xy = ifstream2xy(ifs, start_line);//ifsからXYオブジェクト取得
213
+ auto xy = ifstream2xy(ifs, start_line);//ifsからXY情報取得
214
214
 
215
215
 
216
216
 

1

誤解を招きそうな部分修正

2020/07/16 00:25

投稿

退会済みユーザー
test CHANGED
@@ -106,7 +106,7 @@
106
106
 
107
107
  //
108
108
 
109
- // std::ifstreamからxy型への変換
109
+ // std::ifstreamからXY型への取得
110
110
 
111
111
  //
112
112
 
@@ -210,7 +210,7 @@
210
210
 
211
211
  std::ofstream ofs(output_path);//出力ストリーム
212
212
 
213
- auto xy = ifstream2xy(ifs, start_line);//ifsからxy型への変換
213
+ auto xy = ifstream2xy(ifs, start_line);//ifsからXYオブジェクト取得
214
214
 
215
215
 
216
216