[LANG-1679] StringUtils.unwrap() may be implemented faster

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.13.0
• Assignee: Unassigned
• Reporter: clover
• Created: 30/Dec/21 15:45
• Updated: 03/Apr/22 14:20
• Resolved: 03/Apr/22 14:20

Description

public static String unwrap(final String str, final String wrapToken) {
        if (isEmpty(str) || isEmpty(wrapToken) || str.length() < 2 * wrapToken.length())

{             return str;         }

        if (startsWith(str, wrapToken) && endsWith(str, wrapToken)) {

            // When we come here, startIndex must be 0. Thus, startIndex is redundant.
            final int startIndex = str.indexOf(wrapToken);
            final int endIndex = str.lastIndexOf(wrapToken);
            final int wrapLength = wrapToken.length();

           // (startIndex != -1 && endIndex != -1) is always true

            if (startIndex != -1 && endIndex != -1) {
                return str.substring(startIndex + wrapLength, endIndex);
            }
        }

        return str;
    }

 

As commented above, unwrap() could be improved:

public static String unwrap(final String str, final String wrapToken)

{         if (isEmpty(str) || isEmpty(wrapToken) || str.length() < 2 * wrapToken.length()) \{             return str;         }

        if (startsWith(str, wrapToken) && endsWith(str, wrapToken)) {
                return str.substring(wrapToken.length(), str.lastIndexOf(wrapToken));
            }
        }

        return str;
    }