sonar-dotnet

sonar-dotnet copied to clipboard

NET-1997 New rule for checking an enum flag is set verifies enum value

2

### Description Where an attempt is made to check if an enum value has a flag set, then SonarC# should verify that this can be done correctly. This issue suggests...

alexangas

NET-1746 New Rule Idea: Multiple loops over the same set should be combined (S3047)

1

RSPEC draft of [S3047](https://sonarsource.github.io/rspec/#/rspec/S3047/csharp) was deleted for C# and VB.NET as non-implemented. The rule is implemented for Java only.

antonioaversa

NET-1747 New Rule Idea: Functions returns should not be invariant (S3516)

1

RSPEC draft of [S3516](https://sonarsource.github.io/rspec/#/rspec/S2752/csharp) was deleted as non-implemented. The rule is already implemented for Python, Java, and JS, in the context of https://sonarsource.atlassian.net/browse/MMF-1813, and makes sense for both C# and...

antonioaversa

NET-1956 Fix S4275 FP: with private setter and collection deserialization

4

### Description This scenario should be valid for deserialization purposes. Coming from the community - https://community.sonarsource.com/t/s4275-getters-and-setters-should-access-the-expected-fields/24581/3 ``` public class ReproDeserializationScenario { private readonly List _data = new List(); public List...

andrei-epure-sonarsource

NET-1966 New Rule Idea: Classes should not have too many methods (S1448)

1

Requested by community: https://community.sonarsource.com/t/c-rule-request-classes-should-not-have-too-many-methods/17978

costin-zaharia-sonarsource

NET-1996 Fix S3900 FN: Constructor initializer

2

### Description The rule incorrectly suggests to refactor constructor by adding argument checks for arguments that are dereferenced in the this/base initializer. We should consider changing the rule message to...

valhristov

NET-1745 New Rule Idea: Only the elements of control have to be in the test block of the instruction for (S1247)

1

RSPEC draft of [S1247](https://sonarsource.github.io/rspec/#/rspec/S1247/csharp) was deleted for C# and VB.NET as non-implemented. ```cs for(int i = 0, sum = 0; i < array.Length; i++) //Noncompliant { sum += worth[i]; }...

cristian-ambrosini-sonarsource

NET-1931 Improve S2234: Detect nullable values

2

### Description False negative with nullable argument value. ### Repro steps This code doesn't throw by rule, but rule has been violated. ```csharp void NotNullableParamVoid(int a, int b) { //...

H4pH

NET-1970 Fix S1172 FN: Rule should handle all accessibilities

2

### Description Currently, the rule will report an issue only on effectively `private` methods. The idea of this improvement is to report on methods whatever their accessibility. ### TODO *...

Evangelink

NET-1744 New Rule Idea: Comments should not be located at the end of lines of code (S139)

1

RSPEC draft of [S139](https://sonarsource.github.io/rspec/#/rspec/S139/csharp) was deleted for C# as non-implemented (the rule is already implemented for VB6 and VBNET). **Noncompliant** int a1 = b + c; // This is a...

cristian-ambrosini-sonarsource