fix: [network] add network full support

[jeffshuai]

add network full support

Log: add network full support

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

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 代码重复：

   • isValidLogicalName 和 isValidMAC 函数中，QDir 对象的创建和 exists 方法的调用是重复的，可以考虑将这部分逻辑提取出来作为一个单独的函数。

2. 命名规范：

   • isValidLogicalName 和 isValidMAC 函数的命名不够明确，建议使用更具描述性的命名，例如 isLogicalNameValid 和 isMacAddressValid。

3. 日志记录：

   • 在 isValidLogicalName 函数中，当目录不存在时，使用了 qCInfo 记录日志。建议使用 qCWarning 或 qCDebug，以便在调试时更容易找到问题。

4. 正则表达式：

   • isValidMAC 函数中的正则表达式可以简化，例如 ^([0-9A-Fa-f]{2}[:-]){5}([0-9A-Fa-f]{2})$，这样可以确保整个字符串完全匹配。

5. 资源管理：

   • 在 generatorNetworkDevice 函数中，DeviceNetwork 对象在创建后直接添加到 lstDevice 列表中，但没有看到对这些对象的删除或释放操作。建议在适当的时候释放这些对象，以避免内存泄漏。

6. 逻辑判断：

   • 在 isValidLogicalName 和 isValidMAC 函数中，逻辑判断可以简化，例如使用 QString::compare 方法而不是 contains 方法。

7. 代码风格：

   • 代码中存在一些不一致的缩进和空格使用，建议统一代码风格，以提高可读性。

8. 错误处理：

   • 在 isValidLogicalName 和 isValidMAC 函数中，当输入参数为空时，函数没有返回值。建议在函数开始时检查输入参数是否为空，并在为空时抛出异常或返回一个错误码。

9. 注释：

   • 代码中缺少一些必要的注释，特别是对于复杂的逻辑判断和函数，建议添加注释以提高代码的可维护性。

10. 性能优化：

    • 在 generatorNetworkDevice 函数中，对 lstDevice 的遍历和查找操作可能会影响性能，特别是当 lstDevice 非常大时。可以考虑使用更高效的数据结构或算法来优化这部分逻辑。

综上所述，代码在逻辑、性能、安全性和可维护性方面存在一些问题，建议根据以上意见进行修改和优化。

[jeffshuai]

/merge

[deepin-bot]

This pr cannot be merged! (status: unstable)

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jeffshuai, lzwind, max-lvs

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