Control structures should use curly braces - except for single-line statements

  • More a variation of the existing rule (S121) than a new one
  • Allow single-line structures without curly braces.
  • snippet of Noncompliant Code
if (condition)
   DoSomething();
  • snippet of Compilant Code (fixing the above noncompliant code)
if (condition) DoSomething();

preferred.
At the moment this would be required:

if (condition) { DoSomething(); }

or

if (condition)
{
   DoSomething();
}

Hi,

It’s not obvious to me what language this is (other than that it’s C-ish)…?

 
Thx,
Ann

C# in my specific case but really any C-ish language that allows you to have single line statements in this style

1 Like

Hi,

Thanks. I get that the idea is somewhat generic, but new rules ideas are evaluated by language-specific teams (and propagated to other languages as appropriate) & we don’t have a “C-ish” team. :laughing:

I’ve added the csharp tag and the right folks should be along.

 
:smiley:
Ann

Without curly braces control structures deal with single-line statements. So, if that is your preferred way, why not just disable the rule?!

@ganncamp Thanks for tagging the issue - I am still a n00b here :wink:

@Corniel I want these to be valid:

if (condition) DoSomething();
if (condition)
{
    DoSomething();
}
if (condition) 
{
    DoSomething();
    DoSomethingElse();
}

What I do not want to allow is:

if (condition)
    DoSomething();

I thought for this last case I need curly braces control?

No, the last one is - from the compiler’s perspective - equal to if (condition) DoSomething();

Sonar has a rule by the way that allows:

if (condition) DoSomething(); // Compliant

if (condition)
    DoSomething(); // Compliant

if (condition)
DoSomething(); // Noncompliant

E.a.w, if the indenting gives the wrong impression, it will raise a warning. That is Rule S2681, and there is a PR pending to extend the coverage.

@Corniel so basically I should turn off csharpsquid:S121 and I will get closer to what I want.
But

if (condition)
    DoSomething();

will still be allowed, which I don’t really want…because of exactly the problem your PR is trying to address - I don’t want people to use this style because they could accidentally execute unconditional code. I guess with your PR in then it would be okay to have this style since the accidental indentation case will be caught

That might be an option, or you write your own. The thing is, off course, that rules that are more mainly about style (like these ones) are always up for debate. You can’t make everybody happy.

I personally only allow if statements without curly braces once on a single line when reviewing code, but I would also not advise Sonar to change the behavior of any of these rules.