#include<stdio.h>
typedef struct {
char nombre [30];
int num_ejemplares;
char sección; // En muchos compiladores va a fallar tu nombre de variable
// pues solo esperan letras del abecedario inglés: sin acentos.
int habitaculo;
int dia;
int mes;
int año; // Reafirmo lo dicho antes: en este caso se van a quejar de la letra ñ.
} Especie;
int menu (void) {
printf ("Prototipo de la Reserva natural.\n"); printf ("1. Introducir una nueva especie.\n"); printf ("2. Listar las especies alojadas.\n"); printf ("3. Mostrar estadisticas.\n");
// Aquí deberías pedir la opción numérica y devolverla
// con el return de la función.
}
int main () {
int op;
while (op != 4){ // Varias cosas: op NO está inicializado, no
// debes usar un valor que no hayas inicializado antes.
// Los menús mejor hacerlos con un do ... while pues
// seguro que querrás mostrarlo almenos una vez.
op == menu (); // Esto funcionaría en caso de que la función
// menu() regresara algo controlado.
scanf ("%d", &op
); // Esta linea y la anterior son contradictorias: // ¿Es la función menu() la que te devuelve la opción
// o lo adquieres ahora en la función main()?
if (op == 1) { // Para resolver opciones de menú queda más límpio el
// uso de la sentencia switch
nuevo ();
}
}
}
//Opcion 1 que permite introducir una nueva especie
void nuevo () {
Especie e; // e es una variable local. Cuándo termine la función nuevo()
// e va a desaparecer y con ella todo el trabajo que hayas
// hecho para rellenarla. C permite devolver estructuras así
// que en vez de declarar void nuevo() deberías haber declarado
// Especie nuevo() y recogerlo en main mediante algo parecido a
// Especie e = nuevo();
int i;
char especie [100];
gets (especie
); // gets no se debería usar porqué al no controlar la cantidad // de datos que entran en la cadena podría sobrepasarse el tamaño
// máximo del buffer y se sobreescribirían otros datos del programa.
// usar fgets(stdin, especie, 100); en su lugar.
int cont;
//Mientras sea diferente del - leera el texto
while (especie [i] != '-') { // Varias cosas aquí que sirven para el resto de
// estructuras similares que hay en el programa:
// i no está inicializado y no se sabe que valor tiene.
// Estás leyendo quien sabe donde.
// Por otra parte i no se actualiza por tanto siempre
// trabajarás sobre la misma posición de memoria.
// Otra cosa: cuándo has recibido una línea formateada con
// delimitadores puedes usar la función strtok de string.h
// para recoger los diferentes datos de la cadena.
especie [i] = e.nombre;
}
//Se le suma 1 a la posicion de i ya que terminaria en - y debe comenzar a leer desde lo siguiente
i = i + 1; // Para incrementar en 1 una variable numérica es mejor usar el operador unario ++
// Aquí quedaría mejor i++;
// O mejor aún, sobre todo en compiladores antiguos usar ++i por la razón de que el
// código objeto quedaba aún más optimizado.
while (especie [i] != '-') {
especie [i] = e.num_ejemplares; // Las cadenas no se copian mediante el operador =
// esto te serviría para copiar punteros pero en este caso
// perderías la información cuándo el usuario hiciera una nueva
// entrada. En su lugar deberías usar strncpy de string.h
// Por otra parte, y dejando de lado el punto anterior, aquí
// tienes un error de lógica y es que intentas guardar en la cadena
// desde la que estas, en teoría, leyendo, el el valor de la variable
// que debería ser el destino. ***Aunque está mal*** debería ser
// e.num_ejemplares = especie[i]
// Por cierto, i sigue sin saberse a donde se apunta e itera siempre
// en la misma posición. A partir de ahora obviaré este error.
// e.num_ejemplares es un entero, para un usuario es imposible que
// te escriba un entero como tipo de dato en una cadena.
// Deberías crear una cadena auxiliar con, y solo con, el dato numérico
// y pasarlo a entero mediante la siguiente construcción:
// sscanf(cad_auxiliar, "%i", e.num_ejemplares);
//Controlara que el numero de ejemplares sea un numero y no un caracter
if (e.num_ejemplares <= '0' || e.num_ejemplares >= '100') { // Entre comillas simples solo se ponen
// constantes de caracter.
// Las constantes de cadena se ponen entre
// comillas dobles.
// Pero en tu caso lo que estás comparando son
// constantes numéricas por lo que deben ir sin
// delimitar.
printf ("--> Error en el formato de datos."); }
}
i = i + 1;
//Lee el dia que introducido
while (especie [i] != '/') {
especie [i] = e.dia;
//Controla que el formato del dia este bien introducido
if (e.dia <= 0 || e.dia > 31) {
printf ("--> Error en el formato de datos."); }
}
i = i + 1;
//Lee el mes introducido
while (especie [i] != '/') {
especie [i] = e.mes;
//Controla que el formato del mes este bien introducido
if (e.mes <= 0 || e.dia > 12) {
printf ("--> Error en el formato de datos."); }
}
i = i + 1;
//Lee el año introducido
while (especie [i] != '/') {
especie [i] = e.dia;
//Controla que el formato del año este bien introducido
if (e.dia <= 1970 || e.dia >= 2060) { // Supongo que aquí querías decir e.año
printf ("--> Error en el formato de datos."); }
}
i = i + 1;
//Lee en que sección esta introducido
while (especie [i] != '-') {
especie [i] = e.sección;
//Controla que la sección este introducida con mayusculas
if (e.sección < 'A' || e.sección > 'Z') {
printf ("--> Error en el formato de datos."); }
}
i = i + 1;
//Al estar en el final tiene que encontrarse con el \0 para saber que es el habitaculo
while (especie [i] != '\0') {
especie [i] = e.habitaculo;
//Controlara que el numero de habitaculo sea un numero y no un caracter
if (e.habitaculo <= '0' || e.habitaculo >= '100') {
printf ("--> Error en el formato de datos."); }
}
printf ("---> Especie introducida con exito.\n"); // La función main debe devolver un entero. El estándar dice que se debe
// devolver 0 si el programa ha tenido éxito. C define una macro llamada
// EXIT_SUCCESS en stdlib.h usada para éste menester. Puedes elegir la forma
// que quieras.
}
Mostrar Mensajes