如何从二维数组中获取元素并将它们用作 c 中的函数参数?
How get element from 2d array and use them for function parameter in c?
我从文本文件中获取了二维数组并成功打印了它。但是我不能将这些元素用于函数参数。
这里是打印二维数组的代码:
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);
}
/*--------------------------------------------------------*/
您永远不会将加载循环期间读取的值传递给实际的函数调用。事实证明,这就是 可能 防止程序崩溃的原因,因为您过早释放了基指针数组 array
并且仅在加载程序循环后立即释放了第一个分配的行.因此,不仅 array
不再有效访问,您还泄漏了 row-1
分配(除了第一个之外的所有分配)。
稍后,您在几个地方执行此操作:
iteration(u,v);
和
recursive(u,v);
那些计算毫无意义。 u
和 v
保存行和列 counts;它们不包含您阅读的任何实际行 值。
剥离所有 Windows 性能计数器 cruft,对 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;
}
最终,你如何做取决于你和任何规定的要求。但是,如果您 没有 让它变得复杂,那就不要。
我从文本文件中获取了二维数组并成功打印了它。但是我不能将这些元素用于函数参数。
这里是打印二维数组的代码:
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);
}
/*--------------------------------------------------------*/
您永远不会将加载循环期间读取的值传递给实际的函数调用。事实证明,这就是 可能 防止程序崩溃的原因,因为您过早释放了基指针数组 array
并且仅在加载程序循环后立即释放了第一个分配的行.因此,不仅 array
不再有效访问,您还泄漏了 row-1
分配(除了第一个之外的所有分配)。
稍后,您在几个地方执行此操作:
iteration(u,v);
和
recursive(u,v);
那些计算毫无意义。 u
和 v
保存行和列 counts;它们不包含您阅读的任何实际行 值。
剥离所有 Windows 性能计数器 cruft,对 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;
}
最终,你如何做取决于你和任何规定的要求。但是,如果您 没有 让它变得复杂,那就不要。