fix: Update command execution for document processing

[dengzhongyuan365-dev]

• Changed command execution from string concatenation to QStringList for better argument handling in Model.cpp.
• Improved logging for command execution in DocTabBar.cpp and FindWidget.cpp.
• Ensured null checks before invoking methods in FindWidget.cpp.

This refactor enhances code readability and maintainability.

[deepin-ci-robot]

deepin pr auto review

我来对这几处代码修改进行详细审查：

1. htmltopdf/main.cpp 中的修改：

-    qInfo() << "Input file:" << requiredArguments.at(0) << "Output PDF:" << requiredArguments.at(1);
     if (requiredArguments.size() != 2) {
         qCritical() << "Invalid number of arguments, expected 2 but got" << requiredArguments.size();
         parser.showHelp(1);
     }
+    qInfo() << "Input file:" << requiredArguments.at(0) << "Output PDF:" << requiredArguments.at(1);

改进意见：

• 优点：将日志输出移到参数验证之后是正确的做法，避免了在参数不足时访问数组越界
• 建议：可以考虑使用 qInfo().noquote() 来避免输出中的引号，使日志更清晰

2. reader/document/Model.cpp 中的修改：

-        QString unzipCommand = "unzip " + targetDoc;
-        qCDebug(appLog) << "执行命令: " << unzipCommand;
-#if (QT_VERSION < QT_VERSION_CHECK(6, 0, 0))
-        decompressor.start(unzipCommand);
-#else
-        decompressor.startCommand(unzipCommand);
-#endif
+        QStringList unzipArgs;
+        unzipArgs << targetDoc;
+        qCDebug(appLog) << "执行命令: unzip" << unzipArgs;
+        decompressor.start("unzip", unzipArgs);

改进意见：

• 优点：使用 QStringList 参数列表而不是字符串拼接，更安全且避免了命令注入风险
• 建议：可以在执行前检查 unzip 命令是否存在，提高程序健壮性

3. reader/uiframe/DocTabBar.cpp 中的修改：

-    qCDebug(appLog) << "DocTabBar::insertSheet start - inserting sheet:" << sheet->filePath();
     if (sheet == nullptr) {
         qCWarning(appLog) << "Cannot insert null sheet";
         return;
     }
+    qCDebug(appLog) << "DocTabBar::insertSheet start - inserting sheet:" << sheet->filePath();

改进意见：

• 优点：将日志移到空指针检查之后，避免潜在的空指针解引用
• 建议：可以考虑添加对 filePath() 返回值的检查，确保路径有效

4. reader/widgets/FindWidget.cpp 中的修改：

-        m_docSheet->startSearch(searchText);
+        if (m_docSheet) {
+            m_docSheet->startSearch(searchText);
+        }

改进意见：

• 优点：添加了空指针检查，提高了代码安全性
• 建议：可以在 else 分支添加警告日志，帮助调试

总体改进建议：

1. 安全性：所有修改都提高了代码的安全性，特别是命令注入防护和空指针检查
2. 可维护性：代码结构更清晰，错误处理更完善
3. 日志记录：建议统一日志格式，使用结构化日志记录
4. 错误处理：建议添加更多的错误检查和异常处理机制
5. 代码复用：可以考虑将命令执行相关的代码封装成通用函数，减少重复代码

这些修改都是正向的改进，提高了代码的安全性和可靠性。建议继续保持这种代码审查和改进的实践。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dengzhongyuan365-dev, 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

[dengzhongyuan365-dev]

/forecemerge