如何在PyGetSetDef中使用closure复用属性getter与setter？

你对closure参数的理解存在偏差，它本身支持传入任意自定义数据的指针，不需要局限于函数指针用来做分支判断标记，用元数据结构体承载每个属性的配置信息是官方推荐的标准实现方案，具体实现如下：

步骤1：定义属性元数据结构体 #

把每个属性的公共配置信息抽离成独立结构体，一次定义全局复用：

#include <stddef.h>

// 自定义属性元数据结构体，可根据业务需求扩展字段
typedef struct {
    size_t offset; // 属性在FOO结构体中的内存偏移量
    int (*validator)(PyObject *value); // 对应属性的校验函数，无校验可填NULL
    const char *err_msg; // 校验失败的报错提示
} FooAttrMeta;

步骤2：实现各属性独立的校验逻辑 #

每个属性的校验逻辑单独实现，完全解耦：

// bar_1属性的校验逻辑
static int validate_bar1(PyObject *value) {
    // 替换为实际校验规则，合法返回0，不合法返回-1
    if (/* 校验不通过条件 */) return -1;
    return 0;
}

// bar_2属性的校验逻辑
static int validate_bar2(PyObject *value) {
    // 替换为实际校验规则
    if (/* 校验不通过条件 */) return -1;
    return 0;
}

步骤3：预定义所有属性的元数据实例 #

所有属性的配置统一在此处管理，新增属性只需新增对应实例即可：

static const FooAttrMeta meta_bar1 = {
    .offset = offsetof(FOO, bar_1),
    .validator = validate_bar1,
    .err_msg = "invalid value for bar_1."
};
static const FooAttrMeta meta_bar2 = {
    .offset = offsetof(FOO, bar_2),
    .validator = validate_bar2,
    .err_msg = "invalid value for bar_2."
};

步骤4：改写通用getter/setter #

不需要任何分支判断，直接从closure中取元数据操作对应属性：

static PyObject *
FOO_getter(FOO* self, void *closure) {
    const FooAttrMeta *meta = (const FooAttrMeta *)closure;
    // 通过偏移量直接定位到对应属性的内存地址
    PyObject **attr_ptr = (PyObject **)((char *)self + meta->offset);
    // 按Python C API规范返回新引用，根据实际属性类型调整逻辑
    Py_INCREF(*attr_ptr);
    return *attr_ptr;
}

static int
FOO_setter(FOO* self, PyObject *value, void *closure) {
    const FooAttrMeta *meta = (const FooAttrMeta *)closure;
    // 执行校验逻辑
    if (meta->validator != NULL) {
        if (meta->validator(value) < 0) {
            PyErr_SetString(PyExc_ValueError, meta->err_msg);
            return -1;
        }
    }
    // 安全更新属性引用
    PyObject **attr_ptr = (PyObject **)((char *)self + meta->offset);
    Py_XINCREF(value);
    Py_XSETREF(*attr_ptr, value);
    return 0;
}

步骤5：简化PyGetSetDef定义 #

属性表结构极度精简，新增属性仅需新增一行配置：

static PyGetSetDef FOO_getsetters[] = {
    {"bar_1", (getter)FOO_getter, (setter)FOO_setter, "bar_1 attribute", (void *)&meta_bar1},
    {"bar_2", (getter)FOO_getter, (setter)FOO_setter, "bar_2 attribute", (void *)&meta_bar2},
    {NULL}  /* Sentinel */
};

该方案完全符合Python C API设计规范，消除了所有冗余分支，新增属性不需要修改原有getter/setter逻辑，维护成本极低。如果你的属性不是PyObject*类型，仅需调整元数据结构体字段和getter/setter中的类型转换逻辑即可。

内容的提问来源于stack exchange，提问作者M47