NodeGraphQt中value_changed信号重复触发问题求助

问题场景 #

输入20/25后，计算得到需要创建5个输入端口，执行创建操作后value_changed信号会再次触发，此时existing_inputs变为5，命中elif分支尝试重复创建端口，导致报错。尝试手动断开再重连信号无法解决问题。

核心原因 #

NodeGraphQt的文本输入组件在节点结构（如添加输入端口）发生变化时，可能会触发内部状态同步，进而额外发射一次value_changed信号；或者手动断开重连的方式无法覆盖所有信号绑定场景，导致重复触发。

解决方案 #

方案1：使用blockSignals临时阻塞信号发射 #

这是最可靠的方式，通过临时阻止组件发射所有信号，避免操作过程中触发额外的value_changed。

修改后的代码示例：

def __init__(self):
    super(RelayIONode, self).__init__()
    self.add_text_input('Inputs', 'IN: Start/End Values', "##/##", tab='Widget')
    self.get_widget('Inputs').value_changed.connect(self.on_input_changed)


def on_input_changed(self, name, value):
    input_widget = self.get_widget('Inputs')
    # 临时阻塞信号，避免操作触发二次发射
    input_widget.blockSignals(True)

    io_series = self.get_widget('IOSeries').get_value()
    existing_inputs = len(self.inputs())

    try:
        input_start, input_end = value.split('/')
        input_start = int(input_start)
        input_end = int(input_end)
        input_counts = input_end - input_start + 1
    except ValueError:
        print("Error: End value must be greater than Start value")
        input_widget.blockSignals(False)
        return

    # 修复原条件判断的可读性问题
    if existing_inputs == 0 and input_counts > 0:
        for i in range(input_counts):
            try:
                self.add_input(f'{io_series}{str(i + input_start).zfill(2)}', multi_input=True)
            except Exception as e:
                print(f"Error adding input: {e}")
    elif 0 < existing_inputs < input_start:
        # 先检查端口是否已存在，避免重复创建报错
        for i in range(input_counts):
            port_name = f'{io_series}{str(i + input_start).zfill(2)}'
            if not any(port.name() == port_name for port in self.inputs()):
                try:
                    self.add_input(port_name, multi_input=True)
                except Exception as e:
                    print(f"Error adding input: {e}")
                    print(input_start, input_end, existing_inputs)

    # 恢复信号发射
    input_widget.blockSignals(False)
    return

方案2：添加状态标记避免重复执行 #

通过实例变量标记当前是否正在处理信号，重复触发时直接跳过：

def __init__(self):
    super(RelayIONode, self).__init__()
    self.add_text_input('Inputs', 'IN: Start/End Values', "##/##", tab='Widget')
    self.get_widget('Inputs').value_changed.connect(self.on_input_changed)
    self._is_updating = False  # 添加状态标记


def on_input_changed(self, name, value):
    if self._is_updating:
        return
    self._is_updating = True

    # 原逻辑代码...

    self._is_updating = False
    return

额外优化点 #

• 原条件input_counts > existing_inputs == 0可读性差，建议拆分为清晰的逻辑判断；
• 避免使用裸except，捕获具体异常类型（如Exception）并打印错误详情，方便调试；
• 创建端口前先检查是否已存在，从根源避免重复创建报错。

内容的提问来源于stack exchange，提问作者user23373326