svn commit: r1896259 - in /poi/trunk: poi-examples/src/main/java/org/apache/poi/examples/xssf/eventusermodel/ poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/ poi-ooxml/src/test/java/org/apache/poi/xssf/eventusermodel/

[fanningpj]

Author: fanningpj
Date: Wed Dec 22 03:51:51 2021
New Revision: 1896259

URL: http://svn.apache.org/viewvc?rev=1896259&view=rev
Log:
small refactor of XSSFReader

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

Modified: 
poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xssf/eventusermodel/FromHowTo.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xssf/eventusermodel/FromHowTo.java?rev=1896259&r1=1896258&r2=1896259&view=diff
==============================================================================
--- 
poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xssf/eventusermodel/FromHowTo.java
 (original)
+++ 
poi/trunk/poi-examples/src/main/java/org/apache/poi/examples/xssf/eventusermodel/FromHowTo.java
 Wed Dec 22 03:51:51 2021
@@ -27,7 +27,7 @@ import org.apache.poi.openxml4j.opc.OPCP
 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;
         }
 

Modified: 
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java?rev=1896259&r1=1896258&r2=1896259&view=diff
==============================================================================
--- 
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java
 (original)
+++ 
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java
 Wed Dec 22 03:51:51 2021
@@ -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));
     }

Modified: 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/eventusermodel/TestXSSFReader.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/eventusermodel/TestXSSFReader.java?rev=1896259&r1=1896258&r2=1896259&view=diff
==============================================================================
--- 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/eventusermodel/TestXSSFReader.java
 (original)
+++ 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/xssf/eventusermodel/TestXSSFReader.java
 Wed Dec 22 03:51:51 2021
@@ -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());
         }
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@poi.apache.org
For additional commands, e-mail: commits-h...@poi.apache.org