実行したが何も表示されていない、エラーメッセージもなし

Question

前提

Pythonを用いて、Excelで集計を行っています。
フォルダ内にある複数（1000以上）のExcelファイルを1つのExcelファイルにまとめたいと考えています。
１段目では、ライブラリをインポートをしています。
２段目では、対象フォルダとその保存先、保存ファイル名を指定しています。
３段目では、フォルダ内にあるExcelファイルを列挙、各Excelを１つずつ、中身を次々と読んでいく。
４段目では、実行するプログラムです。
フォルダ内にある全てのExcelファイルの中身は52行×30列のデータで、このExcelファイルが1000個以上あります。このデータを新規Excelに1ファイルを1行で集計したいと考えています。成功例は画像のように1行目に1ファイル目、2行目に2ファイル目、・・・となるようにしたいです。
このプログラミング自体はネットから取ってきており、実際に使えることは1つのファイルを用いて確認済みなのですが、フォルダ内にある複数のExcelファイルで実行できるようにしたいのですが、上手くいきません。

実現したいこと

まとめとして、
フォルダ内にある全てのExcelファイルに4段目のプログラミングを実行できるようにしたいです。また、成功例として、画像のように1行に1ファイルとなるように集計したいと考えています。
どのようにすれば出来るのでしょうか。教えていただきたいです。

該当のソースコード

コード
import openpyxl as px
import os
import glob
from openpyxl import Workbook

#対象ファイルが格納されているフォルダ
target_dir = 'C:/Users/student/Documents/MyPython/hukusu'
#保存先
save_file = 'C:/Users/student/Documents/MyPython/hukusu/まとめ.xlsx'
#「まとめ.xlsx」作成
wb = px.workbook()
ws = wb.active

#「target_dir」にあるエクセルファイル一覧を取得
files = glob.glob(target_dir + '/*.xlsx')
#「target_dir」にあるエクセルファイルを読み込む
for fname in files:
    book = px.load_workbook(fname, data_only=True)
    sheet = book.active

#行をループして、コピー＆ペースト
    n=1
    t=30
    for i in range(1,53):
        for j in range(1,31):
            copy = ws.cell(row=i,column=j).value
            #j+t*(n-1)これにより1列目からスタートできる。２列目は+1する。
            ws.cell(row=1,column=j+t*(n-1)+1,value=copy)
        n+=1

wb.save(save_file)

Comments

[TakaiY]

「上手くいきません。」とはどういう状況なのでしょう。

「実際に使えることは1つのファイルを用いて確認済み」とあるときのプログラムも見せていただけますか。

[esg22]

import openpyxl as px
import os
import glob
from openpyxl import Workbook
​
path_name ='./**.xlsx'
load_path_name = './20140910-2300.xlsx'
​
for f in glob.glob(path_name):
wb = px.workbook()
ws = wb.active
ws.title='Sheet1'

wb1 = px.load_workbook(load_path_name)
ws1 = wb1['Sheet1']
​
n=1
t=30
for i in range(1,53):
for j in range(1,31):
copy = ws1.cell(row=i,column=j).value
#j+t*(n-1)これにより1列目からスタートできる。２列目は+1する。
ws.cell(row=1,column=j+t*(n-1)+1,value=copy)
n+=1
wb.save('matome.xlsx')

このコードで、1つのExcelファイルを用いたときは、1行に並びました。
これをフォルダ内にある複数のファイルで実行したいのですが、ソースコードに載せているコードでは作成されたExcelファイルに何も出力されていませんでした。これを参考に実行するプログラム以外を、フォルダ内にある複数のファイルで実行するように変更させました。

Answer 1

import openpyxl as px
import os
import glob
from openpyxl import Workbook

#対象ファイルが格納されているフォルダ
target_dir = 'C:/Users/student/Documents/MyPython/hukusu'
#保存先
save_file = 'C:/Users/student/Documents/MyPython/hukusu/まとめ.xlsx'
#「まとめ.xlsx」作成
wb = px.workbook()
ws = wb.active

#「target_dir」にあるエクセルファイル一覧を取得
files = glob.glob(target_dir + '/*.xlsx')
#「target_dir」にあるエクセルファイルを読み込む

cnt=1#ファイルを一つ処理するごろに１行ずらす
for fname in files:
    book = px.load_workbook(fname, data_only=True)
    sheet = book.active

#行をループして、コピー＆ペースト
    n=0
    t=30
    for i in range(1,53):
        for j in range(1,31):
            copy = sheet.cell(row=i,column=j).value

            ws.cell(row=cnt,column=j+(t*n)+1,value=copy)
        n+=1
    cnt+=1
wb.save(save_file)

Comments

[esg22]

wsの部分はそのままでも問題ないと思います。転記先についてはrow=iにしてしまうと1ファイルが1行でまとまらなくなってしまいます。
フォルダ内にある全てのExcelファイルに同じ処理させる方法を教えていただきたいです。

[pig_vba]

見落としてました。確かにiではなかったです。今なんファイル目かを表す変数が新たに必要ですね。

wsがそのままで問題ないとおっしゃる意味は理解しかねます。wsは新規作成した「まとめ.xlsx」のシートであって実際のデータが入っているエクセルファイルを管理しているのはbookとsheetなのですが…今のままだと新規作成した空ブックのデータの入ってないセルを延々とコピペする処理に見えるのですが

[esg22]

確かにwsはsheetに直さなければなりませんでした。訂正、ありがとうございます