Closed strubium closed 4 hours ago
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
21603 new problems were found
Inspection name | Severity | Problems |
---|---|---|
Unchecked warning |
🔶 Warning | 10614 |
Unnecessary 'this' qualifier |
🔶 Warning | 4012 |
Deprecated API usage |
🔶 Warning | 1510 |
Auto-boxing |
🔶 Warning | 886 |
@NotNull/@Nullable problems |
🔶 Warning | 417 |
Statement with empty body |
🔶 Warning | 361 |
Field can be local |
🔶 Warning | 313 |
Lombok @Getter may be used |
🔶 Warning | 276 |
Nullability and data flow problems |
🔶 Warning | 183 |
Auto-unboxing |
🔶 Warning | 144 |
Unused assignment |
🔶 Warning | 126 |
Non-constant field with upper-case name |
🔶 Warning | 114 |
Constant values |
🔶 Warning | 108 |
Redundant local variable |
🔶 Warning | 101 |
Javadoc declaration problems |
🔶 Warning | 95 |
Lombok @Setter may be used |
🔶 Warning | 89 |
Questionable name |
🔶 Warning | 76 |
Possibly lossy implicit cast in compound assignment |
🔶 Warning | 46 |
Call to 'printStackTrace()' |
🔶 Warning | 36 |
Declaration has problems in Javadoc references |
🔶 Warning | 24 |
Result of method call ignored |
🔶 Warning | 18 |
Deprecated member is still used |
🔶 Warning | 14 |
'size() == 0' can be replaced with 'isEmpty()' |
🔶 Warning | 14 |
Dynamic regular expression can be replaced by compiled 'Pattern' |
🔶 Warning | 13 |
Link specified as plain text |
🔶 Warning | 13 |
Method is identical to its super method |
🔶 Warning | 13 |
Suspicious collection method call |
🔶 Warning | 13 |
'Set' can be replaced with 'EnumSet' |
🔶 Warning | 11 |
Integer division in floating-point context |
🔶 Warning | 9 |
Mismatched query and update of collection |
🔶 Warning | 9 |
Unused import |
🔶 Warning | 9 |
'Map' can be replaced with 'EnumMap' |
🔶 Warning | 8 |
String concatenation as argument to 'StringBuilder.append()' call |
🔶 Warning | 8 |
Class is exposed outside of its visibility scope |
🔶 Warning | 7 |
Condition is covered by further condition |
🔶 Warning | 7 |
Octal integer |
🔶 Warning | 7 |
Suspicious variable/parameter name combination |
🔶 Warning | 7 |
Vulnerable imported dependency |
🔶 Warning | 7 |
Functional expression can be folded |
🔶 Warning | 6 |
Copy of existing static method body |
🔶 Warning | 6 |
Unnecessary unary minus |
🔶 Warning | 6 |
Unstable API Usage |
🔶 Warning | 6 |
Bulk operation can be used instead of iteration |
🔶 Warning | 6 |
Catch block may ignore exception |
🔶 Warning | 5 |
Redundant type cast |
🔶 Warning | 5 |
Variable is assigned to itself |
🔶 Warning | 5 |
Stream API call chain can be simplified |
🔶 Warning | 5 |
Dangling Javadoc comment |
🔶 Warning | 4 |
Duplicate condition |
🔶 Warning | 4 |
Inner class may be 'static' |
🔶 Warning | 4 |
Manual min/max calculation |
🔶 Warning | 4 |
AutoCloseable used without 'try'-with-resources |
🔶 Warning | 3 |
Too weak variable type leads to unnecessary cast |
🔶 Warning | 3 |
Mismatched read and write of array |
🔶 Warning | 3 |
Value passed as parameter never read |
🔶 Warning | 3 |
String concatenation in loop |
🔶 Warning | 3 |
Local variable used and declared in different 'switch' branches |
🔶 Warning | 3 |
Unnecessary 'continue' statement |
🔶 Warning | 3 |
'default' not last case in 'switch' |
🔶 Warning | 2 |
Call math rounding with 'int' argument |
🔶 Warning | 2 |
Redundant operation on empty container |
🔶 Warning | 2 |
Redundant usage of unmodifiable collection wrappers |
🔶 Warning | 2 |
Method does not call super method |
🔶 Warning | 2 |
Caught exception is immediately rethrown |
🔶 Warning | 1 |
Redundant 'Collection.addAll()' call |
🔶 Warning | 1 |
'static' method declared 'final' |
🔶 Warning | 1 |
Call to 'toString()' on array |
🔶 Warning | 1 |
Method name same as class name |
🔶 Warning | 1 |
String comparison using '==', instead of 'equals()' |
🔶 Warning | 1 |
'String.equals()' can be replaced with 'String.isEmpty()' |
🔶 Warning | 1 |
Minimum 'switch' branches |
🔶 Warning | 1 |
Unary plus |
🔶 Warning | 1 |
Commented out code |
◽️ Notice | 1004 |
Duplicated code fragment |
◽️ Notice | 676 |
Non-distinguishable logging calls |
◽️ Notice | 33 |
Non-constant string concatenation as argument to logging call |
◽️ Notice | 17 |
'if' statement with identical branches or common parts |
◽️ Notice | 15 |
Constant values |
◽️ Notice | 14 |
Method can be extracted |
◽️ Notice | 12 |
Multiple occurrences of the same expression |
◽️ Notice | 4 |
Duplicate branches in 'switch' |
◽️ Notice | 3 |
Non-strict inequality '>=' or '<=' can be replaced with '==' |
◽️ Notice | 1 |
☁️ View the detailed Qodana report
📝 Description
Fixes the swapping images in the changelog
🎯 Goals
❌ Non Goals
🖼️ Screenshots/Recordings
Before:
After:
📖 Added to documentation?