pythonのdel文で消えない文字があるのですが

Question

実現したいこと

以下のコードを実行すると"a"が一つ消えません。
どうすれば消すことができるでしょうか？

発生している問題・エラーメッセージ

該当のソースコード

python
s=list(input())
for j,i in enumerate(s):
if i=="a" or i=="i" or i=="u" or i=="e" or i=="o":
del s[j]
print(s)

Comments

[meg_]

コードはマークダウンで記入しましょう。
https://teratail.com/help#about-markdown

Answer 1

削除されない要素が出るのは、ループの途中で、ループ対象のリストを変更していることが原因です。

試しに、ご提示のコードに「aac」という文字列を入力した際の動作を追ってみましょう。

1. 入力された文字列をリストに変換してsに代入してるので、s = ['a', 'a', 'c']となる
2. sの先頭要素'a'とインデックス0をそれぞれi, jに代入してループ内部が実行され、s = ['a', 'c']となる
3. sの2番目の要素'c'とインデックス1をそれぞれi, jに代入してループ内部が実行され、s = ['a', 'c']となる

これにより、もともと2番目にあった要素の判定が飛ばされて、消えずに残ってしまうのです。

対象要素をすべて削除する方法ですが、公式のFAQには、内包表記を使う方法などが紹介されています。

s=[i for i in s if not(i=="a" or i=="i" or i=="u" or i=="e" or i=="o")]

どうしてもループとdelを使いたいのであれば、逆順に実行するという方法もあります。

for j,i in reversed(list(enumerate(s))):
    if i=="a" or i=="i" or i=="u" or i=="e" or i=="o":
        del s[j]

Comments

[kokyo-ai]

回答ありがとうございます。
ベストアンサーは別の方を選ばせていただきましたが、こちらの回答も非常に参考になりました。

Answer 2

リストや辞書の内容でループしながら、そのループしているリストなどを書き換えることはすべきではありません。通常は予期しない結果が出ます。
例外の例としては、巨大なデータを扱っていてメモリを節欲するために内容見ながら書き換えるようなことをする必要がある場合などですね。その場合も、その言語がどのようにデータを管理しているかを考えながら処理を行なう必要があります。

通常の対応方法はいろいろありますが、質問のように不要なものを削除する場合、必要なものだけ抽出するのように読み替えることで、ループの中でリストを直接変更することを避けられます。

ans = []
for i in s:
    if not(i=="a" or i=="i" or i=="u" or i=="e" or i=="o"):
        ans.append(i)
print(i)
print(ans)

(他の回答の内包表記も同じ考え)

ちなみに、判定で 「 複数のうちのどれか」を書くとき、orでつなぐよりも、「in」を使った方や読みやすくなります。

    if i not in ["a", "i", "u", "e", "o"]:

Comments

[kokyo-ai]

回答ありがとうございます。
試したところ問題が解決しました！
ベストアンサーに選ばせていただきました。