fix: Setting notifications number less than 3, do not display overlay

[yixinshark]

as title

Log: as title
Bug: https://pms.uniontech.com/bug-view-290185.html

[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. 代码可读性：

   • 在bubblemodel.cpp文件中，变量lastBubbleMaxIndex的命名应该更加直观，例如lastBubbleIndex，以表明它是一个索引值。

2. 常量使用：

   • 在bubblemodel.h文件中，LastBubbleMaxIndex被移除，但BubbleMaxCount仍然被使用。建议在bubblemodel.h中定义LastBubbleMaxIndex为常量，以保持一致性。

3. 代码风格：

   • 在bubblemodel.cpp文件中，item->setLevel(i == lastBubbleMaxIndex ? 1 + overlayCount() : 1);这行代码的格式可以调整，使其更易读，例如：

     item->setLevel(i == lastBubbleMaxIndex ? 1 + overlayCount() : 1);

   • 可以考虑使用std::max函数来确保i不会超过BubbleMaxCount - 1，以避免潜在的越界访问。

4. 性能考虑：

   • 如果m_bubbles列表很大，可以考虑在循环开始前使用size()方法获取列表大小，以避免每次迭代都调用displayRowCount()方法。

5. 错误处理：

   • 在updateLevel方法中，没有对m_bubbles列表为空的情况进行错误处理。虽然这种情况已经在方法开始时进行了检查，但建议添加注释说明为什么不需要额外的错误处理。

6. 代码注释：

   • 在bubblemodel.h文件中，BubbleMaxCount和OverlayMaxCount的注释应该更详细，说明这些常量的用途和含义。

综上所述，代码的改进主要集中在提高可读性、保持一致性、调整代码风格、考虑性能和错误处理等方面。

[yixinshark]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)