fix: optimize dbus monitor

[zy-seven]

optimize dbus monitor

Log: optimize dbus monitor

Bug: https://pms.uniontech.com/bug-view-291055.html

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 重复定义宏：

   • 在multitaskview.cpp文件中，DBUS_APPEARANCE_SERVICE、DBUS_APPEARANCE_OBJ和DBUS_APPEARANCE_INTF宏在多个文件中重复定义，建议将这些宏移动到一个公共头文件中，以避免重复定义。

2. 不必要的QDBusInterface调用：

   • 在outline.cpp文件中，ActiveColor函数中直接使用QDBusInterface获取属性值，建议使用m_colorConfigReader来获取属性值，以保持一致性。

3. 未使用的变量：

   • 在chameleonbutton.cpp文件中，qDebug宏被注释掉了，如果不需要调试信息，建议删除这些代码以保持代码的整洁。

4. 潜在的内存泄漏：

   • 在workspace.cpp文件中，m_fontSizeConfigReader和m_fontFamilyConfigReader被定义为指针，但在析构函数中并没有释放这些指针，建议使用std::unique_ptr来管理这些指针的生命周期，以避免潜在的内存泄漏。

5. 代码风格一致性：

   • 在workspace.cpp文件中，slotActiveColorChanged和slotIconThemeChanged函数的参数类型不一致，建议统一使用QVariant类型作为参数。

6. 未使用的变量：

   • 在workspace.h文件中，KWinDBusService、KWinDBusPath和KWinDBusInterface宏在文件中未使用，建议删除这些宏以保持代码的整洁。

7. 潜在的空指针解引用：

   • 在workspace.cpp文件中，m_colorConfigReader和m_iconConfigReader在ActiveColor函数中被使用前没有检查是否为空，建议在访问这些指针之前添加空指针检查。

8. 代码重复：

   • 在workspace.cpp文件中，getFontSizeScale和getFontFamily函数中重复了类似的代码来获取属性值，建议将这些代码封装到一个公共函数中，以减少代码重复。

总体来说，代码中存在一些重复定义、未使用的变量、潜在的内存泄漏和代码风格不一致的问题，建议进行相应的修改和优化。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: haidong-602, zy-seven

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