[openpyxl] Excelシート内で任意の文字と完全一致するセルが存在する列を格納したい

Question

###実現したいこと

excelのシート内で、ある文字列と完全一致するセルがある列を格納するコードについて

コピーしたいものはセル列で、それの区別分けをE列のセルに入っている文字列でしたいです。
下記のコードでは区別分けがうまくできませんでした。

最後ですべての列がコピーされるのですが、この部分がよく分かりません。
検索したものをそのまま使い、いろいろいじってみましたがうまくいかないので有識者の方、ご意見いただければ幸いです。

恐らく初歩的な問題だと思いますが、よろしくお願いいたします。
参照ページ『https://gammasoft.jp/support/openpyxl-iter-rows/』

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

Excelシートに入っているセルの文字列

[None, None, None, None, None]
[None, None, None, None, None]
[None, None, None, None, None]
['名前', '年齢', '性別', '出身', '部活']
['佐藤', 12, '男', '福島', '写真クラブ']
['渡辺', 23, '女', '愛媛', '陸上部']
['山田', 22, 'なし', '京都', '野球部']
['相田', 111, '男', '鹿児島', '陸上部']

-------------------------------------------------

実行結果
['山田', 22, 'なし', '京都', '野球部']
['渡辺', 23, '女', '愛媛', '陸上部', '相田', 111, '男', '鹿児島', '陸上部']
['佐藤', 12, '男', '福島', '写真クラブ', '渡辺', 23, '女', '愛媛', '陸上部', '山田', 22, 'なし', '京都', '野球部', '相田', 111, '男', '鹿児島', '陸上部']

該当のソースコード

import os
import openpyxl as px
import re

wb = px.load_workbook(r"C:\Users\---\Desktop\MyPandas\test.xlsx")
ws = wb["Sheet1"]

print("Excelシートに入っているセルの文字列")
print("")
active_sheet = wb.active
for row in active_sheet.rows:
    excel = []
    for cell in row:
        excel.append(cell.value)
    print(excel)
#格納するリスト
bb_clb = []
tf_clb = []
pic_clb = []


#空行の定義
def is_empty(cell):
    return cell.value is None or not str(cell.value).strip()

#Excelシートで一列ごとに対応するリストへ格納
#部活で格納するリストを分ける
for row in ws.iter_rows(min_row = 5):
    if all(is_empty(c) for c in row):
        break
    elif (row[4].value == "野球部"):
        for col in row:
            bb_clb.append(col.value)
    elif (row[4].value == "陸上部"):
        for col in row:
            tf_clb.append(col.value)
    else: row[4].value == "写真クラブ"
    for col in row:
        pic_clb.append(col.value)
        
print("")
print("-------------------------------------------------")
print("")
print("実行結果")
print(bb_clb)
print(tf_clb)
print(pic_clb)
    

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

写真クラブの判定のところがelseになっているので他の判定で省かれたデータがすべて追加されてしまっています。
以下のようにelifに修正すれば上手く条件分けできるようになります。

from openpyxl import load_workbook

wb = load_workbook(
    r"C:\Users\---\Desktop\MyPandas\test.xlsx")
ws = wb["Sheet1"]

print("Excelシートに入っているセルの文字列")
print("")
active_sheet = wb.active
for row in active_sheet.rows:
    excel = []
    for cell in row:
        excel.append(cell.value)
    print(excel)
# 格納するリスト
bb_clb = []
tf_clb = []
pic_clb = []


# 空行の定義
def is_empty(cell):
    return cell.value is None or not str(cell.value).strip()


# Excelシートで一列ごとに対応するリストへ格納
# 部活で格納するリストを分ける
for row in ws.iter_rows(min_row=5):
    if all(is_empty(c) for c in row):
        break
    elif (row[4].value == "野球部"):
        for col in row:
            bb_clb.append(col.value)
    elif (row[4].value == "陸上部"):
        for col in row:
            tf_clb.append(col.value)
    elif (row[4].value == "写真クラブ"):  # elseをelifに修正
        for col in row:
            pic_clb.append(col.value)

print("")
print("-------------------------------------------------")
print("")
print("実行結果")
print(bb_clb)
print(tf_clb)
print(pic_clb)

追記

因みにiter_rows()の引数にvalues_only=Trueを設定すると、cellオブジェクトではなくセルの中の値を受け取るようになるので代入の部分を簡単に書くことができます。
iter_rows() | openpyxlドキュメント

# 空行の定義
def is_empty(cell):
    return cell is None or not str(cell).strip()


# Excelシートで一列ごとに対応するリストへ格納
# 部活で格納するリストを分ける
for row in active_sheet.iter_rows(min_row=5, values_only=True):
    if all(is_empty(c) for c in row):
        break
    if row[4] == "野球部":
        bb_clb.extend(row)
    elif row[4] == "陸上部":
        tf_clb.extend(row)
    elif row[4] == "写真クラブ":
        pic_clb.extend(row)

Comments

[daichiddd]

ありがとうございます！無事解決できました。
こんな初歩的な所でつまずいてしまいお恥ずかしい限りです。。。

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

[退会済みユーザー]

お役に立てたようで何よりです。
期待通りに動かなかったときはデバッグして確認してみるといいかもしれませんね。

[daichiddd]

追記までしていただいて本当にありがとうございます。。。

pythonはおろかプログラミングも初学者なため、このような情報はとても参考になります。