-1

我有两个代码几乎相同的示例。我可以从 C# 中检索一个的数据,但我没有正确的另一个数据。他们来了:

效果很好

C++部分:

__declspec(dllexport) void** enumerateDevices(int *dIsize){
    Array<DeviceInfo> dIArray;
    Framewoek::enumerateDevices(&dIArray);
    *dIsize = dIArray.getSize();
    DeviceInfo dP[255];
    for (int i = 0; i < dIArray.getSize(); i++)
        dP[i] = dIArray[i];
    void* p = dP;
    return &p;
}

C#部分:

    [DllImport("Wrapper.dll")]
    static extern IntPtr enumerateDevices(out int devicesSize);
    public static DeviceInfo[] EnumerateDevices()
    {
        int devicesSize;
        IntPtr arrayPointer = enumerateDevices(out devicesSize);
        IntPtr[] array = new IntPtr[devicesSize];
        Marshal.Copy(arrayPointer, array, 0, devicesSize);
        DeviceInfo[] arrayObjects = new DeviceInfo[devicesSize];
        for (int i = 0; i < devicesSize; i++)
            arrayObjects[i] = new DeviceInfo(array[i]);
        return arrayObjects;
    }

不按预期工作

C++部分:

__declspec(dllexport) void** SensorInfo_getSupportedVideoModes(SensorInfo* si, int *dIsize){
    const Array<VideoMode>& dIArray = si->getSupportedVideoModes();
    *dIsize = dIArray.getSize();
    VideoMode dP[255];
    for (int i = 0; i < dIArray.getSize(); i++)
        dP[i] = dIArray[i];
    void* p = dP;
    return &p;
}

C#部分:

    [DllImport("Wrapper.dll")]
    static extern IntPtr SensorInfo_getSupportedVideoModes(IntPtr objectHandler, out int arraySize);
    public VideoMode[] getSupportedVideoModes()
    {
        int arraySize;
        IntPtr arrayPointer = SensorInfo_getSupportedVideoModes(this.Handle, out arraySize);
        IntPtr[] array = new IntPtr[arraySize];
        Marshal.Copy(arrayPointer, array, 0, arraySize);
        VideoMode[] arrayObjects = new VideoMode[arraySize];
        for (int i = 0; i < arraySize; i++)
            arrayObjects[i] = new VideoMode(array[i]);
        return arrayObjects;
    }

如您所见,我只想要对象的指针。但是第二个例子给了我无效的指针。我不知道为什么。这是一个很大的库,我正在尝试创建一个 .net 包装器,除了这个功能之外,一切都很好。

注意:Array<> 是一个模板类,包含两个字段、大小和数组以及几个方法。


*编辑:我知道它是一个局部变量(甚至 VS 都会给我警告),但是为什么第一个有效呢?!


*编辑2:

我使用@JefferyThomas 回答解决了第二个问题。它现在看起来像这样,没有问题: C++ 方面:

__declspec(dllexport) void* SensorInfo_getSupportedVideoModes(SensorInfo* si, int *dIsize){
    const Array<VideoMode>& dIArray = si->getSupportedVideoModes();
    *dIsize = dIArray.getSize();
    VideoMode** dP = new VideoMode*[255];
    for (int i = 0; i < dIArray.getSize(); i++)
        dP[i] = const_cast<VideoMode*>(&(dIArray[i]));
    return dP;
}
__declspec(dllexport) void SensorInfo_destroyVideoModesArray(VideoMode** dP){
    delete[] dP;
}

目前我在更改我的第一个函数时遇到问题,就像@JefferyThomas 建议的那样。让我们看一下代码,然后我会告诉你问题出在哪里。C++ 方面:

__declspec(dllexport) void** Framework_enumerateDevices(int *dIsize){
    Array<DeviceInfo> dIArray;
    Framework::enumerateDevices(&dIArray);
    *dIsize = dIArray.getSize();
    DeviceInfo* dP = new DeviceInfo[255];
    for (int i = 0; i < dIArray.getSize(); i++)
        dP[i] = dIArray[i];
    void** p = new void*;
    *p = dP;
    return p;
}
__declspec(dllexport) void Framework_destroyDevicesArray(void **p){
    DeviceInfo *dP = (DeviceInfo *)*p;
    //delete [] dP;
    delete p;
}

C# 方面:

    [DllImport("Wrapper.dll")]
    static extern IntPtr Framework_enumerateDevices(out int devicesSize);
    [DllImport("Wrapper.dll")]
    static extern IntPtr Framework_destroyDevicesArray(IntPtr arrayPointer);
    public static DeviceInfo[] EnumerateDevices()
    {
        int devicesSize;
        IntPtr arrayPointer = Framework_enumerateDevices(out devicesSize);
        IntPtr[] array = new IntPtr[devicesSize];
        Marshal.Copy(arrayPointer, array, 0, devicesSize);
        DeviceInfo[] arrayObjects = new DeviceInfo[devicesSize];
        for (int i = 0; i < devicesSize; i++)
            arrayObjects[i] = new DeviceInfo(array[i]);
        Framework_destroyDevicesArray(arrayPointer);
        return arrayObjects;
    }

和问题:

  1. 我不能在销毁函数中保留“删除 [] dP”。它不仅删除数组,还删除数据。
  2. 我只能正确地拥有第一个指针(数组的第一项)。其他指针均不正确。我不知道为什么。尝试将 DeviceInfo 直接更改为指针数组而不是对象数组(如第二个函数),但仍然没有成功。

目前我认为这是因为 Array<> 类处理和销毁所有东西。


编辑3:我自己找到了第一个问题的解决方案。感谢杰夫让我朝着正确的方向前进。

4

1 回答 1

5

这里的问题是您正在返回一个堆栈变量。这是一个坏主意。

当函数返回时,两者VideoMode dP[255];都被销毁。DeviceInfo dP[255];碰巧在第一种情况下内存没有被覆盖,在第二种情况下你就没有那么幸运了。

在过去,我从堆中分配并提供了第二个 releaseXXXX 方法来释放该内存。

__declspec(dllexport) void** enumerateDevices(int *dIsize){
    Array<DeviceInfo> dIArray;
    Framewoek::enumerateDevices(&dIArray);
    *dIsize = dIArray.getSize();
    DeviceInfo *dP = new DeviceInfo[255];
    for (int i = 0; i < dIArray.getSize(); i++)
        dP[i] = dIArray[i];
    void** p = new void*;
    *p = dP;
    return p;
}

__declspec(dllexport) void releaseEnumerateDevices(void **p){
    DeviceInfo *dP = (DeviceInfo *)*p;
    delete [] dP;
    delete p;
}

C#

    [DllImport("Wrapper.dll")]
    static extern IntPtr enumerateDevices(out int devicesSize);
    static extern void releaseEnumerateDevices(IntPtr arrayPointer);
    public static DeviceInfo[] EnumerateDevices()
    {
        int devicesSize;
        IntPtr arrayPointer = enumerateDevices(out devicesSize);
        IntPtr[] array = new IntPtr[devicesSize];
        Marshal.Copy(arrayPointer, array, 0, devicesSize);
        DeviceInfo[] arrayObjects = new DeviceInfo[devicesSize];
        for (int i = 0; i < devicesSize; i++)
            arrayObjects[i] = new DeviceInfo(array[i]);
        releaseEnumerateDevices(arrayPointer);
        return arrayObjects;
    }
于 2013-02-13T20:33:13.037 回答