ZBLOG

ui->label_7->installEventFilter(this); ui->label_8->installEventFilter(this); ui->label_9->installEventFilter(this); ui->label_10->installEventFilter(this); ui->label_11->installEventFilter(this); ui->label_12->installEventFilt...

这段代码存在几个潜在问题和可以优化的地方:

  1. 内存泄漏问题

    • 在电子白板部分创建了QProcess *process但没有管理其生命周期,应该设置为类的成员变量或使用QProcess::startDetached()
  2. 路径拼接问题

    • 使用字符串拼接路径不够健壮,建议改用QDir::toNativeSeparators()QFileInfo处理路径。
  3. 重复代码问题

    • 多个label的事件处理逻辑高度相似,可以考虑用映射表(QHash)统一管理。
  4. 资源释放问题

    • SVG/PNG资源文件没有检查是否存在就直接使用。
  5. 事件处理返回值

    • return false表示事件未被完全处理,可能影响其他事件过滤器。
  6. 硬编码问题

    • 颜色值和路径都是硬编码的,建议定义为常量或配置文件读取。

优化建议版本:

// 头文件中定义
const QString COLOR_NORMAL = "rgb(255, 255, 255)";
const QString COLOR_HOVER = "rgb(235, 235, 235)";

// 初始化时建立映射
QHash<QLabel*, std::pair<QString, QByteArray>> labelActions = {
    {ui->label_7, {currentPath + "/img/rollcall.svg", vSendArrayHeard_Roll_call}},
    {ui->label_19, {currentPath + "/img/whiteBoard.png", vSendArrayHeartWhiteBoard}}
};

bool YourClass::eventFilter(QObject *obj, QEvent *event)
{
    if (auto label = qobject_cast<QLabel*>(obj)) {
        if (labelActions.contains(label)) {
            auto [iconPath, cmdData] = labelActions[label];
            
            if (event->type() == QEvent::Leave) {
                setLabelStyle(label, COLOR_NORMAL, iconPath);
            } 
            else if (event->type() == QEvent::Enter) {
                setLabelStyle(label, COLOR_HOVER, iconPath);
            }
            else if (event->type() == QEvent::MouseButtonPress) {
                setLabelStyle(label, COLOR_HOVER, iconPath);
                
                // 特殊处理白板
                if (label == ui->label_19) {
                    QString appPath = QDir::cleanPath(
                        QCoreApplication::applicationDirPath() + 
                        "/WhiteBoard/WhiteBoard.exe");
                    
                    if (!QProcess::startDetached(appPath)) {
                        qWarning("Failed to start WhiteBoard");
                    }
                }
                
                SendCMD(cmdData);
            }
            return true; // 已处理
        }
    }
    return QObject::eventFilter(obj, event);
}

主要改进点:

  1. 使用智能指针管理QProcess
  2. 统一化事件处理逻辑
  3. 更安全的路径处理方式
  4. 更好的资源管理
  5. 更清晰的责任划分

注意:如果这些标签需要不同的特殊行为,可以在映射表中增加回调函数字段来实现差异化处理。


内容由零声教学AI助手提供,问题来源于学员提问

本站部分文章来源于网络,版权归原作者所有,如有侵权请联系站长删除。
转载请注明出处:https://sdn.0voice.com/?id=8033

分享:
扫描分享到社交APP
上一篇
下一篇
发表列表
游客 游客
此处应有掌声~
评论列表

还没有评论,快来说点什么吧~

联系我们

在线咨询: 点击这里给我发消息

微信号:3007537140

上班时间: 10:30-22:30

关注我们
x

注册

已经有帐号?