Анаграмма Python 3 [закрыта]

Вы определили функцию почти правильно, но есть несколько проблем.

Сначала вы запрашиваете s1 и s2 в качестве параметров. Это хорошо. Теперь используйте эти значения, а не глобальные переменные word1 и word2.

Во-вторых, если эти значения являются строками, вы не можете вызвать для них sort, потому что у строк нет метода sort. Но вы можете вызывать функцию sorted для любой последовательности, даже для строк.

В-третьих, есть простая опечатка, отсутствие " во втором print.

Возможно, было бы лучше вернуть значение True или False и поместить print вне функции, но пока оставим это.

Собрав это вместе, вот рабочая функция:

def isAnagram(s1, s2):
    s1 = sorted(s1)
    s2 = sorted(s2)
    if s1 == s2:
       print("This is an anagram")
    else:
       print("This is not an anagram")

Но теперь вам также нужно правильно вызвать функцию. Вы определили функцию, которая принимает два параметра, s1 и s2. Это означает, что вам нужно вызвать функцию с двумя аргументами.

Так откуда у вас такие аргументы? Что ж, у вас уже есть эти переменные word1 и word2, и они кажутся именно тем, что вам нужно.

Итак, измените последнюю строку на:

isAnagram(word1, word2)

И вы сделали.

Вы правильно поняли, но поскольку word1 и word2 являются строками, у них нет атрибута .sort(). Вы все еще можете сортировать буквы в этой строке:

>>> w='hello'
>>> sorted(w)
['e', 'h', 'l', 'l', 'o']

Результатом sorted() является список, мы можем превратить его обратно в строку, объединив их с помощью пустой строки:

>>> ''.join(sorted(w))
'ehllo'

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

word1 = input("Enter a string: ")
word2 = input("Enter a second string: ")

def isAnagram(s1, s2):
    s1 = ''.join(sorted(word1))
    s2 = ''.join(sorted(word2))
    if s1 == s2:
       print("This is an anagram")
    else:
       print("This is not an anagram")

isAnagram(word1, word2)

Ваш подход к поиску анаграмм вполне разумен. Сортировка слов и их сравнение — самый простой способ определить, являются ли два слова анаграммами друг друга.

Однако я думаю, что вас смущает идея параметров функции. Когда вы определяете

foo(x1, x2)

когда вызывается foo, ожидается, что он будет вызван с двумя параметрами. Вы определяете

anagram(s1, s2)

но никогда не снабжайте его s1 и s2. Список параметров — это не список имен переменных, которые вы используете в функции — вы можете назначать новые переменные по желанию. Вместо этого это список входных данных, которые принимает функция.

поэтому anagram() неверно. Вам нужно позвонить anagram(input1, input2). (Предполагая, что у вас нет значений по умолчанию, в которые я не буду вдаваться.

def isAnagram(s1, s2):
    sortedWord1 = sorted(s1) # s1 is word1! (sorted instead of sort, strings are immutable)
    #what should you do here?
    if sortedWord1 == sortedWord2:
       print("This is an anagram")
    else:
       print("This is not an anagram") # you forgot a closing quote!

word1 = input("Enter a string: ")
word2 = input("Enter a second string: ")

isAnagram(word1, word2)

Я немного изменил ваш код, чтобы он работал правильно. Тем не менее, я рекомендую вам немного больше прочитать о функциях, прежде чем продолжить.

Думайте о них так же, как о функциях в математике! f(x) имеет смысл, f, хотя и имеет смысл, вероятно, не то, что вы искали.

>>> isAnagram("anagram", "nagaram")
This is an an anagram
>>> isAnagram("anagram", "woohoo")
This is not an anagram
>>> isAnagram("a", "a")
This is an an anagram