就職活動前のポートフォリオ作品のコードレビュー

コードの処理内容

1. ショップから出力した orders.csv を読み込む
2. 注文行ごとに stock.xlsx からSKUで商品を探し、数量を差し引く
3. 更新した在庫を書き戻し、集計を import_log.txt に記録する

構成はわかりやすく、関数名も適切です。問題の多くは、想定と違う形式のデータが入ってきたときに起きています。

バグと弱点

No.	行	重要度	内容
1	48	高	同じCSVを2回取り込めてしまう。処理済みの注文IDを確認していない。在庫がマイナスになる最も有力な原因
2	72	高	返品注文の空欄や「2.0」のような値で int(row["qty"]) が落ちる
3	95	中	SKUの照合が大文字・小文字を区別するため「gt-101」と「GT-101」が別商品扱い
4	130	中	encoding="utf-8" を指定せずにファイルを開いており、Windowsで日本語の商品名が文字化けする
5	201	低	素の except: が本当のエラーを隠し、ログが役に立たない

修正コード（抜粋）

問題1：取り込み済みの注文をスキップする

processed = load_processed_ids("processed_orders.txt")
for row in reader:
    if row["order_id"] in processed:
        continue
    # ... 既存の処理 ...
    processed.add(row["order_id"])
save_processed_ids("processed_orders.txt", processed)

理由：同じファイルでもう一度実行しても、各注文は1回しか反映されません。

問題2：数量を安全に読み取る

raw = (row.get("qty") or "").strip()
try:
    qty = int(float(raw))
except ValueError:
    log.warning("注文 %s：数量 %r が不正のためスキップ", row["order_id"], raw)
    continue

理由：返品や一部キャンセルの行は、取り込み全体を止めずに、わかりやすいメモを残してスキップされます。

問題3：SKUを正規化する

sku = row["sku"].strip().upper()

在庫の辞書を作る側にも同じ .strip().upper() を適用してください。

書き方と読みやすさ

• process() が90行あります。read_orders、apply_to_stock、write_report に分けましょう。
• row[7] のような列番号の直書きは、見出し名での参照に置き換えてください。

ご注意： 各修正は必ずファイルのコピーで試してください。今回はコードを読んでレビューしたもので、実行はしていません。