Notes项目中的高CPU占用问题分析与修复

问题背景

在Notes项目的最新版本中，用户报告了一个性能问题：当鼠标悬停在文件夹和标签区域时，应用程序的CPU使用率会显著升高至30-50%，远高于其他界面区域的5-10%使用率。这个问题在Wayland和XWayland环境下均能复现。

问题分析

经过深入调查，发现问题根源在于TagTreeDelegateEditor类中的paintEvent事件处理函数。该函数内部调用了updateDelegate()方法，而后者又会通过setStyleSheet()设置样式表，从而触发新的绘制事件，形成了无限递归的绘制循环。

具体来看，updateDelegate()方法会根据当前选择状态动态设置标签文本的颜色样式表。每次调用setStyleSheet()都会导致Qt框架重新计算样式并触发重绘，而重绘又会再次调用paintEvent，如此循环往复。

解决方案

针对这个问题，我们采用了以下修复方案：

1. 在设置样式表前，先检查新样式表是否与当前样式表相同
2. 只有在样式表确实需要更新时才调用setStyleSheet()
3. 避免了不必要的样式表设置操作，从而打破了绘制循环

关键代码修改如下：

QString ss;
if (m_view->selectionModel()->selectedIndexes().contains(m_index)) {
    ss = QStringLiteral("QLabel{color: rgb(%1, %2, %3);}")
             .arg(QString::number(m_titleSelectedColor.red()),
                  QString::number(m_titleSelectedColor.green()),
                  QString::number(m_titleSelectedColor.blue()));
} else {
    ss = QStringLiteral("QLabel{color: rgb(%1, %2, %3);}")
             .arg(QString::number(m_titleColor.red()),
                  QString::number(m_titleColor.green()),
                  QString::number(m_titleColor.blue()));
}

if (m_label->styleSheet() != ss) {
    m_label->setStyleSheet(ss);
}

优化效果

修复后，CPU使用率显著下降：

• 悬停在标签区域时的CPU使用率从30-50%降至正常水平
• 消除了不必要的样式计算和重绘操作
• 保持了原有的视觉表现和功能完整性

后续优化方向

虽然当前修复方案解决了性能问题，但从架构角度考虑，仍有进一步优化的空间：

1. 考虑使用Qt的样式表伪类选择器替代动态样式设置
2. 将颜色设置逻辑与文本更新逻辑分离
3. 减少在绘制事件中的计算操作

这个案例也提醒我们，在GUI编程中要特别注意避免在绘制事件中执行可能触发新绘制操作的功能，以免形成性能瓶颈。