]> source.dussan.org Git - poi.git/commitdiff
replace "".equals(string) with string.isEmpty() to avoid null strings silently evalua...
authorJaven O'Neal <onealj@apache.org>
Wed, 18 Oct 2017 08:41:42 +0000 (08:41 +0000)
committerJaven O'Neal <onealj@apache.org>
Wed, 18 Oct 2017 08:41:42 +0000 (08:41 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1812476 13f79535-47bb-0310-9956-ffa450edef68

20 files changed:
src/java/org/apache/poi/hpsf/Variant.java
src/java/org/apache/poi/poifs/crypt/CryptoFunctions.java
src/java/org/apache/poi/poifs/crypt/DataSpaceMapUtils.java
src/java/org/apache/poi/sl/draw/DrawTextParagraph.java
src/java/org/apache/poi/ss/formula/functions/Countblank.java
src/java/org/apache/poi/ss/usermodel/FractionFormat.java
src/java/org/apache/poi/util/StringUtil.java
src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
src/ooxml/java/org/apache/poi/openxml4j/opc/PackagePartName.java
src/ooxml/java/org/apache/poi/openxml4j/opc/ZipPackage.java
src/ooxml/java/org/apache/poi/poifs/crypt/dsig/OOXMLURIDereferencer.java
src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/RelationshipTransformService.java
src/ooxml/java/org/apache/poi/ss/extractor/EmbeddedExtractor.java
src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFHyperlink.java
src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFTextRun.java
src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFPivotTable.java
src/ooxml/java/org/apache/poi/xssf/usermodel/helpers/XSSFPasswordHelper.java
src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java
src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFFontInfo.java
src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSimpleShape.java

index b93419545aa422712dab57204e0c8bee2177e432..87acdc066a47862bb8cdfd384f235fea2cedc672 100644 (file)
@@ -420,7 +420,7 @@ public class Variant
         }
         
         name += numberToName.get(vt);
-        return !"".equals(name) ? name : "unknown variant type";
+        return !name.isEmpty() ? name : "unknown variant type";
     }
 
     /**
index af829839f896002335a2f110b31b0ceb916dbcf4..9273f53e0e818d4862f4a1b07dde82483d06a02a 100644 (file)
@@ -379,7 +379,7 @@ public class CryptoFunctions {
         // SET Verifier TO 0x0000
         short verifier = 0;
 
-        if (!"".equals(password)) {
+        if (!password.isEmpty()) {
             // FOR EACH PasswordByte IN PasswordArray IN REVERSE ORDER
             for (int i = arrByteChars.length-1; i >= 0; i--) {
                 // SET Verifier TO Intermediate3 BITWISE XOR PasswordByte
@@ -418,7 +418,7 @@ public class CryptoFunctions {
         //Maximum length of the password is 15 chars.
         final int maxPasswordLength = 15; 
         
-        if (!"".equals(password)) {
+        if (!password.isEmpty()) {
             // Truncate the password to 15 characters
             password = password.substring(0, Math.min(password.length(), maxPasswordLength));
 
index 620b168eb0d8324399083db8f28a1bbdfead582a..38f0b533c4b4e738dd69b0f7c010813233ea9f50 100644 (file)
@@ -356,7 +356,7 @@ public class DataSpaceMapUtils {
     }
     
     public static void writeUtf8LPP4(LittleEndianOutput os, String str) {
-        if (str == null || "".equals(str)) {
+        if (str == null || str.isEmpty()) {
             os.writeInt(str == null ? 0 : 4);
             os.writeInt(0);
         } else {
index bec5d898f4eff383d0b4d51b7a9a97702806decb..e93a3a5fb0cf82d3d08a08103d9bc546dfe04186 100644 (file)
@@ -257,7 +257,7 @@ public class DrawTextParagraph implements Drawable {
         fact.fixFonts(graphics);
         StringBuilder text = new StringBuilder();
         AttributedString at = getAttributedString(graphics, text);
-        boolean emptyParagraph = ("".equals(text.toString().trim()));
+        boolean emptyParagraph = text.toString().trim().isEmpty();
 
         AttributedCharacterIterator it = at.getIterator();
         LineBreakMeasurer measurer = new LineBreakMeasurer(it, graphics.getFontRenderContext());
index 348a23e12e6e967e57179ccc123e91e85a9628a8..af2016f807e3fb953d9c07d55b2049b239977f4f 100644 (file)
@@ -54,7 +54,7 @@ public final class Countblank extends Fixed1ArgFunction {
                        return valueEval == BlankEval.instance ||
                                        // see https://support.office.com/en-us/article/COUNTBLANK-function-6a92d772-675c-4bee-b346-24af6bd3ac22
                                        // "Cells with formulas that return "" (empty text) are also counted."
-                                       (valueEval instanceof StringEval && "".equals(((StringEval)valueEval).getStringValue()));
+                                       (valueEval instanceof StringEval && ((StringEval)valueEval).getStringValue().isEmpty());
                }
        };
 }
index e4e7d61f37f3a27221802a489f89a8b263cbfd9f..76834a7ed8d93de27acdba159af03efa62fa11d5 100644 (file)
@@ -150,7 +150,7 @@ public class FractionFormat extends Format {
         }
         
         //if whole part has to go into the numerator
-        if ("".equals(wholePartFormatString)){
+        if (wholePartFormatString.isEmpty()){
             int trueNum = (fract.getDenominator()*(int)wholePart)+fract.getNumerator();
             sb.append(trueNum).append("/").append(fract.getDenominator());
             return sb.toString();
index b9b904d669eda867ace2be92d9ed0f3bae942d0a..32b5acf9abab295bf6bc8b38aba4468d970d6171 100644 (file)
@@ -348,7 +348,7 @@ public class StringUtil {
     * @see <a href="http://www.alanwood.net/demos/symbol.html">Symbol font - Unicode alternatives for Greek and special characters in HTML</a>
     */
    public static String mapMsCodepointString(String string) {
-       if (string == null || "".equals(string)) return string;
+       if (string == null || string.isEmpty()) return string;
        initMsCodepointMap();
        
        StringBuilder sb = new StringBuilder();
index 5c9a02e86e4b63b72cd4dd543b30aa0bd8bb213f..901829d0f2dbb53d12a9b97f852b6b4b5c029d34 100644 (file)
@@ -242,7 +242,7 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
         */
        public static OPCPackage open(String path, PackageAccess access)
                        throws InvalidFormatException, InvalidOperationException {
-               if (path == null || "".equals(path.trim())) {
+               if (path == null || path.trim().isEmpty()) {
                        throw new IllegalArgumentException("'path' must be given");
                }
                
@@ -459,7 +459,7 @@ public abstract class OPCPackage implements RelationshipSource, Closeable {
                try {
                        l.writeLock().lock();
                        if (this.originalPackagePath != null
-                                       && !"".equals(this.originalPackagePath.trim())) {
+                                       && !this.originalPackagePath.trim().isEmpty()) {
                                File targetFile = new File(this.originalPackagePath);
                                if (!targetFile.exists()
                                                || !(this.originalPackagePath
index 099382d3bec9e6da4a771adafec4cff963bedf5d..7658e62590448738408d309adc9668656b9dece5 100644 (file)
@@ -260,7 +260,7 @@ public final class PackagePartName implements Comparable<PackagePartName> {
                                                                + partUri.getPath());
                        }
 
-                       if ("".equals(seg.replaceAll("\\\\.", ""))) {
+                       if (seg.replaceAll("\\\\.", "").isEmpty()) {
                                // Normally will never been invoked with the previous
                                // implementation rule [M1.9]
                                throw new InvalidFormatException(
index 16285615dfc1944abccf5b16892a464d11a9d108..7dcf8b5f9156bfc7be22b22057a68ac18e41180d 100644 (file)
@@ -430,7 +430,7 @@ public final class ZipPackage extends OPCPackage {
         // Flush the package
         flush();
 
-               if (this.originalPackagePath == null || "".equals(this.originalPackagePath)) {
+               if (this.originalPackagePath == null || this.originalPackagePath.isEmpty()) {
                    return;
                }
 
index 07711d2b844770ccb772ab8c1f40e880cf895f42..6a9cf7685324574898e691f72a72d3138f513b25 100644 (file)
@@ -112,7 +112,7 @@ public class OOXMLURIDereferencer implements URIDereferencer, SignatureConfigura
         LOG.log(POILogger.DEBUG, "dereference", uri);
 
         String path = uri.getPath();
-        if (path == null || "".equals(path)) {
+        if (path == null || path.isEmpty()) {
             LOG.log(POILogger.DEBUG, "illegal part name (expected)", uri);
             return null;
         }
index 79c2b496801b6c19ca325c1290bf338639ded1a8..c4d767fac59626d5be53c019319cf186d7aa608e 100644 (file)
@@ -205,7 +205,7 @@ public class RelationshipTransformService extends TransformService {
                 String id = el.getAttribute("Id");
                 if (sourceIds.contains(id)) {
                     String targetMode = el.getAttribute("TargetMode");
-                    if ("".equals(targetMode)) {
+                    if (targetMode.isEmpty()) {
                         el.setAttribute("TargetMode", "Internal");
                     }
                     rsList.put(id, el);
index 5c235267ca3343e7bf98f5220777f700e673c32f..1b2fa0e489a180d092d01faf21cf2d6cc78ebd10 100644 (file)
@@ -138,14 +138,14 @@ public class EmbeddedExtractor implements Iterable<EmbeddedExtractor> {
             String extension = (filename == null || filename.lastIndexOf('.') == -1) ? ".bin" : filename.substring(filename.lastIndexOf('.'));
             
             // try to find an alternative name
-            if (filename == null || "".equals(filename) || filename.startsWith("MBD") || filename.startsWith("Root Entry")) {
+            if (filename == null || filename.isEmpty() || filename.startsWith("MBD") || filename.startsWith("Root Entry")) {
                 filename = shape.getShapeName();
                 if (filename != null) {
                     filename += extension;
                 }
             }
             // default to dummy name
-            if (filename == null || "".equals(filename)) {
+            if (filename == null || filename.isEmpty()) {
                 filename = "picture_" + embeddings.size() + extension;
             }
             filename = filename.trim();
index d5d0d4c6cb6e8e6df14e26b1983f7500f3d69656..e806861d29d7ae14b2f9d2e70b68bcce34952eaa 100644 (file)
@@ -51,7 +51,7 @@ public class XSLFHyperlink implements Hyperlink<XSLFShape,XSLFTextParagraph> {
     @Override
     public String getAddress() {
         String id = _link.getId();
-        if (id == null || "".equals(id)) {
+        if (id == null || id.isEmpty()) {
             return _link.getAction();
         }
 
@@ -163,4 +163,4 @@ public class XSLFHyperlink implements Hyperlink<XSLFShape,XSLFTextParagraph> {
         _link.setId("");
         _link.setAction("ppaction://hlinkshowjump?jump="+jump);
     }
-}
\ No newline at end of file
+}
index ea533600840cbe234585b9b7862bc31be3b79675..a1a3065a1031291e3fb5a6a114565bbdfb2f2afb 100644 (file)
@@ -845,7 +845,7 @@ public class XSLFTextRun implements TextRun {
                 }
                 // SYMBOL is missing
                 
-                if (font == null || !font.isSetTypeface() || "".equals(font.getTypeface())) {
+                if (font == null || !font.isSetTypeface() || font.getTypeface().isEmpty()) {
                     font = coll.getLatin();
                 }
             }
index c0e0fc8551b0e698d86ec112de4fcbaea91d67a7..e60c4fca1882c0f5226f45f262101ec864965d89 100644 (file)
@@ -312,7 +312,7 @@ public class XSSFPivotTable extends POIXMLDocumentPart {
 
         pivotField.setAxis(STAxis.AXIS_COL);
         pivotField.setShowAll(false);
-        if (valueFormat != null && !"".equals(valueFormat.trim())) {
+        if (valueFormat != null && !valueFormat.trim().isEmpty()) {
             DataFormat df = parentSheet.getWorkbook().createDataFormat();
             pivotField.setNumFmtId(df.getFormat(valueFormat));
         }
@@ -440,7 +440,7 @@ public class XSSFPivotTable extends POIXMLDocumentPart {
         cell.setCellType(CellType.STRING);
         dataField.setName(valueFieldName);
         dataField.setFld(columnIndex);
-        if (valueFormat != null && !"".equals(valueFormat.trim())) {
+        if (valueFormat != null && !valueFormat.trim().isEmpty()) {
             DataFormat df = parentSheet.getWorkbook().createDataFormat();
             dataField.setNumFmtId(df.getFormat(valueFormat));
         }
index 46e47f688ddae5724191c867910c514f58c14ea0..471a48c713ded997e5585358b986be859a79e160 100644 (file)
@@ -127,7 +127,7 @@ public final class XSSFPasswordHelper {
     
     
     private static QName getAttrName(String prefix, String name) {
-        if (prefix == null || "".equals(prefix)) {
+        if (prefix == null || prefix.isEmpty()) {
             return new QName(name);
         } else {
             return new QName(prefix+Character.toUpperCase(name.charAt(0))+name.substring(1));
index cf5d74b1808273a03856848ae3aa9b5bdb74e50c..dcdf9c699ade9d0c43b93bfab192e09d455b0c4f 100644 (file)
@@ -230,7 +230,7 @@ public final class PowerPointExtractor extends POIOLE2TextExtractor {
                         if(sh instanceof HSLFTextShape){
                             HSLFTextShape hsh = (HSLFTextShape)sh;
                             final String text = hsh.getText();
-                            if (text == null || "".equals(text) || "*".equals(text)) {
+                            if (text == null || text.isEmpty() || "*".equals(text)) {
                                 continue;
                             }
                             
index f2cc64c5156ce3006900d5a0b86ff48aae2002f4..57c473173a458567afbe7a4d810a667d8689eaca 100644 (file)
@@ -125,7 +125,7 @@ public class HSLFFontInfo implements FontInfo {
 
     @Override
     public void setTypeface(String typeface){
-        if (typeface == null || "".equals(typeface)) {
+        if (typeface == null || typeface.isEmpty()) {
             throw new IllegalArgumentException("typeface can't be null nor empty");
         }
         this.typeface = typeface;
index 6152920ade86931bf6980535a26585948f0295fc..13b2806cd7eaab497349e4b3000ea4587905f857 100644 (file)
@@ -323,7 +323,7 @@ public abstract class HSLFSimpleShape extends HSLFShape implements SimpleShape<H
         }
 
         name = name.replace("adj", "");
-        if ("".equals(name)) {
+        if (name.isEmpty()) {
             name = "1";
         }
 
@@ -727,4 +727,4 @@ public abstract class HSLFSimpleShape extends HSLFShape implements SimpleShape<H
     protected void setHyperlink(HSLFHyperlink link) {
         _hyperlink = link;
     }
-}
\ No newline at end of file
+}