PyQt5中的按钮连接有什么问题？

Question

我的问题是我无法连接两个按钮之间的实现。在我按下“灰度”按钮并获得灰度图像后，我按下“Canny”按钮但用户界面突然关闭。 我不知道我的代码有什么问题。

def getImage(self):
    global fname
    fname = QFileDialog.getOpenFileName(self, 'Open file', 
       'C:\\Users\binil-ping\Desktop\CODE',"Image Files (*.jpg *.gif *.bmp *.png)")
    pixmap = QPixmap(fname[0])
    self.label.setPixmap(QPixmap(pixmap))
    self.resize(pixmap.width(), pixmap.height())

def Grayscale(self):
    global edges
    edges = cv2.imread(fname[0], 0)
    edges = cv2.GaussianBlur(edges, (5, 5), 0)
    height, width = edges.shape[:2]
    ret,edges = cv2.threshold(edges,150,255,cv2.THRESH_BINARY)
    kernel = np.ones((5,5),np.uint8)
    edges = cv2.morphologyEx(edges, cv2.MORPH_OPEN, kernel)
    edges = cv2.morphologyEx(edges, cv2.MORPH_OPEN, kernel)

    edges = QImage(edges, width, height, QImage.Format_Grayscale8)
    pixmap = QPixmap.fromImage(edges)
    self.label.setPixmap(pixmap)
    self.resize(pixmap.width(), pixmap.height())

def Canny(self):
    edges2 = cv2.imread(edges[0],-1)
    edges2 = cv2.Canny(edges2,180,200)

    edges2 = QImage(edges2, width, height, QImage.Format_Grayscale8)
    pixmap = QPixmap.fromImage(edges2)
    self.label.setPixmap(pixmap)
    self.resize(pixmap.width(), pixmap.height())

Answer 1

你的代码有很多问题（包括你没有提供minimal, reproducible example），其中一些解释了崩溃或可能导致另一个问题（我用 [*] 标记了它们):

• 如前所述，尽可能避免使用全局变量（几乎总是如此），而是使用类属性；
• 您不断地覆盖这些全局变量，这也让调试变得非常混乱；实际上，您将edges 用于numpy 数组和 QImage；
• [*] 您正在尝试 cv2.imread(edges[0],-1)，但不仅 imread 需要一个字符串，而且由于前一点，此时 edges 甚至是一个 QImage；
• [*] 一些变量未在函数范围内声明（Canny 函数中的宽度/高度）；
• 您确实应该避免对函数和变量使用大写的名称，因为它们通常只用于类名和内置常量；
• 在转换回 QImage 的过程中存在一些问题，我想这是由于您对数组应用的各种转换（其中一些似乎也不必要）对 Format_Grayscale8 格式不太友好，也许您应该对此进行更好的调查；

这是对您的代码的可能改进。我正在添加小部件创建部分，因为原始示例中缺少它。

class ShowImage(QWidget):
    def __init__(self):
        QWidget.__init__(self)
        layout = QVBoxLayout(self)
        self.label = QLabel()
        layout.addWidget(self.label)

        self.getImageBtn = QPushButton()
        layout.addWidget(self.getImageBtn)
        self.getImageBtn.clicked.connect(self.getImage)

        self.grayBtn = QPushButton('gray')
        layout.addWidget(self.grayBtn)
        self.grayBtn.clicked.connect(self.grayScale)

        self.cannyBtn = QPushButton('canny')
        layout.addWidget(self.cannyBtn)
        self.cannyBtn.clicked.connect(self.canny)

        self.fileName = None
        self.edges = None

    def getImage(self):
        fname = QFileDialog.getOpenFileName(self, 'Open file', 
           'C:\\Users\binil-ping\Desktop\CODE',"Image Files (*.jpg *.gif *.bmp *.png)")
        if fname[0]:
            self.fileName = fname[0]
            pixmap = QPixmap(self.fileName)
            self.label.setPixmap(pixmap)

    def grayScale(self):
        if not self.fileName:
            return
        edges = cv2.imread(self.fileName, 0)
        edges = cv2.GaussianBlur(edges, (5, 5), 0)
        ret,edges = cv2.threshold(edges, 150, 255, cv2.THRESH_BINARY)
        kernel = np.ones((5, 5), np.uint8)
        edges = cv2.morphologyEx(edges, cv2.MORPH_OPEN, kernel)
        edges = cv2.morphologyEx(edges, cv2.MORPH_OPEN, kernel)

        self.edges = edges
        height, width = edges.shape[:2]
        image = QImage(edges, width, height, QImage.Format_Grayscale8)
        pixmap = QPixmap.fromImage(image)
        self.label.setPixmap(pixmap)

    def canny(self):
        if self.edges is None:
            return
        edges2 = cv2.Canny(self.edges, 180, 200)
        height, width = edges2.shape[:2]

        edges2 = QImage(edges2, width, height, QImage.Format_Grayscale8)
        pixmap = QPixmap.fromImage(edges2)
        self.label.setPixmap(pixmap)

Answer 2

规则 #1：不要使用全局变量。

现在来看看真正的答案。

PyQt 有一个非常非常糟糕、令人讨厌的习惯，如果您的代码中有未捕获的异常，它就会崩溃。这使得调试变得困难。就个人而言，我在调试时在槽周围使用了一个装饰器，它会在 PyQt 杀死应用程序之前打印我的异常。

import logging
from functools import wraps
logger = logging.getLogger(__name__)
# ...lots of other code...
def debugFunc(*errors):
    def decorator(func):
        @wraps(func)
        def runner(*args, **kwargs):
            try:
                return func(*args, **kwargs)
            except errors as err:
                logger.debug(f"Function {func.__name__} raised a {err.__class__.__name__}, which was ignored.",
                             exc_info=True)
            except Exception as err:
                logger.critical(f"{err.__class__.__name__} - {err}", exc_info=True)
        return runner
    return decorator

在调试时，这允许我忽略某些错误，并将所有其他错误发送到logging 系统。如果您自己不使用它并且不愿意开始使用它，您可以改为导入 traceback 并使用它的功能，而不是调用我的 logger 对象。

只需将此函数用作最里面的装饰器，如下所示：

@debugFunc()
def Canny(self):
    edges2 = cv2.imread(edges[0],-1)
    edges2 = cv2.Canny(edges2,180,200)

    edges2 = QImage(edges2, width, height, QImage.Format_Grayscale8)
    pixmap = QPixmap.fromImage(edges2)
    self.label.setPixmap(pixmap)
    self.resize(pixmap.width(), pixmap.height())[enter image description here][1]

虽然我对 cv2 不够熟悉，无法肯定地说，但我认为您的问题可能是 cv2.Canny。它看起来不像其他 cv2 函数的命名，如果我是对的，可能会导致 AttributeError。