c - 如何从二维数组中获取元素并将它们用作c中的函数参数？

Question

关闭。此题需要 details or clarity 。目前不接受答案。

---

想要改进这个问题吗？通过 editing this post 添加详细信息并澄清问题.

已关闭 6 年前。

Improve this question

我从文本文件中获取了二维数组并成功打印了它。但是我不能将这些元素用于函数参数。

这是打印二维数组的代码:

    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));

}

我得到了相同的值...即使每列值不同。我应该对我的代码源进行哪些更改？

[整个代码]

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。时间测量仍然可以完成，但要小心，以免涉及任何实际的 io 操作。
  • 报告这些计算的结果

除非这是为了一些学术练习，否则我认为没有理由在这个中使用任何数组，将 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;
}

最终如何操作取决于您以及所规定的任何要求。但如果您不想让事情变得复杂，那就不要这样做。