Skip to content

Update is_anagram.py #5

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.

Sign up for GitHub

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

Closed

Krasiu wants to merge 1 commit into devAmoghS:master from Krasiu:patch-1

Closed

Update is_anagram.py #5

Krasiu wants to merge 1 commit into devAmoghS:master from Krasiu:patch-1

Conversation

Krasiu

Copy link

@Krasiu Krasiu commented Jun 5, 2019

No description provided.


 Update is_anagram.py

fa42b58

Copy link

Contributor

luiz-surian commented Jun 6, 2019

This only invert the string, won't check if its na anagram :(
print(is_anagram('binary', 'brainy')) should be True, but returns False

Copy link

Owner

devAmoghS commented Jun 7, 2019

@Krasiu thank you for raising the pull request. But Anagrams doesn't work this way. I think what your code is doing: is actually checking whether the strings are palindrome or not ?

Two words are anagrams of one another if their letters can be rearranged to form the other word.

Two words are palindrome if you reverse one to get another e.g. radar

@devAmoghS devAmoghS closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment