sync: from linuxdeepin/dtkgui

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#348

[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

[deepin-ci-robot]

deepin pr auto review

代码审查意见

1. 语法逻辑

1. foundSize 函数实现了从文件路径中提取图标尺寸的功能，但逻辑可以更清晰：

   • 目前使用了两次 parseSize 调用，可以考虑合并逻辑
   • 对于目录名解析，可以增加更多的错误检查

2. 在 main 函数中：

   • dirName.prepend("/") 在条件判断中执行，可能导致路径格式不一致
   • iconSize > 0 ? QString("/%1").arg(iconSize) : dirName.prepend("/") 这行代码可能产生不规范的路径

2. 代码质量

1. foundSize 函数：

   • 建议添加注释说明函数的用途和返回值含义
   • 可以增加对目录名格式的更多验证，确保返回的尺寸是有效的

2. 错误处理：

   • dciChecker 函数的使用很好，但可以考虑增加更多的错误场景处理
   • 对于文件操作，建议添加更多的异常处理

3. 代码可读性：

   • 一些魔法数字（如256）应该定义为常量
   • 变量名可以更具描述性，例如 isNum 可以改为 isNumeric

3. 代码性能

1. foundSize 函数：

   • 多次调用 dir.dirName() 和 dir.cdUp() 可能影响性能
   • 可以考虑缓存目录结构信息

2. 文件操作：

   • 在循环中频繁创建和销毁对象可能影响性能
   • 建议批量处理文件操作

4. 代码安全

1. 路径处理：

   • 当前代码对路径的处理不够严格，可能存在路径遍历风险
   • 建议添加路径规范化处理，防止 ../ 等危险字符

2. 输入验证：

   • 对于用户输入的尺寸值，应该进行范围检查
   • 文件操作前应该验证文件权限

改进建议

1. 重构 foundSize 函数：

static uint foundSize(const QFileInfo &fileInfo) {
    const QStringList validSizeFormats = {"256x256", "128x128", "64x64"}; // 可以扩展
    
    QDir dir = fileInfo.absoluteDir();
    QString dirName = dir.dirName();
    
    // 尝试直接解析尺寸
    bool ok;
    uint size = dirName.toUInt(&ok);
    if (ok && size > 0 && size <= 512) { // 添加合理范围检查
        return size;
    }
    
    // 尝试解析格式如 "256x256"
    if (dirName.contains('x')) {
        auto parts = dirName.split('x');
        if (parts.size() == 2) {
            size = parts.first().toUInt(&ok);
            if (ok && size > 0 && size <= 512 && validSizeFormats.contains(dirName)) {
                return size;
            }
        }
    }
    
    // 检查上级目录
    if (dir.cdUp()) {
        dirName = dir.dirName();
        size = dirName.toUInt(&ok);
        if (ok && size > 0 && size <= 512) {
            return size;
        }
    }
    
    return 0;
}

2. 增加常量定义：

namespace Constants {
    const uint MAX_ICON_SIZE = 512;
    const uint DEFAULT_ICON_SIZE = 256;
    const QString DEFAULT_SIZE_DIR = "/256";
}

3. 改进路径处理：

QString normalizePath(const QString &path) {
    QString normalized = path;
    normalized = normalized.replace("\\", "/");
    normalized = normalized.remove(QRegularExpression("/+"));
    if (!normalized.startsWith("/")) {
        normalized.prepend("/");
    }
    return normalized;
}

4. 增加输入验证：

bool isValidIconSize(uint size) {
    return size > 0 && size <= Constants::MAX_ICON_SIZE;
}

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