使用istringstream读取文件,意外输出

时间:2017-03-31 18:55:00

标签: c++ string multidimensional-array file-io

我有一个.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的这些和的不止一个实例。)

enter image description here

1 个答案:

答案 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 ++标准库,容器和泛型函数会更有用。