python 三重ループの値が更新されない

Question

実現したいこと・問題点

PythonでOpenCVを用いて画像からRGB値により特定の色を精度良く抽出したいです。
そのため、最も精度良く抽出できている最適なパラメータを取得するために、下限のパラメータを１ずつ増加させて画像を保存させています。

そのとき、ファイル名からパラメータがわかるよう画像を保存しているのですが、変数a,b,cがなぜか更新されません。
その結果、同じ画像名の画像が連続で出力され続けています。
しかし、フォルダを見ていると画像は少しずつ変化しているのでパラメータはおそらく更新できているはずです。

どなたか原因が分かる方教えていただけないでしょうか。

該当のソースコード

import os
import cv2
import shutil
import glob
import numpy as np
from time import sleep
from PIL import Image

cwd = os.getcwd()

#file name
img_dir = "E:/b/"
out = "output"

#Lower param
lb = 34
lg = 3
lr = 53

#Upper param
ub = 114
ug = 75
ur = 150

#Threshold
lower = 200
upper = 1500

def test():
    path = glob.glob(img_dir + "/*png")
    outPath = img_dir + "/" + out
    os.makedirs(outPath, exist_ok=True)
    for aaa in range (55):
        for bbb in range(36):
            for ccc in range(48):
                for i in path:
                    a = lb + aaa
                    b = lg + bbb
                    c = lr + ccc
                    name = os.path.basename(i)
                    image = cv2.imread(i) # ファイル読み込み
                    bgrLower = np.array([a, b, c])    # 抽出する色の下限
                    bgrUpper = np.array([ub, ug, ur])          # 抽出する色の上限
                    bgrResult = bgrExtraction(image, bgrLower, bgrUpper)
                    w,h,c = bgrResult.shape
                    
                    count = 0
                    #全画素を探索してクロ以外の画素をカウント　upper以上になったらストップ
                    for i in range(w):
                        for j in range(h):
                            imgdata = bgrResult[i,j]
                            b,g,r = imgdata
                            if b != 0 and g != 0 and r !=0: 
                                count += 1
                            elif count == upper:
                                break
                        else:
                            continue
                        break

                    #lower以上 uppr以下　であれば画像を保存
                    if count < lower and count > upper:
                        continue
                    cv2.imwrite(os.path.join(outPath , str(a) + "." + str(b) + "." + str(c) + "-" + name ),bgrResult)
                    print(os.path.join(outPath , str(a) + "." + str(b) + "." + str(c) + "-"  + name ))

# BGRで特定の色を抽出する関数
def bgrExtraction(image, bgrLower, bgrUpper):
    img_mask = cv2.inRange(image, bgrLower, bgrUpper) # BGRからマスクを作成
    result = cv2.bitwise_and(image, image, mask=img_mask) # 元画像とマスクを合成
    return result

if __name__ == '__main__':
    test()

試したこと

下から二行目のところで計算式を入れていたのがいけないのかと思い、変数に代入してからファイル名に入力：結果は変わらず
ファイル名に連番を追加：画像は別々に保存できるが肝心のパラメータがわからないため失敗

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

ここにより詳細な情報を記載してください。

Answer 1

コードを見た上での見解

少なくともb,cに関して言えば、ループ内の処理で上書きが起こりうるコードになっています。

質問上に書かれているコードから抜粋して、コメントとして状況を説明しています。

for i in path:
    a = lb + aaa
    b = lg + bbb
    c = lr + ccc
    # (略)
    w,h,c = bgrResult.shape  # <- ここで、cを上書きしている

    count = 0
    for i in range(w):
        for j in range(h):
            imgdata = bgrResult[i,j]
            b,g,r = imgdata  # <- このブロックに入ると、bを上書きする

aが更新されない可能性はこのコードからは読み取れませんでしたが、
bbb,cccまでのループ内での処理時点でb,cに変化がないことに気づいて処理をやめているだけで、
最後までループさせ続けるとaは変化している可能性があります。
(bbb,cccのループだけで1728周あることから、自分なら事象に気づいた時点で止めるかなと想像しています)

どうするべきか

• それぞれに意味の通りやすい変数にする
• 変数にする必要のない箇所を変数にしない

それぞれに意味の通りやすい変数にする

一例ですが、a = lb + aaaのような書き方だと「aは何か？」「aaaは何か？」がわかりにくくなります。
変数には役割があるはずなので、役割に応じた命名を適切にすることで、
「違う役割の変数には違う役割が割り当てられる」ため、重複が起きにくくなります。

for plus_b in range(36):  # 加算用の数値であることがわかるように、仮でplus_という接頭辞にする
   # (略)
   cur_lb = lb + plus_b  #「ループ内の現在のlowerなbをlbとループ時に使うbの補正値」

変数にする必要のない箇所を変数にしない

for i in pathのループ内の最小に定義しているa,b,cは下限値用のbgrLowerのためだけに使っているように見えます。
であれば、わざわざ一度変数にするのではなく、直接bgrLowerを用意してしまったほうがわかりやすいケースがあります。

for i in path:
-    a = lb + aaa
-    b = lg + bbb
-    c = lr + ccc
    name = os.path.basename(i)
    image = cv2.imread(i) # ファイル読み込み
-    bgrLower = np.array([a, b, c])    # 抽出する色の下限
+    bgrLower = np.array([lb + aaa, lg + bbb, lr + ccc])

Comments

[H4shikaN]

うまく処理が回りました！
ありがとうございます！！

自分の中で誰かに渡すプログラムでもないから、変数名を適当に決めてしまっていたのですが、こういった重複に気づけなかったりといいことがないのでこれからはきちんと役割に応じた変数名をつけるようにしていきます。

ご回答いただきありがとうございました????

Answer 2

ループで上書きが起こっています

a = lb + aaa
b = lg + bbb #bに代入
c = lr + ccc #cに代入

ここで定義をしていますが、

w,h,c = bgrResult.shape #cを上書きしている

や、

b,g,r = imgdata #bを上書きしている

で上書きをしています。
変数名の重複を防ぐため、変数名を変えてみてはどうでしょうか。