Pythonでwhile文のループ時に挙動が変わってしまう

Question

実現したいこと

Python初心者です。初書き込み失礼します。
Pythonを使用してエクセルを開いたままエクセル上で更新されるデータを同じシートの別のセルに転記してそのシートをcsvファイルとして保存し、csvデータを読み込むためのプログラムを作成中です。

発生している問題・分からないこと

while文にてClass1～８をループさせているのですが、Class6の挙動が１回目と２回目で違ってしまい、２回目の挙動が望まない動作をしてしまいます。
具体的にはClass5が実行されたあとはエクセルへのデータ転記→Sheet0.csvの保存→Sheet1.csvの保存となるはずなのですが、２回目のwhile文の動作でSheet0、Sheet1の保存→データの転記→もう一度Sheet0、Sheet1の保存となり同名データがあるため上書き保存の警告で動作が中断します。1回目の動作と同じ挙動にしたいです。宜しくお願いします。

エラーメッセージ

エラーメッセージはありません。

該当のソースコード

import os
import xlwings as xw
import win32com.client


# Class1
class FileDeleterSheet2:
    def __init__(self, file_path_sheet2):
        self.state = 'initial'  # 初期状態
        self.file_path_sheet2 = file_path_sheet2

    # Sheet2.csvを削除
    def delete_file_sheet2(self):
        try:
            if os.path.exists(self.file_path_sheet2):
                os.remove(self.file_path_sheet2)
                print(f"File {self.file_path_sheet2} has been deleted successfully.")
            else:
                print(f"File {self.file_path_sheet2} does not exist.")
        except PermissionError:
            print(f"Permission denied. Cannot delete {self.file_path_sheet2}.")
        except Exception as e:
            print(f"An error occurred: {e}")

        self.state = 'updated'  # インスタンス変数を更新

        print("Class1実行完了")
            

# 使用例
file_path_sheet2= "C:\\Users\\PC_User\\Desktop\\test_file_dummy\\Sheet2.csv"
deleter = FileDeleterSheet2(file_path_sheet2)
deleter.delete_file_sheet2()



# Class2
class DummyFileCreator:
    def __init__(self, sheet_name, dummy_folder, save_folder):
        self.state = 'initial'  # 初期状態
        self.wb = xw.books.active
        self.sht = xw.sheets[sheet_name]
        self.dummy_folder = dummy_folder
        self.save_folder = save_folder
        self.dummy_file_path = None
        self.csv_file_path = None

    # フォルダ内のファイル数をカウントする
    def count_files(self):
        try:
            file_list = os.listdir(self.dummy_folder)
            return len(file_list)
        except FileNotFoundError:
            return 0
        
    # Sheet(ダミーファイル　+　２)の名前に変更するための設定    
    def save_dummy_csv(self):
        file_count = self.count_files()
        dummy_file_name = f'Sheet{file_count + 2}.csv'
        self.dummy_file_path = os.path.join(self.dummy_folder, dummy_file_name)
        self.sht.api.SaveAs(self.dummy_file_path, FileFormat=6)
        # ここで CSV ファイルを保存する処理を実装してください
        print(f"ダミーファイルSheet{file_count +2}を作成しました")

        self.state = 'updated'  # インスタンス変数を更新

        print("Class2実行完了")


# 使用例
if __name__ == "__main__":
    dummy_folder_path = "C:\\Users\\PC_User\\Desktop\\test_file_dummy"
    save_folder_path = "C:\\Users\\PC_User\\Desktop\\test_file_save"
    handler = DummyFileCreator("Sheet1", dummy_folder_path, save_folder_path)
    handler.save_dummy_csv()




# Class3
class FileDeleterSheet0:
    def __init__(self, file_path_sheet0):
        self.state = 'initial'  # 初期状態
        self.file_path_sheet0 = file_path_sheet0

    # Sheet0.csvを削除
    def delete_file_sheet0(self):
        try:
            if os.path.exists(self.file_path_sheet0):
                os.remove(self.file_path_sheet0)
                print(f"File {self.file_path_sheet0} has been deleted successfully.")
            else:
                print(f"File {self.file_path_sheet0} does not exist.")
        except PermissionError:
            print(f"Permission denied. Cannot delete {self.file_path_sheet0}.")
        except Exception as e:
            print(f"An error occurred: {e}")

        self.state = 'updated'  # インスタンス変数を更新

        print("Class3実行完了")    

# 使用例
file_path_sheet0= "C:\\Users\\PC_User\\Desktop\\test_file_save\\Sheet0.csv"
deleter = FileDeleterSheet0(file_path_sheet0)
deleter.delete_file_sheet0()



# Class4
class FileDeleterSheet1:
    def __init__(self, file_path_sheet1):
        self.state = 'initial'  # 初期状態
        self.file_path_sheet1 = file_path_sheet1

    # Sheet1.csvを削除
    def delete_file_sheet1(self):
        try:
            if os.path.exists(self.file_path_sheet1):
                os.remove(self.file_path_sheet1)
                print(f"File {self.file_path_sheet1} has been deleted successfully.")
            else:
                print(f"File {self.file_path_sheet1} does not exist.")
        except PermissionError:
            print(f"Permission denied. Cannot delete {self.file_path_sheet1}.")
        except Exception as e:
            print(f"An error occurred: {e}")

        self.state = 'updated'  # インスタンス変数を更新
        print("Class4実行完了")    

# 使用例
file_path_sheet1 = "C:\\Users\\PC_User\\Desktop\\test_file_save\\Sheet1.csv"
deleter = FileDeleterSheet1(file_path_sheet1)
deleter.delete_file_sheet1()




# Class5
class SheetRenamer:
    def __init__(self, sheet_index, new_name):
        self.state = 'initial'  # 初期状態
        self.xl_app = win32com.client.GetObject(Class='Excel.Application')
        self.workbook = self.xl_app.Workbooks(1)  # 1 番目のワークブックを指定
       

    def rename_sheet(self, sheet_index, new_name):
        try:
            sheet = self.workbook.Sheets(sheet_index)
            sheet.Name = new_name
            print(f"Sheet {sheet_index} の名前を {new_name} に変更しました。")
        except Exception as e:
            print(f"エラー: {e}")

        self.state = 'updated'  # インスタンス変数を更新
        print("Class5実行完了")

# クラスをインスタンス化
renamer = SheetRenamer(sheet_index=1, new_name='Sheet1')
renamer.rename_sheet(sheet_index=1, new_name='Sheet1')





# Class6
class Class6Test6:
    def __init__(self, sheet_name, output_folder):
        self.state = 'initial'  # 初期状態
        
        self.wb = xw.books.active
        self.sht = xw.sheets[sheet_name]
        self.output_folder = output_folder
        self.file_number = 1
        self.count = 0
    

    def job(self):
        a1value = self.sht.range('A1').value
        self.sht.range('A' + str(self.sht.cells.last_cell.row)).end('up').offset(1, 0).value = a1value # 1
        a1value = self.sht.range('B1').value
        self.sht.range('B' + str(self.sht.cells.last_cell.row)).end('up').offset(1, 0).value = a1value # 2
        
        

        print(f"Job executed: {a1value}")  # デバッグ用プリント文
      
        
    def save_csv(self):
        csv_file_name1 = f'Sheet{self.file_number - 1}.csv'  #Sheet0のCSV保存設定
        csv_file_path1 = os.path.join(self.output_folder, csv_file_name1)

        csv_file_name2 = f'Sheet{self.file_number}.csv'  #Sheet1のCSV保存設定
        csv_file_path2 = os.path.join(self.output_folder, csv_file_name2)


        # シート保存とクリアタイミングの設定
        self.sht.api.SaveAs(csv_file_path1, FileFormat=6) #Sheet0のCSV保存
        self.sht.api.SaveAs(csv_file_path2, FileFormat=6) #Sheet1のCSV保存
        print(f"Saved CSV: {csv_file_path1}, {csv_file_path2}")  # デバッグ用プリント文

        last_row = self.sht.cells.last_cell.row 
        if last_row >= 3:
            self.sht.range(f'A2:B{last_row}').clear_contents()
        self.file_number += 1
        self.count = 0


    def run(self):  #実行ファイルメソッド
        for _ in range(11):  # 11回のイテレーションを実行
            self.job()  # self.job メソッドまたは関数を実行
            print(f"Iteration: {_}")  # デバッグ用プリント文

        # 最後にシートを保存
        self.save_csv()
        self.state = 'updated'  # インスタンス変数を更新

        print("Class6実行完了")



# 使用
if __name__ == "__main__":
    exporter = Class6Test6(sheet_name="Sheet1", output_folder="C:\\Users\\PC_User\\Desktop\\test_file_save")
    exporter.run()



# Class7
class Class7Test7:
    def __init__(self):
        self.state = 'initial'

    def test7(self):

        print("test7")

        self.state = 'updated'  # インスタンス変数を更新

        print("Class7実行完了")

class7test7 = Class7Test7()
class7test7.test7()


# Class8
class Class8Test8:
    def __init__(self):
        self.state = 'initial'  

    def test8(self):

        print("test8")

        self.state = 'noneupdated'  # インスタンス変数を更新しない

        print("Class8実行完了")

class8test8 = Class8Test8()
class8test8.test8()


def main():
    class1_instance = FileDeleterSheet2(file_path_sheet2)
    class2_instance = DummyFileCreator("Sheet1", dummy_folder_path, save_folder_path)
    class3_instance = FileDeleterSheet0(file_path_sheet0)
    class4_instance = FileDeleterSheet1(file_path_sheet1)
    class5_instance = SheetRenamer(sheet_index=1, new_name='Sheet1')
    class6_instance = Class6Test6(sheet_name="Sheet1", output_folder="C:\\Users\\PC_User\\Desktop\\test_file_save")
    class7_instance = Class7Test7()
    class8_instance = Class8Test8() 


    while class8_instance.state != 'updated':
        class1_instance.delete_file_sheet2()
        class2_instance.count_files()
        class2_instance.save_dummy_csv()
        class3_instance.delete_file_sheet0()
        class4_instance.delete_file_sheet1()
        class5_instance.rename_sheet(sheet_index=1, new_name='Sheet1')
        class6_instance.job()
        class6_instance.save_csv()
        class6_instance.run()
        class7_instance.test7()
        class8_instance.test8()
        
        break
 

    print("Done!")

if __name__ == "__main__":
    main()

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

原因が不明で、ＡＩチャットでもお手上げでした。

補足

特になし

Comments

[TakaiY]

質問にあるコードは1つのファイルですか？
そして、このファイルを実行していますか？

[pythonwakaran]

コメントありがとうございます。該当のソースコードで完結した一つの.pyファイルです。実行するとwhileによるループ時にClass6にて最初の挙動と違う動作をします。

[pythonwakaran]

質問者です。文字数オーバーで記載できなかったので問題箇所のターミナル表示をここのコメント欄に記載させて頂きます。（class5からclass6までの動作）

一回目の動作
Class5実行完了
Job executed: test2
Iteration: 0
Job executed: test2
Iteration: 1
Job executed: test2
Iteration: 2
Job executed: test2
Iteration: 3
Job executed: test2
Iteration: 4
Job executed: test2
Iteration: 5
Job executed: test2
Iteration: 6
Job executed: test2
Iteration: 7
Job executed: test2
Iteration: 8
Job executed: test2
Iteration: 9
Job executed: test2
Iteration: 10
Saved CSV: C:\Users\PC_User\Desktop\test_file_save\Sheet0.csv, C:\Users\PC_User\Desktop\test_file_save\Sheet1.csv
Class6実行完了

2回目の動作
Class5実行完了
Job executed: test2
Saved CSV: C:\Users\PC_User\Desktop\test_file_save\Sheet0.csv, C:\Users\PC_User\Desktop\test_file_save\Sheet1.csv
Job executed: test2
Iteration: 0
Job executed: test2
Iteration: 1
Job executed: test2
Iteration: 2
Job executed: test2
Iteration: 3
Job executed: test2
Iteration: 4
Job executed: test2
Iteration: 5
Job executed: test2
Iteration: 6
Job executed: test2
Iteration: 7
Job executed: test2
Iteration: 8
Job executed: test2
Iteration: 9
Job executed: test2
Iteration: 10

ここでエクセルの上書き保存警告にて動作が中断されます。

以上になります。引き続きよろしくお願いします。

[bsdfan]

プログラム全体での問題部分についてはすでに回答ついている通りと思いますが、上書き保存警告でプログラムが止まることについては、display_alerts が、関係すると思います。
https://docs.xlwings.org/ja/latest/api/app.html#xlwings.App.display_alerts

[pythonwakaran]

コメントありがとうございます。display_alertsというものがあったのですね。知りませんでした。今回は解決したので、次回上書き保存警告で困った場合は一度検討してみます！

Answer 1

該当のソースコードで完結した一つの.pyファイルです

とのことなので、まずは、ざっと見たところのコードの問題点を指摘します。

このコードは複数のソースコードを1つにまとめたものだと推測します。
元になったファイルは、ぞれぞれのパートを実行可能なように作ってあったのだと 思われます。このようなファイルは単に結合してしまうと、元のファイルの実行部分をひきづっているため、予期しない動作をします。

なので、まずは、余計な動作をしていると思われる部分をすべて削除する必要があると思います。
また、削除候補にした箇所には、main関数で使っている変数の定義も含まれているので、それらの定義はmain関数の中に移すのがいいでしょう。

削除部分の例をあげると

30行目から

# 使用例
file_path_sheet2= "C:\\Users\\PC_User\\Desktop\\test_file_dummy\\Sheet2.csv"
deleter = FileDeleterSheet2(file_path_sheet2)
deleter.delete_file_sheet2()

70行目から

# 使用例
if __name__ == "__main__":
    dummy_folder_path = "C:\\Users\\PC_User\\Desktop\\test_file_dummy"
    save_folder_path = "C:\\Users\\PC_User\\Desktop\\test_file_save"
    handler = DummyFileCreator("Sheet1", dummy_folder_path, save_folder_path)
    handler.save_dummy_csv()

間にもいくつもあり、最後は260行目

class8test8 = Class8Test8()
class8test8.test8()

これらの部分は、それぞれのClassの必要な処理をこのタイミングで起動しています。
それは本来ここでは実施すべきでないところだと思います。

これらを消すことで、やっと main関数にある処理のみ実施されるようになります。

Comments

[pythonwakaran]

コメントありがとうございます。すぐにはできそうにないので明日以降上記の削除・修正を試してみます！解決もしくは疑問あったらまたコメント等（解決したらベストアンサー）させて頂きます！

[pythonwakaran]

遅くなりましたが、問題は解決しました！不要部分を削除してmain関数にできるだけ以降したところ、while文のループ挙動だけになり、そこでwhile文の挙動のほうがおかしいことに気付いたので、ためしにclass6の
class6_instance.job()
class6_instance.save_csv()
の二つを停止させて
class6_instance.run()
だけを動作させたら、望んでいた挙動になりました！
ご助力頂き、ありがとうございました！！