python3:ファイル内の特定文字列があったら処理を分けたい。

Question

##実現したいこと
（C:\tool\配下）ファイル名を取得し、テキストファイル（test01.txt）を開いて、そのファイル名があったら処理を分けたい。このようなことは可能でしょうか。途中まで検討しました。（????）の部分や全体的にご指摘あれば、お力沿いお願いいたします。

import os
import pathlib

for JsonConf in glob.glob(r'C:\tool*.json'):
    FileName = os.path.splitext(os.path.basename(JsonConf))[0]

    with open(test01.txt, 'r') as lists:
        if ????:
            print("ファイル名あり")
        else:
            print("ファイル名無し"

[C:\tool*]

aaaa.json
bbbb.json
eeee.json

[test01.txt]

aaa
bbbb
cccc
eeee
dddd

Answer 1

こんな感じでしょうか？

import os
import glob

for json_conf in glob.glob(r'tool/*.json'):
    file_name = os.path.basename(json_conf)
    droped_extension = os.path.splitext(file_name)[0]

    with open("test01.txt") as lists:
        lines = map(lambda x: x.strip(), lists.readlines())
        if droped_extension in lines:
            print(f"{droped_extension}({file_name})はファイル名が記載されいてます")
        else:
            print(f"{droped_extension}({file_name})はファイル名が記載されていません")

可能な限り元のコードの構成に近い状態で書いてみました。
ただ上記だと、各JSONファイル名がtest01.txtに記載されているか、という確認のために、各JSONファイルごとにtest01.txtをオープン、全行読み込み、余計な改行の削除、という処理を毎回行ってしまっています。
1回だけやっておけば十分な処理なので、自分なら以下のように書きます。

import os
import glob

with open("test01.txt") as lists:
    lines = list(map(lambda x: x.strip(), lists.readlines()))

for json_conf in glob.glob(r'tool/*.json'):
    file_name = os.path.basename(json_conf)
    dropped_extension = os.path.splitext(file_name)[0]

    if dropped_extension in lines:
        print(f"{dropped_extension}({file_name})はファイル名が記載されいてます")
    else:
        print(f"{dropped_extension}({file_name})はファイル名が記載されていません")

なお、もしPythonにまだ慣れていなくてmap(lambda...)のところがよく分からん、ということであれば、以下のようにコードを書き直すことも可能ですのでご参考までに。

with open("test01.txt") as lists:
    # test01.txtの全行を読み込み。
    lines_work = lists.readlines()
    lines = list()

    # ただし、各行の末尾には改行コードが含まれているので、それを削除する。
    # for文で各行毎に処理を回す
    for line in lines_work:
        # 各行（文字列）がline変数に渡されるので、strip関数を使って改行コードを削除
        striped = line.strip()
        # 結果格納用のlinesに結果を追加
        lines.append(striped)

Comments

[LouiS0616]

list(map(lambda x: x.strip(), lists.readlines())) は list(map(str.strip, lists)) あるいは [s.strip() for s in lists] で充分かなと思います。
読み込みが遅延されるためちょっとだけ効率的です。

また、後にinで帰属判定することを考えれば、linesはリストではなく集合の方が良さそうです。

[ITOMO5963]

お二人の方、回答いただき大変助けりました。
ありがとうございました。

[ITOMO5963]

ありがとうございました。