fix: 窗口特效支持三挡调节

[fly602]

窗口特效支持三挡调节

Log: 窗口特效支持三挡调节
pms: BUG-286797

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 代码逻辑：

   • 在RequestSwitchWM函数中，原先的逻辑是先检查是否可以关闭特效，然后再检查是否可以开启特效。如果特效无法开启，则显示错误通知。现在的修改直接显示一个通用的"SwitchWM"通知，这可能会掩盖实际的问题。建议保留原有的逻辑，或者添加更详细的错误信息。

2. 代码可读性：

   • 新的代码简化了逻辑，但可能会降低代码的可读性。建议保留原有的注释，以便其他开发者理解代码的意图。

3. 错误处理：

   • 如果特效无法开启，应该提供更详细的错误信息，而不是仅仅显示一个通用的通知。这有助于调试和用户理解问题。

4. 代码重复：

   • 如果showOSD函数在其他地方也有使用，并且需要传递不同的错误信息，建议将错误信息作为参数传递给showOSD函数，而不是直接传递字符串。

5. 代码维护：

   • 如果将来需要添加更多的错误处理逻辑，建议将错误处理部分提取到一个单独的函数中，以保持RequestSwitchWM函数的简洁性。

改进后的代码示例：

void WMSwitcher::RequestSwitchWM() const
{
    if (m_wmInter->compositingEnabled()) {
        m_wmInter->setCompositingEnabled(false);
        return;
    }

    // 开启特效前需要先判断是否支持开启
    if (m_wmInter->compositingPossible()) {
        m_wmInter->setCompositingEnabled(true);
        return;
    }

    // 无法开启特效，发送osd通知
    showOSD(OSD_Switch_WM_ERROR);
}

void WMSwitcher::showOSD(const QString &message) const
{
    // 实现显示OSD的逻辑
}

这样，RequestSwitchWM函数的逻辑更加清晰，并且showOSD函数可以接受不同的错误信息，提高了代码的可维护性和可读性。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fly602, mhduiy

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