Question

我从文本文件中获得了2d数组并成功打印出来。但是，我不能将这些元素用于函数参数。

这是打印2d数组的代码：

    fscanf(fp, "%d", &row);
    //printf("%d\n", row);
    array = (int**)malloc(sizeof(int*)*row);

    for (u = 0; u < row; u++) {
        array[u] = (int*)malloc(sizeof(int)*col);
        for (v = 0; v < col; v++) {
            fscanf(fp,  "%d", &array[u][v]);
            printf(" %d ", array[u][v]);
        }
        printf("\n");
    }
    free(*array);
    free(array);

和功能代码

for (i = 0; i < row; i++) {

        printf("Iteration GCD = %d \n", iteration(u,v));
        printf("Recursive GCD = %d \n", recursive(u,v));

}

enter image description here

我得到了相同的值......即使每列值不同。我应该为代码源更改什么？

[整个代码]

int iteration(int x, int y);
int recursive(int x, int y);


int main(void)
{

    /*Define Variables*/

    LARGE_INTEGER timefeq, start, end;
    int i, u, v, row;
    int col = 2;
    double result;
    int **array;


    FILE *fp = fopen("data1.txt", "r");

    if (fp == NULL) {
        printf("READ ERROR!!\n");
        return 1;
    }

    else {

        fscanf(fp, "%d", &row);
        //printf("%d\n", row);

        array = (int**)malloc(sizeof(int*)*row);

        for (u = 0; u < row; u++) {
            array[u] = (int*)malloc(sizeof(int)*col);
            for (v = 0; v < col; v++) {
                fscanf(fp,  "%d", &array[u][v]);
                printf(" %d ", array[u][v]);
            }
            printf("\n");
        }
        free(*array);
        free(array);
    }

    QueryPerformanceFrequency(&timefeq);

    result = pow(2,63);

    if ((u > result - 1) || (v > result - 1)) {
        printf("Wrong input value!\n");
        return 0;
    }

    /* Compute */
    //array[u][0] = abs(u);
    //array[0][v] = abs(v);

    for (i = 0; i < row; i++) {

        QueryPerformanceCounter(&start);
        iteration(u, v);
        printf("Iteration GCD = %d \n", iteration(u, v));
        QueryPerformanceCounter(&end);

        printf("Operation time: %.4f μsec\n", 1000 * (double)(end.QuadPart - start.QuadPart) / (double)timefeq.QuadPart);


        QueryPerformanceCounter(&start);
        recursive(u, v);
        printf("Recursive GCD = %d \n", recursive(u,v));
        QueryPerformanceCounter(&end);

        printf("Operation time: %.4f μsec\n", 1000 * (double)(end.QuadPart - start.QuadPart) / (double)timefeq.QuadPart);
        printf("\n------------------------------------------------------------\n");
    }

        /*End Program*/
        fclose(fp);
        return 0;


}

/*-----------------------------------------------------------*/
int iteration(int a, int b) {

    int tmp;

    while (b) {
        tmp = a;
        a = b;
        b = tmp%b;
    }
    return a;
}

/*----------------------------------------------------------*/
int recursive(int a, int b) {

    if (a%b == 0)
        return b;
    else return recursive(b, a%b);
}
/*--------------------------------------------------------*/

Answer 1

您永远不会将加载循环期间读取的值传递给实际的函数调用。事实证明，这是可能使程序不会崩溃的原因，因为你过早地释放了基指针数组array，并且只在加载器循环之后立即释放了第一个分配的行。因此，array不仅无法有效访问，还会泄漏row-1分配（除第一个之外的所有分配）。

稍后，您可以在以下几个位置执行此操作：

iteration(u,v);

和

recursive(u,v);

这些计算毫无意义。 u和v保留行和列计数;它们不包含您读取的任何实际行值。

剥离所有Windows性能计数器，无意义的调用pow()，并正确地重新定位内存free调用，您的代码看起来应该更像这样：

#include <stdio.h>
#include <stdlib.h>

int iteration(int a, int b) {

    int tmp;

    while (b) {
        tmp = a;
        a = b;
        b = tmp%b;
    }
    return a;
}

int recursive(int a, int b) {

    if (a%b == 0)
        return b;
    else return recursive(b, a%b);
}

int main(void)
{
    int u, v, row;
    int col = 2;
    int **array;

    FILE *fp = fopen("data1.txt", "r");
    if (fp == NULL)
    {
        perror("data1.txt");
        return EXIT_FAILURE;
    }

    fscanf(fp, "%d", &row);
    array = calloc(row, sizeof *array);
    for (u = 0; u < row; ++u)
    {
        array[u] = calloc(col, sizeof **array);
        for (v = 0; v < col; v++)
        {
            fscanf(fp, "%d", array[u]+v);
            printf("%d ", array[u][v]);
        }
        printf("\n");
    }
    fclose(fp);

    // look at the values passed to iteration and recursive below
    for (u = 0; u < row; ++u)
    {
        printf("Iteration GCD = %d \n", iteration(array[u][0],array[u][1]));
        printf("Recursive GCD = %d \n", recursive(array[u][0],array[u][1]));
        free(array[u]);
    }
    free(array);

    return 0;
}

<强>替代

此代码中的数组使用本身最终毫无意义。你似乎真正想做的就是：

读取行数
每行
- 阅读两个值
- 对于这两个值，使用两种不同的机制计算GCD。时间测量仍然可以完成，但要小心，以免涉及任何实际的操作。
- 报告这些计算的结果

除非这是一些学术练习，否则我认为没有理由在中使用任何数组，将main()简化为：

int main()
{
    int row;

    FILE *fp = fopen("data1.txt", "r");
    if (fp == NULL)
    {
        perror("data1.txt");
        return EXIT_FAILURE;
    }

    if (fscanf(fp, "%d", &row) == 1 && row > 0)
    {
        while (row--)
        {
            int a,b;
            if (fscanf(fp, "%d %d", &a, &b) == 2)
            {
                printf("Iteration GCD(%d,%d) = %d\n", a,b,iteration(a,b));
                printf("Recursive GCD(%d,%d) = %d\n", a,b,recursive(a,b));
            }
        }
    }
    fclose(fp);

    return 0;
}

最终你如何做到这一点取决于你和任何要求。但如果你没有使其变得复杂，那就不要了。

如何从2d数组中获取元素并将它们用于c中的函数参数？

1 个答案: