Artemis: refactor: Optimize algorithms and add safety checks #53

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.

Sign up for GitHub

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

Jump to bottom

Open

mike-turintech wants to merge 1 commit into main from artemis-b82ca054-bbed-4f85-9287-ab0561be9643

Member

mike-turintech commented Jan 22, 2025

This PR improves performance and adds robustness to several algorithms:

Replace iterative solutions with mathematical formulas in sumSquare and sumTriangle for better performance
Optimize countPairs using HashMap to reduce complexity from O(n²) to O(n)
Add null checks and array bounds validation in countDuplicates
Modernize sumMatrix with Java 8 Stream API features

Technical changes:

Added java.util dependencies (HashMap, Map, Arrays)
Replaced nested loops with mathematical formulas where possible
Implemented safety checks for null inputs and array bounds
Leveraged Stream API for cleaner array processing


          Artemis Changes

c428ad9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet