Revert "fix: 解决dconfig属性类型不对导致崩溃的问题"

[fly602]

This reverts commit 42c4ea8.

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fly602

The full list of commands accepted by this bot can be found here.

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. 类型转换：

   • 在代码中，将 dbus.Variant 的值从 int64 转换为 float64，这可能会导致数据丢失或精度问题。如果 dbus.Variant 的值确实应该是 float64 类型，那么这个改动是合理的。如果原本应该是 int64，那么应该保持类型一致，避免潜在的错误。

2. 日志记录：

   • 在日志记录中，直接将 m.CustomShutdownWeekDays 和 m.ShutdownRepetition 输出可能会导致日志信息难以阅读。建议使用格式化字符串来提高日志的可读性。

3. 代码风格：

   • 在 initDsg 函数中，case 分支的代码风格不一致，有些分支使用了大括号，有些没有。建议统一使用大括号来提高代码的可读性。

4. 错误处理：

   • 在 setPropShutdownRepetition 方法调用后，没有检查其返回值是否为 false，如果 setPropShutdownRepetition 方法失败，应该有相应的错误处理逻辑。

5. 类型断言：

   • 在类型断言时，没有进行错误处理。如果 data.Value().(float64) 断言失败，会导致程序崩溃。建议使用类型断言的另一种形式，即 value, ok := data.Value().(float64)，并在 ok 为 false 时进行错误处理。

6. 注释：

   • 代码中没有足够的注释来解释 initDsg 函数的作用和每个 case 分支的具体逻辑。建议添加注释来提高代码的可维护性。

综上所述，建议在保持原有逻辑不变的前提下，对代码进行以下改进：

• 确认类型转换的合理性，并保持类型一致。
• 统一使用大括号来处理 case 分支。
• 添加错误处理逻辑，特别是在类型断言和 setPropShutdownRepetition 方法调用后。
• 使用格式化字符串来提高日志的可读性。
• 添加必要的注释来解释代码逻辑。

[deepin-bot]

TAG Bot

New tag: 6.1.13
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #712

[deepin-bot]

TAG Bot

New tag: 6.1.14
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #715

[deepin-bot]

TAG Bot

New tag: 6.1.15
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #718

[deepin-bot]

TAG Bot

New tag: 6.1.16
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #725