MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-05-01 02:38:59 +07:00
Code Issues Packages Projects Releases Wiki Activity

IDEA-217572 Inspection to replace String.toLowerCase().equals() with String.equalsIgnoreCase()

Browse Source 
Contributed by pyltsin <pyltsin-m@yandex.ru> (PR#1398)

GitOrigin-RevId: 8f8582ce822deb0dadb6ba89834e39f3b5db3aa6
This commit is contained in:
pyltsin
2020-07-06 12:49:10 +07:00
committed by intellij-monorepo-bot
parent 2d6d8cb12f
commit d808337407
4 changed files with 305 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

78
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantStringOperation/afterChangeCase.java Normal file
View File

@@ -0,0 +1,78 @@
// "Fix all 'Redundant String operation' problems in file" "true"
import java.util.Locale;
public class Main {
public static void main(String[] args) {
boolean value = args[0].equalsIgnoreCase("foo");
boolean value2 ="foo".equalsIgnoreCase(args[0]) ;
boolean value3 = args[0].toLowerCase().equalsIgnoreCase(args[0].toLowerCase());
boolean value1WithBrackets = (args[0]).equalsIgnoreCase(("foo"));
boolean value2WithBrackets = ("foo").equalsIgnoreCase((args[0]));
boolean value3WithBrackets = (args[0].toLowerCase()).equalsIgnoreCase((args[0].toLowerCase()));
boolean valueUpperCase1 = args[0].equalsIgnoreCase("foo");
boolean valueUpperCase2 ="foo".equalsIgnoreCase(args[0]) ;
boolean valueUpperCase3 = args[0].equalsIgnoreCase(args[0]);
boolean valueIgnoreCase = args[0].equalsIgnoreCase("foo");
boolean valueIgnoreCase2 ="foo".equalsIgnoreCase(args[0]) ;
boolean valueIgnoreCase3 = args[0].toLowerCase().equalsIgnoreCase(args[0].toLowerCase());
boolean valueIgnoreCaseUpperCase1 = args[0].equalsIgnoreCase("foo");
boolean valueIgnoreCaseUpperCase2 ="foo".equalsIgnoreCase(args[0]) ;
boolean valueIgnoreCaseUpperCase3 = args[0].equalsIgnoreCase(args[0]);
boolean nonChangeValue1 = args[0].toLowerCase(Locale.ROOT).equalsIgnoreCase("foo");
boolean nonChangeValue2 = args[0].toLowerCase(Locale.ENGLISH).equalsIgnoreCase("foo");
boolean nonChangeValue3 ="foo".equalsIgnoreCase(args[0].toUpperCase().toLowerCase(Locale.ENGLISH)) ;
boolean nonChangeValue4 = args[0].toLowerCase().toLowerCase(Locale.ENGLISH).equalsIgnoreCase(args[0].toLowerCase().toLowerCase());
boolean nonChangeValue5 = args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toLowerCase(Locale.ENGLISH));
boolean nonChangeValue6 = args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toUpperCase());
boolean nonChangeValue7 = args[0].toLowerCase(Locale.ROOT).equalsIgnoreCase("foo");
boolean nonChangeValue8 = args[0].toLowerCase(Locale.ENGLISH).equalsIgnoreCase("foo");
boolean nonChangeValue9 ="foo".equalsIgnoreCase(args[0].toUpperCase().toLowerCase(Locale.ENGLISH)) ;
boolean nonChangeValue10 = args[0].toLowerCase().toLowerCase(Locale.ENGLISH).equalsIgnoreCase(args[0].toLowerCase().toLowerCase());
boolean nonChangeValue11 = args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toLowerCase(Locale.ENGLISH));
boolean nonChangeValue12 = args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toUpperCase());
if(args[0].equalsIgnoreCase("foo")){}
if("foo".equalsIgnoreCase(args[0].toUpperCase())){}
if( args[0].toLowerCase().equalsIgnoreCase(args[0].toLowerCase())){}
if(!args[0].equalsIgnoreCase("foo")){}
if(!"foo".equalsIgnoreCase(args[0].toUpperCase())){}
if(! args[0].toLowerCase().equalsIgnoreCase(args[0].toLowerCase())){}
if(!!args[0].equalsIgnoreCase("foo")){}
if(!!"foo".equalsIgnoreCase(args[0].toUpperCase())){}
if(!! args[0].toLowerCase().equalsIgnoreCase(args[0].toLowerCase())){}
if(args[0].equalsIgnoreCase("foo")){}
if("foo".equalsIgnoreCase(args[0].toUpperCase())){}
if( args[0].toUpperCase().equalsIgnoreCase(args[0].toLowerCase())){}
if(args[0].equalsIgnoreCase("foo")){}
if("foo".equalsIgnoreCase(args[0].toUpperCase())){}
if( args[0].toLowerCase().equalsIgnoreCase(args[0].toLowerCase())){}
if(args[0].equalsIgnoreCase("foo")){}
if("foo".equalsIgnoreCase(args[0].toUpperCase())){}
if( args[0].toUpperCase().equalsIgnoreCase(args[0].toLowerCase())){}
/* five */
if(/* one */args/* two */[/* three */0/* four */]/* six */./* seven */equalsIgnoreCase(/* eight */"foo" /* nine */)){}
/* eight */
/* nine */
if(/* one */"foo"/* two */./* three */equalsIgnoreCase(/* four */args/* five */[/* six */0/* seven */]/* ten */)){}
/* twelve */
/* thirteen */
/* five */
if(/* one */args/* two */[/* three */0/* four */]/* six */./* seven */equalsIgnoreCase(/* eight */args/* nine */[/* ten */1/* eleven */]/* fourteen */)){}
/* five */
if(!/* one */args/* two */[/* three */0/* four */]/* six */./* seven */equalsIgnoreCase(/* eight */"foo" /* nine */)){}
}
}

71
java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/redundantStringOperation/beforeChangeCase.java Normal file
View File

@@ -0,0 +1,71 @@
// "Fix all 'Redundant String operation' problems in file" "true"
import java.util.Locale;
public class Main {
public static void main(String[] args) {
boolean value = args[0].toLowe<caret>rCase().equals("foo");
boolean value2 ="foo".equals(args[0].toLowerCase()) ;
boolean value3 = args[0].toLowerCase().toLowerCase().equals(args[0].toLowerCase().toLowerCase());
boolean value1WithBrackets = (args[0]).toLowerCase().equals(("foo"));
boolean value2WithBrackets = ("foo").equals((args[0].toLowerCase()));
boolean value3WithBrackets = (args[0].toLowerCase().toLowerCase()).equals((args[0].toLowerCase().toLowerCase()));
boolean valueUpperCase1 = args[0].toUpperCase().equals("foo");
boolean valueUpperCase2 ="foo".equals(args[0].toUpperCase()) ;
boolean valueUpperCase3 = args[0].toUpperCase().equals(args[0].toUpperCase());
boolean valueIgnoreCase = args[0].toLowerCase().equalsIgnoreCase("foo");
boolean valueIgnoreCase2 ="foo".equalsIgnoreCase(args[0].toLowerCase()) ;
boolean valueIgnoreCase3 = args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toLowerCase());
boolean valueIgnoreCaseUpperCase1 = args[0].toUpperCase().equalsIgnoreCase("foo");
boolean valueIgnoreCaseUpperCase2 ="foo".equalsIgnoreCase(args[0].toUpperCase()) ;
boolean valueIgnoreCaseUpperCase3 = args[0].toUpperCase().equalsIgnoreCase(args[0].toUpperCase());
boolean nonChangeValue1 = args[0].toLowerCase(Locale.ROOT).equalsIgnoreCase("foo");
boolean nonChangeValue2 = args[0].toLowerCase(Locale.ENGLISH).equalsIgnoreCase("foo");
boolean nonChangeValue3 ="foo".equalsIgnoreCase(args[0].toUpperCase().toLowerCase(Locale.ENGLISH)) ;
boolean nonChangeValue4 = args[0].toLowerCase().toLowerCase(Locale.ENGLISH).equalsIgnoreCase(args[0].toLowerCase().toLowerCase());
boolean nonChangeValue5 = args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toLowerCase(Locale.ENGLISH));
boolean nonChangeValue6 = args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toUpperCase());
boolean nonChangeValue7 = args[0].toLowerCase(Locale.ROOT).equalsIgnoreCase("foo");
boolean nonChangeValue8 = args[0].toLowerCase(Locale.ENGLISH).equalsIgnoreCase("foo");
boolean nonChangeValue9 ="foo".equalsIgnoreCase(args[0].toUpperCase().toLowerCase(Locale.ENGLISH)) ;
boolean nonChangeValue10 = args[0].toLowerCase().toLowerCase(Locale.ENGLISH).equalsIgnoreCase(args[0].toLowerCase().toLowerCase());
boolean nonChangeValue11 = args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toLowerCase(Locale.ENGLISH));
boolean nonChangeValue12 = args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toUpperCase());
if(args[0].toLowerCase().equals("foo")){}
if("foo".equals(args[0].toUpperCase().toLowerCase())){}
if( args[0].toLowerCase().toLowerCase().equals(args[0].toLowerCase().toLowerCase())){}
if(!args[0].toLowerCase().equals("foo")){}
if(!"foo".equals(args[0].toUpperCase().toLowerCase())){}
if(! args[0].toLowerCase().toLowerCase().equals(args[0].toLowerCase().toLowerCase())){}
if(!!args[0].toLowerCase().equals("foo")){}
if(!!"foo".equals(args[0].toUpperCase().toLowerCase())){}
if(!! args[0].toLowerCase().toLowerCase().equals(args[0].toLowerCase().toLowerCase())){}
if(args[0].toUpperCase().equals("foo")){}
if("foo".equals(args[0].toUpperCase().toUpperCase())){}
if( args[0].toUpperCase().toUpperCase().equals(args[0].toLowerCase().toUpperCase())){}
if(args[0].toLowerCase().equalsIgnoreCase("foo")){}
if("foo".equalsIgnoreCase(args[0].toUpperCase().toLowerCase())){}
if( args[0].toLowerCase().toLowerCase().equalsIgnoreCase(args[0].toLowerCase().toLowerCase())){}
if(args[0].toUpperCase().equalsIgnoreCase("foo")){}
if("foo".equalsIgnoreCase(args[0].toUpperCase().toUpperCase())){}
if( args[0].toUpperCase().toUpperCase().equalsIgnoreCase(args[0].toLowerCase().toUpperCase())){}
if(/* one */args/* two */[/* three */0/* four */]./* five */toLowerCase()/* six */./* seven */equals(/* eight */"foo" /* nine */)){}
if(/* one */"foo"/* two */./* three */equals(/* four */args/* five */[/* six */0/* seven */]/* eight */./* nine */toLowerCase()/* ten */)){}
if(/* one */args/* two */[/* three */0/* four */]./* five */toLowerCase()/* six */./* seven */equals(/* eight */args/* nine */[/* ten */1/* eleven */]/* twelve */./* thirteen */toLowerCase()/* fourteen */)){}
if(!/* one */args/* two */[/* three */0/* four */]./* five */toLowerCase()/* six */./* seven */equals(/* eight */"foo" /* nine */)){}
}
}