PySide6部件交换动画完成后程序崩溃问题排查与修复

问题描述 #

作为PySide6新手，编写两个Widget的交换动画时，使用列表存储Widget指针用于后续操作。向下移动Widget时代码运行正常，但向上移动时，交换完成后程序可能崩溃或出现部件交换错误；崩溃时终端无输出，程序会冻结或直接关闭。原代码如下：

def widgets_swap(self,widget1,mode:int):

    index1 = self.scripts.index(widget1)
    #move up
    if mode == 0 and index1 > 0:
        index2 = index1 - 1
        widget2 = self.scripts[index2]
    #move down
    elif mode == 1 and index1 < len(self.scripts) -1:
        index2 = index1 + 1
        widget2 = self.scripts[index2]
    
    else:
        print("Not moving")
        return
    
    #animation
    ani1 = QPropertyAnimation(widget1,b"pos")
    ani2 = QPropertyAnimation(widget2,b"pos")

    ani1.setDuration(50)
    ani1.setStartValue(widget1.pos())
    ani1.setEndValue(widget2.pos())
    ani2.setDuration(50)
    ani2.setStartValue(widget2.pos())
    ani2.setEndValue(widget1.pos())

    
    loop = QEventLoop()
    ani1.start()
    ani2.start()
    ani2.finished.connect(loop.quit)
    loop.exec()

    self.scripts[index1], self.scripts[index2] = self.scripts[index2], self.scripts[index1]

    self.ui.verticalLayout_6.removeWidget(widget1)
    self.ui.verticalLayout_6.removeWidget(widget2)
    self.ui.verticalLayout_6.insertWidget(index1,widget2,alignment=Qt.AlignTop)
    self.ui.verticalLayout_6.insertWidget(index2,widget1,alignment=Qt.AlignTop)

崩溃原因分析 #

1. 布局操作索引错误：交换列表后，index1和index2是交换前的旧索引，直接用旧索引插入布局会导致部件位置逻辑混乱。向上移动时，这种索引不匹配的问题会触发布局管理器的内部错误，进而引发崩溃。
2. 事件循环与对象生命周期冲突：QEventLoop是局部变量，动画对象ani1、ani2也是局部变量，事件循环结束后这些对象可能被Python垃圾回收，但Qt内部可能仍有未处理的信号关联，导致内存访问错误。
3. 手动修改pos与布局管理器冲突：动画直接修改Widget的pos属性，但布局管理器本身负责管理部件位置，两者的位置控制逻辑冲突，会导致布局状态异常，引发程序崩溃或冻结。

修复方案 #

关键修改点 #

• 使用QParallelAnimationGroup同步管理两个动画，确保动画同时结束，避免单独处理finished信号的问题。
• 动画结束后，基于交换后的列表索引正确操作布局，避免旧索引导致的位置错误。
• 移除阻塞式的QEventLoop，改用信号槽处理动画结束后的布局更新，避免事件循环阻塞引发的问题。
• 动画结束后强制布局更新，确保位置与列表状态一致。

修复后的代码 #

from PySide6.QtCore import QParallelAnimationGroup, QPropertyAnimation, Qt

def widgets_swap(self, widget1, mode: int):
    index1 = self.scripts.index(widget1)
    widget2 = None
    index2 = -1

    # 确定交换目标
    if mode == 0 and index1 > 0:
        index2 = index1 - 1
        widget2 = self.scripts[index2]
    elif mode == 1 and index1 < len(self.scripts) - 1:
        index2 = index1 + 1
        widget2 = self.scripts[index2]
    else:
        print("Not moving")
        return

    # 获取动画起始和结束位置（基于当前布局中的位置）
    start_pos1 = widget1.pos()
    start_pos2 = widget2.pos()

    # 创建动画组同步两个动画
    animation_group = QParallelAnimationGroup()

    ani1 = QPropertyAnimation(widget1, b"pos")
    ani1.setDuration(50)
    ani1.setStartValue(start_pos1)
    ani1.setEndValue(start_pos2)

    ani2 = QPropertyAnimation(widget2, b"pos")
    ani2.setDuration(50)
    ani2.setStartValue(start_pos2)
    ani2.setEndValue(start_pos1)

    animation_group.addAnimation(ani1)
    animation_group.addAnimation(ani2)

    # 动画结束后处理布局和列表更新
    def on_animation_finished():
        # 先更新列表
        self.scripts[index1], self.scripts[index2] = self.scripts[index2], self.scripts[index1]
        
        # 移除部件后，按交换后的位置重新插入
        layout = self.ui.verticalLayout_6
        layout.removeWidget(widget1)
        layout.removeWidget(widget2)
        
        # 使用交换后的索引插入，确保布局位置与列表一致
        layout.insertWidget(index2, widget1, alignment=Qt.AlignTop)
        layout.insertWidget(index1, widget2, alignment=Qt.AlignTop)
        
        # 强制布局更新，消除位置冲突
        layout.update()

    animation_group.finished.connect(on_animation_finished)
    animation_group.start()

额外优化建议 #

• 可以为动画添加缓动曲线提升视觉效果，比如ani1.setEasingCurve(QEasingCurve.InOutQuad)。
• 若布局中有拉伸因子，交换时需同步调整部件的拉伸属性，避免布局变形。

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