sync: from linuxdeepin/dtkwidget

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkwidget.

Source-pull-request: linuxdeepin/dtkwidget#696

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot

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

我来对这个diff进行审查：

1. 语法逻辑：

• 代码语法正确，符合C++标准
• 函数命名遵循了项目的命名规范（驼峰式）
• 使用了Qt的属性系统，实现方式合理
• 添加了Q_FALLTHROUGH()宏，确保switch语句的正确执行

2. 代码质量：

• 代码结构清晰，注释说明了新功能的应用场景
• 使用了合理的参数类型（int margin）
• 添加了适当的边界检查（margin >= 0）
• 使用了QVariant的isValid()和toInt()进行类型安全检查

3. 代码性能：

• 实现简单高效，没有性能问题
• 使用了Qt的属性系统，避免了额外的数据结构

4. 代码安全：

• 添加了margin >= 0的检查，防止负值
• 使用了QVariant的toInt()进行安全转换
• 通过isValid()检查属性是否存在

改进建议：

1. 在头文件中添加更详细的函数文档注释：

/**
 * @brief 设置行编辑器图标边距
 * @param object 要设置的对象指针
 * @param margin 边距值，必须大于等于0
 */
static void setLineEditIconMargin(QObject *object, int margin);

2. 在setLineEditIconMargin函数中添加参数验证：

void DStyle::setLineEditIconMargin(QObject *object, int margin)
{
    if (!object || margin < 0) {
        return;
    }
    object->setProperty("_d_dtk_lineeditIconMargin", margin);
}

3. 考虑将属性名"_d_dtk_lineeditIconMargin"定义为常量，便于维护：

static const char* const LINE_EDIT_ICON_MARGIN_PROPERTY = "_d_dtk_lineeditIconMargin";

4. 在pixelMetric函数中，可以将margin的获取和验证逻辑提取为单独的函数，提高代码复用性：

static int getLineEditIconMargin(const QWidget *widget)
{
    if (!widget) {
        return -1;
    }
    const QVariant &margin_value = widget->property(LINE_EDIT_ICON_MARGIN_PROPERTY);
    if (!margin_value.isValid()) {
        return -1;
    }
    bool ok = false;
    int margin = margin_value.toInt(&ok);
    return (ok && margin >= 0) ? margin : -1;
}

这些改进可以提高代码的可维护性、可读性和健壮性。