python - この関数を書くより Pythonic な方法

Question

この関数を書くためのもっとpythonicな方法があるのだろうか：

def parse(filename):
    with open(filename, 'r', encoding='koi8-r') as f:
        for log_line in f:
            for s in services:
                if ' ' + s + ' ' in log_line:
                    print(s)
                    services.remove(s)

Answer 1

使用セット:

def parse(filename):
    servicesset = set(services)
    with open(filename, 'r', encoding='koi8-r') as f:
        for log_line in f:
            words = set(log_line.split())
            servicesset -= words

    services[:] = list(servicesset)

servicesこれは、グローバルがリストであることを前提としています。サービスを渡してから結果を返す方がはるかに良いことに注意してください。

def parse(filename, services):
    services = set(services)
    with open(filename, 'r', encoding='koi8-r') as f:
        for log_line in f:
            words = set(log_line.split())
            services -= words

    return list(servicesset)