]> source.dussan.org Git - poi.git/commitdiff
use isEmpty()
authorPJ Fanning <fanningpj@apache.org>
Thu, 30 Dec 2021 20:34:10 +0000 (20:34 +0000)
committerPJ Fanning <fanningpj@apache.org>
Thu, 30 Dec 2021 20:34:10 +0000 (20:34 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1896541 13f79535-47bb-0310-9956-ffa450edef68

27 files changed:
poi-ooxml/src/main/java/org/apache/poi/openxml4j/opc/ZipPackage.java
poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFBReader.java
poi-ooxml/src/main/java/org/apache/poi/xssf/eventusermodel/XSSFReader.java
poi-ooxml/src/main/java/org/apache/poi/xssf/streaming/SXSSFSheet.java
poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFRow.java
poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFAbstractFootnoteEndnote.java
poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFDocument.java
poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFHeaderFooter.java
poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFTable.java
poi-ooxml/src/main/java/org/apache/poi/xwpf/usermodel/XWPFTableCell.java
poi-ooxml/src/test/java/org/apache/poi/xssf/TestXSSFXLookupFunction.java
poi-scratchpad/src/main/java/org/apache/poi/hwpf/model/CHPBinTable.java
poi-scratchpad/src/main/java/org/apache/poi/hwpf/model/PAPBinTable.java
poi-scratchpad/src/main/java/org/apache/poi/hwpf/usermodel/Range.java
poi/src/main/java/org/apache/poi/extractor/ExtractorFactory.java
poi/src/main/java/org/apache/poi/hssf/record/AbstractEscherHolderRecord.java
poi/src/main/java/org/apache/poi/hssf/record/DrawingGroupRecord.java
poi/src/main/java/org/apache/poi/hssf/record/common/UnicodeString.java
poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
poi/src/main/java/org/apache/poi/poifs/eventfilesystem/POIFSReaderRegistry.java
poi/src/main/java/org/apache/poi/poifs/filesystem/POIFSFileSystem.java
poi/src/main/java/org/apache/poi/ss/format/CellFormatPart.java
poi/src/main/java/org/apache/poi/ss/formula/EvaluationConditionalFormatRule.java
poi/src/main/java/org/apache/poi/ss/formula/atp/TextJoinFunction.java
poi/src/main/java/org/apache/poi/ss/formula/functions/LookupUtils.java

index af1a4c07b1343aff5ba2e5088aa5c5a14101a5b4..bf91ad17b07b56be9a831be787d1618c4bad46fe 100644 (file)
@@ -537,8 +537,8 @@ public final class ZipPackage extends OPCPackage {
         try {
             // If the core properties part does not exist in the part list,
             // we save it as well
-            if (this.getPartsByRelationshipType(PackageRelationshipTypes.CORE_PROPERTIES).size() == 0 &&
-                this.getPartsByRelationshipType(PackageRelationshipTypes.CORE_PROPERTIES_ECMA376).size() == 0    ) {
+            if (this.getPartsByRelationshipType(PackageRelationshipTypes.CORE_PROPERTIES).isEmpty() &&
+                this.getPartsByRelationshipType(PackageRelationshipTypes.CORE_PROPERTIES_ECMA376).isEmpty()) {
                 LOG.atDebug().log("Save core properties part");
 
                 // Ensure that core properties are added if missing
index d9a24bb855dd1d648cd5d14ccf2ed42589d577ea..9a02bcb4a0659d6f5194453e8fb026d716de17de 100644 (file)
@@ -108,7 +108,7 @@ public class XSSFBReader extends XSSFReader {
 
     public XSSFBStylesTable getXSSFBStylesTable() throws IOException {
         ArrayList<PackagePart> parts = pkg.getPartsByContentType(XSSFBRelation.STYLES_BINARY.getContentType());
-        if(parts.size() == 0) return null;
+        if(parts.isEmpty()) return null;
 
         // Create the Styles Table, and associate the Themes if present
         try (InputStream stream = parts.get(0).getInputStream()) {
index 220f5b89c48aedd34252367cbdaa7be317cf38da..64ed7297eee047e06f3e0ca4674fa6defb1e003f 100644 (file)
@@ -144,7 +144,7 @@ public class XSSFReader {
     public SharedStrings getSharedStringsTable() throws IOException, InvalidFormatException {
         ArrayList<PackagePart> parts = pkg.getPartsByContentType(XSSFRelation.SHARED_STRINGS.getContentType());
         try {
-            return parts.size() == 0 ? null :
+            return parts.isEmpty() ? null :
                     useReadOnlySharedStringsTable ? new ReadOnlySharedStringsTable(parts.get(0)) :
                             new SharedStringsTable(parts.get(0));
         } catch (SAXException se) {
@@ -158,7 +158,7 @@ public class XSSFReader {
      */
     public StylesTable getStylesTable() throws IOException, InvalidFormatException {
         ArrayList<PackagePart> parts = pkg.getPartsByContentType(XSSFRelation.STYLES.getContentType());
-        if (parts.size() == 0) return null;
+        if (parts.isEmpty()) return null;
 
         // Create the Styles Table, and associate the Themes if present
         StylesTable styles = new StylesTable(parts.get(0));
index d4e22810b21560d70a0a716fd0550d472f9052e5..38ff3d3e09558ea9ae58a8c8c7c00642f8aefdbb 100644 (file)
@@ -198,7 +198,7 @@ public class SXSSFSheet implements Sheet, OoxmlSheetExtensions {
         if(_writer.getNumberOfFlushedRows() > 0) {
             return _writer.getLowestIndexOfFlushedRows();
         }
-        return _rows.size() == 0 ? -1 : _rows.firstKey();
+        return _rows.isEmpty() ? -1 : _rows.firstKey();
     }
 
     /**
@@ -208,7 +208,7 @@ public class SXSSFSheet implements Sheet, OoxmlSheetExtensions {
      */
     @Override
     public int getLastRowNum() {
-        return _rows.size() == 0 ? -1 : _rows.lastKey();
+        return _rows.isEmpty() ? -1 : _rows.lastKey();
     }
 
     /**
index 85543d5ff029c74ed8a5d4be87d0612e8ad93baf..15859f4fdbcdc48d09e232b45380d63f92b77a29 100644 (file)
@@ -304,7 +304,7 @@ public class XSSFRow implements Row, Comparable<XSSFRow> {
      */
     @Override
     public short getFirstCellNum() {
-        return (short)(_cells.size() == 0 ? -1 : _cells.firstKey());
+        return (short)(_cells.isEmpty() ? -1 : _cells.firstKey());
     }
 
     /**
@@ -328,7 +328,7 @@ public class XSSFRow implements Row, Comparable<XSSFRow> {
      */
     @Override
     public short getLastCellNum() {
-        return (short)(_cells.size() == 0 ? -1 : (_cells.lastKey() + 1));
+        return (short)(_cells.isEmpty() ? -1 : (_cells.lastKey() + 1));
     }
 
     /**
index 9f69900ec808b0f8a231f1ba6fcd2a5b46c373ee..50ab442b15a1aa7913cd5aceca3958768b8471b1 100644 (file)
@@ -2835,7 +2835,7 @@ public class XSSFSheet extends POIXMLDocumentPart implements Sheet, OoxmlSheetEx
      */
     @Beta
     public void copyRows(List<? extends Row> srcRows, int destStartRow, CellCopyPolicy policy) {
-        if (srcRows == null || srcRows.size() == 0) {
+        if (srcRows == null || srcRows.isEmpty()) {
             throw new IllegalArgumentException("No rows to copy");
         }
         final Row srcStartRow = srcRows.get(0);
index ec17fa8809345a60531c91e46ce8ab6f749485e6..0552d8182f2d56521dda0d1c0fc49efb5958951f 100644 (file)
@@ -1383,7 +1383,7 @@ public class XSSFWorkbook extends POIXMLDocument implements Workbook, Date1904Su
         sheets.remove(index);
 
         // only set new sheet if there are still some left
-        if(sheets.size() == 0) {
+        if(sheets.isEmpty()) {
             return;
         }
 
index 7c97b908cd8512072d0723d2aef7c17574cdd364..0be1a2f74bc06afb2be7d8143713f2addc046ce9 100644 (file)
@@ -503,7 +503,7 @@ public abstract class XWPFAbstractFootnoteEndnote  implements Iterable<XWPFParag
      */
     public XWPFTable createTable() {
         XWPFTable table = new XWPFTable(ctFtnEdn.addNewTbl(), this);
-        if (bodyElements.size() == 0) {
+        if (bodyElements.isEmpty()) {
             XWPFParagraph p = createParagraph();
             ensureFootnoteRef(p);
         }
index f65667c55de7579d40858a2a200f05e53fc8f802..7274f9bbe7bd3f9c668118b8aa8b85273b31f42b 100644 (file)
@@ -618,7 +618,7 @@ public class XWPFDocument extends POIXMLDocument implements Document, IBody {
      */
     private int getBodyElementSpecificPos(int pos, List<? extends IBodyElement> list) {
         // If there's nothing to find, skip it
-        if (list.size() == 0) {
+        if (list.isEmpty()) {
             return -1;
         }
 
index fea3e33b48028ef90ed77304d7973a4cb77b9b0a..a839c587cfcff57c49a24e499da63e1efad08565 100644 (file)
@@ -603,7 +603,7 @@ public abstract class XWPFHeaderFooter extends POIXMLDocumentPart implements IBo
     @Override
     protected void prepareForCommit() {
         // must contain at least an empty paragraph
-        if (bodyElements.size() == 0) {
+        if (bodyElements.isEmpty()) {
             createParagraph();
         }
 
@@ -611,7 +611,7 @@ public abstract class XWPFHeaderFooter extends POIXMLDocumentPart implements IBo
         for (XWPFTable tbl : tables) {
             for (XWPFTableRow row : tbl.tableRows) {
                 for (XWPFTableCell cell : row.getTableCells()) {
-                    if (cell.getBodyElements().size() == 0) {
+                    if (cell.getBodyElements().isEmpty()) {
                         cell.addParagraph();
                     }
                 }
index 971ea0ffd5550af3ac98a0974a2c2532121fc694..00247afc92a0d097d9fc2bf0ae3075f4cfcae2d8 100644 (file)
@@ -237,10 +237,10 @@ public class XWPFTable implements IBodyElement, ISDTContents {
     /**
      * Add a new cell at the end of each row in this table, creating a new column.
      * If rows have different numbers of columns, will still append a cell to each row.
-     * Currently does not match the width of existing columns.
+     * Currently, does not match the width of existing columns.
      */
     public void addNewCol() {
-        if (tableRows.size() == 0) {
+        if (tableRows.isEmpty()) {
             createRow();
         }
         for (XWPFTableRow tableRow : tableRows) {
index b81cde1eee99067237f0dda1be9135a605161904..fe3140613a1124b439b2c47934eb5a79cd181a83 100644 (file)
@@ -441,7 +441,7 @@ public class XWPFTableCell implements IBody, ICell {
     }
 
     public void setText(String text) {
-        XWPFParagraph par = (paragraphs.size() == 0) ? addParagraph() : paragraphs.get(0);
+        XWPFParagraph par = paragraphs.isEmpty() ? addParagraph() : paragraphs.get(0);
         par.createRun().setText(text);
     }
 
index dbe7c956fc5dce5a428b825fe7f9a2b7f73e84e3..81fe37b62354faf505c7973a90502a79667a2b10 100644 (file)
@@ -31,7 +31,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
 /**
  * Testcase for function XLOOKUP()
  */
-public class TestXSSFXLookupFunction {
+class TestXSSFXLookupFunction {
 
     //https://support.microsoft.com/en-us/office/xlookup-function-b7fd680e-6d10-43e6-84f9-88eae8bf5929
 
index 1d0a5707911fb5ad46ce1d30660498c52654d901..3f2bd010814b861a35cc20744c7e5fbb4fdb867a 100644 (file)
@@ -242,7 +242,7 @@ public class CHPBinTable
                 }
             }
 
-            if ( chpxs.size() == 0 )
+            if ( chpxs.isEmpty() )
             {
                 LOG.atWarn().log("Text piece [{}; {}) has no CHPX. Creating new one.", box(startInclusive),box(boundary));
                 // create it manually
index 675b62e8bef51481ac930037b6d4a732f85e3aad..917c7d895812f93079beca3da7306ca1c0d25fcc 100644 (file)
@@ -215,7 +215,7 @@ public class PAPBinTable
                 lastPapxIndex = oldPapxSortedByEndPos.size() - 1;
             }
 
-            if ( papxs.size() == 0 )
+            if ( papxs.isEmpty() )
             {
                 LOG.atWarn().log("Paragraph [{}; {}) has no PAPX. Creating new one.", box(startInclusive),box(endExclusive));
                 // create it manually
index a3fd882d645724ffdd8af8e1ec277c1ca5b687c3..08d3840e5637044803ab92199b5882dc3764a684 100644 (file)
@@ -889,7 +889,7 @@ public class Range {
     private static int binarySearchStart( List<? extends PropertyNode<?>> rpl,
             int start )
     {
-        if ( rpl.size() == 0 )
+        if ( rpl.isEmpty())
             return -1;
         if ( rpl.get( 0 ).getStart() >= start )
             return 0;
index 03a8278077c8c6f0a991923aabe7d27147b3f067..759e75e068b90f647a57a0f0bc1c5a048beb94b7 100644 (file)
@@ -347,7 +347,7 @@ public final class ExtractorFactory {
         }
 
         // Create the extractors
-        if(dirs.size() == 0 && nonPOIFS.size() == 0){
+        if(dirs.isEmpty() && nonPOIFS.isEmpty()){
             return new POITextExtractor[0];
         }
 
index 5d022a26052735edba97c1ad5f0e205de79dc79d..0f1377aa03f37e0b448b8e3b2977a0e8d12e7a35 100644 (file)
@@ -117,7 +117,7 @@ public abstract class AbstractEscherHolderRecord extends Record {
     @Override
     public int getRecordSize() {
         byte[] rawData = getRawData();
-        if (escherRecords.size() == 0 && rawData != null) {
+        if (escherRecords.isEmpty() && rawData != null) {
             // XXX: It should be possible to derive this without concatenating the array, too.
             return rawData.length;
         }
index 91d805a381f9c7371dc7caa1411995e593e65936..ee301fce452090a77e04731f07b99e2674c02f97 100644 (file)
@@ -82,7 +82,7 @@ public final class DrawingGroupRecord extends AbstractEscherHolderRecord {
     public int serialize(int offset, byte[] data)
     {
         byte[] rawData = getRawData();
-        if (getEscherRecords().size() == 0 && rawData != null)
+        if (getEscherRecords().isEmpty() && rawData != null)
         {
             return writeData( offset, data, rawData );
         }
@@ -117,8 +117,7 @@ public final class DrawingGroupRecord extends AbstractEscherHolderRecord {
     private int getRawDataSize() {
         List<EscherRecord> escherRecords = getEscherRecords();
         byte[] rawData = getRawData();
-        if (escherRecords.size() == 0 && rawData != null)
-        {
+        if (escherRecords.isEmpty() && rawData != null) {
             return rawData.length;
         }
         int size = 0;
index 37fd7e0ec5e79dd24f9146bfde523c0741bae662..9769f3cb1fc43c50ee9e26c5fcef6bdfb3ac4782 100644 (file)
@@ -337,7 +337,7 @@ public class UnicodeString implements Comparable<UnicodeString>, Duplicatable, G
 
     public void removeFormatRun(FormatRun r) {
       field_4_format_runs.remove(r);
-      if (field_4_format_runs.size() == 0) {
+      if (field_4_format_runs.isEmpty()) {
         field_4_format_runs = null;
         field_2_optionflags = richText.clearByte(field_2_optionflags);
       }
index f77cc162053bcbb208614392b16f6db3316fc442..0bc614264ee764011ac3bfc31a180a9570c2e26c 100644 (file)
@@ -316,7 +316,7 @@ public final class HSSFSheet implements Sheet {
             _sheet.removeRow(hrow.getRowRecord());
 
             // if there are no more rows, then reset first/last
-            if(_rows.size() == 0) {
+            if(_rows.isEmpty()) {
                 _firstrow = -1;
                 _lastrow = -1;
             }
index 2091eec5126368aa65ff45c81b0ed87d047b1ab3..3b2af081be1b222f9586ed602abb542402bd6618 100644 (file)
@@ -150,13 +150,11 @@ class POIFSReaderRegistry
     }
 
     private void dropDocument(final POIFSReaderListener listener,
-                              final DocumentDescriptor descriptor)
-    {
+                              final DocumentDescriptor descriptor) {
         Set<POIFSReaderListener> listeners = chosenDocumentDescriptors.get(descriptor);
 
         listeners.remove(listener);
-        if (listeners.size() == 0)
-        {
+        if (listeners.isEmpty()) {
             chosenDocumentDescriptors.remove(descriptor);
         }
     }
index eba62568b904961436144099df0c9d6812856499..6194efab289fb3baea1f982bc97197abb7123c23 100644 (file)
@@ -583,7 +583,7 @@ public class POIFSFileSystem extends BlockStore
                 offset++;
 
                 // Chain it
-                if (_xbat_blocks.size() == 0) {
+                if (_xbat_blocks.isEmpty()) {
                     _header.setXBATStart(offset);
                 } else {
                     _xbat_blocks.get(_xbat_blocks.size() - 1).setValueAt(
index c350f5f363a311aa8b5dfc7bd250c8cbb010fe68..9ecc5ce53dff7a610db21448367fe2e4d9ac36b6 100644 (file)
@@ -48,7 +48,7 @@ import static org.apache.poi.ss.format.CellFormatter.quote;
  */
 @SuppressWarnings("RegExpRepeatedSpace")
 public class CellFormatPart {
-    private static final Logger LOG = LogManager.getLogger(CellFormat.class.getName());
+    private static final Logger LOG = LogManager.getLogger(CellFormatPart.class);
 
     static final Map<String, Color> NAMED_COLORS;
 
index b1c3504f2b80b491f16069a371f004517a5d017f..b3fb95f5939068e647441569f978066d0f631a55 100644 (file)
@@ -598,7 +598,7 @@ public class EvaluationConditionalFormatRule implements Comparable<EvaluationCon
         }
 
         final Set<ValueAndFormat> avgSet = new LinkedHashSet<>(1);
-        avgSet.add(new ValueAndFormat(allValues.size() == 0 ? 0 : total / allValues.size(), null, decimalTextFormat));
+        avgSet.add(new ValueAndFormat(allValues.isEmpty() ? 0 : total / allValues.size(), null, decimalTextFormat));
 
         final double stdDev = allValues.size() <= 1 ? 0 : ((NumberEval) AggregateFunction.STDEV.evaluate(pop, 0, 0)).getNumberValue();
         avgSet.add(new ValueAndFormat(stdDev, null, decimalTextFormat));
index 14a17854f3d6e6eef09a8536ee34a066a192c4d7..52fcb05cc00ed5c298cc6f48fb6d443bc6b1ea5b 100644 (file)
@@ -93,7 +93,7 @@ final class TextJoinFunction implements FreeRefFunction {
             }
 
             // Join the list of values with the specified delimiter and return
-            if (delimiterArgs.size() == 0) {
+            if (delimiterArgs.isEmpty()) {
                 return new StringEval(String.join("", textValues));
             } else if (delimiterArgs.size() == 1) {
                 String delimiter = laxValueToString(delimiterArgs.get(0));
index 648250beca75f14d28b2e646cb2c37056c75559a..c76bfc0f4e5acba582446483d9a2dfb135d0d8c1 100644 (file)
@@ -22,8 +22,6 @@ import java.util.HashSet;
 import java.util.Iterator;
 import java.util.Map;
 import java.util.NoSuchElementException;
-import java.util.Spliterator;
-import java.util.Spliterators;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;