python Программа, чтобы проверить, являются ли два слова анаграммами

Question

Я написал небольшую программу, чтобы проверить, являются ли два слова анаграммами. Если два слова являются анаграммами, он должен возвращать «истину», иначе он должен возвращать «False», но я не получаю правильный вывод. Пожалуйста, скажите мне, что является ошибкой в ​​нижеприведенной программе.

def anagram(s1,s2): 
    for x in s1: 
     if (x in s2) and (s2.count(x)==s1.count(x)): 
      pass 
     return(True) 
    else: 
     return(False) 

Answer 1

Просьба отформатировать это более удобочитаемым способом. Тем не менее, это выглядит, как вы вызываете return True внутри цикла, а это означает, что если какой-либо символ встречается одинаковое количество раз в каждой строке, ваша функция будет return True

Попробуйте это:

def anagram(s1,s2): 
    for x in s1: 
     if (x in s2) and (s2.count(x) == s1.count(x)): 
      pass 
     else: 
      return False 
    for x in s2: 
     if (x in s1) and (s1.count(x) == s2.count(x)): 
      pass 
     else: 
      return False 
    return True 

Answer 2

Вы итерация слово, а затем ничего не делает (с pass, пустой оператор). Затем вы возвращаете True безоговорочно, не оставляя никаких шансов на возврат False.

Вместо этого, вы можете просто сортировать два слова, а затем увидеть, если они в конечном итоге то же самое:

def anagram(s1, s2): 
    return sorted(s1) == sorted(s2) 

Answer 3

Вы были очень близки. Ваш отступ был плохим, но, вероятно, из-за форматирования текста здесь в SO.

Ошибка в коде заключалась в том, что вы вернули True слишком скоро. То, что вам нужно сделать, это пройти все письма и проверить наличие и подсчет. Ниже вы можете найти исправленную и несколько оптимизированную версию того, что вы пытались сделать.

def anagram(s1, s2): 
    if set(s1) == set(s2) and all(s2.count(x) == s1.count(x) for x in set(s1)): 
     return True 
    return False 

Но опять же решение Tigerhawk стало лучше.