fix: x11 shutdown menu can not lock screen

[yixinshark]

as title

Log: as title

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 代码重复：在requestShutdown函数中，type为空时，直接调用x11LockScreen()方法，而在x11LockScreen方法内部，当originMap为空时，没有进行任何操作。这可能导致逻辑上的不一致，建议在x11LockScreen方法中处理originMap为空的情况。

2. 错误处理：在x11LockScreen方法中，当process.waitForFinished()返回false时，只打印了警告信息，但没有采取任何措施来处理错误。建议添加错误处理逻辑，比如重试或者记录错误日志。

3. 资源管理：QProcess对象在x11LockScreen方法中创建，但没有看到对应的销毁操作。虽然QProcess对象在超出作用域时会自动销毁，但为了代码的可读性和维护性，建议显式地调用process.close()和process.waitForFinished()。

4. 安全性：在x11LockScreen方法中，直接执行了外部命令，这可能会引入安全风险。建议使用更安全的方法来执行外部命令，比如使用QProcess的startDetached()方法。

5. 代码风格：x11LockScreen方法中的注释使用了中文，而在其他地方使用了英文注释。建议保持代码注释的风格一致。

6. 代码可读性：x11LockScreen方法中的代码可以进一步拆分成更小的函数，以提高代码的可读性和可维护性。

7. 异常处理：在x11LockScreen方法中，没有对QProcess的执行结果进行异常处理。建议添加异常处理逻辑，比如捕获QProcess的执行异常。

8. 代码重复：在requestShutdown函数中，type为空时，直接调用x11LockScreen()方法，而在x11LockScreen方法内部，当originMap为空时，没有进行任何操作。这可能导致逻辑上的不一致，建议在x11LockScreen方法中处理originMap为空的情况。

综上所述，建议对代码进行重构，以提高代码的可读性、可维护性和安全性。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, yixinshark

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

[yixinshark]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)