bd2019us created FINERACT-747:
---------------------------------

             Summary: change String.replaceAll() to replace() if the replaced 
one is plain string.
                 Key: FINERACT-747
                 URL: https://issues.apache.org/jira/browse/FINERACT-747
             Project: Apache Fineract
          Issue Type: Bug
            Reporter: bd2019us


Locations: 

(1) 
fineract-provider/src/main/java/org/apache/fineract/accounting/rule/api/AccountingRuleJsonInputParams.java
(2) 
fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/populator/fixeddeposits/

FixedDepositTransactionWorkbookPopulator.java
(3) 
fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/populator/loanrepayment/LoanRepaymentWorkbookPopulator.java
(4) 
fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/populator/savings/SavingsTransactionsWorkbookPopulator.java
(5) 
fineract-provider/src/main/java/org/apache/fineract/portfolio/common/domain/ConditionType.java
(6) 
fineract-provider/src/main/java/org/apache/fineract/portfolio/group/service/CenterReadPlatformServiceImpl.java
(7) 
fineract-provider/src/main/java/org/apache/fineract/portfolio/group/service/GroupReadPlatformServiceImpl.java
(8) 
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/service/LoanReadPlatformServiceImpl.java

 

The String.replaceAll(regex, other) is usually used to replace strings 
represented by regular expression "regex". When the replaced string is a plain 
string but regex form, the String.replace(origin, other) is recommended as it 
does not need to compile and can improve the performance.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to