Python for文について

Question

下記プログラム内容ですが、コピー先ブックを開いた後に、別ブックの行を上から順番にfor文で回し、一行目の値を抽出したらその値をコピー先ブックへ貼付けし、上書き保存するということをしたいと考えています。コピー元の値はあくまで、一行ずつで、一行コピー貼付け、保存、次のブックを開いて、先ほどコピーした次の行を、コピー貼付け、保存・・・という流れを作成したいです。
私が考えた内容だと、コピー元の行の値をすべて拾ってきて、各ファイルに貼付けするような結果しか得られませんでした。if文を使用して条件指定をし、ファイルを開くたび、行を指定したいのですが、どのようにしていいかわかりません。

このように記述すると、for文で読んできた番号の行のみを出力することはわかりましたが、このif b != 4:の数字の4の部分をfor b in range(4,Sheet_Max2,1):このfor文で値をとってくるたび同じように数字を4,5,6,7・・・と増やしていければいいのかと思い色々試行錯誤して実行しましたが結局自分ではできませんでした。
for b in range(4,Sheet_Max2,1):
if b != 4:
break

お手数おかけしますが、どのようにしたらいいかご教授お願いいたします。

#コピー先ファイル一つずつ開く
for key, value in so_dict2.items():
    for v in value:
        wb = openpyxl.load_workbook(f'C:\テスト用\個別表2\{key}\{v}.xlsx')
        ws = wb[sheet_name]#コピー先シート取得
        Sheet_Max2 = ws1.max_row#シートの最終行を取得する コピー元シート
 #コピー元ファイルの列行を取得する
        for b in range(4,Sheet_Max2,1):
            if ws1.cell(row=b, column=3).value is None:
                break
            Sheet_Max1 = ws.max_row#シート最終行を取得　コピー先シート
            for a in range(Sheet_Max1, 0, -1):
                if ws.cell(row=a, column=2).value is not None:
                    ws.cell(row=a+1,column=2).value = ('料金')
                    kensin = ws1.cell(row=b,column=3).value#セルコピー
                    ws.cell(row=a+1,column=3,value=kensin)#セル貼付け
                    break
        wb.save(f'C:\テスト用\個別表2\{key}\{v}.xlsx')#上書き保存

Answer 1

ループの入れ子として実装するのは無理があると思いますよ。先に進む前に一呼吸置いて、根本的にもっと分かりやすい方法がないか、見直すべきだと思います。

処理が入れ子になるのはなるべく避けて、独立した小さな処理ブロックに分けるようにした方が良いと思います。
そして個々の処理ブロックは、自分が理解できる範囲の処理の大きさに収めるようにした方が良いと思います。
そうでないと、先に行って、改修したくなったり不具合があったりしたときに、自分では何もできなくなってしまうと思いますよ。

質問文に記載されているコードの範囲で思いつくのは zip 関数を使う方法ですが、全体的に見直したときにそれが一番いい方法かどうかも、私には分かりません。参考までに考え方のみ下に示しておきますが…
（コピー先ファイルの数と、転記したい値の数は同じで、抽出したリストの中での位置も一致しているという前提です）
（説明のために一部変数名を日本語にしています）
（コピー先ファイルの最終行を見つけるコードも関数にして、本体のロジックから切り離した方がいいと思います）

# Excel ワークシートの特定列を指定して値のある最終行を見つける関数
def max_row_with_value(ws, column):
    for i in range(ws.max_row, 0, -1):
        if ws.cell(row=i, column=column).value is not None:
            return i

# コピー先ファイルパスを取得
コピー先ファイルパスのリスト = []
for key, value in so_dict2.items():
    for v in value:
        コピー先ファイルパスのリスト.append(f'C:\テスト用\個別表2\{key}\{v}.xlsx')

# 転記する値を取得
転記する値のリスト = []
Sheet_Max2 = ws1.max_row
for b in range(4, Sheet_Max2 + 1, 1):
    copy_value = ws1.cell(row=b, column=3).value
    if copy_value is None:
        break
    転記する値のリスト.append(copy_value)

# コピー先ファイルに値を転記
for ファイルパス, 転記する値 in zip(コピー先ファイルパスのリスト, 転記する値のリスト):
    wb = openpyxl.load_workbook(ファイルパス)
    ws = wb[sheet_name]
    max_row = max_row_with_value(ws, column=2)
    ws.cell(row=max_row+1, column=2).value = ('料金')
    ws.cell(row=max_row+1, column=3, value=転記する値)
    wb.save(ファイルパス)

Comments

[退会済みユーザー]

ethe rbeg様
お世話になっております。
出先ですので、戻り次第パソコンで調べながら内容の確認いたします。
見ただけでは全然理解出来ませんでした。
私の知識不足です。
ご指摘誠にありがとうございます。
勉強させて頂きます。

Answer 2

for b in range(4,Sheet_Max2,1):の4がどこから来たか分かりませんが…

4が初期値であれば、forの一番外側で、int_var = 3として置いて、for文で4から増やしたいたびにint_var+=1とすれば、4の部分が都度増えていきます。
for b in range(int_var,Sheet_Max2,1):
という形で試してください。

3で初期化して、forの度に1足したらforの初期値は4になりますね。

int_var = 3 # <---- 追加 


#コピー先ファイル一つずつ開く
for key, value in so_dict2.items():
    for v in value:
        wb = openpyxl.load_workbook(f'C:\テスト用\個別表2\{key}\{v}.xlsx')
        ws = wb[sheet_name]#コピー先シート取得
        Sheet_Max2 = ws1.max_row#シートの最終行を取得する コピー元シート
 #コピー元ファイルの列行を取得する


        int_var+=1 # <---- 追加
        # for b in range(4,Sheet_Max2,1):
        for b in range(int_var ,Sheet_Max2,1): # <---- 修正
            if ws1.cell(row=b, column=3).value is None:
                break
            Sheet_Max1 = ws.max_row#シート最終行を取得　コピー先シート
            for a in range(Sheet_Max1, 0, -1):
                if ws.cell(row=a, column=2).value is not None:
                    ws.cell(row=a+1,column=2).value = ('料金')
                    kensin = ws1.cell(row=b,column=3).value#セルコピー
                    ws.cell(row=a+1,column=3,value=kensin)#セル貼付け
                    break
        wb.save(f'C:\テスト用\個別表2\{key}\{v}.xlsx')#上書き保存

Comments

[退会済みユーザー]

fourteenlength様
ご返信ありがとうございます。
4の値は、行を４行目から取得したかったためにそのようにしています。
早速教えていただいた内容を実行しました。
指定行からの値を拾ってきてはいるのですが、その後ろの値も入ってきてしまいます。
１番目のブック　には４行目を貼付け
２番目のブック　には５行目を貼付け
としたいのですが、今の内容だと、

１番目のブックには　４行目以降すべての行が貼付けされ
２番目のブックには　５行目以降すべての行が貼付け
３番目のブックには　６行目以降すべての行が貼付け

という形になっています。
どのように修正したらよろしいでしょうか。
お手数おかけしますがご教授お願いいたします。

[退会済みユーザー]

fourteenlength様
for b in range(int_var,Sheet_Max2,1):#<---- 修正
if b != int_var:
break

上記内容で、一行ずつ取得できました。
数日悩んでおりましたので大変うれしいです。

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