Pythonコードを実行しても思い通りにならない

Question

実現したいこと

Excelのデータリストから、各条件にあった行を新規シートに抽出したい

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

新規シートは作成できているが、ヘッダーのみしか表示されない

エラーメッセージ

エラーはありませんが、結果が違います

該当のソースコード

import openpyxl
import os

# Excelファイルのパス
file_path = r"C:\Users\ozawa\Documents\python練習コード.xlsx"

# ファイルの存在を確認
if not os.path.exists(file_path):
    print(f"エラー: ファイル '{file_path}' が存在しません。")
    exit()

try:
    # 読み込み
    workbook = openpyxl.load_workbook(file_path)

    # コピー元シートの取得
    source_sheet = workbook["データ一覧"] # シート名は必要に応じて変更

    # コピー先シートの作成
    if "抽出結果" not in workbook.sheetnames:
        target_sheet = workbook.create_sheet("抽出結果")
        header = [cell.value for cell in source_sheet[1]] # ヘッダー行をコピー
        target_sheet.append(header)
    else:
        target_sheet = workbook["抽出結果"]
        if target_sheet.max_row == 0:
            header = [cell.value for cell in source_sheet[1]]
            target_sheet.append(header)


            data9_value = row[10].value # 目的 (11列目)

            # data9が空白の場合に9を設定
            if data9_value is None or str(data9_value).strip() == "":
                data9 = 9
            else:
                try:
                    data9 = int(data9_value)
                except ValueError:
                    data9 = 0 # 数値に変換できない場合は0を設定

            data4 = row[4].value #  (5列目)
            data7 = row[8].value #  (9列目)

            # 値を数値に変換 (エラー処理付き)
            data8 = row[9].value #  (10列目)

            if data8 is None or str(data8_value).strip() == "":
                data8 = 9
            else:
                try:
                    data8 = int(data8_value)
                except ValueError:
                    data8 = 0 # 数値に変換できない場合はスキップ

            if data7 is None or str(data7).strip() == "":
                data7 = 9
            else:
                try:
                    data7 = int(data7)
                except ValueError:
                    data7 = 0
                    
            # 条件に合致する行を抽出
            if data9 == 1 and data4 == 1 and data7 == 15001 and data9 == 15001:
                for i, value in enumerate(values): # enumerateを使ってindexを取得
                    target_sheet.cell(row=row_num, column=i + 1).value = value
                row_num += 1

    # 保存
    workbook.save(file_path)

    print("抽出とコピーが完了しました。")

except KeyError as e:
    print(f"エラー: シート '{e}' が存在しません。")
except Exception as e:
    print(f"予期せぬエラー: {e}")

試したこと・調べたこと

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

上記の詳細・結果

自分になりにGeminiを使って修正したりして、エラー表示はなくなりましたが、ヘッダーのみしか表示されませんでした

補足

どんなコードであれば、キチンと実行できるのかも知りたいです.
よろしくお願いします

Comments

[YAmaGNZ]

「抽出結果」というシートがない状態で実行しているかと思います。
その場合はif文にてどこが実行されるのかを考えてみるといいかと思います。

[k161811]

ありがとうございます。
つまり、今のコードのままだと、＃コピー先シートの作成欄のelse以降にコードを記入しているため、if文のヘッダー作成のみで終了している、ということでしょうか。

よろしくお願いします

[meg_]

質問タグの「VBA」はどう関係していますか？関係しているのであれば質問文に書いてください。無関係であれば質問タグから削除してください。

[YAmaGNZ]

>つまり、今のコードのままだと、＃コピー先シートの作成欄のelse以降にコードを記入しているため、if文のヘッダー作成のみで終了している、ということでしょうか。

そうですね。
「抽出結果」というシートが存在した場合にどのような動作を求めているのかは分かりませんが、シートの有無にかかわらず抽出したいのであればif文での条件とは別に実行すべきかと思います。

[k161811]

ありがとうございます。

自分としては、
else:
target_sheet = workbook["抽出結果"]
以降にあるif文を「抽出結果」というシートが存在した場合に実行したいと考えております。
よろしくお願いします

[k161811]

ありがとうございます。

「VBA」は、VBA・マクロで行うとすれば、どのようなコードになるのかを知りたいためです。
質問タグの操作が分かりません。

よろしくお願いします

[YAmaGNZ]

「抽出結果」というシートが存在した場合はそのシートの最後尾から追記するといったことでしょうか？
それともそのシートをクリアして最初からということでしょうか？
また
else:
　　target_sheet = workbook["抽出結果"]
以降にあるif文というのは「抽出結果」というシートが存在しなかった場合には実行しなくていいのでしょうか？
そのあたりを考えてif文を構築する必要があるかと思います。

if 「抽出結果」というシートがない
　　# 「抽出結果」というシートを作成し見出し行を追加
else
　　# 「抽出結果」というシートがある場合の処理
　　#　シートをクリアするなりの処理

# 以降シートがある場合でもない場合でも実行する処理
# 今回は抽出処理となるのではないかと思います。
# ようはtarget_sheet = workbook["抽出結果"]以降の処理だと思います

といった形になるのではないですか？

Answer 1

Windows 11、Python 3.13.3で確認しました。

• target_sheet.max_rowは1開始なので、26行目以降のif文が実行されていません。
• そのif文の中で、row、data8_value、values、row_numが未定義(値が無い)です。

上記を見直してみてください。

Comments

[k161811]

ご対応ありがとうございます。

自分は、Pythonを利用し始めたばかりのため、
・そのif文の中で、row、data8_value、values、row_numが未定義(値が無い)です。
をどう記入すればよいか、ヒントをいただきたいです。

rowは、row[]のように記入すれば”〇番目の列の値”として出来るのと、未定義（値がない）のとでは違うのでしょうか
よろしくお願いします。

[hiroki-o]

rowが最初に出てくるのは、31行目です。
それ以前に値を代入しないで、いきなり参照しているため、未定義になります。
他の変数も同様です。

どういう条件でコピーしたいか教えてもらえたら、具体的なソースを回答できるかもしれません。

[k161811]

返信が遅れてしまい、すみません。
ありがとうございます

・それ以前に値を代入しないで、いきなり参照しているため、未定義になります。
は、自分の考えとして、row[10]は、シート「データ一覧」にある11列目の値らを代入しているつもりだったのですが、記入ミスでしたでしょうか。

丁度、エクセルのフィルター機能のようにしたいと考えています。
具体的には、
データ９…１
データ４…１
データ７…10
データ８…20000～20100
の条件の値が合致する行をシート「抽出結果」にコピーする、
という事を実行したいです。

よろしくお願いします