-
-
Notifications
You must be signed in to change notification settings - Fork 360
Cleaned up the code a bit #227
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
Conversation
made it use a one print instead of two and deleted the function that was handling the two prints and made it print directly and used a -= b, b -= a instead of a = a - b, b= b - a
The |
I don't have a main function here at all if__name__==" __ main __ ": is the way of python to say run this part only if this file was run as a main process, which means if you import this as module you won't see it print any thing which can be helpful @jiegillet |
I am aware of how |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Address the changes and I'll merge the PR
|
||
main() | ||
if __name__=="__main__": | ||
print('Eucledean Mod: {}\nEucledean sub: {}'.format(euclid_mod(64 * 67, 64 * 81), euclid_sub(128 * 12, 128 * 77))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- It's spelled Euclidian
- split that line into 2 lines to make it easier to read, something like
print('Euclidian Mod: {}'.format(euclid_mod(64 * 67, 64 * 81))
print('Euclidian sub: {}'.format(euclid_sub(128 * 12, 128 * 77))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
made it use a one print instead of two and deleted the function that was handling the two prints and made it print directly and used a -= b, b -= a instead of a = a - b, b= b - a