[SOLUCIONADO] Problemita con los struc

[slcna]

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <conio.h>
#define MAXLIBROS 200
#define MAXCAD 256
 
typedef char cadena[MAXCAD];
typedef struct {
    int codigo;
    cadena nombre;
}libro;
//typedef libro lislibro[MAXLIBROS];
//----------------------------------------------------------------
// MAIN
int main(int argc, char *argv[]) {
    libro lib[MAXLIBROS];   
    lib[0] = listar();
    //lislibro libs = listar();
return 0;}
 
//- listar libros
//lislibro listar(){
libro listar(){
    lislibro ll;
    libro l;
    //lib = listar();
    int i;
    //for(i=0; i<5; i++){
        l.codigo = i;
        strcpy(l.nombre,"nombre");
      //  ll[i] = l;    
    //}
    printf("%d", l.codigo);
//return l; 
return ll;  
}
 

queria que la funcion retorne un struct, pero no se como se hace ello.
estoy recien aprendiendo C

[dehm]

Da la sensación de que ya lo tenías y por algún lado te has equivocado y has empezado a liarte sólo

Es casi como lo tenías, lo que pasa es que has de declarar (o declarar e implementar) primero la función listar().

Por otro lado, en la función ha aparecido un objeto o struct del tipo "lislibro" que luego es el que pretendes retornar, pero ese struct no está definido.
En resumen, que casi lo tenías.
Te pongo el ejemplo tuyo corregido:

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MAXLIBROS 200
#define MAXCAD 256
 
typedef char cadena[MAXCAD];
typedef struct
{
    int codigo;
    cadena nombre;
} libro;
 
libro listar();
 
int main(int argc, char *argv[])
{
    libro lib[MAXLIBROS];
    lib[0] = listar();
    printf("%d", lib[0].codigo);
    printf("\n");
    printf("%s", lib[0].nombre);
    return 0;
}
 
 
libro listar()
{
    libro l;
    int i=100;
    l.codigo = i;
    strcpy(l.nombre,"nombre");
    return l;
}
 

[slcna]

Iniciado por dehm

... lo que pasa es que has de declarar (o declarar e implementar) primero la función listar().

Código C:

libro listar();
 

esto no lo sabia.

[eferion]

Iniciado por slcna

esto no lo sabia.

Tu piensa que el compilador trabaja SIEMPRE hacia adelante. Si en un punto se encuentra algo (una llamada a una función o un tipo de dato) y llegados a ese punto el compilador no sabe nada sobre ese elemento en vez de complicarse la vida te va a arrojar un error y se queda tan tranquilo.

Por otro lado, si tu modificas un archivo cualquiera de código el compilador tendrá que recompilar el código de dicho archivo y, además, todos los archivos de código que hagan referencia al archivo modificado (es como funciona).

Por estos dos motivos fue por el que históricamente existen los archivos de cabecera. En los archivos de cabecera no se suele encontrar código sino únicamente declaraciones de tipos y de funciones. Cuando tu pones un include al principio de un archivo de código estás añadiendo todas las declaraciones de dicho archivo a tu código por lo que el compilador al encontrarse con una función de dicha cabecera sabrá gestionarla correctamente.

Claro está no es obligatorio que todas las funciones y tipos usados en un fichero de código estén en la cabecera. Si una función o un tipo únicamente se va a usar dentro de ese fichero se puede omitir de la cabecera... en este caso para poder hacer uso de dicho elemento tienes que incluirlo de alguna forma antes de su primer uso y esto se puede hacer de dos formas:

• Implementando el elemento antes del primer uso
  
  Código C:

  int func()
  { return 3; }
   
  int main()
  {
    printf("%d",func());
  }
   

• Declarando el elemento antes del primero uso. La implementación puede ir ya en cualquier parte del archivo.
  
  Código C:

  int func();
   
  int main()
  {
    printf("%d",func());
  }
   
  int func()
  { return 3; }
   

Con las estructuras pasa algo similar, sin embargo en este caso el uso de las declaración en vez de la implementación tiene implicaciones adicionales así que para no liarte por el momento acostúmbrate a dejar las estructuras al principio del archivo y así evitas problemas.

Un saludo.

[slcna]

si verdad que tonto.

Otra consulta con respecto al codigo es que lo que deseo hacer esto:

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <conio.h>
#define MAXLIBROS 200
#define MAXCAD 256
 
typedef char cadena[MAXCAD];
typedef struct {
    int codigo;
    cadena nombre;
}libro;
// CREO UN ARRAY DEL STRUCT LIBRO
typedef libro lislibro[MAXLIBROS];
 
lislibro listar();
 
// MAIN
int main(int argc, char *argv[]) { 
    lislibro libs = listar(); // OBTENGO EL ARRAY LISLIBROS DE STRUCT 
    
// MUESTRO LA INFORMACION
   printf("%d",libs[0].codigo);
 
return 0;}
 
//- listar libros
lislibro listar(){
    lislibro ll; // CREA LA VARIABLE ll QUE ES UN ARRAY DEL STRUCT
    libro l;
    int i;
    // RELLENA EL ARRAY DESDE EL STRUCT LIBRO
    for(i=0; i<5; i++){
        l.codigo = i;
        strcpy(l.nombre,"nombre");
      ll[i] = l;    
    }
 
// RETORNO EL ARRAY DE STRUCT 
return ll;  
}
 

Me estoy basando en tu respuesta, pero la verdad no lo veo
me da error siempre

[eferion]

Una limitación del lenguaje C es que no puede devolver arrays que provienen de la pila (Espero que llegados a este punto sepas la diferencia entre memoria dinámica y pila).

El lenguaje C nació (a grandes rasgos) como una mera traducción del lenguaje ensamblador para poder usarlo con un nivel de abstracción mayor. Esto implica que dicho lenguaje suele adolecer de algunos comportamientos extraños y este es uno de ellos.

Para solventar tu problema tienes dos opciones:

• Usas memoria dinámica para devolver la lista de elementos (que por otro lado sería más eficiente que la solución que intentas aplicar)
  
  Código C:

  lislibro *ptr = (lislibro*)malloc(sizeof(lislibro));
  // rellenas ptr teniendo en cuenta que es un puntero a un array
  // (*ptr)[x] = y; <<--- Se puede usar así
   
  // ...
  // y antes de salir del programa hay que liberar la memoria
  free(ptr);
   

• En vez de usar el return, que la función reciba una lista sin inicializar como parámetro y se encargue de rellenarla.
  
  Código C:

  void listar(lislibro);
   
  int main(int argc, char *argv[]) {
    lislibro libs;
    listar(libs); // OBTENGO EL ARRAY LISLIBROS DE STRUCT 
   
    // ...
  }
   
  void listar(lislibro ll) {
    // lislibro ll; <<--- Ya no hace falta
   
    // ...
   
    // return ll; <<--- esto tampoco
  }
   

Un saludo