我有一个.txt文件,其中包含制表符分隔的数据(多行):
42 2 62
2 2 42
2 2 62
我尝试使用istringstream从文件中读取数据,然后将每个列的数据用于每个相应的行作为将1添加到2-D数组中的位置。
这是循环,直到2-D数组用循环(行)和文件数据(列)指定的每个位置填充1个。
注意:第二列未使用,但已放入,因为这是.txt文件的显示方式。
我逐个打印出每一行,并期望每行中有三个1,对于所有行。
然而,当我打印输出为2-D阵列(底部附近的代码块)时,我得到了意想不到的结果。对于某些行,行内只有两个1,而对于某些行则会有4个1#s。
我不是一个非常好的编码员,所以任何建议都将不胜感激,代码发布在下面。
#include "stdafx.h"
using namespace std;
//prints the diagonal of array
void printarray(int arg[][300], int length) {
for (int n = 0; n<length; ++n)
cout << arg[n][n] << ' ';
cout << '\n';
}
//makes 2d array values all 0
void zeros(int a[][300], int size)
{
cout << "Setting all elements of array to zero...." << endl;
for (int k = 0; k < (size*size); k++)
{
for (int i = 0; i<size; i++)
{
for (int j = 0; j<size; j++)
{
a[i][j] = 0;
}
}
}
}
int main()
{
//2D array of 300by300
int adMatrix [300][300];
zeros(adMatrix, 300);
int counter = 0;
string line;
int firstVar,secondVar,thirdVar; // variables to hold the bond neighbors while looping
// Read sample file
ifstream infile("myTestFile.txt");
while (counter < 300 ) {
getline(infile, line);
istringstream stream(line);
stream >> firstVar >> secondVar >> thirdVar;
adMatrix[counter][firstVar] = 1;
adMatrix[counter][thirdVar] = 1;
stream.str("");
stream.clear();
getline(infile, line);
istringstream stream2(line);
stream2 >> firstVar >> secondVar >> thirdVar;
adMatrix[counter][firstVar] = 1;
adMatrix[counter][thirdVar] = 1;
stream2.str("");
stream2.clear();
getline(infile, line);
istringstream stream3(line);
stream3 >> firstVar >> secondVar >> thirdVar;
adMatrix[counter][firstVar] = 1;
adMatrix[counter][thirdVar] = 1;
stream3.str("");
stream3.clear();
counter++;
}
// iterates through entire matrix, prints values, also tallys sum of each row, should equal 3 for each row.
int var = 0;
for (int i = 0; i < 300; ++i)
{
int sum = 0;
for (int i = 0; i < 300; ++i)
{
cout << adMatrix[var][i];
sum = adMatrix[var][i] + sum;
}
var++;
cout << endl << "sum is : " << sum << endl;
}
return 0;
}
以下是我使用的.txt文件的数据:
43 1 61
2 1 43
2 1 61
1 2 3
1 2 11
3 2 11
2 3 45
4 3 45
2 3 4
3 4 13
5 4 13
3 4 5
4 5 6
6 5 162
4 5 162
15 6 123
5 6 15
5 6 123
59 7 77
8 7 59
8 7 77
7 8 9
7 8 21
9 8 21
8 9 61
10 9 61
8 9 10
9 10 23
11 10 23
9 10 11
2 11 12
10 11 12
2 11 10
13 12 25
11 12 25
11 12 13
12 13 14
4 13 14
4 13 12
15 14 27
13 14 27
13 14 15
14 15 16
6 15 16
6 15 14
29 16 133
15 16 29
15 16 133
75 17 93
18 17 75
18 17 93
17 18 19
17 18 34
19 18 34
18 19 77
20 19 77
18 19 20
19 20 36
21 20 36
19 20 21
8 21 22
20 21 22
8 21 20
23 22 38
21 22 38
21 22 23
22 23 24
10 23 24
10 23 22
25 24 40
23 24 40
23 24 25
24 25 26
12 25 26
12 25 24
27 26 42
25 26 42
25 26 27
26 27 28
14 27 28
14 27 26
27 28 29
29 28 44
27 28 44
16 29 28
28 29 158
16 29 158
91 30 109
31 30 91
31 30 109
30 31 32
30 31 50
32 31 50
31 32 93
33 32 93
31 32 33
32 33 52
34 33 52
32 33 34
18 34 35
33 34 35
18 34 33
36 35 54
34 35 54
34 35 36
35 36 37
20 36 37
20 36 35
38 37 56
36 37 56
36 37 38
37 38 39
22 38 39
22 38 37
40 39 58
38 39 58
38 39 40
39 40 41
24 40 41
24 40 39
40 41 42
42 41 60
40 41 60
26 42 41
41 42 43
26 42 43
42 43 44
1 43 42
1 43 44
28 44 43
28 44 45
43 44 45
3 45 44
44 45 157
3 45 157
107 46 122
47 46 107
47 46 122
46 47 48
46 47 66
48 47 66
47 48 109
49 48 109
47 48 49
48 49 68
50 49 68
48 49 50
31 50 51
49 50 51
31 50 49
52 51 70
50 51 70
50 51 52
51 52 53
33 52 53
33 52 51
54 53 72
52 53 72
52 53 54
53 54 55
35 54 55
35 54 53
56 55 74
54 55 74
54 55 56
55 56 57
37 56 57
37 56 55
56 57 58
58 57 76
56 57 76
39 58 57
57 58 59
39 58 59
58 59 60
7 59 58
7 59 60
41 60 59
41 60 61
59 60 61
1 61 60
9 61 60
1 61 9
120 62 132
63 62 120
63 62 132
62 63 64
62 63 82
64 63 82
63 64 122
65 64 122
63 64 65
64 65 84
66 65 84
64 65 66
47 66 67
47 66 65
65 66 67
68 67 86
66 67 86
66 67 68
67 68 69
49 68 69
49 68 67
70 69 88
68 69 88
68 69 70
69 70 71
51 70 71
51 70 69
72 71 90
70 71 90
70 71 72
71 72 73
53 72 73
53 72 71
72 73 74
74 73 92
72 73 92
73 74 75
55 74 73
55 74 75
74 75 76
17 75 74
17 75 76
57 76 75
57 76 77
75 76 77
7 77 76
19 77 76
7 77 19
130 78 138
79 78 130
79 78 138
78 79 80
78 79 98
80 79 98
79 80 132
81 80 132
79 80 81
80 81 100
80 81 82
82 81 100
63 82 83
63 82 81
81 82 83
84 83 102
82 83 102
82 83 84
83 84 85
65 84 85
65 84 83
86 85 104
84 85 104
84 85 86
85 86 87
67 86 87
67 86 85
88 87 106
86 87 106
86 87 88
87 88 89
69 88 89
69 88 87
90 89 108
88 89 90
88 89 108
71 90 89
89 90 91
71 90 91
90 91 92
30 91 90
30 91 92
73 92 91
73 92 93
91 92 93
17 93 92
32 93 92
17 93 32
95 94 136
136 94 160
95 94 160
94 95 96
94 95 111
96 95 111
95 96 138
97 96 138
95 96 97
96 97 113
96 97 98
98 97 113
79 98 99
79 98 97
97 98 99
100 99 115
98 99 115
98 99 100
99 100 101
81 100 101
81 100 99
102 101 117
100 101 117
100 101 102
101 102 103
83 102 103
83 102 101
104 103 119
102 103 119
102 103 104
103 104 105
85 104 105
85 104 103
106 105 121
104 105 106
104 105 121
105 106 107
87 106 105
87 106 107
106 107 108
46 107 106
46 107 108
89 108 107
89 108 109
107 108 109
30 109 108
48 109 108
30 109 48
111 110 123
123 110 161
111 110 161
95 111 112
95 111 110
110 111 112
113 112 125
111 112 125
111 112 113
112 113 114
97 113 114
97 113 112
115 114 127
113 114 127
113 114 115
114 115 116
99 115 116
99 115 114
117 116 129
115 116 129
115 116 117
116 117 118
101 117 118
101 117 116
119 118 131
117 118 119
117 118 131
103 119 118
118 119 120
103 119 120
119 120 121
62 120 119
62 120 121
105 121 120
105 121 122
120 121 122
46 122 121
64 122 121
46 122 64
6 123 124
110 123 124
6 123 110
125 124 133
123 124 133
123 124 125
124 125 126
112 125 126
112 125 124
127 126 135
125 126 135
125 126 127
126 127 128
114 127 128
114 127 126
127 128 129
129 128 137
127 128 137
128 129 130
116 129 128
116 129 130
129 130 131
78 130 129
78 130 131
118 131 130
118 131 132
130 131 132
62 132 131
80 132 131
62 132 80
16 133 134
124 133 134
16 133 124
133 134 135
135 134 159
133 134 159
126 135 134
134 135 136
126 135 136
135 136 137
94 136 135
94 136 137
128 137 136
128 137 138
136 137 138
78 138 137
96 138 137
78 138 96
141 139 156
141 139 220
156 139 220
141 140 157
141 140 158
157 140 158
139 141 142
140 141 142
139 141 140
141 142 144
141 142 283
144 142 283
144 143 158
144 143 159
158 143 159
142 144 145
142 144 143
143 144 145
144 145 147
147 145 284
144 145 284
147 146 159
147 146 160
159 146 160
145 147 148
146 147 148
145 147 146
147 148 150
147 148 215
150 148 215
150 149 160
150 149 161
160 149 161
148 150 151
149 150 151
148 150 149
150 151 153
153 151 164
150 151 164
153 152 161
153 152 162
161 152 162
151 153 154
151 153 152
152 153 154
153 154 156
153 154 163
156 154 163
156 155 157
156 155 162
157 155 162
139 156 154
154 156 155
139 156 155
140 157 155
45 157 155
45 157 140
140 158 143
29 158 143
29 158 140
143 159 146
134 159 143
134 159 146
146 160 149
94 160 149
94 160 146
149 161 152
110 161 149
110 161 152
152 162 155
5 162 155
5 162 152
165 163 184
154 163 165
154 163 184
166 164 185
151 164 166
151 164 185
163 165 181
163 165 187
181 165 187
164 166 182
164 166 188
182 166 188
169 167 183
169 167 189
183 167 189
170 168 186
170 168 190
186 168 190
167 169 191
167 169 193
191 169 193
168 170 192
168 170 194
192 170 194
173 171 195
173 171 196
195 171 196
174 172 197
174 172 198
197 172 198
171 173 199
171 173 201
199 173 201
172 174 200
172 174 203
200 174 203
177 175 202
177 175 205
202 175 205
178 176 204
178 176 206
204 176 206
175 177 207
175 177 211
207 177 211
176 178 208
176 178 212
208 178 212
180 179 209
180 179 213
209 179 213
179 180 210
179 180 214
210 180 214
165 181 183
183 181 220
165 181 220
166 182 186
186 182 215
166 182 215
167 183 181
181 183 216
167 183 216
163 184 185
185 184 218
163 184 218
164 185 184
184 185 219
164 185 219
168 186 182
182 186 217
168 186 217
165 187 189
189 187 221
165 187 221
166 188 190
190 188 222
166 188 222
167 189 187
187 189 225
167 189 225
168 190 188
188 190 226
168 190 226
169 191 195
169 191 223
195 191 223
170 192 197
197 192 224
170 192 224
169 193 196
196 193 227
169 193 227
170 194 198
198 194 228
170 194 228
171 195 191
191 195 231
171 195 231
171 196 193
193 196 229
171 196 229
172 197 192
192 197 232
172 197 232
172 198 194
194 198 230
172 198 230
173 199 202
202 199 233
173 199 233
174 200 204
204 200 234
174 200 234
173 201 205
205 201 235
173 201 235
175 202 199
199 202 237
175 202 237
174 203 206
206 203 236
174 203 236
176 204 200
200 204 238
176 204 238
175 205 201
201 205 243
175 205 243
176 206 203
203 206 244
176 206 244
177 207 209
177 207 239
209 207 239
178 208 210
210 208 240
178 208 240
179 209 207
207 209 241
179 209 241
180 210 208
208 210 242
180 210 242
177 211 213
177 211 245
213 211 245
178 212 214
214 212 246
178 212 246
179 213 211
211 213 247
179 213 247
180 214 212
212 214 248
180 214 248
182 215 250
148 215 182
148 215 250
183 216 223
183 216 251
223 216 251
186 217 224
224 217 254
186 217 254
184 218 221
184 218 252
221 218 252
185 219 222
222 219 253
185 219 253
181 220 249
139 220 181
139 220 249
187 221 218
218 221 255
187 221 255
188 222 219
219 222 256
188 222 256
191 223 216
216 223 259
191 223 259
192 224 217
217 224 260
192 224 260
189 225 227
227 225 257
189 225 257
190 226 228
228 226 258
190 226 258
193 227 225
225 227 261
193 227 261
194 228 226
226 228 262
194 228 262
196 229 233
196 229 264
233 229 264
198 230 234
234 230 266
198 230 266
195 231 235
195 231 263
235 231 263
197 232 236
236 232 265
197 232 265
199 233 229
229 233 267
199 233 267
200 234 230
230 234 268
200 234 268
201 235 231
231 235 269
201 235 269
203 236 232
232 236 271
203 236 271
202 237 239
239 237 270
202 237 270
204 238 240
204 238 272
240 238 272
207 239 237
237 239 275
207 239 275
208 240 238
238 240 276
208 240 276
209 241 242
209 241 277
242 241 277
210 242 241
241 242 278
210 242 278
205 243 245
205 243 273
245 243 273
206 244 246
246 244 274
206 244 274
211 245 243
243 245 279
211 245 279
212 246 244
244 246 280
212 246 280
213 247 248
213 247 281
248 247 281
214 248 247
247 248 282
214 248 282
220 249 251
251 249 285
220 249 285
215 250 254
215 250 286
254 250 286
216 251 249
249 251 287
216 251 287
218 252 253
218 252 283
253 252 283
219 253 252
252 253 284
219 253 284
217 254 250
250 254 288
217 254 288
221 255 257
221 255 285
257 255 285
222 256 258
222 256 286
258 256 286
225 257 255
255 257 287
225 257 287
226 258 256
256 258 288
226 258 288
223 259 263
223 259 289
263 259 289
224 260 265
265 260 290
224 260 290
227 261 264
227 261 289
264 261 289
228 262 266
228 262 290
266 262 290
231 263 259
259 263 291
231 263 291
229 264 261
261 264 291
229 264 291
232 265 260
260 265 292
232 265 292
230 266 262
262 266 292
230 266 292
233 267 270
270 267 293
233 267 293
234 268 272
234 268 294
272 268 294
235 269 273
235 269 293
273 269 293
237 270 267
267 270 295
237 270 295
236 271 274
236 271 294
274 271 294
238 272 268
268 272 296
238 272 296
243 273 269
269 273 295
243 273 295
244 274 271
271 274 296
244 274 296
239 275 277
277 275 297
239 275 297
240 276 278
278 276 298
240 276 298
241 277 275
275 277 299
241 277 299
242 278 276
276 278 300
242 278 300
245 279 281
245 279 297
281 279 297
246 280 282
282 280 298
246 280 298
247 281 279
279 281 299
247 281 299
248 282 280
280 282 300
248 282 300
252 283 285
142 283 252
142 283 285
253 284 286
145 284 253
145 284 286
255 285 283
249 285 283
249 285 255
256 286 284
250 286 284
250 286 256
251 287 257
257 287 289
251 287 289
258 288 290
254 288 290
254 288 258
259 289 287
261 289 287
259 289 261
262 290 288
260 290 288
260 290 262
263 291 264
263 291 293
264 291 293
265 292 266
266 292 294
265 292 294
267 293 291
269 293 291
267 293 269
268 294 292
271 294 292
268 294 271
270 295 273
273 295 297
270 295 297
274 296 298
272 296 298
272 296 274
279 297 295
275 297 295
275 297 279
276 298 296
280 298 296
276 298 280
281 299 300
277 299 300
277 299 281
278 300 299
282 300 299
278 300 282
这是我得到的奇怪输出的图片。 (当我想要总和为3时,总和为2和4的输出。)(另请注意,出现2和4的这些和的不止一个实例。)
答案 0 :(得分:2)
文件的格式看起来有点奇怪且肯定是多余的(即使不考虑结果矩阵的对称性)。 选择一个二维的int数组来表示对称的稀疏(1%的非零元素,全部是值1)矩阵也可能是浪费空间,但问题并没有指明潜在的问题,所以我会不要再推测了。
似乎在输入文件中,行和列的索引从1开始,而在C ++中,数组的indeces从0开始,所以当像
这样的行时281 299 300
OP的代码读取:
stream >> firstVar >> secondVar >> thirdVar;
adMatrix[counter][firstVar] = 1;
adMatrix[counter][thirdVar] = 1;
值300
关闭一个并且在内存中连续是2D数组,而不是行的最后一个元素(第二个值代表行号,显然)下一行的第一个元素设置为1。
如果格式良好的输入文件(如提供的那样),使用operator>>
直接从文件流中读取值也更容易,而不使用字符串流。
相同的代码段可以像这样重写和“简化”:
#include <iostream>
#include <fstream>
template <size_t Dim>
bool is_symmetric(int (&arr)[Dim][Dim]);
template <size_t Dim>
bool are_there_exactly_3_nonzeroes_each_row(int (&arr)[Dim][Dim]);
template <size_t Dim>
int nonzeroes_on_diagonal(int (&arr)[Dim][Dim]);
const size_t size = 300;
int main()
{
// Declares a 2D array of 300 by 300 int and initializes it to zero:
int adMatrix [size][size] = {};
std::ifstream infile("myTestFile.txt");
if ( !infile )
{
std::cerr << "Unable to open input file.\n";
return EXIT_FAILURE;
}
// The value in the second column is the line number.
// All the the values in the file are indeces in the range 1 - 300
int first_col, row, second_col;
// This will read all the file, if it is well formatted.
while ( infile >> first_col >> row >> second_col )
{
if ( first_col < 1 or first_col > size
or row < 1 or row > size
or second_col < 1 or second_col > size )
{
std::cerr << "Wrong file format.\n The line with data: "
<< first_col << ' ' << row << ' ' << second_col
<< " have out of range indeces.\n";
return EXIT_FAILURE;
}
adMatrix[row - 1][first_col - 1] = 1;
adMatrix[row - 1][second_col - 1] = 1;
}
// Print values or ...
if ( is_symmetric(adMatrix) == false )
{
std::cerr << "The matrix isn't symmetric.\n";
}
if ( are_there_exactly_3_nonzeroes_each_row(adMatrix) == false )
{
std::cerr << "There aren't exactly three non zero elements each row.\n";
}
std::cout << "Number of non zero elements in the main diagonal: "
<< nonzeroes_on_diagonal(adMatrix) << '\n';
}
template <size_t Dim>
bool is_symmetric(int (&arr)[Dim][Dim])
{
for ( size_t i = 0; i < Dim; ++i )
{
for ( size_t j = i + 1; j < Dim; ++j )
{
if ( arr[i][j] != arr[j][i] )
return false;
}
}
return true;
}
template <size_t Dim>
bool are_there_exactly_3_nonzeroes_each_row(int (&arr)[Dim][Dim])
{
for ( size_t i = 0; i < Dim; ++i )
{
int count = 0;
for ( size_t j = 0; j < Dim; ++j )
{
if ( arr[i][j] != 0 )
++count;
}
if ( count != 3 )
return false;
}
return true;
}
template <size_t Dim>
int nonzeroes_on_diagonal(int (&arr)[Dim][Dim])
{
int count = 0;
for ( size_t i = 0; i < Dim; ++i )
{
if ( arr[i][i] != 0 )
{
++count;
}
}
return count;
}
这将解决问题,但是对于更通用的解决方案并真正提高您的语言知识,开始学习类,C ++标准库,容器和泛型函数会更有用。