fix: add QML annotations to eliminate compilation warnings

[18202781743]

Added QML_ELEMENT and QML_UNCREATABLE macros to PluginSurface and
PluginPopup classes to resolve QString::arg compilation warnings. These
annotations explicitly declare the QML element registration and prevent
QML instantiation, ensuring proper type registration and eliminating the
missing argument warnings during compilation.

Log: Fixed compilation warnings related to QML type registration

Influence:

1. Verify that the dock plugin compiles without warnings
2. Test that PluginSurface and PluginPopup classes still function
   correctly in C++ context
3. Confirm that QML cannot instantiate these classes as intended
4. Check that all existing dock functionality remains unaffected

fix: 添加QML注解消除编译警告

为PluginSurface和PluginPopup类添加QML_ELEMENT和QML_UNCREATABLE宏，解决
QString::arg编译警告。这些注解显式声明QML元素注册并防止QML实例化，确保正
确的类型注册并消除编译时的参数缺失警告。

Log: 修复了与QML类型注册相关的编译警告

Influence:

1. 验证dock插件编译时无警告
2. 测试PluginSurface和PluginPopup类在C++上下文中仍能正常工作
3. 确认QML无法实例化这些类（符合设计意图）
4. 检查所有现有dock功能不受影响

Summary by Sourcery

Add QML annotations to PluginSurface and PluginPopup classes to resolve compilation warnings and enforce non-creatable QML elements

Bug Fixes:

• Fix compilation warnings related to missing QML type registration in PluginSurface and PluginPopup

Enhancements:

• Add QML_ELEMENT and QML_UNCREATABLE macros to PluginSurface and PluginPopup to explicitly register types and prevent QML instantiation

[deepin-ci-robot]

deepin pr auto review

我对这段代码进行审查，主要关注语法逻辑、代码质量、性能和安全性等方面：

语法逻辑

1. 语法正确，没有明显的语法错误。
2. 类继承关系合理，使用了模板类QWaylandShellSurfaceTemplate和QtWaylandCompositor。
3. Q_PROPERTY宏的使用正确，属性声明完整。

代码质量

1. 添加了QML_ELEMENT和QML_UNCREATABLE是一个好的改进，这明确表示了这些类是QML元素但不能直接创建。
2. 类的声明遵循了良好的C++实践，如删除了拷贝构造函数（= delete）。
3. 属性命名清晰，遵循了驼峰命名法。

改进建议

1. 文档注释：建议为类和关键方法添加详细的文档注释，说明其用途和使用方法。
2. 接口设计：
   • 考虑为PluginSurface和PluginPopup添加纯虚接口类，以提高代码的可扩展性和解耦性。
   • 可以考虑使用Pimpl模式实现，减少头文件依赖，提高编译效率。
3. 错误处理：
   • 在构造函数中添加参数有效性检查，例如x, y坐标是否在合理范围内。
   • 考虑添加更多的错误处理机制，特别是在属性设置时。

性能考虑

1. 对于频繁调用的属性访问（如height、width），可以考虑添加缓存机制，避免重复计算。
2. 如果这些类会被频繁创建和销毁，可以考虑实现对象池来管理这些实例。
3. 注意信号和槽的连接方式，避免不必要的信号触发。

安全性

1. 确保所有输入参数都经过验证，特别是来自QML的输入。
2. 对于setX、setY等设置坐标的方法，应该添加边界检查，防止坐标值超出合理范围。
3. 考虑添加更多的const正确性，确保对象状态不会被意外修改。

其他建议

1. 考虑添加更多的单元测试，确保各种边界条件下的行为正确。
2. 可以考虑使用智能指针来管理对象生命周期，避免内存泄漏。
3. 对于QML属性，考虑添加验证器（Validator）来确保输入值的合法性。

总体而言，这段代码结构清晰，添加的QML元信息提高了代码的可用性。通过实施上述建议，可以进一步提高代码的健壮性、可维护性和性能。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, mhduiy, wjyrich

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

[18202781743]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)

[sourcery-ai]

Hey there - I've reviewed your changes and they look great!

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}