EricGao888 opened a new issue, #15599:
URL: https://github.com/apache/dolphinscheduler/issues/15599

   ### Search before asking
   
   - [X] I had searched in the 
[issues](https://github.com/apache/dolphinscheduler/issues?q=is%3Aissue) and 
found no similar feature requirement.
   
   
   ### 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.
   
   
![image](https://github.com/apache/dolphinscheduler/assets/34905992/e28180d1-01b1-40f3-b235-8a808a32e4e7)
   
   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?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of 
Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: 
commits-unsubscr...@dolphinscheduler.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to