Alvin Tan Jian Jia created GUACAMOLE-1630: ---------------------------------------------
Summary: Potential Refactoring Estimate Key: GUACAMOLE-1630 URL: https://issues.apache.org/jira/browse/GUACAMOLE-1630 Project: Guacamole Issue Type: Improvement Reporter: Alvin Tan Jian Jia Attachments: Guacamole-client_1.4.0.csv **Summary** I have identified a few classes with refactoring potential along with its estimated time to refactor, specifically ActiveConnectionPermissionService.java. The estimated time is based on historical commits containing refactoring and the complexity of the involved classes. The class was refactored once on Sep 20, 2018 along with other classes in commit 69f58c8ca314c44822e0eaab354b5f722a01ac89. Given the duration of the last refactoring, could it be possible to improve this class further? I am probably not qualified and experienced enough to point out the exact code needs to be refactored, given my limited knowledge of the project and codebase. **Urgency** Low priority. For maintenance of code in the long run. Hi, I am currently doing some research on Estimating Time Taken for Software Refactoring. Guacamole is one of my case studies being a highly popular project. Would appreciate your kind feedback on the appropriateness of the estimated time to refactor. I am specifically looking at potential for refactoring, such as breaking functions that are too long into shorter functions, or to remove code duplication. Would you agree that there could be some modification to the code to make it better in the listed classes? And if so, is the estimated time appropriate to carry out the improvement? I will be grateful for any help you can provide. # ActiveConnectionPermissionService.java - 2 hours # EntityMapper.java - 2 hours # EntityService.java - 3 hours Thank you. Regards, Alvin -- This message was sent by Atlassian Jira (v8.20.10#820010)