Skip to content

Created harshad_numbers.py #9023

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.

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

Merged
merged 20 commits into from
Sep 6, 2023
Merged
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
132 changes: 132 additions & 0 deletions maths/harshad_numbers.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,132 @@
"""
A Harshad number is divisible by the sum of its digits in any base n.
Copy link
Contributor

Choose a reason for hiding this comment

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

Nitpicks:

  1. "Harshad" should be lowercase ("harshad" is not a name).
  2. What you're describing here is an all-harshad number. In general, a harshad number (or more specifically an n-harshad number) is a number that's divisible by the sum of its digits in some given base n.

Reference: https://en.wikipedia.org/wiki/Harshad_number
"""


def int_to_base(number, base_of_interest):

Choose a reason for hiding this comment

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

As there is no test file in this pull request nor any test function or class in the file maths/harshad_numbers.py, please provide doctest for the function int_to_base

Please provide return type hint for the function: int_to_base. If the function does not return a value, please provide the type hint as: def function() -> None:

Please provide type hint for the parameter: number

Please provide type hint for the parameter: base_of_interest

digits = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ"
result = ""

while number > 0:
number, remainder = divmod(number, base_of_interest)
result = digits[remainder] + result

if result == "":
result = "0"

return result


def sum_of_digits(num: int, base_of_interest: int):

Choose a reason for hiding this comment

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

Please provide return type hint for the function: sum_of_digits. If the function does not return a value, please provide the type hint as: def function() -> None:

"""
Calculate the sum of digit values in a positive integer
converted to the given 'base_of_interest'.
Where 'base_of_interest' ranges from 2 to 36.

Examples:
>>> sum_of_digits(103, 12)
'13'
>>> sum_of_digits(1275, 4)
'30'
>>> sum_of_digits(6645, 2)
'1001'
>>> # bases beyond 36 and below 2 will error
>>> sum_of_digits(543, 1)
Traceback (most recent call last):
...
ValueError: 'base_of_interest' must be between 36 and 2 inclusive
>>> sum_of_digits(543, 37)
Traceback (most recent call last):
...
ValueError: 'base_of_interest' must be between 36 and 2 inclusive
"""

if (base_of_interest > 36) or (base_of_interest < 2):
raise ValueError("'base_of_interest' must be between 36 and 2 inclusive")

num = int_to_base(num, base_of_interest)
res = 0
for char in num:
res += int(char, base_of_interest)
res = int_to_base(res, base_of_interest)
return res


def all_harshad_numbers(num: int, base_of_interest: int):

Choose a reason for hiding this comment

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

Please provide return type hint for the function: all_harshad_numbers. If the function does not return a value, please provide the type hint as: def function() -> None:

"""
Finds all Harshad numbers smaller than num in base 'base_of_interest'.
Where 'base_of_interest' ranges from 2 to 36.

Examples:
>>> all_harshad_numbers(15, 2)
(7, ['1', '10', '100', '110', '1000', '1010', '1100'])
>>> all_harshad_numbers(12, 34)
(11, ['1', '2', '3', '4', '5', '6', '7', '8', '9', 'A', 'B'])
>>> all_harshad_numbers(12, 4)
(7, ['1', '2', '3', '10', '12', '20', '21'])
>>> # bases beyond 36 and below 2 will error
>>> all_harshad_numbers(234, 37)
Traceback (most recent call last):
...
ValueError: 'base_of_interest' must be between 36 and 2 inclusive
>>> all_harshad_numbers(234, 1)
Traceback (most recent call last):
...
ValueError: 'base_of_interest' must be between 36 and 2 inclusive
"""

if (base_of_interest > 36) or (base_of_interest < 2):
raise ValueError("'base_of_interest' must be between 36 and 2 inclusive")

result = 0
numbers = []
if num >= 0:
for i in range(1, num):
y = sum_of_digits(i, base_of_interest)
if i % int(y, base_of_interest) == 0:
result += 1
numbers.append(int_to_base(i, base_of_interest))

return result, numbers


def is_harshad_number(num: int, base_of_interest: int):

Choose a reason for hiding this comment

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

Please provide return type hint for the function: is_harshad_number. If the function does not return a value, please provide the type hint as: def function() -> None:

"""
Determines whether n in base 'base_of_interest' is a harshad number.
Where 'base_of_interest' ranges from 2 to 36.

Examples:
>>> is_harshad_number(18, 10)
True
>>> is_harshad_number(21, 10)
True
>>> is_harshad_number(-21, 5)
False
>>> # bases beyond 36 and below 2 will error
>>> is_harshad_number(45, 37)
Traceback (most recent call last):
...
ValueError: 'base_of_interest' must be between 36 and 2 inclusive
>>> is_harshad_number(45, 1)
Traceback (most recent call last):
...
ValueError: 'base_of_interest' must be between 36 and 2 inclusive
"""

if (base_of_interest > 36) or (base_of_interest < 2):
raise ValueError("'base_of_interest' must be between 36 and 2 inclusive")

if num >= 0:
n = int_to_base(num, base_of_interest)
d = sum_of_digits(num, base_of_interest)
if int(n, base_of_interest) % int(d, base_of_interest) == 0:
return True

return False


if __name__ == "__main__":
import doctest

doctest.testmod()