]> source.dussan.org Git - poi.git/commitdiff
small refactor of XSSFReader
authorPJ Fanning <fanningpj@apache.org>
Wed, 22 Dec 2021 03:51:51 +0000 (03:51 +0000)
committerPJ Fanning <fanningpj@apache.org>
Wed, 22 Dec 2021 03:51:51 +0000 (03:51 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1896259 13f79535-47bb-0310-9956-ffa450edef68

poi-examples/src/main/java/org/apache/poi/examples/xssf/eventusermodel/FromHowTo.java
poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java
poi-ooxml/src/test/java/org/apache/poi/xssf/eventusermodel/TestXSSFReader.java

index 9a40e23f401010c70316c63cf52d55d4144962c6..60246e07a36bbd65d2ec0b0a6f4e917ae45a8069 100644 (file)
@@ -27,7 +27,7 @@ import org.apache.poi.openxml4j.opc.OPCPackage;
 import org.apache.poi.openxml4j.opc.PackageAccess;
 import org.apache.poi.util.XMLHelper;
 import org.apache.poi.xssf.eventusermodel.XSSFReader;
-import org.apache.poi.xssf.model.SharedStringsTable;
+import org.apache.poi.xssf.model.SharedStrings;
 import org.xml.sax.Attributes;
 import org.xml.sax.ContentHandler;
 import org.xml.sax.InputSource;
@@ -45,7 +45,7 @@ public class FromHowTo {
     public void processFirstSheet(String filename) throws Exception {
         try (OPCPackage pkg = OPCPackage.open(filename, PackageAccess.READ)) {
             XSSFReader r = new XSSFReader(pkg);
-            SharedStringsTable sst = r.getSharedStringsTable();
+            SharedStrings sst = r.getSharedStringsTable();
 
             XMLReader parser = fetchSheetParser(sst);
 
@@ -60,7 +60,7 @@ public class FromHowTo {
     public void processAllSheets(String filename) throws Exception {
         try (OPCPackage pkg = OPCPackage.open(filename, PackageAccess.READ)) {
             XSSFReader r = new XSSFReader(pkg);
-            SharedStringsTable sst = r.getSharedStringsTable();
+            SharedStrings sst = r.getSharedStringsTable();
 
             XMLReader parser = fetchSheetParser(sst);
 
@@ -76,7 +76,7 @@ public class FromHowTo {
         }
     }
 
-    public XMLReader fetchSheetParser(SharedStringsTable sst) throws SAXException, ParserConfigurationException {
+    public XMLReader fetchSheetParser(SharedStrings sst) throws SAXException, ParserConfigurationException {
         XMLReader parser = XMLHelper.newXMLReader();
         ContentHandler handler = new SheetHandler(sst);
         parser.setContentHandler(handler);
@@ -87,7 +87,7 @@ public class FromHowTo {
      * See org.xml.sax.helpers.DefaultHandler javadocs
      */
     private static class SheetHandler extends DefaultHandler {
-        private final SharedStringsTable sst;
+        private final SharedStrings sst;
         private String lastContents;
         private boolean nextIsString;
         private boolean inlineStr;
@@ -107,7 +107,7 @@ public class FromHowTo {
             }
         }
 
-        private SheetHandler(SharedStringsTable sst) {
+        private SheetHandler(SharedStrings sst) {
             this.sst = sst;
         }
 
index bb795333ed95894310b3baecc116d7ea50595186..bc1bad8abf48182c165c92b63339af4e71032f25 100644 (file)
@@ -120,7 +120,7 @@ public class XSSFReader {
      * returns a handy object for working with
      * shared strings.
      */
-    public SharedStringsTable getSharedStringsTable() throws IOException, InvalidFormatException {
+    public SharedStrings getSharedStringsTable() throws IOException, InvalidFormatException {
         ArrayList<PackagePart> parts = pkg.getPartsByContentType(XSSFRelation.SHARED_STRINGS.getContentType());
         return parts.size() == 0 ? null : new SharedStringsTable(parts.get(0));
     }
index 3e1bfa50f6cb2977bd454efe123a44aba872d630..1b02f66108ebd70a4a81e3ecf559e816293efdd0 100644 (file)
@@ -102,7 +102,7 @@ public final class TestXSSFReader {
 
             XSSFReader r = new XSSFReader(pkg);
 
-            assertEquals(11, r.getSharedStringsTable().getSharedStringItems().size());
+            assertEquals(11, r.getSharedStringsTable().getCount());
             assertEquals("Test spreadsheet", r.getSharedStringsTable().getItemAt(0).toString());
         }
     }