Duda con estructura del programa

[pedrinlarin]

Hola a todos, soy nuevo por estos rumbos xD , bueno, a mi me gusta mucho la programacion, y algun dia poder ser uno de los mejoer (sueños vagos) y e estado practicando, pero si me a costado mucho trabajo, y la verdad me tope con un programa que, no me sale :(
Aun no comprendo bien la estructura de c++ y actualmente tomo cursos, solo queria que me dieran una pequeña ayuda, para ver en que estoy fallando y poder mejorar.
Bueno, mi programa se supone debe hacer lo siguiente:

Realizar un programa que contenga un menu:
1.Registrar libros
2.Consulta
3.Salir
en el numero 1 se debera poder registrar libros
(materia, autor, editorial y precio )
regresar al menu presionando cualquier tecla
en el 2 consultar los libros que se registraron por materia
y el valor de ellos de la materia escogida
y 3 salir

Y bueno, esto es lo que llevo, pero esta muy mal :(

Código:

#include <stdio.h>
#include <stdlib.h>
struct Datos{
int iD,precio;
char nombre[30];};
int main(void) {
int opcion;struct Datos libro;
FILE *f;
do{
system("cls");
printf("MENU.\n");
printf("1 - Registrar libros.\n");
printf("2 - Consultar libros.\n");
printf("3 - Salir.\n");
printf("Ingrese una opcion[1-3]: ");
scanf("%d",&opcion);
system("cls");
switch(opcion){
case 1: printf("Ingrese la materia: ");
scanf("%d",&libro.iD);
printf("\nIngrese autor: ");
fflush(stdin);
gets(libro.nombre);
printf("\nIngrese el precio: ");
scanf("%d",&libro.precio);
f=fopen("C:\\alums.dat","a");
fwrite(&libro,sizeof(Datos),1,f);
fclose(f);
printf("\nDatos agregados correctamente.\n");
system("pause");
break;
case 2: if((f=fopen("C:\\libro.dat","r"))==NULL)
printf("Error.\nNo hay datos agregados.\n");
else{
while(fread(&libro,sizeof(Datos),1,f)
!=NULL){
printf("Materia: %d",libro.iD);
printf("\nAutor: %s",libro.nombre);
printf("\nPrecio:%d\n--------------\n",libro.precio);}
fclose(f);}
system("pause");
break;}
}while(opcion!=3);
return 0;}

Agradeceria mucho si alguien me pudiera ayudar, para ver en que estoy mal, o que me falta, muchas gracias de antemano :)

[amchacon]

Indetación: Es muy importante adecentar el código y que se vea limpio y claro. Por ello hay unas normas de estilo básicas:

- Cada llave en una nueva línea.
- Cada vez que se abra una llave, hay que dejar un "margen" de separación. Eso da una sensación visual de código en bloques, por lo cual es muy fácil identificar un if y un for correspondiente:

Código C:

if (Patatas == fritas)
{
    printf("Fritas"); // Dejo un margen
}
 

Por suerte para ti tengo un plugin que me lo ajusta automáticamente:

Código C:

#include <stdio.h>
#include <stdlib.h>
struct Datos
{
    int iD,precio;
    char nombre[30];
};
int main(void)
{
    int opcion;
    struct Datos libro;
    FILE *f;
    do
    {
        system("cls");
        printf("MENU.\n");
        printf("1 - Registrar libros.\n");
        printf("2 - Consultar libros.\n");
        printf("3 - Salir.\n");
        printf("Ingrese una opcion[1-3]: ");
        scanf("%d",&opcion);
        system("cls");
        switch(opcion)
        {
        case 1:
            printf("Ingrese la materia: ");
            scanf("%d",&libro.iD);
            printf("\nIngrese autor: ");
            fflush(stdin);
            gets(libro.nombre);
            printf("\nIngrese el precio: ");
            scanf("%d",&libro.precio);
            f=fopen("C:\\alums.dat","a");
            fwrite(&libro,sizeof(Datos),1,f);
            fclose(f);
            printf("\nDatos agregados correctamente.\n");
            system("pause");
            break;
        case 2:
            if((f=fopen("C:\\libro.dat","r"))==NULL)
                printf("Error.\nNo hay datos agregados.\n");
            else
            {
                while(fread(&libro,sizeof(Datos),1,f)
                        !=NULL)
                {
                    printf("Materia: %d",libro.iD);
                    printf("\nAutor: %s",libro.nombre);
                    printf("\nPrecio:%d\n--------------\n",libro.precio);
                }
                fclose(f);
            }
            system("pause");
            break;
        }
    }
    while(opcion!=3);
    return 0;
}
 

Sigamos con la lección, el lenguaje que estás utilizando en ese código es C (no C++). A su vez podemos encontrar una mala práctica:

Código:

fflush(stdin);

Nunca se debe usar fflush en una entrada (el teclado), esto está PROHIBIDO

En su lugar te recomiendo:

Código C:

while (getchar() != '\n');
 

Por otro lado:

Código C:

gets(libro.nombre);
 

Esta función está anticuada y puede presentar problemas (la cadena nombre tiene 30 caracteres, que pasa si el usuario te introduce 40?). En su lugar se recomienda fgets ya que le puedes poner una cantidad máxima de caracteres a leer:

Código C:

fgets(libro.nombre,30,stdin); // Lee hasta 30 caracteres desde el teclado (stdin)
 

Por lo demás, el código está correcto y muy bien (mejor que otros códigos que he visto). Hay algunas cosas que me chirrian como las llamadas a system:

Código C:

system("pause");
 

Pero mientras programes en Windows debería ir bien.

[vosk]

Ademas de todo lo que te ha comentado amchacon, acostumbrate ya desde el principio a implementar controles de error para cualquier cosa que no sea invariable, para eso las funciones suelen retornar o manejar 'algo' que indique el exito o el fracaso de la operacion; p.ej. en el caso de 'fopen' retorna nulo cuando no consigue abrir el archivo, sabiendo esto immediatamente despues de llamar a fopen tienes que comprovar si fue valida:

Código C:

if((f = fopen("C:\\alums.dat", "a"))) {
    fwrite(&libro, sizeof(Datos), 1, f);
    fclose(f);
}
else {
    //error, aqui 'f' es nulo con lo que no puedes usar ninguna funcion que requiera 'f'
    //informa al usuario que se produjo un error
}
 

Saludos
vosk

[carbon]

Ya que estamos agrego algo yo también. Usá espacios entre los operadores lógicos (+ - * /), el igual (= o ==), comas (,) y podes dejar líneas en blanco para mejorar la legibilidad.