sync: from linuxdeepin/dtkcore

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkcore.

Source-pull-request: linuxdeepin/dtkcore#502

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot

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

[github-actions]

Note

[静态代码检查]

• 敏感词检查失败, 检测到1个文件存在敏感词

详情

{
    "src/dsysinfo.cpp": [
        {
            "line": "    QString fallbackSiteUrl = type == Distribution ? QStringLiteral(\"https://www.deepin.org\") : QString();",
            "line_number": 962,
            "rule": "S35",
            "reason": "Url link | 8ab1f6f000"
        }
    ]
}

[deepin-ci-robot]

deepin pr auto review

我来对这段代码进行审查，并提供改进意见：

1. 语法逻辑

1. 在 DSysInfo 类中新增了 UosDefense 枚举值，这是合理的，用于标识国防版系统。

2. 在 OSBuild 结构体中增加了详细的注释说明版本号的构成规则，这对维护很有帮助。

3. 改进了版本号解析逻辑：

   • 使用 parseDigitOrAZ lambda 函数统一处理数字和字母的转换
   • 使用 QChar 的 isDigit() 和比较运算符替代了原来的正则表达式
   • 代码更加清晰和健壮

2. 代码质量

1. 版本解析逻辑改进：

   • 原来的代码混合使用了 toUInt() 和字符处理，逻辑不够清晰
   • 新的 parseDigitOrAZ lambda 函数统一了解析逻辑，提高了代码一致性
   • 建议将这个 lambda 函数提取为类的私有方法，提高代码复用性

2. 版本号检查：

   • 建议在解析每个版本号部分后增加范围检查，确保符合文档中描述的位数和值范围
   • 例如：A、B、C、D、E 应该在 0-35 之间（0-9 和 A-Z）

3. 错误处理：

   • 目前的错误处理比较简单，只是打印警告和设置默认值
   • 建议考虑抛出异常或返回错误码，让调用者能够更好地处理错误情况

3. 代码性能

1. 字符串处理：

   • 当前代码多次调用 left.value() 和字符串操作，可能影响性能
   • 建议一次性获取所有需要的字符串，减少重复操作

2. 字符转换：

   • 使用 QChar 的 unicode() 方法进行转换是高效的
   • 但可以考虑使用查找表（lookup table）来优化字母到数字的转换

4. 代码安全

1. 输入验证：

   • 当前代码对输入字符串的验证不够严格
   • 建议增加对输入字符串长度的检查
   • 建议验证每个字符是否在允许的范围内（0-9, A-Z）

2. 类型转换：

   • 在进行类型转换时，建议增加范围检查，防止溢出
   • 例如：static_cast<uint>(ch.unicode() - 'A') 的结果应该在 0-35 之间

3. 枚举值处理：

   • 在 uosEditionType() 函数中，使用了字符 'A' 作为枚举值，这不太符合常规
   • 建议使用明确的枚举值，而不是依赖字符编码

改进建议

1. 将 parseDigitOrAZ 提取为类的私有方法：

private:
    static uint parseDigitOrAZ(const QString &s, bool *outOk) {
        // 实现代码
    }

2. 增加版本号范围检查：

void checkVersionRange(uint value, const char* name) {
    if (value > 35) {  // 0-9 + A-Z = 36个值
        qWarning() << "Invalid" << name << "version:" << value;
        // 处理错误
    }
}

3. 改进 uosEditionType() 函数：

DSysInfo::UosEdition DSysInfo::uosEditionType() {
    switch (uosEdition) {
        case 3: return UosEuler;
        case 4: return UosMilitaryS;
        case 5: return UosDeviceEdition;
        case 9: return UosEducation;
        case 10: return UosDefense;  // 使用明确的枚举值而不是 'A'
        default: return UosUnknown;  // 添加默认情况
    }
}

4. 增加输入验证：

bool DSysInfoPrivate::validateVersionString(const QString &version) {
    if (version.isEmpty() || version.size() != 5) {
        return false;
    }
    for (const QChar &ch : version) {
        if (!ch.isDigit() && (ch < 'A' || ch > 'Z')) {
            return false;
        }
    }
    return true;
}

这些改进将提高代码的可维护性、性能和安全性。

[github-actions]

Note

[静态代码检查]

• 敏感词检查失败, 检测到1个文件存在敏感词

详情

{
    "src/dsysinfo.cpp": [
        {
            "line": "    QString fallbackSiteUrl = type == Distribution ? QStringLiteral(\"https://www.deepin.org\") : QString();",
            "line_number": 962,
            "rule": "S35",
            "reason": "Url link | 8ab1f6f000"
        }
    ]
}