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
use format to remove '0b' #11307
base: master
Are you sure you want to change the base?
use format to remove '0b' #11307
Conversation
for more information, see https://pre-commit.ci
@@ -35,8 +35,8 @@ def binary_and(a: int, b: int) -> str: | |||
if a < 0 or b < 0: | |||
raise ValueError("the value of both inputs must be positive") | |||
|
|||
a_binary = str(bin(a))[2:] # remove the leading "0b" |
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.
Why we need to more the comments??
In the given line of code, the comment is explaining what the line does, which is to remove the leading "0b" from the binary representation of the integer a. Comments are useful for making the code more understandable to other developers (or to your future self) who might read the code later.
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.
When using the format, "0b" is not included in the output, as it generates a string of 0s and 1s directly. As for the comment, would "converting to binary string" accurately reflect the operation? If so, I would be happy to implement these changes. Your feedback is valuable.
Describe your change:
use format to remove '0b'
Checklist: