qodana cleaning #803
1236 new problems found by Qodana for .NET
Qodana for .NET
1236 new problems were found
Inspection name | Severity | Problems |
---|---|---|
Redundant type arguments of method |
🔶 Warning | 111 |
Assignment is not used |
🔶 Warning | 42 |
Namespace does not correspond to file location |
🔶 Warning | 33 |
Unused local variable |
🔶 Warning | 33 |
Expression is always 'true' or always 'false' |
🔶 Warning | 15 |
Possible 'System.NullReferenceException' |
🔶 Warning | 14 |
Auto-property accessor is never used (non-private accessibility) |
🔶 Warning | 14 |
Redundant member initializer |
🔶 Warning | 13 |
Virtual member call in constructor |
🔶 Warning | 9 |
Invalid XML documentation comment |
🔶 Warning | 7 |
Redundant using directive |
🔶 Warning | 7 |
Possible 'null' assignment to non-nullable entity |
🔶 Warning | 5 |
Non-readonly type member referenced in 'GetHashCode()' |
🔶 Warning | 5 |
Redundant 'object.ToString()' call |
🔶 Warning | 5 |
Unused parameter (private accessibility) |
🔶 Warning | 4 |
Collection content is never queried (private accessibility) |
🔶 Warning | 3 |
Unused type parameter |
🔶 Warning | 3 |
Co-variant array conversion |
🔶 Warning | 2 |
Conditional access qualifier expression is known to be null or not null |
🔶 Warning | 2 |
Member initialized value ignored |
🔶 Warning | 2 |
Possible multiple enumeration |
🔶 Warning | 2 |
Redundant cast |
🔶 Warning | 2 |
Dictionary lookup can be simplified with 'TryAdd' |
🔶 Warning | 1 |
Empty constructor |
🔶 Warning | 1 |
Expression is always 'null' |
🔶 Warning | 1 |
Local variable hides member |
🔶 Warning | 1 |
Non-accessed local variable |
🔶 Warning | 1 |
Mismatch of optional parameter value in overridden method |
🔶 Warning | 1 |
Redundant argument with default value |
🔶 Warning | 1 |
Redundant explicit type in array creation |
🔶 Warning | 1 |
Redundant control flow jump statement |
🔶 Warning | 1 |
Redundant lambda expression parameter type specification |
🔶 Warning | 1 |
Useless arithmetical operation |
🔶 Warning | 1 |
Type member is never used (non-private accessibility) |
◽️ Notice | 112 |
Member can be made private (non-private accessibility) |
◽️ Notice | 104 |
Use preferred 'var' style (for built-in types) |
◽️ Notice | 90 |
Invert 'if' statement to reduce nesting |
◽️ Notice | 87 |
Auto-property can be made get-only (non-private accessibility) |
◽️ Notice | 58 |
Use object or collection initializer when possible |
◽️ Notice | 43 |
Unused parameter (non-private accessibility) |
◽️ Notice | 34 |
Use preferred 'var' style (elsewhere) |
◽️ Notice | 25 |
Redundant 'else' keyword |
◽️ Notice | 21 |
Use preferred 'var' style (when type is simple) |
◽️ Notice | 21 |
Use preferred style of default value expression when type is not evident |
◽️ Notice | 19 |
Type is never used (non-private accessibility) |
◽️ Notice | 19 |
'if-return' statement can be rewritten as 'return' statement |
◽️ Notice | 17 |
Member can be made protected (non-private accessibility) |
◽️ Notice | 15 |
'if' statement can be rewritten as '?:' expression |
◽️ Notice | 14 |
Member can be made static (shared) (private accessibility) |
◽️ Notice | 14 |
Method return value is never used (non-private accessibility) |
◽️ Notice | 14 |
Convert delegate variable into local function |
◽️ Notice | 13 |
Field can be made readonly (non-private accessibility) |
◽️ Notice | 11 |
Part of foreach loop can be converted into LINQ-expression but another 'GetEnumerator' method will be used |
◽️ Notice | 9 |
Inline 'out' variable declaration |
◽️ Notice | 8 |
Parameter can be declared with the base type |
◽️ Notice | 8 |
Add/remove 'this.' qualifier |
◽️ Notice | 7 |
Class is never instantiated (non-private accessibility) |
◽️ Notice | 7 |
Local variable has too wide declaration scope |
◽️ Notice | 7 |
Type member is never accessed via base type (non-private accessibility) |
◽️ Notice | 7 |
Field can be made readonly (private accessibility) |
◽️ Notice | 6 |
Foreach loop can be converted into LINQ-expression but another 'GetEnumerator' method will be used |
◽️ Notice | 6 |
Convert constructor into member initializers |
◽️ Notice | 5 |
Inline temporary variable |
◽️ Notice | 5 |
Parameter type can be IEnumerable<T> (private accessibility) |
◽️ Notice | 5 |
Some values of the enum are not processed inside 'switch' statement and are handled via default section |
◽️ Notice | 5 |
Virtual (overridable) member is never overridden (non-private accessibility) |
◽️ Notice | 5 |
Convert 'if' statement into 'switch' |
◽️ Notice | 4 |
Non-accessed field (non-private accessibility) |
◽️ Notice | 4 |
Redundant empty argument list on object creation expression |
◽️ Notice | 4 |
Class with virtual (overridable) members never inherited (non-private accessibility) |
◽️ Notice | 3 |
Convert local variable or field into constant (private accessibility) |
◽️ Notice | 3 |
Merge conditional ?: expression into conditional access |
◽️ Notice | 3 |
Parameter type can be IEnumerable<T> (non-private accessibility) |
◽️ Notice | 3 |
Tail recursive call can be replaced with loop |
◽️ Notice | 3 |
Remove redundant parentheses |
◽️ Notice | 2 |
Auto-property can be made get-only (private accessibility) |
◽️ Notice | 2 |
Incorrect indent (around statement braces) |
◽️ Notice | 2 |
Convert lambda expression into method group |
◽️ Notice | 2 |
For-loop can be converted into foreach-loop |
◽️ Notice | 2 |
Loop can be converted into LINQ-expression |
◽️ Notice | 2 |
Member can be made static (shared) (non-private accessibility) |
◽️ Notice | 2 |
Redundant [AttributeUsage] attribute property assignment |
◽️ Notice | 2 |
Redundant explicit array creation in argument of 'params' parameter |
◽️ Notice | 2 |
Redundant lambda signature parentheses |
◽️ Notice | 2 |
Return type can be IEnumerable<T> (non-private accessibility) |
◽️ Notice | 2 |
Simplify LINQ expression (use 'All') |
◽️ Notice | 2 |
Replace if statement with null-propagating code |
◽️ Notice | 2 |
Literal length can be reduced by using verbatim string |
◽️ Notice | 2 |
Use preferred body style (convert into property, indexer, or event with preferred body style) |
◽️ Notice | 1 |
Use preferred style for trailing comma before new line in multiline lists |
◽️ Notice | 1 |
Replace built-in type reference with a CLR type name or a keyword |
◽️ Notice | 1 |
Use compound assignment |
◽️ Notice | 1 |
Convert into lambda expression |
◽️ Notice | 1 |
Introduce optional parameters (non-private accessibility) |
◽️ Notice | 1 |
Join local variable declaration and assignment |
◽️ Notice | 1 |
Merge sequential checks into single conditional access check |
◽️ Notice | 1 |
Redundant string interpolation |
◽️ Notice | 1 |
Replace auto-property with computed property |
◽️ Notice | 1 |
Replace with single call to Any(..) |
◽️ Notice | 1 |
Some values of the enum are not processed inside 'switch' statement |
◽️ Notice | 1 |
Unassigned field (non-private accessibility) |
◽️ Notice | 1 |
Use collection's count property |
◽️ Notice | 1 |
Use deconstruction |
◽️ Notice | 1 |
Use string interpolation expression |
◽️ Notice | 1 |
💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report
Contact Qodana team
Contact us at [email protected]
- Or via our issue tracker: https://jb.gg/qodana-issue
- Or share your feedback: https://jb.gg/qodana-discussions
Details
This result was published with Qodana GitHub Action
Annotations
Check notice on line 55 in src/sly/lexer/TokenChannels.cs
github-actions / Qodana for .NET
Use preferred body style (convert into property, indexer, or event with preferred body style)
Inconsistent body style: use expression-bodied indexer
Check notice on line 211 in src/sly/lexer/GenericLexer.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 643 in src/sly/lexer/LexerBuilder.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 162 in src/sly/lexer/GenericLexer.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 102 in src/sly/parser/generator/visitor/GraphVizEBNFSyntaxTreeVisitor.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 22 in src/sly/parser/syntax/grammar/TerminalClause.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 650 in src/sly/lexer/LexerBuilder.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 142 in src/sly/lexer/fsm/FSMLexer.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 312 in src/sly/lexer/fsm/FSMLexer.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 51 in src/sly/parser/generator/visitor/GraphVizEBNFSyntaxTreeVisitor.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 36 in src/sly/parser/parser/Group.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 286 in src/sly/lexer/fsm/FSMLexer.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 636 in src/sly/lexer/LexerBuilder.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 326 in src/sly/lexer/fsm/FSMLexer.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 68 in src/sly/parser/generator/visitor/ConcreteSyntaxTreeWalker.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 455 in src/sly/lexer/fsm/FSMLexer.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 22 in src/sly/EnumConverter.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 24 in src/sly/lexer/fsm/FSMLexerBuilder.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 108 in src/sly/parser/syntax/grammar/TerminalClause.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 54 in src/sly/parser/generator/visitor/ConcreteSyntaxTreeWalker.cs
github-actions / Qodana for .NET
Use preferred style of default value expression when type is not evident
Redundant type specification
Check notice on line 540 in src/sly/lexer/GenericLexer.cs
github-actions / Qodana for .NET
Remove redundant parentheses
Redundant parentheses
Check notice on line 58 in src/sly/parser/syntax/grammar/OptionClause.cs
github-actions / Qodana for .NET
Remove redundant parentheses
Redundant parentheses
Check notice on line 252 in src/sly/lexer/fsm/FSMLexer.cs
github-actions / Qodana for .NET
Add/remove 'this.' qualifier
Qualifier 'this.' is redundant
Check notice on line 74 in src/sly/parser/syntax/grammar/LeadingToken.cs
github-actions / Qodana for .NET
Add/remove 'this.' qualifier
Qualifier 'this.' is redundant
Check notice on line 172 in src/sly/parser/parser/llparser/RecursiveDescentSyntaxParserStarter.cs
github-actions / Qodana for .NET
Add/remove 'this.' qualifier
Qualifier 'this.' is redundant