索引超出范围异常:重复多次相同的行

时间:2013-11-22 14:56:05

标签: java excel apache-poi indexoutofboundsexception

这是我用来获取Excel工作表的代码,读取其内容,每行我从列表中添加一个GUID。

更新 代码:

public class PublishOutputFile {

    public void publishOutputWithGuids(List<String> guids, File file)
            throws Exception {
        FileInputStream fileStream = new FileInputStream(file);
        HSSFWorkbook workbook = new HSSFWorkbook(fileStream);
        HSSFSheet sheet = workbook.getSheetAt(0);
        Row toprow = sheet.getRow(0);
        int cellsNum = toprow.getLastCellNum();
        Iterator<Row> rowIterator = sheet.iterator();
        while (rowIterator.hasNext()) {
            Row row = rowIterator.next();
            int rownum = row.getRowNum();
            System.out.println(rownum);
            Iterator<Cell> cellIterator = row.cellIterator();
            while (cellIterator.hasNext()) {
                Cell cell = cellIterator.next();
                switch (cell.getCellType()) {
                case Cell.CELL_TYPE_BOOLEAN:
                    break;
                case Cell.CELL_TYPE_NUMERIC:
                    break;
                case Cell.CELL_TYPE_STRING:
                    break;
                }
                if (rownum == 0) {
                    row.createCell(cellsNum).setCellValue("GUID");
                } else {
//                  System.out.println(rownum);
                    row.createCell(cellsNum).setCellValue(guids.get(rownum-1));
                }
            }
        }
        fileStream.close();
        FileOutputStream out = new FileOutputStream(
                new File(
                        "C:/Users/U0172959/Desktop/Themes_20131120_234645-hierarchy with GUIDS.xls"));
        workbook.write(out);
        out.close();
    }
}

更新输出:

0
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 286, Size: 286
    at java.util.ArrayList.rangeCheck(Unknown Source)
    at java.util.ArrayList.get(Unknown Source)
    at mappingForEtl.PublishOutputFile.publishOutputWithGuids(PublishOutputFile.java:43)
    at mappingForEtl.InputFileParsing.main(InputFileParsing.java:75)

在输出中,每行重复5次,我认为这是导致越界异常的原因。但我无法弄清楚导致这种情况发生的原因。

(我昨天使用了相同的代码用于不同的文件,并且工作正常......)它可以与我作为输入的Excel工作表中的列数有关。我昨天使用的那个有8列,我现在使用的是20列。

任何人都知道可能导致这种情况的原因是什么?谢谢你的帮助。

3 个答案:

答案 0 :(得分:2)

您正在迭代行中的单元格,并为每个单元格打印行号。您是否确定在else案例中打印正确值的数据?

if (row.getRowNum() == 0) {
    row.createCell(cellsNum).setCellValue("GUID");
} else {
    System.out.println((row.getRowNum())); // row number for each cell?
    row.createCell(cellsNum).setCellValue(guids.get((row.getRowNum())));
}

更新

重复输出的问题是你的行上有一个循环。每行包含许多单元格。你也可以循环遍历细胞。重复输出来自这样一个事实:当你循环遍历行中的单元格时,行不会改变。 为了更好地理解,您应该将输出更改为:

System.out.println("Being at row: " + row.getRowNum() + " creating cell at cellnum: " + cellsNum);

这应该清除对正在发生的事情的理解。

更新2:

抛出IndexOutOfBoundsException是因为guid的大小是286但是索引是基于零并且不是以1.开始。当您将第一行设置为不同时,您应该将guids.get(row.getRowNum())更改为guids.get(row.getRowNum() - 1)。这可以工作,但它不必。只要我们不知道详细的数据结构,它对你来说就没有价值......

答案 1 :(得分:2)

关于例外

您可能意外地创建了一些新的Row可能为空白),GUID上的List没有Row。这可能导致此异常。

也许这个添加可以帮助你:

while (rowIterator.hasNext()) {
    Row row = rowIterator.next();
    if (row.getLastCellNum() < cellsNum)
        continue;

关于重复输出

您正在迭代每一行,然后遍历每个单元格,然后打印行号。考虑到一行有4个单元格,最终会将行号打印4次。

答案 2 :(得分:-1)

问题在于:

} else {
    row.createCell(cellsNum).setCellValue(
        guids.get((row.getRowNum())));
}

您的guids.get正在抛出异常,因为它在索引286处不包含元素。

解决方案:检索row.getRowNum - 1中索引处的元素,现在您将检索0到285之间的元素:

} else {
    row.createCell(cellsNum).setCellValue(
        guids.get((row.getRowNum() - 1)));
}

更新

因为它看起来你并没有试图解决IndexOutOfBoundsException(非常奇怪),而是更多关于为什么数字被打印4到5次,然后检查你当前的算法:

Iterator<Row> rowIterator = sheet.iterator();
//verify if there's a row to read in the current Excel sheet
while (rowIterator.hasNext()) {
    //get the row
    Row row = rowIterator.next();
    //get an iterator of the cells
    Iterator<Cell> cellIterator = row.cellIterator();
    //while there's a cell to read...
    while (cellIterator.hasNext()) {
        //get the cell
        //note: if the cell is empty, it won't be considered by the CellIterator, so even
        //if you open the file and see 20 cells, this Iterator will seek for cells with some value
        //from your current output, this looks to be only 5 cells
        Cell cell = cellIterator.next();
        //verify the type of the cell...
        switch (cell.getCellType()) {
        case Cell.CELL_TYPE_BOOLEAN:
            //do nothing?
            break;
        case Cell.CELL_TYPE_NUMERIC:
            //do nothing?
            break;
        case Cell.CELL_TYPE_STRING:
            //do nothing?
            break;
        }
        if (row.getRowNum() == 0) {
            row.createCell(cellsNum).setCellValue("GUID");
        } else {
            //prints the number of the row (per cell)
            //this causes the 1\n1\n1\n1\n2\n2\n... output
            System.out.println((row.getRowNum()));
            //writes the element in guids at index of row num
            row.createCell(cellsNum).setCellValue(
                    //guids.get((row.getRowNum())));
                    guids.get((row.getRowNum() - 1)));
        }
    }
}