Question

我在Ubuntu中使用SWIG 2.0.10来用Java调用C ++代码。

我的C ++代码是：

//ImgPro.h:
#include <vector>

typedef struct _bin
{
    char* name;
    float value;
} Bin;

typedef struct imgprops
{
    std::vector<Bin> color;
    int width;
    int height;
    char *print;
} ImageProperties;

class ImgPro
{
public:
    ImgPro();

    ImageProperties *processImage(char* imagePath);
};

processImage函数定义是：

ImageProperties* ImgPro::processImage(char *imagePath)
{
    ImageProperties* imgProp = new ImageProperties();
    imgProp->width = 200;
    imgProp->height = 200;

    char* fp = new char(5);
    strcpy(fp, "abc!");
    imgProp->print = fp;

    Bin outputBin1;
    char *name1 = new char(strlen("red")+1);
    strcpy(name1, "red");
    outputBin1.name = name1;
    outputBin1.value = 0.125;

    Bin outputBin2;
    char *name2 = new char(strlen("blue")+1);
    strcpy(name2, "blue");
    outputBin2.name = name1;
    outputBin2.value = 0.27;

    vector<Bin> tempVec;
    tempVec.push_back(outputBin1);
    tempVec.push_back(outputBin2);
    imgProp->color = tempVec;
    return imgProp;

}

因此，要使用swig生成jni代码，我使用了以下swig文件（注意：vector.i文件是使用此example创建的）：

%module CBIR

// to handle char** has String_Array in Java
%include <various.i>
%include "vector.i"

%{
   #include "ImgPro.h"
%}

// to handle char** has String_Array in Java
%apply char **STRING_ARRAY { char ** };

// memory release
%extend imgprops {
       ~imgprops(){
    if($self != NULL)
    {
        // releasing print element
        if($self->print != NULL)
            delete[] $self->print;
        // releasing vector elements
        for(uint x = 0; x < $self->color.size(); x++)
        {
                Bin currentBin = $self->color[x];
                if(currentBin.name != NULL)
                    delete[] currentBin.name;
        }
        // releasing stuct Pointer
        delete $self;
    }
}
}

%include "ImgPro.h"

%template(BinVec) std::vector<Bin>;

这会在swig_wrap文件中生成下一个函数：

SWIGINTERN void delete_imgprops(imgprops *self){
    if(self != NULL)
    {
        // releasing print element
        if(self->print != NULL)
            delete[] self->print;
        // releasing vector elements
        for(uint x = 0; x < self->color.size(); x++)
        {
                Bin currentBin = self->color[x];
                if(currentBin.name != NULL)
                    delete[] currentBin.name;
        }
        // releasing stuct Pointer
        delete self;
    }
}

在删除ImageProperties c ++函数中调用。

但是，在Java中运行以下代码永远不会释放在C ++中分配的内存（调用函数delete_imgprops）：

ImgPro imgObject = new ImgPro();
ImageProperties propObject = imgObject.processImage("imagem123-jpg");            

int width = propObject.getWidth();
int height = propObject.getHeight();
String fingerPrint = propObject.getPrint();

propObject.delete();            
imgObject.delete();

因此，在分析了代码流之后，我发现了没有释放内存的原因。由SWIG生成的ImageProperties.Java文件包含删除功能：

public synchronized void delete() {
   if (swigCPtr != 0) {
     if (swigCMemOwn) {
       swigCMemOwn = false;
       CBIRJNI.delete_ImageProperties(swigCPtr);
     }
     swigCPtr = 0;
   }
}

“CBIRJNI.delete_ImageProperties（swigCPtr）;”永远不会调用因为var swigCMemOwn始终为false。

据我所知，因为Java端没有分配内存所以它也没有释放它，所以我该怎么做才能确保java释放内存而不对swig生成的java文件进行任何修改？

我发现释放内存的解决方案是在delete（）函数上注释if（swigCMemOwn）测试，但我不认为这是最好的方法！

谢谢，塞尔吉奥

Answer 1

你可以查看

%newobject

带有工厂方法的

指令（在swig 2.0中）。它告诉swig一个特定的函数将返回一个新对象，并且java代理类也应该负责清理c ++内存。生成的代码将swigCMemOwn设置为true，从而导致调用c ++析构函数。

如果您自己调用了删除方法，那很好 - 您只需要改变编程风格，将swig'ed对象视为文件句柄或数据库连接。当你完成这些对象时，你会调用close（），因为你不想等待java的GC在稍后的某个未知点开始并收集这个昂贵的资源 - 你想手动管理它。

但你显然必须记住要运用良好的代码规则来确保在调用delete（）之后不在任何地方使用java对象

Answer 2

您不应该手动拨打delete()。如果您正确实现了C ++析构函数（或者释放内存的任何地方），一旦Java包装器对象被释放，内存将被释放，然后SWIG包装器代码将自动调用适当的方法。阅读更多SWIG doc for Java。

SWIG java：释放用c ++分配的内存

2 个答案: