pythonでfor文を使ってインスタンスの作成をしたいのです。

Question

pythonでfor文を用いてインスタンスの作成をしようとしているのですが

# coding: UTF-8
import cv2
import numpy as np

class Roi():
    def __init__(self):
        pass

    def rect(self,roi,col,img):

        x1 = roi[0]
        y1 = roi[1]
        x2 = roi[2]
        y2 = roi[3]
        cv2.rectangle(img, (x1, y1), (x2, y2), col, 2)

#クリックイベントのコード省略

def web_cam():
    while True:
        ret, frame = cap.read()
        img = frame
        cv2.waitKey(100)
        wname = 'Web Cam Image'
        cv2.imshow(wname, img)

        name.rect(a[0:4:1], (0, 255, 0),img)
        ver.rect(a[4:8:1], (255, 0, 0),img)
        lat.rect(a[8:12:1], (255, 255, 0),img)
        long.rect(a[12:16:1], (0, 0, 255),img)


if __name__ == '__main__':
    cap = cv2.VideoCapture(0)
    a=[]

    roi_name = ['name','ver','lat','long']
    for i in roi_name:
        i = Roi()

    ret, frame = cap.read()
    img = frame
    wname = "select area"

    cv2.imshow(wname, img)
    cv2.setMouseCallback(wname, onMouse, [wname, img])

    while cv2.waitKey(0) == 27:
        break

上記コードのfor文を使ってインスタンス作成を試みているのですが
その後、インスタンス使用時　web_camの関数内で

NameError: name 'name' is not defined

のエラーが出てしまいます。
どなたか、ご教授願えないでしょうか。
まだ、作成半ばですので上記質問部分以外にも、おかしなコードがあるかもしれませんがよろしくお願いいたします。

Answer 1

roi_name = ['name','ver','lat','long']
for i in roi_name:
    i = Roi()

↑この記述ですが、

i = 'name'
i = Roi()

みたいな代入を4回繰り返す処理であり、インスタンスがnameではなくiに格納されて、nameという変数自体は未定義になっているのが原因です。

exec('%s = Roi()' % i)

とすれば name = Roi() を実行して変数nameにインスタンスを格納することができますが、怪しいのでおすすめしません。
見た感じ、矩形がどんどん増えていくものではなさそうなので、for文を使わず素朴に書くのが良いと思います。

name = Roi()
ver = Roi()
lat = Roi()
long = Roi()

Comments

[dendendenjirou]

早速の回答ありがとうございます。
アドバイス通り素朴にコードを書くことにしました。
また、exec('%s = Roi()' % i)のコード勉強になりました。
ありがとうございました。