pick develop/eagle to master

[add-uos]

pick develop/eagle to master

[sourcery-ai]

Sorry @add-uos, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

[deepin-ci-robot]

deepin pr auto review

我来对这段代码进行审查，主要关注以下几个方面：

1. 语法逻辑
2. 代码质量
3. 代码性能
4. 代码安全

分析如下：

1. 语法逻辑改进建议：

• DeviceMonitor.cpp 中的 setInfoFromXradr 函数新增的 Display Input 检查逻辑位置不太合理，应该放在函数开始处进行参数验证
• DeviceStorage.cpp 中的比较逻辑使用了浮点数比较，建议使用 qFuzzyCompare 替代手动实现的浮点数比较

2. 代码质量改进建议：

• 多处使用了魔法数字（如 specialComType 的 5 和 6），建议定义常量或枚举
• 字符串替换操作（如 "x" 到 "×"）重复出现，建议封装成工具函数
• DeviceMonitor.cpp 中的 loadBaseDeviceInfo 函数的 if 条件判断可以优化结构

3. 代码性能改进建议：

• DeviceStorage.cpp 中的正则表达式可以预编译，避免重复创建
• HWGenerator.cpp 中的文件遍历可以使用 QDirIterator 替代 entryInfoList，更节省内存

4. 代码安全改进建议：

• DeviceStorage.cpp 中的浮点数比较需要考虑精度问题
• 字符串操作建议增加空值检查
• 文件操作建议增加错误处理

具体修改建议：

1. 将 Display Input 检查逻辑移到函数开始处：

bool DeviceMonitor::setInfoFromXradr(const QString &main, const QString &edid, const QString &name)
{
    // 参数验证
    if (main.isEmpty() || edid.isEmpty() || name.isEmpty()) {
        return false;
    }

    // Display Input 检查
    for(const auto& info : m_LstBaseInfo) {
        if (info.first.contains("Display Input")) {
            if (!main.contains(info.second, Qt::CaseInsensitive)) {
                return false;
            }
        }
    }

    // 其余逻辑...
}

2. 优化字符串替换操作：

// 在工具类中添加
QString formatResolution(const QString& resolution) {
    return resolution.replace("x", "×", Qt::CaseInsensitive);
}

// 使用时
m_CurrentResolution = formatResolution(QString("%1@%2").arg(match.captured(1)).arg(curRate));

3. 改进浮点数比较：

bool DeviceStorage::compareSize(const QString &size1, const QString &size2)
{
    double num1 = 0;
    double num2 = 0;
    QRegularExpression reg(".*\\[(\\d+\\.?\\d+).*\\]");
    
    if (reg.match(size1).hasMatch()) {
        num1 = reg.match(size1).captured(1).toDouble();
    }
    if (reg.match(size2).hasMatch()) {
        num2 = reg.match(size2).captured(1).toDouble();
    }

    return qFuzzyCompare(num1, num2) ? size2 : (num1 > num2 ? size1 : size2);
}

4. 优化文件遍历：

void HWGenerator::generatorMonitorDevice()
{
    QDir toDir(toPath_);
    if (!toDir.exists()) {
        return;
    }

    QDirIterator it(toPath_, QDir::Files, QDirIterator::Subdirectories);
    while (it.hasNext()) {
        QFileInfo fileInfo = it.nextFileInfo();
        if (!fileInfo.fileName().startsWith("card")) {
            continue;
        }
        // 处理文件...
    }
}

5. 添加常量定义：

namespace DeviceConstants {
    enum SpecialComputerType {
        NormalCom = 5,
        SpecialCom = 6
    };
}

这些改进可以提高代码的可维护性、性能和安全性。建议在实施这些更改时进行充分的测试，确保不会影响现有功能。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: add-uos, lzwind

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[add-uos]

/merge