将While循环更改为for循环

Question

您好，这里使用while（true）的extractTable方法的人。 根据方法，如果条件失败，它将是一个无限循环，这是一个糟糕的编程实践，您可以帮助我解决在每次或每次更改时

private List<Map<String, Map<String, Integer>>> extractTable(XSSFSheet sheet, Cell tablePosition) {
    List<Map<String, Map<String, Integer>>> table = new ArrayList<>();

    int headerRow = tablePosition.getRowIndex();
    int valuesHeaderColumn = tablePosition.getColumnIndex();

    // read all columns and rows until finding an empty one
    int row = headerRow + 1;
    int col = valuesHeaderColumn + 1;

    // read all columns starting at col
    while (true) {
        Map<String, Map<String, Integer>> tableColumn = new HashMap<>();
        Cell headerCell = sheet.getRow(headerRow).getCell(col);
        if (headerCell == null)
            break;
        String columnName = headerCell.getStringCellValue().toUpperCase();
        if ("".equals(columnName))
            break;

        // read all rows starting at row
        Map<String, Integer> values = new HashMap<>();
        while (true) {
            Row valueHeaderRow = sheet.getRow(row);
            if (valueHeaderRow == null)
                break;
            Cell valueHeaderCell = valueHeaderRow.getCell(valuesHeaderColumn);
            String valueHeader = valueHeaderCell.getStringCellValue();
            if ("".equals(valueHeader))
                break;
            Cell valueCell = sheet.getRow(row).getCell(col);
            Integer value = (int) valueCell.getNumericCellValue();
            values.put(valueHeader, value);
            row++;
        }

        tableColumn.put(columnName, values);
        // Add the columns map to the list
        table.add(tableColumn);
        col++;
        row = headerRow + 1;
    }
    return table;
}

Answer 1

我不相信while（true）... break是不好的编程习惯，请看这里 。 但是，要使用POI迭代电子表格的行和列， 建议您采用以下方式：

for (Row row: sheet) {
  for (Cell cell: row) {
    // do something here
  }
}