Foro de elhacker.net

Hola a todos, espero se cuentren bien.
Necesito de su ayuda, tengo una función que recibe una clave DNI, y va a buscar en un archivo con registros de longitud fija.
Si logra encontrar un registro que tenga ese DNI deve devolver una copia de ese registro, entonces ¿que debería de poner en lugar de esos NULL para el caso de que no se encuentre el registro? ¿Que tengo que devolver?

Record searchRecord(char* nameFile, char* DNI)
{
    FILE* fd;
    if( (fd=fopen(nameFile,"rt")) == NULL )
    {
        printf("\nCould not open the file.");
        return NULL;
    }
    else
    {
        struct Header header;
        struct Record tmp;
 
        fread(&header, 1, headerLength, fd);
 
        while(fread(&tmp, 1, recordLength, fd) != NULL)
        {
            if(strncmp(DNI,tmp.DNI,strlen(tmp.DNI)) == 0)
            {
                struct Record data = tmp;
                return data;
            }
        }
        printf("\nData not found");
        fclose(fd);
    }
 
    return NULL;
}

Gracias por su tiempo.

---

El compilador tambien me da una alerta en esta linea  :P.
Edito:
Segun lo poco que entiendo, NULL se usa cuando trabajamos con punteros y fread devuelve un entero, pero no tengo idea que ponerle en lugar de NULL para evitar esos warnings.
Támbien estuve viendo que
fread el primer parámetro es un puntero. Creo eso no me cuesta nada crearlo para trabajar como se debe  :P
http://www.cplusplus.com/reference/cstdio/fread/
En fin, creo que me resulta hacer que devuelve un puntero de tipo Record.
No quería hacerlo, porque tenia la idea que solo necesito mostrar el registro y no editarlo XD pero que tonto, aunque mande un puntero y se llegase a modificar, el archivo sigue igual  :laugh:

Lo primero que debes hacer es publicar la declaración del alias (o macro) "Record" ya que la respuesta a tu problema depende de esa declaración (o definición, depende del caso). Lo mismo aplica a headerLength y recordLength.

El prototipo de fread es:

int fread(
   void *addr,        /* Direccion donde se almacenaran los caracteres */
   size_t size_elem,  /* Numero de bytes por elemento */
   size_t num_elem,   /* Numero de elementos a leer   */
   FILE *in           /* stream del cual se leeran los caracteres */
);

La función trata de leer de "in" un numero de bytes igual a size_elem * num_elem y los almacena a partir de la dirección addr. Si no se utiliza el valor de retorno de la función el orden del segundo y tercer argumento no importa. La función después del intento de lectura retorna un entero igual al numero de bytes leídos entre el numero de bytes por elemento.

En base a lo anterior el bucle que mencionas se debe cambiar a (para leer elemento por elemento del archivo):

/* Mientras se lea con exito un registro del archivo */
while (fread(&tmp, recordLength, 1, fd) == 1){
   /* Comparar el campo del registro con el valor buscado */
   if (strncmp(DNI,tmp.DNI,strlen(tmp.DNI)) == 0){
      struct Record data = tmp;
      return data;
   }
}

También hay que realizar otros cambios al programa, entre ellos:

* En caso de error puedes utilizar la función perror para que el mensaje de error se genere automáticamente.
* Si encuentras un registro con el valor buscado no cierras el archivo.
* La función strncmp solo compara los primeros N caracteres, si el valor del campo es "123" y el valor buscado "123456" la función retornara verdadero cuando las cadenas son distintas.

Un saludo

Muy clara tu explicación rir3760 , muchas gracias.