-
Notifications
You must be signed in to change notification settings - Fork 4.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Improvement] Refactoring Code For Better Comprehension #15599
Comments
Is it only for a specific package, or all of them? |
I can't agree more with your advice ,really need refactoring code. |
We could do it iteratively, with priority. |
@EricGao888 seems like i do not have write access to this repo. Due to this I am unable to push my feature branch for this issue. Could you help here |
Hi @akshitk20 , you don't need direct write access to this repo to contribute. You could submit a PR and maintainers will review it and then help merge it. |
This issue has been automatically marked as stale because it has not had recent activity for 30 days. It will be closed in next 7 days if no further activity occurs. |
This issue has been closed because it has not received response for too long time. You could reopen it if you encountered similar problems in the future. |
Search before asking
Description
Currently some parts of code are poorly constructed, leading to difficult comprehension, trouble of testing and maintenance.
With an IntelliJ plugin,
metrics reloaded
, we could get the complexity of methods in this project.If the cognitive complexity of a method is more than 30, basically we treat it as incomprehensible, untestable and unmaintainable. We would like to have these methods refactored and achieve a significant decrease in complexity, maybe 50% or more.
Any help will be appreciated : )
https://plugins.jetbrains.com/plugin/93-metricsreloaded
https://www.hatica.io/blog/code-complexity/
Are you willing to submit a PR?
Code of Conduct
The text was updated successfully, but these errors were encountered: