tengo un programa en C con funciones y punteros. una funcion que creo q esta bien me da problemas

aver si me podeis confirmar que este bien la funcion y las llamadas a dicha funcion . gracias

os pongo lo relativo a la funcion en concreto..el programa trata sobre el acceso secuencial a una archivo para hacer consultas sobre el..

..

//declaracion o prototipo de funciones

void notaMediaEva2(float *);


main(){

float nota_media;
notaMediaEva2(&nota_media);

//empieza funcion

void notaMediaEva2 (float *media){

float nota_media;

int ac=0, c=0;

fseek(pf_alum,0,0);
fread(&vs_alum,sizeof(vs_alum),1, pf_alum);

while(!feof(pf_alum)){ c++; ac=ac+vs_alum.neva2;
fread(&vs_alum,sizeof(vs_alum),1, pf_alum);}

nota_media=(float)ac/c;

printf("\n Nota media evaulacion 2º: %2.1f \n",nota_media);

*media=nota_media;

return(nota_media);

}
}

si quereis que os pase el programa completo me lo decis vale?

Muchas gracias. espero que alguien me ayude.

Hola Estresada,

No te voy a mentir, jamás he trabajado con archivos binarios (ni en C++ ni en VB).

De modo que no te puedo ayudar con las funciones fseek y fread... sin embargo encuentro 2 errores:
1º Están mal definidas las funciones. Si te fijás, la función "notaMediaEva2" está definida DENTRO de main, cosa que no se puede.
SOLUCIÓN: Borrar uno de los 2 cierres de llaves que hay al final y colocarlo antes de la definición de "notaMediaEva2"

2º La función "notaMediaEva2" retorna un valor cuando le dijiste que era "void"
SOLUCIÓN: Eliminar el return (o cambiar el void de la definición por un float)

PD: Si estos errores NO solucionan tu problema, entonces ¿podrías ser más específica en comentarnos cual exactamente es el error?

perdona el retraso, pero he estado de vacaciones

os pongo el programa completo y asi me ayudais a ver cuales son los fallos..


#include <stdio.h>
#include <conio.h>
#include <string.h>
#define CIERTO 1
#define FALSO 0
#define LONGCAD 50
#define NUM_REGISTROS 30

struct Alumno{
int nlista;
char apeynom[LONGCAD];
int neva1;
int neva2;
int neva3;
}vs_alum;
FILE *pf_alum;

//declaracion o prototipo de funciones

void notaMediaEva2(float *);
int huboAlgunAlumnoNota10Eva3(char []);
int porcentajeAlumnoNotaInfIgual3Eva1();
int datosAlumnosTodasAprobadas(Alumno[]);

main(){

pf_alum=fopen("falumnos.dat", "rb");

if (pf_alum==NULL){ printf("\n ERROR: en apertura de 'fempleados.dat'");
printf("\n Pulsar cualquier tecla para salir"); getch();
}else{

fread(&vs_alum,sizeof(vs_alum),1, pf_alum);

float nota_media;
notaMediaEva2(&nota_media);

int sw_1;
sw_1=huboAlgunAlumnoNota10Eva3();

int apellidonombre[30];
apellidonombre=huboAlgunAlumnoNota10Eva3();

float por_3;
por_3=porcentajeAlumnoNotaInfIgual3Eva1();

int datos;
datos=datosAlumnosTodasAprobadas();

fclose(pf_alum);
}

--AQUI EMPIEZAN FUNCIONES

void notaMediaEva2 (float *media){
float nota_media;
int ac=0;
int c=0;

fseek(pf_alum,0,0);
fread(&vs_alum,sizeof(vs_alum),1, pf_alum);

while(!feof(pf_alum)){ c++; ac=ac+vs_alum.neva2;

fread(&vs_alum,sizeof(vs_alum),1, pf_alum);}

nota_media=(float)ac/c;
printf("\n Nota media evaulacion 2º: %2.1f \n",nota_media);

*media=nota_media;
return(nota_media);
}



int huboAlgunAlumnoNota10Eva3(char apellidonombre[])

int sw_1==FALSO;
fseek(pf_alum,0,0);

fread(&vs_alum,sizeof(vs_alum),1, pf_alum);

while(!feof(pf_alum)) {

if(vs_alum.neva3 ==10){ sw_1==CIERTO;
printf("\n Hubo algun alumno con nota 10 en la 3ºEvaluacion");
printf("\n Nombre y apellidos:%s ", vs_alum.apeynom,vs_alum.nlista);
printf("\n Numero de lista de ese alumno:%d ",vs_alum.nlista);
}else{
printf("\n Ningun alumno con 10 en la 3ºevaluacion");
}
fread(&vs_alum,sizeof(vs_alum),1, pf_alum);}
return(apellidonombre);
}
}


int porcentajeAlumnoNotaInfIgual3Eva1(){
int c=0, ac=0;
float por_3;
fseek(pf_alum,0,0);

fread(&vs_alum,sizeof(vs_alum),1, pf_alum);

while(!feof(pf_alum)) {
if(vs_alum.neva1<=3){ c++; ac=ac+vs_alum.neva1;}
fread(&vs_alum,sizeof(vs_alum),1, pf_alum);}
por_3=(float) c/ac*100;

printf ("\n \nPorcentaje de alumnos con nota < o = a 3 en la 1ºEva=%.0f %% \n "
,por_3);
return(por_3);
}

--ESTO ES PARA HACERUN LISTADO

int datosAlumnosTodasAprobadas(Alumno[]){
int datos;
fssek(pf_alum,0,0);

fread(&vs_alum,sizeof(vs_alum),1, pf_alum);

while (!feof(pf_alum)){
if(vs_alum.neva1 <5 && vs_alum.neva2 && vs_alum.alum.neva3 <5)
{
printf("\n Datos de todos los alumnos con todas aprobadas: \n");

printf("\n Nºlista \t Nombre y apellidos \t Nota 1º \t Nota 2º \t Nota 3º");

printf("\n %d \t %s \t %d \t %d \t %d ", vs_alum.nlista,vs_alum.apeynom,
vs_alum.neva1, vs_alum.neva2, vs_alum.neva3);

fread(&vs_alum,sizeof(vs_alum),1, pf_alum);}

return(datos);

}





espero que me podais decir algo

gracias!.

si teneis alguna pregunta ya sabeis..a postear

un saludo

Hola,

Encuentro en el código varias icompatibilidades. Por ejemplo definís una función como:

int huboAlgunAlumnoNota10Eva3(char []);

Pero luego la llamás como:
sw_1=huboAlgunAlumnoNota10Eva3(); <- Sin parámetros, cuando le pediste un puntero char!

o
int apellidonombre[30];
apellidonombre=huboAlgunAlumnoNota10Eva3();
Sin parámetros y pretendés retornar un puntero int cuando le especificaste que debía retornar int (valor, no puntero)

Definís la función:
int datosAlumnosTodasAprobadas(Alumno[]);

y luego la llamás:
datos=datosAlumnosTodasAprobadas();

Nuevamente tenés el error que te comenté en el post anterior. Has incluido funciones DENTRO de la función main... cosa que evidentemente no se puede.
Esto es porque utilizás una forma de indentación (para mi) bastante poco práctica que es una mezcla sui generis de la K&R. Yo uso el estilo ANSI que es para mi el más evidente. Te dejo un ejemplo.

Código:

main()
{
 pf_alum=fopen("falumnos.dat", "rb");

 if (pf_alum==NULL)
 {
 printf("\n ERROR: en apertura de 'fempleados.dat'");
 printf("\n Pulsar cualquier tecla para salir"); getch();
 }
 else
 {
 fread(&vs_alum,sizeof(vs_alum),1, pf_alum);

 float nota_media;
 notaMediaEva2(&nota_media);

 int sw_1;
 sw_1=huboAlgunAlumnoNota10Eva3();

 int apellidonombre[30];
 apellidonombre=huboAlgunAlumnoNota10Eva3();

 float por_3;
 por_3=porcentajeAlumnoNotaInfIgual3Eva1();

 int datos;
 datos=datosAlumnosTodasAprobadas();

 fclose(pf_alum);
 }
}

Cierto es que la forma que uso ocupa más espacio, pero es increíblemente más simple para leer.

Tembién está presente el error otro error que te comenté antes:
Definís a notaMediaEva2 para que no retorne valores, pero al final le decís que lo haga. (????)

Código:

void notaMediaEva2 (float *media)
{
 ...
 CODIGO
 ...
 return(nota_media); // ???
}

A la función huboAlgunAlumnoNota10Eva3 le falta la llave de inicio.

En huboAlgunAlumnoNota10Eva3 definís a sw_1 asignándole el valor FALSO, pero en vez de utilizar el caracter de asignación "=" has usado el de comparación "=="
int sw_1==FALSO;

NOTA: Esto también pasa en if(vs_alum.neva3 ==10){ sw_1==CIERTO;

Tenés comentarios que no comienzan con el signo "//"
--ESTO ES PARA HACER UN LISTADO

... y así sigue... Te pido que revises todos estos errores, los que no he puesto y los warnings. Si tenés problema con alguno, con mucho gusto te voy a ayudar a resolverlo, pero POR FAVOR, antes de postear, tratá de resolverlo antes por tu cuenta (yo no voy a depurar tu código, tan sólo diré donde falla (y por que)).

Saludos

El problema que te dan las variables es que son muy largas, acortalas a 8 caracteres y tal vez se te solucionen muchos problemas. Creo que c no acepta mas de 8 caracteres para definir el nombre de una variable.


Si me equivoco corregirme.

Saludos.

Iniciado por DuDu

El problema que te dan las variables es que son muy largas, acortalas a 8 caracteres y tal vez se te solucionen muchos problemas. Creo que c no acepta mas de 8 caracteres para definir el nombre de una variable.


Si me equivoco corregirme.

Saludos.

Hola,

Tal vez en Cs muy viejos, porque acabo de compilar esto en GCC sin ningún problema.

Código:

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

int main()
{
 int holacomoestasvosyomuybiengraciasporpreguntarytufamiliatodosbiengracias = 1;
 printf("Hello world! %i\n", holacomoestasvosyomuybiengraciasporpreguntarytufamiliatodosbiengracias);
 return 0;
}