访问循环内定义的变量时报错：cannot access local variable 'x' where it is not associated with a value

嘿，这个问题其实挺常见的，核心原因是你在循环外部访问了只在循环内部定义的变量，咱们一步步理清楚：

你的x、y是在for (x, y, w, h) in faces:这个人脸检测循环里定义的，ex、ey则是在眼睛检测的内层循环里定义的。如果程序运行时没检测到人脸（比如摄像头没拍到脸），那这个for循环根本不会执行，x、y这些变量就从来没被创建过。这时候后面的elif x == 210 or y== 140:去访问它们，Python自然会报错说变量未定义。

另外你的代码还有几个小细节需要调整，我一起给你修正：

解决方案步骤 #

1. 提前初始化变量：在进入人脸检测循环前，先给x、y、ex、ey设置默认值（比如None），这样即使没检测到人脸/眼睛，变量也存在，不会报错。
2. 修正ROI切片错误：你写的roi_gray = gray[y:y + w, x:x + w]是错的，应该是y:y+h（高度方向用h，不是w），否则人脸ROI的范围会出错。
3. 调整逻辑位置：把判断坐标的逻辑放到对应的循环里，或者先判断变量是否存在再进行值的比较，避免无意义的判断。

修改后的完整代码 #

import numpy as np
import cv2

cap = cv2.VideoCapture(0)
face_cascade = cv2.CascadeClassifier(cv2.data.haarcascades + 'haarcascade_frontalface_default.xml')
eye_cascade = cv2.CascadeClassifier(cv2.data.haarcascades + 'haarcascade_eye.xml')

def eye_detector():
    while True:
        ret, frame = cap.read()   
        # 提前初始化变量，避免未定义报错
        x, y = None, None
        ex, ey = None, None

        gray = cv2.cvtColor(frame, cv2.COLOR_BGR2GRAY)
        faces = face_cascade.detectMultiScale(gray, 1.3, 5)
        
        for (x, y, w, h) in faces:
            # 修正ROI切片：高度用h，不是w
            roi_gray = gray[y:y + h, x:x + w]
            roi_color = frame[y:y + h, x:x + w]
            eyes = eye_cascade.detectMultiScale(roi_gray, 1.3, 5 )
            
            for (ex, ey, ew, eh) in eyes:
                cv2.rectangle(roi_color, (ex, ey), (ex + ew, ey + eh), (0, 255, 0), 5)
                # 检测到眼睛后判断坐标，符合条件就关闭窗口继续循环
                if ey == 30 or ex == 50:
                    cv2.destroyWindow("frame")
                    continue
            
            # 检测到人脸后判断坐标
            if x == 210 or y == 140:
                cv2.destroyWindow("frame")
                continue

        cv2.imshow('frame', frame)    
        if cv2.waitKey(1) == ord('q'):
            break

    cap.release()
    cv2.destroyAllWindows()

# import project1_module1 
# project1_module2.eye_detector() 

额外说明 #

• 我把判断坐标的逻辑放到了对应的循环内部，这样只有当检测到人脸/眼睛时才会进行判断，逻辑更合理。
• 初始化变量后，如果你还是想在循环外部判断，记得先加一层判断，比如if x is not None and x == 210，确保变量存在后再比较值。
• 你的原代码里cap.release()和cv2.destroyAllWindows()放在了函数外面，这会导致函数还没执行完就释放资源，我把它们移到了while循环结束后，也就是函数内部的末尾，这样逻辑才正确。

备注：内容来源于stack exchange，提问作者jun