エクセルのコピーについて

Question

前提・実現したいこと

pythonを勉強中です。プログラム自体はじめてなのでいろいろ調べながら勉強していますが，下記について調べ方が悪いのか，原因がわからなかったため質問させてください．

横2列(X軸，Y軸の値)，縦に1040行(xyの座標の個数)があるExcelデータがあるとします．
2行4列(合計8セル，4点の座標)ごとにデータをコピーし，それを同じシートの4列目以降に1行になるように複製して，ペーストする処理を元データの最終行まで行いたいと考えています．2行4列のデータを定数を用いて，一行に並べる処理はできたのですが，それを最終行まで繰り返させる処理がどうしても実行されません．どうすればできるのか，アドバイス頂きたいと思います．よろしくお願いいたします．

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

エラーメッセージは特に出てきませんがコピーできていない状況です．

該当のソースコード

import os
os.chdir('C:/Users/XXX/Python/0407')

import openpyxl as px
wb = px.load_workbook('C:/Users/XXX/Python/0407/DJI_0001_01.xlsx')
ws = wb['Sheet1']

def act(A): #1行分の列部分を取得する関数(一つ目)
    copyx1 = ws.cell(row = A, column =1).value #x1
    copyy1 = ws.cell(row = A, column =2).value #y1       

    for j in range(0,A):#抜き取った部分を複製する　後
        ws.cell(row = j,column = 4).value = copyx1 
        ws.cell(row = j,column = 5).value = copyy1 
         
    max=int(ws.max_row) #最終行数の算出(1040行)
    for A in range(1,max,4): #一行目から4つ飛ばしで最終行まで
        act(A)


def act(B): #1行分の列部分を取得する関数
    copyx2 = ws.cell(row = B, column =1).value #x2
    copyy2 = ws.cell(row = B, column =2).value #y2       

    for j in range(0,B):#抜き取った部分を複製する　後
        ws.cell(row = j,column = 6).value = copyx2 
        ws.cell(row = j,column = 7).value = copyy2 

    max=int(ws.max_row) #最終行数の算出(1040行)
    for B in range(2,max,4): #2行目から最終行まで4行飛ばしながら実行
        act(B)


def act(C): #3つ目
    copyx3 = ws.cell(row = C, column =1).value #x3
    copyy3 = ws.cell(row = C, column =2).value #y3

    for j in range(0,C):#抜き取った部分を複製する　後
        ws.cell(row = j,column = 8).value= copyx3 
        ws.cell(row = j,column = 9).value = copyy3 

    max=int(ws.max_row) #最終行数の算出(1040行)
    for C in range(3,max,4): #3行目から最終行まで4行飛ばしながら実行
        act(C)


def act(D): #4つ目
    copyx4 = ws.cell(row = D, column =1).value #x4
    copyy4 = ws.cell(row = D, column =2).value #y4
       
    for j in range(0,D):#抜き取った部分を複製する　後
        ws.cell(row = j,column = 10).value= copyx4 
        ws.cell(row = j,column = 11).value = copyy4 
         
    max=int(ws.max_row) #最終行数の算出(1040行)
    for z in range(4,max,4): #4行目から最終行まで4行飛ばしながら実行
        act(D)

wb.save(filename = 'New.xlsx')

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

Python 3.7.9

Comments

[KojiDoi]

横2列のデータから4列のデータをコピーするとはどういうことでしょうか?

[退会済みユーザー]

質問文及び質問中のコードだけでは、完成形がわかりません。

「4点の座標ごとにデータをコピーし，それを同じシートの4列目以降に1行になるように複製、それを最終行まで繰り返させる」とは、
最終的には、2列1040行の2080個のデータを、
同じシートのD4セル以降に、1行で2080列にわたって表示させるという意味でしょうか。

それとも、

セルD1～K1　の８セルに、セルA1～B4までの8個のデータ、
セルD5～K5　の８セルに、セルA5～B8までの8個のデータ、
セルD9～K9　の８セルに、セルA9～B12までの8個のデータ・・・

という形で表示したいのでしょうか？

[tohoho3]

不親切な質問の仕方で大変失礼いたしました．
追加掲載の写真の通り，AB列において，4列ずつデータを抽出し，1行に並べる作業をAB列のセルにデータがなくなるまで繰り返し行いたいと考えております．

[tohoho3]

qnior さん
セルD1～K1　の８セルに、セルA1～B4までの8個のデータ、
セルD2～K2　の８セルに、セルA5～B8までの8個のデータ、
セルD3～K3　の８セルに、セルA9～B12までの8個のデータ・・
という形での表示をしたいと考えています．
分かりにくい説明で大変失礼いたしました．

Answer 1

A1セル以降から取得した2列4行ずつのデータを8行ごとに改行して書き込む場合下記のようになります。

import openpyxl
wb = openpyxl.load_workbook('C:/Users/XXX/Python/0407/DJI_0001_01.xlsx')
ws = wb['Sheet1']

max = int(ws.max_row)

w = 0
for row in ws.iter_rows(min_col=1, min_row=1, max_col=2, max_row=max):

    for cell in row:
        value = cell.value
        ws.cell(column=w%8+4, row=w//8+1).value = value
        w+=1
    
wb.save('New.xlsx')

Comments

[tohoho3]

データのコピーができました．ありがとうございます．

Answer 2

とりあえず、「どうしても実行されません．」の直接の答えを書いておくと、関数を定義だけして呼び出していないため、何も起こっていないと思います。関数の使い方が基本的に理解できてないようです。

そして、（質問に書いた通り、何をやりたいのかが今ひとつはっきりしないけれども）おそらくこの処理にわざわざ関数を使う必要はありません。
また、明らかに同じような処理を行う関数を４つも定義しているという構造もちょっとありえないことです。

defのことはいったん忘れて、まずはforだけを使って書いてみてはどうでしょうか。

【Python】ファイルの読み込み - Qiita

ここに書いてある「すべての行を読み込んでリストにし、一行ずつ表示する方法」をとりあえず出発点にしてみるといいのではないかと思います。