在C#中编组C数组 - 简单的HelloWorld

时间:2014-05-01 15:59:18

标签: c# c arrays pointers marshalling

建立我的marshalling helloworld question,我遇到问题,编组用C到C#分配的数组。我花了几个小时研究我可能出错的地方,但我尝试过的所有内容都会出现AccessViolationException等错误。

处理在C中创建数组的函数如下所示。

__declspec(dllexport) int __cdecl import_csv(char *path, struct human ***persons, int *numPersons)
{
    int res;
    FILE *csv;
    char line[1024];
    struct human **humans;

    csv = fopen(path, "r");
    if (csv == NULL) {
        return errno;
    }

    *numPersons = 0; // init to sane value
    /*
     * All I'm trying to do for now is get more than one working.
     * Starting with 2 seems reasonable. My test CSV file only has 2 lines.
     */
    humans = calloc(2, sizeof(struct human *));
    if (humans == NULL)
        return ENOMEM;

    while (fgets(line, 1024, csv)) {
        char *tmp = strdup(line);
        struct human *person;

        humans[*numPersons] = calloc(1, sizeof(*person));
        person = humans[*numPersons]; // easier to work with
        if (person == NULL) {
            return ENOMEM;
        }
        person->contact = calloc(1, sizeof(*(person->contact)));
        if (person->contact == NULL) {
            return ENOMEM;
        }

        res = parse_human(line, person);
        if (res != 0) {
            return res;
        }

        (*numPersons)++;
    }
    (*persons) = humans;

    fclose(csv);

    return 0;
}

C#代码:

IntPtr humansPtr = IntPtr.Zero;
int numHumans = 0;

HelloLibrary.import_csv(args[0], ref humansPtr, ref numHumans);

HelloLibrary.human[] humans = new HelloLibrary.human[numHumans];
IntPtr[] ptrs = new IntPtr[numHumans];
IntPtr aIndex = (IntPtr)Marshal.PtrToStructure(humansPtr, typeof(IntPtr));

// Populate the array of IntPtr
for (int i = 0; i < numHumans; i++)
{
    ptrs[i] = new IntPtr(aIndex.ToInt64() +
            (Marshal.SizeOf(typeof(IntPtr)) * i));
}

// Marshal the array of human structs
for (int i = 0; i < numHumans; i++)
{
    humans[i] = (HelloLibrary.human)Marshal.PtrToStructure(
        ptrs[i],
        typeof(HelloLibrary.human));
}

// Use the marshalled data
foreach (HelloLibrary.human human in humans)
{
    Console.WriteLine("first:'{0}'", human.first);
    Console.WriteLine("last:'{0}'", human.last);

    HelloLibrary.contact_info contact = (HelloLibrary.contact_info)Marshal.
        PtrToStructure(human.contact, typeof(HelloLibrary.contact_info));

    Console.WriteLine("cell:'{0}'", contact.cell);
    Console.WriteLine("home:'{0}'", contact.home);
}

第一个human struct被编组好了。我在第一个之后得到了访问冲突异常。我觉得我错过了使用结构指针编组结构的东西。我希望我有一些我忽视的简单错误。你觉得这段代码有什么问题吗?

有关完整来源,请参阅此GitHub gist

1 个答案:

答案 0 :(得分:2)

  // Populate the array of IntPtr

这是你出错的地方。您正在返回指向指针数组的指针。你得到了第一个正确的,实际上从数组中读取指针值。但是你的for()循环错了,只是在第一个指针值上加上4(或8)。而不是从数组中读取它们。修正:

    IntPtr[] ptrs = new IntPtr[numHumans];

    // Populate the array of IntPtr
    for (int i = 0; i < numHumans; i++)
    {
        ptrs[i] = (IntPtr)Marshal.PtrToStructure(humansPtr, typeof(IntPtr));
        humansPtr = new IntPtr(humansPtr.ToInt64() + IntPtr.Size);
    }

或者更加干净,因为已经支持编组简单类型的数组:

    IntPtr[] ptrs = new IntPtr[numHumans];
    Marshal.Copy(humansPtr, ptrs, 0, numHumans);

我通过使用Debug + Windows + Memory + Memory找到了这个错误1.将 humansPtr 放在Address字段中,切换到4字节整数视图并观察C代码是否正确执行。然后很快发现ptrs []不包含我在Memory窗口中看到的值。

不确定为什么要编写这样的代码,而不是作为心理练习。这不是正确的方法,例如,你完全无视再次释放内存的需要。哪个非常非常重要。在C#中解析CSV文件非常简单,就像在C中一样快,它是I / O绑定的,而不是执行绑定的。您可以轻松地避免这些几乎不可能调试错误并从.NET Framework获取lots of help