fix: 优化treeland启动慢的问题

[lzwind]

优化treeland启动慢的问题

Bug: 优化treeland启动慢的问题

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 文件包含顺序：

   • 在detectcode.cpp文件中，新添加的#include <chardet/chardet.h>应该放在其他标准库和Qt库的包含语句之后，以确保编译器能够正确解析依赖关系。

2. 代码重复：

   • 在Service::slotWMChanged函数中，isWinEffectEnabled变量的赋值逻辑在listenWindowEffectSwitcher和slotWMChanged函数中重复。建议将这部分逻辑提取到一个单独的函数中，以减少代码重复。

3. 未使用的变量：

   • 在Service::slotWMChanged函数中，wmName参数未被使用。如果该参数不再需要，应该将其移除以避免混淆。

4. Qt信号和槽的连接：

   • 在Service::listenWindowEffectSwitcher函数中，connect语句使用了Qt::QueuedConnection，这可能会导致信号处理函数在事件循环的下一个迭代中执行，而不是立即执行。如果slotWMChanged函数需要立即执行，应该使用Qt::DirectConnection。

5. 全局变量：

   • m_mainTerminalIsInitWM变量被添加到Service类中，用于跟踪窗口管理器是否已经初始化。虽然这是一个合理的做法，但是应该确保这个变量的状态在所有相关函数中保持一致，以避免潜在的竞态条件。

6. 日志记录：

   • 在Service::slotWMChanged函数中，日志记录使用了qCInfo和qCWarning宏。建议使用更具体的日志级别，例如qCDebug，以便于调试和问题追踪。

7. 代码注释：

   • 在Service::slotWMChanged函数中，注释应该更详细地解释函数的目的和参数的含义，以便其他开发者更好地理解代码。

8. 代码风格：

   • 在Service::slotWMChanged函数中，isWinEffectEnabled变量赋值后应该添加一个空行，以提高代码的可读性。

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

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, wyu71

The full list of commands accepted by this bot can be found here.

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