Как сделать правильную проверку на палиндром?

Question

[LevG0r]

Здравствуйте, недавно начал изучать python по крижке и наткнулся на задние. Нужно сдлать проверку слова на полиндром, но так, чтобы не учитывались регистр, знаки препинания и пробелы. Программа ниже не работает. Скажите пожалуйста как почему и помогите с решением. Заранее спасибо.

def correction(text):
	for item in text:
		if item == ' ':
			del item
	text = text.lower()

def reverse(text):
	return text[::-1]
	
def is_palindrome(text):
	correction(text)
	if text == reverse(text):
		print("Слово является полиндромом")
	else:
		print("Слово не полиндром")

something = input("Введите текст: ")
is_palindrome(something)

Comments

[AWEme]

correction(text) проблема тут.
Ты вызываешь функцию, но не сохраняешь результат её работы в переменную, да и сама функции ничего не возвращает.
Почитай про строки в питоне - узнаешь, что они неизменяемые и что функция correction неверна т.к. ты не можешь просто удалить букву из строки.

Answer 1

[Andrey Dugin]

# phrase = input('Введите фразу: ')
phrase = 'А роза упала на лапу Азора'
phrase = list(filter(str.isalpha, phrase.casefold()))
if phrase == phrase[::-1]:
    print('Палиндром')
else:
    print('Не палиндром')

А проблема у вас прежде всего здесь:

for item in text:
  if item == ' ':
    del item

1) Нельзя удалять что-то из набора, по которому итерируетесь;
2) Строка является неизменяемым объектом;
3) В вашем исполнении это в принципе не работает.

Comments

[Ivan Yakushenko]

У вас вариант не соответствует описанию:

чтобы не учитывались регистр, знаки препинания и пробелы

[Andrey Dugin]

Ivan Yakushenko, пардон, поправил

Answer 2

[Ivan Yakushenko]

Та другого способа вроде как и нет. Ну только если сократить всё это как-то так:

# Проверка каждого слова из предложения на палиндром
def palindrome_sentence(text):
    for word in ["".join(filter(str.isalpha, w)) for w in text.split(" ")]:
        print("palindrome") if word.lower() == word.lower()[::-1] else print("not palindrome")


# Объединение строки в одно слово и его проверка
def palindrome_word(text):
    word = "".join(["".join(filter(str.isalpha, w)) for w in text])
    print("palindrome") if word.lower() == word.lower()[::-1] else print("not palindrome")


text = input("text: \n")
palindrome_word(text)

А то чет нагородили много лишнего.

Comments

[LevG0r]

Ну так еще надо проверку на знаки препинания.

[Ivan Yakushenko]

LevG0r, обновил.

[Иван Мельников]

Например, для строки 'qww q' не работает.

[Ivan Yakushenko]

Иван Мельников, это два разных слова. Вы значения слова "слово" знаете?
Если нужно всю строку мешать в одно слово, то тогда вместо for использовать строку:

word = "".join([''.join(filter(str.isalpha, w)) for w in text])

[Andrey Dugin]

Чудовищный огород нагородили, не надо так

[Ivan Yakushenko]

Andrey Dugin, покажите как это сделать так, что-бы был не огород и значение возвращалось, или принтилось, как в условии.

[Andrey Dugin]

Ivan Yakushenko,

def is_palindrome(phrase):
    phrase = list(filter(str.isalpha, phrase.casefold()))
    return phrase == phrase[::-1]
    
is_palindrome('А роза упала на лапу Азора')