Python if not文がうまく動作しません

Question

前提・実現したいこと

以下コードのif not文とelseの部分がうまく動作しません。
if文とif not文を同時に満たす条件を定義したい場合、コードをどのように書き換えればいいでしょうか。

【実現したいこと】
エクセルブックの「train」シートの２行目から１行ずつ読み込み、以下の処理を行う
①読み込んだ行の15列目が空白かつ、15列目全体にimage_nameの値が入っていない場合：読み込んだ行の15列目に同行13列目のimage_nameの値を代入する
②読み込んだ行の15列目が空白だが、15列目のいずれかに同じimage_nameの値が入っている場合：読み込んだ行の16列目に同行14列目のbbox_infoの値を代入する

学習させたいのは肺のＸ線画像であり、trainシートには、15000枚分の学習用画像名や、分類ラベルや、疾患部分のbounding box情報が記載されています。必要な情報のみ抽出してテキストファイルに転記したく、その前準備として上記のことを実現したいと思っています。
画像は15000枚ですが、複数の所見がある場合、同じ画像の情報が複数の行に分かれて記載されているため、テキストファイルへの転機の際に重複が起きぬよう、上記のような処理を取りたい次第です。

該当のソースコード

import datetime
import openpyxl

xlsx_file = "train.xlsx"

# エクセルブック
wb = openpyxl.load_workbook(xlsx_file, data_only=True)

# エクセルシート
ws = wb["train"]

max_row = wb["train"].max_row
#print(max_row)

for row in range(1,max_row + 1):
    image_name = ws.cell(row = row+1 , column = 13).value
    bbox_info = ws.cell(row = row+1 , column = 14).value
    #print(image_name)
    #print(bbox_info)
    if ws.cell(row = row+1 , column = 15).value is None:  
        if not image_name in ws['O2':'O67916']:
            ws.cell(row = row+1 , column = 15).value = image_name
            ws.cell(row = row+1, column = 16).value = bbox_info
        else: 
            ws.cell(row = row+1 , column = 16).value = bbox_info

#エクセルの保存
wb.save(filename = xlsx_file)

Comments

[ppaul]

うまくいかないというのは、SuzuAyaさんの期待する動きと、ソースコードの実際の動きが違うということですね。
ソースコードは見ればわかりますが、SuzuAyaさんの期待する動きが書かれていないので、問題点がわかりません。どういう結果を期待しているかを言葉で書いてくれませんか。

[SuzuAya]

>ppaul様
コメントありがとうございます！色々説明が不足しており申し訳ございません。
ws.cell(row = row+1 , column = 15)が空白セルかつ、wsのO2からO67916の範囲にimage_nameの値がない場合、ws.cell(row = row+1 , column = 15)に image_nameの値を代入、ws.cell(row = row+1, column = 16).にbbox_infoの値を代入したいです。
もし、上記の条件に当てはまらない場合、ws.cell(row = row+1, column = 16)にbbox_infoの値を代入したいです。
分かりにくい部分がありましたら、お手数ですが再度お知らせいただけますと幸いです。
どうぞよろしくお願いいたします。

Answer 1

if notとは無関係のように見えます。

image_name in ws['O2':'O67916']

は、セルのリストの中にimage_nameがあるかという文なので、常にFlaseです。
そのために

not image_name in ws['O2':'O67916']:

は常にTrueになります。
これは、SuzuAyaさんの求めるものとは違っているでしょう。

if not image_name in ws['O2':'O67916']:
の部分を
if [cell for cell in ws['O2':'O67916'] if cell.value == image_name] == []:
に変更すれば、性能問題は別にして動くでしょう。

また本題とは関係ないかもしれませんが、

for row in range(1,max_row + 1):
は、1～max_rowでループします。
max_rowのとき、
if ws.cell(row = row+1 , column = 15).value is None:
は、常に成り立ちますが、それで良いのですか。

Comments

[SuzuAya]

>ppaul様
ご丁寧なご回答を誠にありがとうございます！私の説明が不十分であったため、お手間を取らせてしまい申し訳ありません。質問内容に、実現したいことの詳細を追記させていただきましたので、ご確認いただけますと幸いです。どうぞよろしくお願いいたします。

[ppaul]

昨日の回答で、問題点の一つはなくなると思います。
修正したコードで、どうなったかを教えてください。

[SuzuAya]

コメントに気がつくのが遅くなり、申し訳ございません。
if not image_name in ws['O2':'O67916']:
の部分を
if [cell for cell in ws['O2':'O67916'] if cell.value == image_name] == []:
に変更したところ、以下のエラーが出てしまいました。
AttributeError: 'tuple' object has no attribute 'value'

[SuzuAya]

たびたび申し訳ありません、私がやりたいことについてすでに質問をされている方がいらっしゃいました。https://teratail.com/questions/140511
こちらを参考に、一度コードを書きなおしてみます。その際、またエラーや不明点など生じましたらこちらに再度質問をさせていただきたいと思います。
貴重なお時間をたくさん使っていただき、ありがとうございました。引き続きどうぞよろしくお願いいたします。

[ppaul]

あっ、間違っていましたね。すみません。
if [cell for cell in ws['O2':'O67916'] if cell.value == image_name] == []:
ではなく、
if [cell for cell in ws['O2':'O67916'] if cell[0].value == image_name] == []:
でした。
時間があれば、これをやってみるもよし、https://teratail.com/questions/140511をもとにやってみるもよしです。

[SuzuAya]

修正いただいたコードで、うまくいきました！！
この度はご丁寧にありがとうございました。大変助かりました。

Answer 2

同時に満たす条件を定義したい場合

and ってはなしかな

Comments

[SuzuAya]

>y_waiwai様
ご回答ありがとうございます！ご提案いただいたとおり、andでつなげたりもしたのですが、うまくいかなかったんです。
if ws.cell(row = row+1 , column = 15).value is None and not image_name in ws['O2':'O67916']:
何か他にも間違っていることがあるのでしょうか。

[y_waiwai]

カッコで囲いましょう
if(なんたら)and(かんたら):

[SuzuAya]

>y_waiwai様
ありがとうございます。カッコで囲んでもうまくいかなかったので、コードの中身を修正する必要がありそうです。お時間をとっていただき、ありがとうございました。