Add some long methods #56
Conversation
![codescene-staging-delta-analysis[bot]](https://avatars.githubusercontent.com/in/52858?s=60&v=4){kind=small}
There was a problem hiding this comment.
❌ Quality Gates: FAILED
- Declining Code Health: 3 findings(s) 🚩
- Improving Code Health: 0 findings(s) ✅
- Affected Hotspots: 0 files(s) 🔥
Recommended Review Level: Detailed -- Inspect the code that degrades in code health.
View detailed results in CodeScene
🚩 Negative Code Health Impact (highest to lowest):
- Large Method c-example.c: a_long_fn
- Large Method c-example.c: another_long_fn
- Code Duplication c-example.c
| if (a > b) { | ||
| do_e(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| } | ||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
|
|
||
|
|
||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
|
|
||
|
|
||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
|
|
||
| return 0; | ||
| } |
There was a problem hiding this comment.
❌ New issue: Code Duplication
The module contains 2 functions with similar structure: a_long_fn,another_long_fn
Why does this problem occur?
Avoid duplicated, aka copy-pasted, code inside the module. More duplication lowers the code health. Read more.
To ignore this warning click here.
| if (a > b) { | ||
| do_e(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| } | ||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
|
|
||
|
|
||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
|
|
||
|
|
||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
|
|
||
| return 0; | ||
| } |
There was a problem hiding this comment.
❌ New issue: Large Method
a_long_fn has 133 lines, threshold = 70
Why does this problem occur?
Large functions with many lines of code are generally harder to understand and lower the code health. Avoid adding more lines to this function. Read more.
To ignore this warning click here.
| if (a > b) { | ||
| do_e(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| do_d(); | ||
| } | ||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
|
|
||
|
|
||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
|
|
||
| do_a(); | ||
| do_a(); | ||
| do_a(); | ||
|
|
||
|
|
||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
| do_b(); | ||
|
|
||
|
|
||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
| do_c(); | ||
|
|
||
| return 0; | ||
| } |
There was a problem hiding this comment.
❌ New issue: Large Method
another_long_fn has 133 lines, threshold = 70
Why does this problem occur?
Large functions with many lines of code are generally harder to understand and lower the code health. Avoid adding more lines to this function. Read more.
To ignore this warning click here.
Testing Large methods...