5:意味づけできるまとまりで関数化する

無思慮に関数をまとめていませんか？　関数に分離するときは処理のまとまりで分けてはいけません。

具体的な失敗

def main():
    with open(...) as f:
        reader = csv.reader(f)
        for row in reader:
            i = row[1]
            if i < 100:
                print(row[0])

この関数は単に「主な処理」として main() 関数にまとめられているだけです。 これを「処理のまとまり」で分離してしまうと以下のようになります。

def print_row(row):
    i = row[1]
    if i < 100:
        print(row[0])


def main():
    with open(...) as f:
        reader = csv.reader(f)
        for row in reader:
            print_row(row)

関数化することで改善した気持ちになってしまいますが、この分離は問題があります。

cover

（中略）詳細は書籍 自走プログラマー をご参照ください

ベストプラクティス

処理の意味、再利用性で関数や処理は分離しましょう。 「関数に分離しよう」と意気込む前に、処理をそれぞれどう意味づけできるかを考えることが大切です。 CSV読み込み、 100 との比較という処理が、どういう意味なのかを関数で表します。 今回は、「価格が100円未満の場合は、買い合わせ対象商品である」という仕様があったとします。

import csv


def read_items():
    """ 商品一覧のCSVデータを読み込んでタプルのジェネレーターで返す
    各商品は「商品名、価格」のタプルで返される
    """
    with open(...) as f:
        reader = csv.reader(f)
        for row in reader:
            name = row[0]
            price = int(row[1])
            yield name, price


def is_addon_price(price):
    """ 価格が「買い合わせ対象商品」の場合Trueを返す
    """
    return price < 100


def main():
    items = read_items()
    for name, price in items:
        if is_addon_price(price):
            print(name)

処理でなく意味でまとめることで、それぞれの処理が「何のために行われているか」が自明になりました。

cover

（中略）詳細は書籍 自走プログラマー をご参照ください