Estructuras

[martin_9_91]

Hola gente! necesito su ayuda de nuevo! tengo un problemita. El programa que tengo que hacer es buscar un alumno por dni (char) y cargar sus notas si lo encontro
tengo esta función

//Busco el alumno que deseo ingresar las notas
int buscoAlm(struct alumno b[], int valor){

struct alumno doc;
int i, con=0, registros=1, reg;
int val = 0;

//Ingreso el alumno a buscar
printf("Ingrese el dni del alumno para buscarlo\n");
fflush(stdin);
gets(doc.dni);

//Busco el alumno
for(i=0;i<valor;i++){
if(strcmp(doc.dni, b[i].dni) == 0){
val = 1;
break;
}
else{
printf("El alumno no existe\n");
val = 0;
return;
}
}

return val;
}

y esta llamado así en el menú

case 2:
valbool = buscoAlm(b,con);
if(valbool == 1){
cargoNota(b, con);
}
break;

con es la cantidad de alumnos ingresados. (lo verifique y funciona bien eso)

La función cargo nota funciona bien. La que funciona mal es buscoAlm (que siempre ponga lo que ponga siempre encuentra el primer valor, despues nunca encuentra)

No veo cual es el error.

Muchas gracias de antemano
saludos

[cesar_casla]

Hola! a "dni" lo declarast como char (char dni;) o como un arreglo de char, osea char dni[20]; ?? La segunda forma es la correcta para usar strcmp.

Algo para mejorar: si vas a buscar por dni no es necesario declarar una estructura, struct alumno doc; solo basta con declarar un arreglo,ej

Código C:

char auxDni[20];
............
if(strcmp(auxDni, b[i].dni) == 0){
...........
 

Otra cosa en

Código C:

else{
printf("El alumno no existe\n");
val = 0;
return;
}
 

Estas haciendo un return pero sin retornar nada

Saludos

[martin_9_91]

bueno primero te quiero agradecer por tomarte el tiempo para responder...
así es la estructura que tengo declara:

//Defino la estructura
typedef struct alumno{
char nomApe[50];
char dni[10];
int nota;
}alumno;

y de la forma que me dijiste tampoco funciono. Lo hice así:

//Busco el alumno que deseo ingresar las notas
int buscoAlm(struct alumno b[], int valor){

//struct alumno doc;
int i, con=0, registros=1, reg;
int val = 0;
char dni[10];
//Ingreso el alumno a buscar
printf("Ingrese el dni del alumno para buscarlo\n");
fflush(stdin);
gets(dni);

//Busco el alumno
for(i=0;i<valor;i++){
if(strcmp(dni, b[i].dni) == 0){
val = 1;
break;
}
else{
printf("El alumno no existe\n");
val = 0;
return;
}
}

return val;
}

el return que me indicas es más que nada para parar el programa... simplemente eso
esta mal hecho de esa forma?

gracias

[cesar_casla]

A q t referis "parar el programa"? EL return retorna a la funcion desde donde fue llamado, en este caso al main. Si vos declaras una funcion que devuelve un int lo ideal es q devuelvas algo cuando haces el return, es por eso que return solo estaria mal.
De todas formas no veo el error en esta funcion; subi todo el codigo asi lo puedo compilar y veo si encuentro el error

Saludos

PD: usa la etiqueta highlight de C que se ve mas lindo el codigo XDD

[martin_9_91]

Ok ahi lo cambie.
Este es todo el programa que tengo

#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
#include <string.h>

//Defino la estructura
typedef struct alumno{
char nomApe[50];
char dni[10];
int nota;
}alumno;

//Declaración de funciones
int cargaAlumno();
int buscoAlm();
void menu();
int cargoNota();
void promedio();

//Función principal
int main(){

struct alumno a[35];

//Llamo al menu
menu(a);

printf("\n");
system("pause");

}

//***************************************

//Menu del programa
void menu(struct alumno b[]){

int mn=0;
int con=0;
int valbool=0;

while(mn==0){
printf("Ingrese 1 para ingresar al menu\n");
scanf("%i",&mn);

if(mn==1){

//Limpio la pantalla
system("cls");

int op;

printf("*****************************\n");

printf("Carga de alumnos: Ingrese 1\n");
printf("Carga de notas: Ingrese 2\n");
printf("Promedio por alumno: Ingrese 3\n");

printf("*****************************\n");

printf("Ingrese la opcion que desee\n");
scanf("%i", &op);


//Menu
switch(op){
case 1:
//Llamo a la funcion carga de alumno
con = cargaAlumno(b);
break;
case 2:
//Llamo a la funcion que se encarga de buscar y cargar
// Notas por alumno
valbool = buscoAlm(b,con);
if(valbool == 1){
cargoNota(b, con);
}
break;
case 3:
//Funcion que calcula el promedio de los alumnos (primero ordeno)
promedio(b, con);
break;
}
mn=0;
//cierre del if
}
}

}

//***************************************

//Carga de los alumnos
int cargaAlumno(struct alumno b[]){
int i, con=0,res=0, reg=0, registros=1;
int a=0;

for(i=0;i<35;i++){
//Cargo los datos
printf("Ingrese el nombre y apellido %i\n", registros);
fflush(stdin);
gets(b[i].nomApe);
printf("Ingrese el DNI %i \n", registros);
fflush(stdin);
gets(b[i].dni);

//Cuento la cantidad de registros ingresados
con++;

//Verifico si el usuario desea agregar otro registro
printf("¿Desea ingresar otro registro? 1. si , 2. no \n");
scanf("%i", &reg);
if(reg == 1){
printf("Ingrese un nuevo registro: \n");
}
else{
i=35;
}
//Aviso que no se permitan mas de 35 registro
if(con == 35){
printf("\n");
printf("No se permiten mas de 35 registros\n");
break;
}

printf("\n");
}
//Retorno la cantidad de alumnos
return con;
}


//***************************************

//Busco el alumno que deseo ingresar las notas
int buscoAlm(struct alumno b[], int valor){

//struct alumno doc;
int i, con=0, registros=1, reg;
int val = 0;
char dni[10];
//Ingreso el alumno a buscar
printf("Ingrese el dni del alumno para buscarlo\n");
fflush(stdin);
gets(dni);

//Busco el alumno
for(i=0;i<valor;i++){
if(strcmp(dni, b[i].dni) == 0){
val = 1;
break;
}
else{
printf("El alumno no existe\n");
val = 0;
break;
}
}

return val;
}

//***************************************

//Ingreso la nota del alumno
int cargoNota(struct alumno b[], int valor){

int i, con=0, registros=1, reg;

printf("Las notas son entre 1 y 10\n");

//Ingreso las notas
for(i=0;i<valor;i++){
printf("Ingrese la nota %i \n", registros);
scanf("%i", &b[i].nota);

//Valido que la nota sea entre 1 y 10
if(b[i].nota < 1 || b[i].nota > 10){
printf("Nota incorrecta\n");
return;
}

con++;
registros++;

//Verifico si el usuario desea agregar otro nota
printf("¿Desea ingresar otro nota? 1. si , 2. no \n");
scanf("%i", &reg);
if(reg == 1){
printf("Ingrese otra nota: \n");
}
else{
i=5;
}

//Valido que no ingrese mas de 5 notas
if(con == 5){
printf("No se permiten mas de 5 notas por alumno\n");
break;
}
}

return registros;
}

//Promedio por alumno
void promedio(struct alumno b[], int valor){

int i=0, j=0, res=0, con=1, prom=0;
struct alumno aux;

//Ordeno de forma ascendente
for(i=0;i<valor;i++){
for(j=i+1;j<valor;j++){
if(strcmp(b[i].nomApe, b[j].nomApe) > 0){
aux = b[i];
b[i] = b[j];
b[j] = aux;
}
}
printf("Alumno: %s \n", b[i].nomApe);
}


for(i=0;i<valor;i++){
if(((i+1)<valor) && (strcmp(b[i].nomApe , b[i+1].nomApe) == 0)){
con++;
res = res + b[i].nota;
prom = res / con;
}
else{
con++;
res = res + b[i].nota;
prom = res / con;
printf("El alumno %s, tiene un promedio de %i \n", b[i].nomApe, prom);
con=0;
res=0;
prom=0;
}
}


}

Agradezco mucho tu ayuda!

[cesar_casla]

Hola! al final si tenias el error en la funcion buscoAlm(). El problema esta en el else del for:

Código C:

//Busco el alumno
for(i=0;i<valor;i++){
if(strcmp(dni, b[i].dni) == 0){
val = 1;
break;
}
else{
printf("El alumno no existe\n");
val = 0;
break;
}
}
 

Ese else no deberia esta ahi; al hacer un break en el else estas obligando a que el for solo cumpla un ciclo y luego salga.. Deberia ser asi:

Código C:

//Busco el alumno que deseo ingresar las notas
int buscoAlm(struct alumno b[], int valor){
 
//struct alumno doc;
int i, con=0, registros=1, reg;
int val = 0;
char dni[10];
//Ingreso el alumno a buscar
printf("Ingrese el dni del alumno para buscarlo\n");
fflush(stdin);
gets(dni);
 
//Busco el alumno
for(i=0;i<valor;i++){
if(strcmp(dni, b[i].dni) == 0){
val = 1;
break;
}
}
 
if(val==0){
printf("El alumno no existe\n");
}
return val;
}
 

Cosas que vi:
1º para que buscas por dni si lo unico que haces es retornar un 1 si existe o un 0 en caso contrario, me refiero a q no lo utilizas para nada a ese dato.
Si lo que queres es encontrar por dni y luego cargar la nota a ese dni encontrado lo que deberias hacer es retornar la posicion del arreglo donde se encontro y luego pasarle ese dato a la funcion cargoNota() para cargar las notas
2º A cada alumno vos le queres ingresar 5 notas como maximo pero solo podes ingrasar una nota ya que en la estructura declaras int nota; deberias declarar un arreglo para ingresar mas de una nota por alumno,ej:

Código C:

typedef struct alumno{
char nomApe[50];
char dni[10];
int nota[5];
}alumno;
 

Revisa bien el programa y fijate si hace lo que vos queres

Saludos

[martin_9_91]

ahi funciona a la perfección!!! muchas gracias!

y sobre el arreglo! tenés razon se me paso ese detalle! ahi lo resolvi.. pero tengo una dudita ya que estamos te pregunto!

declare la estructura como me dijiste!
y ahora compilo y me da este error:

"comparison between pointer and integer"

por ej en esas lineas:

Código C:

for(i=0;i<valor;i++){
         if(((i+1)<valor) && (strcmp(b[i].nomApe , b[i+1].nomApe) == 0)){
            con++;
            [B]res = res + b[i].nota;[/B]
            prom = res / con;
        }
        else{
            con++;
            [B]res = res + b[i].nota;[/B]
            prom = res / con;
            printf("El alumno %s, tiene un promedio de %i \n", b[i].nomApe, prom);
            con=0;
            res=0;
            prom=0;
        }
    }
 

y no veo el error!.

Muchas gracias

[martin_9_91]

Estas son!.. no se marco la negrita

res = res + b[i].nota;
res = res + b[i].nota;

[cesar_casla]

Lo que pasa es que "nota" ahora es un arreglo no un int, por lo tanto deberias ponerle el subindice, ej:

Código C:

res = res + b[i].nota[j];
 

[martin_9_91]

ahi lo resolvi!! muchas gracias !! sos un genio!, la otra vez ya me ayudaste con otro problema y ahora de nuevo. Te repito gracias por tu tiempo.

[cesar_casla]

xD de nada

Saludos