Cyclomatic complexity (CyclomaticComplexity)¶
Type | Scope | Severity | Activated by default |
Minutes to fix |
Tags |
---|---|---|---|---|---|
Code smell |
BSL OS |
Critical |
Yes |
25 |
brainoverload |
Parameters¶
Name | Type | Description | Default value |
---|---|---|---|
complexityThreshold |
Integer |
Complexity threshold |
20 |
checkModuleBody |
Boolean |
Check module body |
true |
Description¶
Cyclomatic complexity of the program code is one of the oldest metrics, it was first mentioned by Thomas McCab in 1976.
Cyclomatic complexity shows the minimum number of required tests. The most effective way to reduce cyclomatic complexity is to decompose the code, split the methods into simpler ones, and also optimize logical expressions.
Cyclomatic complexity increases by 1 for each of following constructions
For ... To .. Do
For each ... Of ... Do
If ... Then
ElsIf ... Then
Else
Try ... Except ... EndTry
GoTo ~Label
- Binary operations
AND ... OR
- Ternary operator
Procedure
Function
Examples¶
Function ServerModuleManager(Name) // 1
ObjectFounded = False; // 0
// 0
NameParts = StrSplit(Name, "."); // 0
If NameParts.Count() = 2 Then // 1
// 0
TypeName = Upper(NameParts[0]); // 0
ObjectName = NameParts[1]; // 0
// 0
If TypeName = Upper("Constants") Then // 1
If Metadata.Constants.Find(ObjectName) <> Undefined Then // 1
ObjectFounded = True; // 0
EndIf; // 0
ElsIf TypeName = Upper("InformationRegisters") Then // 1
If Metadata.InformationRegisters.Find(ObjectName) <> Undefined Then // 1
ObjectFounded = True; // 0
EndIf; // 0
Else // 1
ObjectFounded = False; // 0
EndIf; // 0
EndIf; // 0
// 0
If Not ObjectFounded Then // 1
Raise СтроковыеФункцииКлиентСервер.ПодставитьПараметрыВСтроку( // 0
НСтр("ru = 'Объект метаданных ""%1"" не найден, // 0
|либо для него не поддерживается получение модуля менеджера.'"), // 0
Name); // 0
EndIf; // 0
SetSafeMode(True); // 0
Module = Eval(Name); // 0
F = ?(SomeCondition1, True, Undefined); // 1
А = ?(SomeCondition1, True, ?(SomeCondition2, False, Undefined)); // 2
M = True Or 7; // 1
Return Module; // 0
EndFunction // Total 12
Sources¶
Snippets¶
Diagnostic ignorance in code¶
// BSLLS:CyclomaticComplexity-off
// BSLLS:CyclomaticComplexity-on
Parameter for config¶
"CyclomaticComplexity": {
"complexityThreshold": 20,
"checkModuleBody": true
}