12aviray opened a new pull request, #428: URL: https://github.com/apache/commons-cli/pull/428
## Summary This pull request improves the maintainability of Apache Commons CLI by applying several small refactorings without changing the external behavior. ## Refactorings ### 1. DefaultParser - Applied Extract Method to reduce cognitive complexity. - Split the large `handleShortAndLongOption()` method into smaller helper methods. ### 2. Option - Refactored `processValue()`. - Extracted separator parsing logic into a dedicated helper method. - Improved readability and separation of responsibilities. ### 3. Options - Simplified required option update logic. - Removed redundant lookup operations by replacing `contains()` + `indexOf()` with a single `remove(Object)` call. ## Validation - Project builds successfully. - All existing tests pass (`mvn test`). - No functional behavior has changed. ## Benefits - Improved readability - Better maintainability - Reduced cognitive complexity - Cleaner collection handling -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
