chore: dde notification dbus adaptor export standard interface

[yixinshark]

as title

Log: as title

[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

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 代码格式和风格：

   • 在dbusadaptor.h文件中，public Q_SLOTS:和Q_SIGNALS:的重复声明应该合并为一个，以避免重复和混淆。

2. 方法命名：

   • GetCapabilities方法名应该遵循驼峰命名法，即getCapabilities。
   • Notify方法名应该遵循驼峰命名法，即notify。
   • CloseNotification方法名应该遵循驼峰命名法，即closeNotification。
   • GetServerInformation方法名应该遵循驼峰命名法，即getServerInformation。

3. 参数命名：

   • appName、appIcon、summary、body、actions、hints等参数名应该使用小写字母和下划线分隔，例如app_name、app_icon等。

4. 注释：

   • 在GetServerInformation方法中，todo注释应该被移除，因为它表示未完成的工作，而当前代码中并没有这样的任务。

5. 代码重复：

   • manager()方法在dbusadaptor.cpp文件中被多次调用，可以考虑将其提取为一个私有成员函数，以减少代码重复。

6. 错误处理：

   • 在Notify、CloseNotification和GetServerInformation方法中，没有对manager()返回的指针进行空指针检查，可能会导致运行时错误。应该添加相应的空指针检查。

7. 信号和槽的命名：

   • ActionInvoked和NotificationClosed信号名应该遵循驼峰命名法，即actionInvoked和notificationClosed。

8. 代码可读性：

   • GetServerInformation方法中的参数应该使用引用传递，而不是值传递，以提高性能。

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

[yixinshark]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)