Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
Accelerator
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
Uh oh!
There was an error while loading.
Please reload this page
.
github
/
codeql
Public
Notifications
You must be signed in to change notification settings
Fork
2.1k
Star
10k
Code
Issues
997
Pull requests
451
Discussions
Actions
Projects
Security and quality
0
Insights
Additional navigation options
Code
Issues
Pull requests
Discussions
Actions
Projects
Security and quality
Insights
Files
Expand file tree
yoff/python-shared-cfg-dataflow-flip
Breadcrumbs
codeql
/
javascript
/
ql
/
src
/
Metrics
/
FunCyclomaticComplexity.qhelp
Copy path
Blame
More file actions
Blame
More file actions
Latest commit
History
History
History
42 lines (35 loc) · 1.75 KB
yoff/python-shared-cfg-dataflow-flip
Breadcrumbs
codeql
/
javascript
/
ql
/
src
/
Metrics
/
FunCyclomaticComplexity.qhelp
Copy path
Top
File metadata and controls
Code
Blame
42 lines (35 loc) · 1.75 KB
Raw
Copy raw file
Download raw file
Open symbols panel
Edit and raw actions
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
<!
DOCTYPE
qhelp
PUBLIC
"-//Semmle//qhelp//EN"
"qhelp.dtd">
<
qhelp
>
<
overview
>
<
p
>This metric measures the cyclomatic complexity of each function in the project.</
p
>
<
p
>The cyclomatic complexity of a function is an indication of the number of paths that can be taken during
the execution of a function. Code with many branches and loops has high cyclomatic complexity. A cyclomatic
complexity above 50 should be considered bad practice and above 75 should definitely be addressed.</
p
>
<
p
>Functions with high cyclomatic complexity are</
p
>
<
ul
>
<
li
>difficult to test since tests should be provided for each possible execution path;</
li
>
<
li
>difficult to understand since a developer needs to understand how all conditions interact;</
li
>
<
li
>difficult to maintain since many execution paths is an indication of functions that perform too many tasks.</
li
>
</
ul
>
</
overview
>
<
recommendation
>
<
p
>The primary way to reduce the complexity is to extract sub-functionality into separate functions.
This improves on all problems described above. If the function naturally breaks up into a sequence of operations it
is preferable to extract each operation as a separate function. Even if that's not the case it is often possible
to extract the body of an iteration into a separate function to reduce complexity. If the complexity can't be
reduced significantly make sure that the function is properly documented and carefully tested.</
p
>
</
recommendation
>
<
references
>
<
li
>
M. Fowler. <
em
>Refactoring</
em
>. Addison-Wesley, 1999.
</
li
>
<
li
>
T. J. McCabe. <
em
>A Complexity Measure</
em
>. IEEE Transactions on Software Engineering, SE-2(4), December 1976.
</
li
>
<
li
>
Wikipedia: <
a
href
=
"
http://en.wikipedia.org/wiki/Cyclomatic_complexity
"
>Cyclomatic complexity</
a
>.
</
li
>
</
references
>
</
qhelp
>
You can’t perform that action at this time.