CodeQL documentation

Complex condition

ID: cs/complex-condition
Kind: problem
Security severity: 
Severity: recommendation
Precision: high
   - testability
   - readability
Query suites:
   - csharp-security-and-quality.qls

Click to see the query in the CodeQL repository

Very complex conditions are hard to understand and therefore are a source of a lot of defects.


Complex expressions can often be made easier to read by separating them into different variables or even giving some parts of the condition their own boolean valued method. Using separate methods both reduces code reuse and simplifies the expression.


This example demonstrates some possible conditions and whether or not they are acceptable. As you can see, the length of the condition is not the only thing that contributes to its complexity.

class Complex
    static bool foo(bool a, bool b, bool c, bool d, bool e, bool f, bool g)
        bool x = a || b || c || d || e || f || g; // OK
        bool y = a && b || !(b && c) || !(d && e) && !(f && g); // NOT OK
        bool z = (a && b || (b && c)) && ((d && e) || (f && g)); // NOT OK
        return x && y && z; // OK


  • Robert C. Martin - Clean Code: A Handbook of Agile Software Craftsmanship, §17.G28

  • Steve McConnell - Code Complete: A Practical Handbook of Software Construction

  • © GitHub, Inc.
  • Terms
  • Privacy