From 647f0a90a2a991bffb7aee56fc20d6d37e6f1883 Mon Sep 17 00:00:00 2001 From: Javen O'Neal Date: Fri, 20 Jan 2017 04:38:18 +0000 Subject: [PATCH] pull out loop invariants, use List.indexOf to look up sheet index git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1779560 13f79535-47bb-0310-9956-ffa450edef68 --- src/java/org/apache/poi/hssf/model/InternalWorkbook.java | 3 ++- src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java | 7 +------ src/java/org/apache/poi/ss/util/WorkbookUtil.java | 2 +- 3 files changed, 4 insertions(+), 8 deletions(-) diff --git a/src/java/org/apache/poi/hssf/model/InternalWorkbook.java b/src/java/org/apache/poi/hssf/model/InternalWorkbook.java index 023007d36..063830d32 100644 --- a/src/java/org/apache/poi/hssf/model/InternalWorkbook.java +++ b/src/java/org/apache/poi/hssf/model/InternalWorkbook.java @@ -770,7 +770,8 @@ public final class InternalWorkbook { public int getSheetIndex(String name) { int retval = -1; - for (int k = 0; k < boundsheets.size(); k++) { + final int size = boundsheets.size(); + for (int k = 0; k < size; k++) { String sheet = getSheetName(k); if (sheet.equalsIgnoreCase(name)) { diff --git a/src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java b/src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java index 08504c3a6..e8be284f1 100644 --- a/src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java +++ b/src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java @@ -806,12 +806,7 @@ public final class HSSFWorkbook extends POIDocument implements org.apache.poi.ss */ @Override public int getSheetIndex(org.apache.poi.ss.usermodel.Sheet sheet) { - for(int i=0; i<_sheets.size(); i++) { - if(_sheets.get(i) == sheet) { - return i; - } - } - return -1; + return _sheets.indexOf(sheet); } /** diff --git a/src/java/org/apache/poi/ss/util/WorkbookUtil.java b/src/java/org/apache/poi/ss/util/WorkbookUtil.java index e14b8e0c5..f5e5f40fe 100644 --- a/src/java/org/apache/poi/ss/util/WorkbookUtil.java +++ b/src/java/org/apache/poi/ss/util/WorkbookUtil.java @@ -182,7 +182,7 @@ public class WorkbookUtil { case Workbook.SHEET_STATE_HIDDEN: break; case Workbook.SHEET_STATE_VERY_HIDDEN: break; default: throw new IllegalArgumentException("Invalid sheet state : " + state + "\n" + - "Sheet state must beone of the Workbook.SHEET_STATE_* constants"); + "Sheet state must be one of the Workbook.SHEET_STATE_* constants"); } }