list型からint型への変換

Question

前提・実現したいこと

あるテキストファイルについて、文字列'XXX'が含まれる行を全て削除したい。

やったこと・これからやる予定のこと

1.ファイルの読み込み。'rfile'にテキストファイル名を指定

with open(rfile) as f:
    lines = f.readlines()

2.'XXX'が含まれる行番号の取得

l_XXX_i = [i for i, line in enumerate(lines) if 'XXX' in line]

3.popでの要素削除（ここでエラーが発生）

lines_after = lines.pop(l_XXX_i)

4.ファイルへの書き込み（これからやること）。'wfile'に出力テキストファイル名を指定

with open(wfile, mode='w') as f:
    f.writelines(lines_after)

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

'list' object cannot be interpreted as an integer

試したこと

変数'l_XXX_i'がリスト型だったため、下記の通りint型への変換を試したものの、

l_XXX_index = int(l_XXX_i)

エラーが表示

int() argument must be a string, a bytes-like object or a number, not 'list'

Answer 1

中間リストを作る必要は無く、判定条件を反転させるだけで良い筈です。

with open(rfile) as f:
    lines = f.readlines()

lines_after = [line for line in lines if 'XXX' not in line]

次のように書くとちょっとだけメモリ効率が良くなります。特に除外される要素が多い場合。

with open(rfile) as f:
    lines_after = [line for line in f if 'XXX' not in line]

おまけ

次のように読み出しと書き出しを同時に行うと、更にメモリ効率が良くなります。

with open(rfile) as fin, open(wfile) as fout:
    for line in fin:
        if 'XXX' in line:
            continue

        fout.write(line)

ここまで来るとシェルスクリプトやバッチの方がすっきりしそうなのはさておき。

Answer 2

このようにしたら行けるはずです。

def remove(x):
    lines.pop(x)
    return lines


with open(rfile) as f:
    lines = f.readlines()
l_XXX_i = [i for i, line in enumerate(lines) if 'XXX' in line]
lines_after = list(map(remove, l_XXX_i))
with open(wfile, mode='w') as f:
    f.writelines(lines_after)

※前提条件
l_XXX_iのリストの中身がすべてint型の場合のみ実行可能
str=>intへの変換方法

list(map(int,l_XXX_i))

Comments

[Y_Miz]

早速ご回答いただき、ありがとうございます。
```
lines_after = list(map(lines.pop,l_XXX_i))
```
を試したところ、下記のエラーが出力されました。

```
IndexError: pop index out of range
```

心当たりございますでしょうか。

[退会済みユーザー]

少々お待ちを

[退会済みユーザー]

修正かけました。ご確認ください。

[Y_Miz]

ご確認ありがとうございます。

```
lines_after = list(map(remove, l_XXX_i))
```
を試したところ、
```
NameError: name 'remove' is not defined
```
が出力されました。

せっかくご提案いただきましたが、下記の通り、強引に（'XXX'が現れる先頭行を消す操作を繰り返す）対処しました。
```
l_XXX_i = [i for i, line in enumerate(lines) if 'XXX' in line]
for j in range(len(l_XXX_i)):
l_XXX_inside = [i for i, line in enumerate(lines) if 'XXX' in line]
lines.pop(l_XXX_inside[0])
print(lines)
```
未だ、丸括弧()を使うべきなのか、角括弧[]を使うべきなのか分かりません。
良い教科書などございましたら、ご教示いただけますと幸いです。

[退会済みユーザー]

教科書ですか...自分が個人的におすすめなのはみんなのpythonという本です。これを読めばpythonを不自由なく使うことは可能になります。評価も高いですし、買って損はないと思います。

[Y_Miz]

ありがとうございます。
触って間もないため、是非購入したいと思います。