This pull request removes the multiply and divide methods that accept a double parameter from the ComplexNumber class in both the reference solution and the main.java file. These methods were deemed unnecessary since the primary focus of the exercise is to operate on complex numbers, and the existing tests do not validate these double overloads.
Changes Made:
Removed multiply(double factor) and divide(double divisor) methods from the ComplexNumber class.
Updated the reference solution accordingly to ensure consistency.
Rationale
The double overloads for multiplication and division were not accompanied by corresponding test cases in the canonical data, which could lead to situations where these methods are not adequately tested. By removing these methods, we streamline the class and enhance clarity, ensuring that the exercise focuses on complex number operations as intended.
Next Steps
Please review the changes and let me know if further modifications are needed.
pull request
Description
This pull request removes the
multiplyanddividemethods that accept a double parameter from theComplexNumberclass in both the reference solution and themain.javafile. These methods were deemed unnecessary since the primary focus of the exercise is to operate on complex numbers, and the existing tests do not validate these double overloads.Changes Made:
multiply(double factor)anddivide(double divisor)methods from theComplexNumberclass.Rationale
The double overloads for multiplication and division were not accompanied by corresponding test cases in the canonical data, which could lead to situations where these methods are not adequately tested. By removing these methods, we streamline the class and enhance clarity, ensuring that the exercise focuses on complex number operations as intended.
Next Steps
Please review the changes and let me know if further modifications are needed.
Reviewer Resources:
Track Policies