fix: DConfig add check for AppId

[18202781743]

user can open non-root controlled files when appId is relative path.

Issue: https://bugzilla.suse.com/show_bug.cgi?id=1211374

[deepin-bot]

Doc Check bot
🟢 Document Coverage Check Passed!

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, kegechen

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

关键摘要:

• isValidAppId 函数中直接使用了 contains(' ') 来检查是否包含空格，这种方法不够健壮，建议使用正则表达式来匹配非法字符。
• isValidAppId 函数的实现依赖于 isValidFilename 函数，如果 appId 仅包含空格但被认为有效，那么 isValidFilename 可能不会被正确调用。
• 在 ut_DConfigFileCheckAppId 测试用例中，guard 变量被声明但未使用，应该移除未使用的变量以提高代码清晰度。
• 测试用例中的 FILE_NAME 和 noAppidMetaPath 变量应该是常量或配置项，而不是硬编码的字符串，以提高代码的可维护性。
• 测试用例 std::tuple{QString("org foo"), false} 中的 false 应该是一个变量或者常量，而不是硬编码的值。

是否建议立即修改:
是

[zccrs]

Maybe should use regexp