在QTreeView::currentChanged()中调用dataChanged()是否合理？

首先明确两个函数的核心作用：

• currentChanged()：仅在视图的选中项发生变更时触发，负责处理选中状态切换的逻辑。
• dataChanged()：用于通知视图模型中指定索引的数据发生了实际修改，需要重新渲染该索引的内容（比如文本、图标、样式等）。

从常规逻辑来看，这段代码里的self.dataChanged(current,current)几乎没有意义，原因如下：

1. 选中项变更并没有修改模型中的实际数据，触发dataChanged()属于虚假通知，会导致视图做无意义的重绘，浪费性能。
2. QTreeView本身已经内置了选中状态的处理逻辑，会自动更新选中项的视觉样式（比如高亮），完全不需要通过dataChanged()来触发刷新。

特殊场景的例外？ #

如果你的视图存在极端特殊的自定义需求——比如某个项的显示样式完全依赖于选中状态，且无法通过重写模型的data()方法（利用Qt.BackgroundRole等角色）来实现，可能有人会用这种方式强行触发重绘。但这属于非常不规范的写法，正确的做法应该是通过模型的角色返回对应样式，而非滥用dataChanged()。

代码优化建议 #

直接删除self.dataChanged(current,current)这一行即可，不会影响原有功能，还能避免不必要的性能损耗。优化后的代码如下：

class MyTreeView(QTreeView):
   def currentChanged(self, current, previous):
        #self.setCurrentIndex(current)
        logging.debug(f"on current index {current}, \
            object {QModelIndex.internalPointer(current)}")
        self.obj_changed_signal.emit(current)

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