fix bugs

[Johnson-zs]

No description provided.

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Johnson-zs

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. 文件：src/dfm-io/dfm-io/denumerator.cpp

变更内容：
修改了 DEnumeratorPrivate::buildUrl 函数中构建 URL 的方式，从使用 QUrl::fromLocalFile 改为保留原始 URL 的 scheme 和 host。

审查意见：

• 语法逻辑：

  • ✅ 通过。修改后的逻辑是正确的。之前的代码假设所有 URL 都是本地文件（file://），这会导致处理网络挂载（如 SMB, FTP, SFTP）或自定义 Scheme 的 URL 时出错。新的代码通过复制原始 URL 并仅修改 Path 部分，保留了上下文信息，逻辑更加严谨。
  • ⚠️ 注意：QUrl::setPath 会自动处理路径的分隔符，但建议确认 path 变量的构造逻辑是否与新的 URL Scheme 兼容（例如某些 Scheme 对路径格式有特殊要求）。

• 代码质量：

  • ✅ 优秀。添加的注释“保留原始 URL 的 scheme 和 host，而不是假定为本地文件”非常清晰地解释了修改的原因，提高了代码的可读性和可维护性。
  • ✅ 优秀。复用了传入的 url 对象进行拷贝修改，代码意图明确。

• 代码性能：

  • ✅ 良好。QUrl 的拷贝和 setPath 操作都非常轻量，不会引入明显的性能开销。

• 代码安全：

  • ✅ 提升。之前的实现可能导致非本地文件路径被错误地解析为本地路径，引发安全风险（如沙箱逃逸或路径混淆）。修改后的代码确保了 URL 的上下文一致性，更加安全。

---

2. 文件：src/dfm-search/dfm-search-lib/utils/filenameblacklistmatcher.cpp

变更内容：
为 QString::split 方法添加了 Qt 版本兼容性宏，以适配 Qt 5 和 Qt 6。

审查意见：

• 语法逻辑：

  • ✅ 通过。Qt 6 中 QString::SplitBehavior 命名空间发生了变化，QString::SkipEmptyParts 变为了 Qt::SkipEmptyParts。使用 #if 宏进行条件编译是处理此类 API 变更的标准做法，逻辑正确。

• 代码质量：

  • ⚠️ 可优化：虽然这种写法是正确的，但在代码中频繁出现 #if 宏会降低可读性。
  • 建议：如果在项目中有多处需要适配 Qt 5/6 的差异，建议在公共头文件中定义一个统一的宏或辅助函数，例如：

    // 在公共头文件中
    #if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0)
    #define DFMSkipEmptyParts Qt::SkipEmptyParts
    #else
    #define DFMSkipEmptyParts QString::SkipEmptyParts
    #endif
    
    // 在代码中使用
    const QStringList pathSegments = normalizedPath.split('/', DFMSkipEmptyParts);

    这样可以将版本适配逻辑集中管理，避免代码散落各处。

• 代码性能：

  • ✅ 无影响。这是编译期分支，不会增加运行时开销。

• 代码安全：

  • ✅ 通过。代码逻辑安全，未引入新的风险。

---

总结

这两处修改都是必要且正确的：

1. 第一处修复了处理非本地文件 URL 时的潜在 Bug，提升了健壮性和安全性。
2. 第二处确保了代码在不同 Qt 版本下的编译兼容性。

建议：
对于第二处关于 Qt 版本适配的修改，建议考虑将其封装为项目级的通用宏或辅助函数，以保持代码整洁并便于后续维护。除此之外，代码质量很高。

[Johnson-zs]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)