使用Cython封装C++类时构造函数被多次调用的问题求助

看起来你已经搞定了C++类到Python模块的封装，能生成.so也能导入，但碰到了构造函数被多次调用的麻烦——这确实挺让人困惑的，我来帮你捋捋常见的原因和解决办法。

先把你提供的现有代码贴出来方便参考：

你的现有C++代码 #

DBReader.h #

#ifndef _DB_READER_H_
#define _DB_READER_H_
#include <iostream>
class DBReader {
public:
    DBReader();
    ~DBReader();
};
#endif /* _DB_READER_H_ */

DBReader.cxx #

#include "DBReader.h"
DBReader::DBReader() {
    std::cout << "Hello\n";
}
DBReader::~DBReader() {
    // 假设析构函数实现在这里
}

最可能的问题：Cython代码中的对象初始化逻辑 #

因为你没贴出Cython的.pyx文件，这通常是构造函数多次调用的重灾区，我先讲几个最常见的错误场景和修复方式：

错误场景1：__init__和__cinit__重复创建对象 #

很多人会在Cython的Python封装类中同时用__init__和__cinit__来初始化C++对象，但__cinit__是C层面的构造函数（只会被调用一次），而__init__是Python层面的初始化方法（可能被手动调用多次）。如果两个方法里都new DBReader()，就会触发多次构造。

错误示例：

cdef extern from "DBReader.h":
    cdef cppclass DBReader:
        DBReader()

cdef class PyDBReader:
    cdef DBReader* ptr

    def __cinit__(self):
        self.ptr = new DBReader()  # 第一次调用构造

    def __init__(self):
        self.ptr = new DBReader()  # 第二次调用构造，还会导致内存泄漏！

正确的初始化方式 #

只在__cinit__中创建C对象，并在__dealloc__中释放内存，确保每个Python实例只对应一个C对象：

cdef extern from "DBReader.h":
    cdef cppclass DBReader:
        DBReader()

cdef class PyDBReader:
    cdef DBReader* ptr

    def __cinit__(self):
        # __cinit__只会被调用一次，完美对应C++对象的构造
        self.ptr = new DBReader()

    def __dealloc__(self):
        # 记得释放内存，避免泄漏
        del self.ptr

错误场景2：全局/静态的C++对象 #

如果你的Cython代码里定义了全局的DBReader对象，比如：

cdef extern from "DBReader.h":
    cdef cppclass DBReader:
        DBReader()

# 全局对象，导入模块时就会调用一次构造
cdef DBReader global_reader

cdef class PyDBReader:
    cdef DBReader* ptr
    def __cinit__(self):
        self.ptr = new DBReader()  # 每次创建Python实例又调用一次

这种情况下，导入模块时会打印一次Hello，每次创建PyDBReader实例又会打印一次，导致多次调用。如果不需要全局对象，直接删掉就行。

错误场景3：隐式的对象拷贝 #

如果在Cython函数中返回DBReader值（而不是指针或引用），编译器会生成默认的拷贝构造函数（如果你没自定义的话）。不过默认拷贝构造不会调用你的无参构造函数，所以如果你的Hello打印次数和Python实例数量不符，要检查是否有这种情况——比如：

cdef DBReader create_reader():
    DBReader r
    return r  # 这里会触发拷贝构造，但不会调用你的无参构造

这种情况不会增加Hello的打印次数，但如果你的类有其他资源初始化逻辑，可能会有问题，最好用指针返回：

cdef DBReader* create_reader():
    return new DBReader()

排查步骤总结 #

• 先检查你的Cython .pyx 文件，看是否在多个地方（__cinit__、__init__、全局变量、函数内部）创建了DBReader实例。
• 确保只在__cinit__中初始化C++对象，不要在__init__里重复创建。
• 检查是否有全局/静态的DBReader对象导致模块导入时就触发构造。
• 确认Python代码中是否多次实例化了封装后的类——如果是正常多次创建实例，那多次打印Hello是正常的，只有在单个Python实例对应多次构造调用时才是问题。

完整的工作示例 #

给你一个完整的可运行示例，包括setup.py：

dbreader.pyx #

cdef extern from "DBReader.h":
    cdef cppclass DBReader:
        DBReader()
        ~DBReader()

cdef class PyDBReader:
    cdef DBReader* ptr

    def __cinit__(self):
        self.ptr = new DBReader()

    def __dealloc__(self):
        del self.ptr

setup.py #

from setuptools import setup, Extension
from Cython.Build import cythonize

ext_modules = [
    Extension(
        "dbreader",
        sources=["dbreader.pyx", "DBReader.cxx"],
        language="c++",
        include_dirs=["."],
    )
]

setup(
    name="dbreader",
    ext_modules=cythonize(ext_modules),
)

编译命令：

python setup.py build_ext --inplace

测试Python代码：

import dbreader
# 只创建一个实例，应该只打印一次Hello
reader = dbreader.PyDBReader()

这样应该就能解决构造函数多次调用的问题了，如果还有问题，可以把你的.pyx文件贴出来再进一步排查。

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