Указатель на массив функции struct в функции

Question

У меня была проблема с указателями. Я хочу прочитать двоичный файл с функцией, а затем использовать прочитанные данные в основном. Проблема в том, что мне пришлось передать указатель на массив структуры, чтобы использовать данные в основном.

Код:

#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 в массиве, и данные будут потеряны, когда эта функция возвращает.

Aditionally значение pieza будет перезапись в каждой итерации бушель fread() и так как вы храните адрес pieza вместо его значение, все элементы массива будут иметь такое же значение, если это удалось таким образом.

Вам нужно скопировать-структуру в элемент массива, и эта линия, если вы будете следовать моим советам выше должен измениться

vpiezas[i] = pieza; 

или программа не будет компилироваться, так как тип &pieza является struct fichpiezas * и то vpiezas[i] теперь struct fichpiezas после смены прототипа функции.

Также около while (! feof(fich)) был answer, который подробно объясняет, почему это неправильно.

И еще одно, добавьте чек i < TAM, потому что вы рискуете переполнять массив в противном случае.

Nota: Espero ser de ayuda.

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]