forループを使用して、文字列内のアルファベット以外の文字をすべて無視し、新しい文字列に追加する方法？ Python 3

Question

文字列中のアルファベット以外の文字をすべて無視してアルファベット文字列を新しい文字列に追加しようとするので、大文字と小文字を区別せずに回文かどうかを調べることができます。

は、これまでのところ、私は私がis_palindromeは小文字の単語が回文であるかどうかを確認する前の関数を呼び出していますプロム

new_str = '' 
for ch in s: 
    ch = ch.lower 
    if s.isalpha(): 
     s = s + new_str 

return s == is_palindrome 

を持っています。ここで

s = 'A man, a plan, a canal: Panama!' 
scrunched_s = ''.join(ch.lower() for ch in s if ch.isalpha()) 
reversed_s = ''.join(ch.lower() for ch in s[::-1] if ch.isalpha()) 
print s 
print scrunched_s 
print reversed_s 
print scrunched_s == reversed_s 

は別です：：

scrunched_s = map(str.lower, filter(str.isalpha, s)) 
reversed_s = map(str.lower, filter(str.isalpha, reversed(s))) 
print scrunched_s == reversed_s 

Answer 1

で複数の問題があります は、すべてのヘルプは大幅にあなたの文字列から非アルファのすべてをドロップする一つの方法です。ここ

Answer 2

をいただければ幸いですあなたのコード。のは、それらを1つずつを行こう：

1. ch = ch.lower

   関数やメソッドを呼び出すためには、あなたが実際にその関数/メソッドを呼び出すためのPythonを伝えるために、関数名の後の括弧()を使用する必要があります。ここでは、ファンクション〜chへの参照をに割り当てます。したがって、chはその後の文字ではありません。あなたがしたいのは、ch = ch.lower()です。

2. if s.isalpha():

   この検査は、しかしsはあなたがそこに何を意味するかはおそらくありませんが、一般的に大丈夫です。 sはフルストリングですが、chは現在の文字です。おそらくif ch.isalpha()を意味します。

3. ​​

   ここでも、sはあなたを介して反復されている文字完全な文字列です。おそらく、ここにsに何かを追加するつもりはありませんでした。おそらく、異なる変数にいくつかの結果を集めたいと思うでしょう。

   new_strは空の文字列''に初期化され、変更されないため、空の文字列を文字列sに追加します（何もしません）。

   おそらく何行うためのものがnew_strを変更し、以前の条件が真だった場合文字を追加します：new_str = new_str + ch前と同じ

4. return s == is_palindrome

   、関数を呼び出すために、かっこを使用する必要があります。それ以外の場合は、sという文字列をここでis_palindrome関数と比較します（これは当てはまりません）。

   さらに、sは入力文字列です（ループ全体で誤って変更したのですか？）。あなたはおそらくここでnew_strをチェックすることを意味しました。

   最後に、is_palindromeはおそらく文字列を入力として受け取り、その文字列が回文かどうかを返します。とにかくその結果を文字列と比較したくないのです。

完全なコードは、おそらく修正した後、次のようになります。

new_str = '' 
for ch in s: 
    ch = ch.lower() 
    if ch.isalpha(): 
     new_str = new_str + ch 

return is_palindrome(new_str)