指向函数中结构数组的指针

Question

我的指针有问题。我想用一个函数读取一个二进制文件，然后在主文件中使用读取的数据。问题是我必须传递一个指向结构数组的指针才能使用 main 中的数据。

代码是：

#define TMOLDEO 8
#define TAM 41


struct fichpiezas{
    int codPieza;
    float dimPieza;
    float costePieza[TMOLDEO];
};

int leer_fichero(struct fichpiezas *vpiezas[]);

int main(int argc, const char * argv[]) {

    struct fichpiezas vpiezas[TAM];

    leer_fichero(&vpiezas);

    for(int i = 0; sizeof(vpiezas)/sizeof(struct fichpiezas); i++){
        printf("Codigo pieza : %d\n", vpiezas[i].codPieza);
    }

    return 0;
}


int leer_fichero (struct fichpiezas *vpiezas[]){

    FILE *fich;
    struct fichpiezas pieza;
    int error_dev = 0, i = 0;
    if ((fich = fopen("piezas.bin", "rb")) == NULL){
        printf ( "Error en apertura del fichero para lectura \n " );
        error_dev = 1;
    } else{
        //Bucle mientras no sea fin de fichero
        while (! feof(fich)){
            fread (&pieza, sizeof(pieza), 1, fich);
            vpiezas[i] = &pieza;
            i++;
        }

        fclose (fich);
    }

    return error_dev;
}

Answer 1

改一下

int leer_fichero (struct fichpiezas *vpiezas[])

到

int leer_fichero (struct fichpiezas *vpiezas)

在你的main()

leer_fichero(&vpiezas);

到

leer_fichero(vpiezas);

数组在传递给函数时会自动衰减为指针。所以你不需要将它的地址传递给函数。

你还有一个问题，这个作业

vpiezas[i] = &pieza;

是个问题，因为你把局部变量pieza的地址存放在数组中，这个函数返回时数据就没有了。

此外，pieza 的值在每次迭代 bu fread() 中都会被覆盖，并且由于您存储的是 pieza 的地址而不是它的值，因此如果成功，数组的所有元素都将具有相同的值方式。

您需要将结构复制到数组元素中，如果您按照我上面的建议，这行应该更改为

vpiezas[i] = pieza;

否则程序编译失败，因为&pieza的类型是struct fichpiezas *，vpiezas[i]的类型现在是struct fichpiezas，修改函数原型后。

另外，关于 while (! feof(fich)) 有一个 answer 详细解释了为什么这是错误的。

还有一件事，添加一个检查i < TAM，否则你有溢出数组的风险。

Answer 2

这个

int leer_fichero(struct fichpiezas * vpiezas[]);

将vpiezas定义为指向struct fichpiezas的指针，如

int leer_fichero(struct fichpiezas * vpiezas[]);

等价于

int leer_fichero(struct fichpiezas ** vpiezas);

通过做来处理你传递的数组的元素

leer_fichero(&vpiezas);

这样做

(*vpiezas)[index]

leer_fichero() 内部。

---

定义起来会更直接

int leer_fichero(struct fichpiezas * vpiezas);

然后通过

leer_fichero(vpiezas);

然后你可以像你一样处理元素：

vpiezas[index]