返回对象指针:'无法将'Coordinate *'转换为Python对象'

时间:2018-04-12 10:02:07

标签: python c++ cython cythonize

我正在尝试将我的c ++类cython化为python中的usabe,并且我用一个简单的类开始,但我仍然坚持返回一个指针。我试图解决这个问题的方法是添加一个拷贝构造函数并返回C ++类的Python版本,但没有任何成功。

我收到以下错误:

Error compiling Cython file:
------------------------------------------------------------
...

    def bearing(self, Coordinate *coordinate):
        return self.thisptr.bearing(coordinate)

    def destination(self, double bearing, double distance):
        return PyCCoordinate(self.thisptr.destination(bearing, distance))
                                                    ^
------------------------------------------------------------

./coordinate.pyx:32:53: Cannot convert 'Coordinate *' to Python object

这是我的文件

coordinate.h

class Coordinate {
private:
    double m_x = 0;
    double m_y = 0;
public:
    Coordinate(const double x, const double y);

    Coordinate(const Coordinate *coord);

    void setX(const double value);

    void setY(const double value);

    double getX();

    double getY();

    double distance(Coordinate *coord);

    double bearing(Coordinate *coord);

    Coordinate *destination(const double bearing, const double distance);
};

coordinate.cpp

#include <cmath>
#include "coordinate.h"

Coordinate::Coordinate(const double x, const double y) {
    m_x = x;
    m_y = y;
}

Coordinate::Coordinate(const Coordinate *coord) {
    m_x = coord->x;
    m_y = coord->y;
}


void Coordinate::setX(const double value) {
    m_x = value;
}

void Coordinate::setY(const double value) {
    m_y = value;
}

double Coordinate::getX() {
    return m_x;
}

double Coordinate::getY() {
    return m_y;
}

double Coordinate::distance(Coordinate *coord) {
    return sqrt(pow(m_x - coord->getX(), 2.0) + pow(m_y - coord->getY(), 2.0));
}

double Coordinate::bearing(Coordinate *coord) {
    const double len_x = coord->getX() - m_x;
    const double len_y = coord->getY() - m_y;
    double res = atan2(len_y, len_x);
    if (res < 0.0) {
        res += 2.0 * M_PI;
    }
    return res;
}

Coordinate *Coordinate::destination(const double bearing, const double distance) {
    double new_x = m_x + cos(bearing) * distance;
    double new_y = m_y + sin(bearing) * distance;
    return new Coordinate(new_x, new_y);
}

coordinate.pxy

cdef extern from "coordinate.h":
    cdef cppclass Coordinate:
        Coordinate(const double x, const double y) except +
        Coordinate(const Coordinate *coord) except +

        void setX(const double value)
        void setY(const double value)
        double getX()
        double getY()
        double distance(Coordinate *coord)
        double bearing(Coordinate *coord)
        Coordinate *destination(const double bearing, const double distance)

cdef class PyCCoordinate:
    cdef Coordinate *thisptr
    def __cinit__(self, double x, double y):
        self.thisptr = new Coordinate(x,y)
    def __cinit__(self, Coordinate* coord):
        self.thisptr = new Coordinate(coord)
    def __dealloc__(self):
        del self.thisptr        

    def distance(self, Coordinate *coordinate):
        return self.thisptr.distance(coordinate)

    def bearing(self, Coordinate *coordinate):
        return self.thisptr.bearing(coordinate)

    def destination(self, double bearing, double distance):
        return PyCCoordinate(self.thisptr.destination(bearing, distance))

1 个答案:

答案 0 :(得分:1)

一个问题是,cython语法有点误导:如果定义def - 函数(因为你的例子不是最小的,我构成一个不相关的函数):

def twice(double d):
   return 2.0*d

然后将参数传递给this(python)函数,而不是作为C-double,而是作为通常的Python对象。但是,早期绑定会导致cython尝试在运行时通过__pyx_PyFloat_AsDouble将此Python对象转换为C-double,这是调用该函数时的第一件事 - 这一切都发生在幕后,所以编码器你有这种欺骗性的填充,你会真正传递一个双重功能。

但是,此自动转换仅适用于某些类型 - 最突出的是doubleintcdef - 类等。对于其他类型,这是不可能的,例如对于原始指针(这也意味着指向自定义cpp类的指针) - 没有像__pyx_PyFloat_AsDouble这样的东西。

例如

def twice(double *d):
   pass

无法进行cython化,因为原始指针不能自动转换为python-object所需的python-object。

可以用原始指针定义cdef函数,因为它们只不过是简单的C函数,因此

cdef twice(double *d):
   pass

会编译。但是,这对__cinit__没有帮助,因为它必须是def - 函数。

不幸的是,Cython没有显示代码中的所有错误,只显示它找到的第一个错误 - 否则它会显示所有def Coordinate *coordinate的{​​{1}}无效。

如何解决它?基本上,您应该在签名中使用cdef - 包装类PyCCoordinate,然后使用thisptr - 进行计算,例如:

def distance(self, PyCCoordinate coordinate):
    return self.thisptr.distance(coordinate.thisptr)

这个解决方案显然不适用于构造对象,就像在方法destination中一样 - 你必须构造一个PyCCoordinate - 对象才能使用它!一个可能的解决方案是有一个构造函数,它将构造一个thisptrNULL的包装器对象,调用它并手动设置此指针,如

cdef class PyCCoordinate:
    cdef Coordinate *thisptr
    def __cinit__(self):
        pass

    def __dealloc__(self):
        del self.thisptr        

    def destination(self, double bearing, double distance):
        cdef PyCCoordinate res=PyCCoordinate()
        res.thisptr=self.thisptr.destination(bearing, distance)
        return res

另一个问题:cython(也是python),与c ++不同,不知道重载,所以你不能定义两个不同的构造函数(并且不能将其中一个定义为私有),因此必须手动完成调度,例如:< / p>

cdef class PyCCoordinate:
    cdef Coordinate *thisptr

    def __cinit__(self, x=None, y=None):
        if x is None or y is None:#default, "private" constructor
           pass     #thisptr is initialized to NULL
        else:
           self.thisptr = new Coordinate(x,y) #

    def __dealloc__(self):
        del self.thisptr        

    def destination(self, double bearing, double distance):
        cdef PyCCoordinate res=PyCCoordinate()
        res.thisptr=self.thisptr.destination(bearing, distance)
        return res