[openpyxl] リストに格納した各セル行の文字列を貼り付けるときに、一つのセルにまとめて張り付けられてしまう

Question

前提・実現したいこと

pythonでExcel作業の自動化に取り組んでいます。
主にコピペ作業をさせたいです。
指定したセル行コピーをリストに格納することはできました。
しかしそれを張り付ける際に元のセル行の様にではなく、一つのセルにまとめて張り付けられてしまいます。

enumerate関数のところでつまずいていると思われますが、内容の理解不足もあり、どこが間違っていてこの出力結果になるのか分からないため質問させていただきます。

有識者の方、どうかお力添えください。

該当のソースコード

import os
import openpyxl as px
import re

print("< Sheet1 に入っているセルの文字列 >")
print("")

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

active_sheet_1 = wb.active
for row in active_sheet_1.rows:
    excel = []
    for cell in row:
        excel.append(cell.value)
    print(excel)

# 格納するリスト
bb_clb = []
tf_clb = []
pic_clb = []

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

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


for i, d in enumerate(tf_clb, start = 1):
    ws.cell(row = i,column = 1,value = str(d))
    
wb.save(file_name)

active_sheet_2 = wb.active
for row in active_sheet_2.rows:
    excel_cmp = []
    for cell in row:
        excel_cmp.append(cell.value)
    print(excel_cmp)

出力結果

< Sheet1 に入っているセルの文字列 >

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

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

< 実行結果 >

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

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

python 3.8.5
windows 10

Answer 1

以下の質問の続きですね。
[openpyxl] Excelシート内で任意の文字と完全一致するセルが存在する列を格納したい | teratail

---

しかしそれを張り付ける際に元のセル行の様にではなく、一つのセルにまとめて張り付けられてしまいます。

enumerate関数のところでつまずいていると思われますが

その前のfor文のところが間違っています。

for row in active_sheet_1.iter_rows(min_row = 5, values_only = True):
    if all(is_empty(c) for c in row):
        break
    if "野球部" in row[4]:
            bb_clb.append(row)
    elif "陸上部" in row[4]:
            tf_clb.append(row)
    elif "写真クラブ" in row[4]:
            pic_clb.append(row)

append()はリストに新しい要素を一つ作成してその要素に入力したデータを格納するメソッドです。
しかし上のコードの5,7,9行目では1行のデータが格納された変数を事前に作成したリストにappend()を使ってリストに追加してしまっています。
そのため行のデータがまとめて一つの要素に入ってしまい、そのまま一つのセルに書き込まれています。

なので以前の回答でも書いていたはずですが、行のデータ（リストやタプル）を要素ごとに分けて追加したい場合はappend()ではなくextend()を使う必要があります。

参考：Pythonでリスト（配列）に要素を追加するappend, extend, insert | note.nkmk.me

以下のように書き直せば要素ごとに分かれて書き込まれるようになります。

for row in sheet_1.iter_rows(min_row=5, values_only=True):
    if all(is_empty(c) for c in row):
        break
    if "野球部" in row[4]:
        bb_clb.extend(row)
    elif "陸上部" in row[4]:
        tf_clb.extend(row)
    elif "写真クラブ" in row[4]:
        pic_clb.extend(row)

　　
また書き込みのところですが、

for i, d in enumerate(tf_clb, start = 1):
    ws.cell(row = i,column = 1,value = str(d))

rowにインデックスの変数iを入れてしまうと縦に書き込まれてしまうので、以下のようにrowとcolumnの値を入れ替える必要があります。
あとstr()で書き込むデータを文字列に変換していますが、数字を文字列として書き込んでしまうとExcelでエラーが表示されてしまうのでstr()は外した方がいいです。

for i, d in enumerate(tf_clb, start=1):
    ws.cell(row=i, column=1, value=d)

---

内容の理解不足もあり、どこが間違っていてこの出力結果になるのか分からないため質問させていただきます。

デバッグという機能を使えばコードの処理を1行ずつ確認することができます。
もしお使いのエディタまたはIDEにデバッグ機能がついているなら、意図しない動作をした際はデバッグして1行ずつ確認することをお勧めします。

---

#####追記（コメントの質問について）

for a, tf_clb_list in enumerate(tf_clb, start = 1):
for i in range(1,3):
for j in range(1,6):
ws.cell(row = i, column = j,value = tf_clb_list)

このコードではリストの最初の要素が指定したすべてのセルに書き込まれ、次の要素がすべてのセルに書き込まれ、その次の要素が・・・となってしまっています。

方法としてはいくつか方法がありますが、上のコードを生かすなら以下のようにカウンタ変数を利用する感じでしょうか。

counter = 0
for i in range(1, 3):
    for j in range(1, 6):
        ws.cell(row=i, column=j, value=tf_clb[counter])
        counter += 1

ただカウンタ変数を使うのはあまりよくありませんし、データの数が変わったらrange()の部分をいちいち変更する必要がります。

個人的には以下のように行のデータをappend()で各要素に格納して二次元のリストにしてからセルに書き込む方がいいと思います。
上の方法では同じ部活に所属している人のデータが一列の格納されていましたが、この方法では行ごとにデータがきちんと分けられているのでお行儀がいいですし、カウンタ変数は必要ありません。

for row in active_sheet_1.iter_rows(min_row=5, values_only=True):
    if all(is_empty(c) for c in row):
        break
    if "野球部" in row[4]:
        bb_clb.append(row)
    elif "陸上部" in row[4]:
        tf_clb.append(row)
    elif "写真クラブ" in row[4]:
        pic_clb.append(row)

print("")
print("-------------------------------------------------")
print("")
print("< 実行結果 >")
print("")


for i, row in enumerate(tf_clb, 1):
    for j, elem in enumerate(row, 1):
        ws.cell(row=i, column=j, value=elem)

最終行に書き込んでもいいなら以下のようにワークシートオブジェクトのappend()を使うこともできます。

for row in sheet_1.iter_rows(min_row=5, values_only=True):
    if all(is_empty(c) for c in row):
        break
    if "野球部" in row[4]:
        bb_clb.append(row)
    elif "陸上部" in row[4]:
        tf_clb.append(row)
    elif "写真クラブ" in row[4]:
        pic_clb.append(row)

print("")
print("-------------------------------------------------")
print("")
print("< 実行結果 >")
print("")


for bb_clb_elem in bb_clb:
    ws.append(bb_clb_elem)

新しいシートに書き込むサンプルも載せておきます（少し修正加えてます）。
参考にしてみてください。

import openpyxl as px


# シートのデータをすべて表示
def print_ws_data(ws):
    for row in ws.iter_rows(values_only=True):
        print(row)


# 空行かどうか判定
def is_empty(cell):
    return cell is None or not str(cell).strip()


print("< Sheet1 に入っているセルの文字列 >")
print("")

file_name = r"C:\Users\---\Desktop\MyPandas\test.xlsx"
wb = px.load_workbook(file_name)

sheet_1 = wb.worksheets[0]

print_ws_data(sheet_1)

# 格納するリスト
bb_clb = []
tf_clb = []
pic_clb = []

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

print("")
print("-------------------------------------------------")
print("")
print("< 実行結果 >")
print("")

ws_2 = wb.create_sheet()

for tf_clb_elem in tf_clb:
    ws_2.append(tf_clb_elem)

wb.save(file_name)

print_ws_data(ws_2)

Comments

[daichiddd]

前回に続き回答ありがとうございます！！！！！

無事一つのセルにすべての要素が転記されることはなくなりました。
extendは見落としていました、、
申し訳ありません。

参考もありがとうございます！

[daichiddd]

お時間あればでよろしいのですが、、、
加えて質問よろしいでしょうか。

一つのセルに転記されることはなくなったのですが、一つの行にすべてのリスト内のデータが転記されるようになりました。
↓実行結果

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

[daichiddd]

これに対し自分はこのようにコードを編集したのですが、結果は陸上部リストに格納した最後のデータが転記されるようになりました。

↓コード

for a, tf_clb_list in enumerate(tf_clb, start = 1):
for i in range(1,3):
for j in range(1,6):
ws.cell(row = i, column = j,value = tf_clb_list)

[daichiddd]

↓出力結果
< 実行結果 >

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

[daichiddd]

理想の実行結果は以下です

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

[daichiddd]

理想の実行結果を得るためにはどのような編集が必要ですか。

enumerateのインデックス番号をうまく利用する方法があるのでしょうか。
調べてもあまりいい例がないので、もしよければ教えていただきたいです。。

[退会済みユーザー]

分かりました。
追記しますね。

[daichiddd]

質問への回答ありがとうございます。

appendとexpendで要所に分けて使い方を変えるんですね、、とても参考になります、、、！

コードで冗長な箇所も修正していただいて、感謝してもし足りないです。。

[daichiddd]

デバッグ等さまざまなスキルアップの知識も頂いたのでもっと勉強したいと思います。
またの機会があればよろしくお願いします。