Skip to content

Add SociableNumber Implementation#7475

Open
Rosander0 wants to merge 1 commit into
TheAlgorithms:masterfrom
Rosander0:add-sociable-number
Open

Add SociableNumber Implementation#7475
Rosander0 wants to merge 1 commit into
TheAlgorithms:masterfrom
Rosander0:add-sociable-number

Conversation

@Rosander0

Copy link
Copy Markdown
Contributor
  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
  • All new algorithms include a corresponding test class that validates their functionality.
  • All new code is formatted with clang-format -i --style=file path/to/your/file.java

@codecov-commenter

codecov-commenter commented Jun 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.86%. Comparing base (6fbbc94) to head (5e20539).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #7475      +/-   ##
============================================
+ Coverage     79.80%   79.86%   +0.05%     
- Complexity     7302     7338      +36     
============================================
  Files           803      808       +5     
  Lines         23751    23836      +85     
  Branches       4671     4693      +22     
============================================
+ Hits          18955    19036      +81     
- Misses         4036     4037       +1     
- Partials        760      763       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Rosander0 Rosander0 force-pushed the add-sociable-number branch 2 times, most recently from d58f8a5 to 6867c86 Compare June 14, 2026 09:09
@Rosander0 Rosander0 changed the title feat: add SociableNumber implementation Add SociableNumber Implementation Jun 14, 2026
@Rosander0 Rosander0 force-pushed the add-sociable-number branch 2 times, most recently from 31f86fa to f37df6b Compare June 14, 2026 10:06
@DenizAltunkapan DenizAltunkapan requested a review from Copilot June 18, 2026 07:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new number-theory algorithm for detecting whether a given integer belongs to a sociable-number cycle of a specified length, along with JUnit tests validating known sociable/amicable examples and invalid inputs.

Changes:

  • Introduce SociableNumber utility with isSociable(number, cycleLength) and a proper-divisor-sum helper.
  • Add SociableNumberTest covering a length-5 sociable cycle, amicable pairs (length 2), non-sociable values, and invalid inputs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
src/main/java/com/thealgorithms/maths/SociableNumber.java New sociable-number implementation (isSociable) and proper-divisor-sum helper.
src/test/java/com/thealgorithms/maths/SociableNumberTest.java New unit tests for sociable-cycle detection and edge cases.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main/java/com/thealgorithms/maths/SociableNumber.java
Comment thread src/main/java/com/thealgorithms/maths/SociableNumber.java
Comment thread src/test/java/com/thealgorithms/maths/SociableNumberTest.java
Comment thread src/test/java/com/thealgorithms/maths/SociableNumberTest.java Outdated
@Rosander0 Rosander0 force-pushed the add-sociable-number branch from f37df6b to b493fec Compare June 18, 2026 12:21
@Rosander0 Rosander0 force-pushed the add-sociable-number branch from b493fec to 5e20539 Compare June 18, 2026 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants