Problema en el cálculo del mínimo de un fichero

[rodrigoemece]

El programa se encarga de abrir un fichero de texto y calcular de entre sus elementos, el máximo, el mínimo y la media de ellos. El problema aparece al calcular el máximo exclusivamente. La respuesta por pantalla siempre resulta ser el primer elemento del fichero y no entiendo por qué. Muchas gracias por vuestra ayuda.

doc.txt

Código:

45 32 332 89 1

main.c

Código:

/* 
 * Nome:      Exercicio 1
 * Arquivo:   main.c
 * Autor:     Rodrigo M.
 * Data:      06 de Novembro de 2012
 */

#include <stdio.h>
#include <stdlib.h>
#define LNX 99999

float mx=0, mn=0, me=0; 

float max(float);
float min(float);
float med(float,int,int);

main(){
int v[LNX], i=0, z=1;
float ma=0, mi=0, md=0; 

FILE *fp;
fp = fopen("doc.txt","rt");
if(fp==NULL){
    printf("Erro na apertura do arquivo.\n");
    exit(1);
}

else{
    
 do{
   fscanf(fp,"%d",&v[i]);
   mn=v[0];
   mx=v[0];
   
   if (feof(fp)!=0 || i>=LNX) z=i+1;
   ma=max(v[i]);
   mi=min(v[i]);
   md=med(v[i],i,z);
   i++;  
    }while(feof(fp)==0 && i<LNX);
          
    printf("O maximo do arquivo e: %f\n", ma);
    printf("O minimo do arquivo e: %f\n", mi);
    printf("A media do arquivo e: %f\n", md);
     }
  fclose(fp);
      }

float max(float vec)
{
    if (vec>mx) mx = vec;              
    return mx;
}

float min(float vec)
{
    if (vec<mn) mn = vec;              
    return mn;    
}

float med(float vec, int a, int x)
{
    me = (me + vec)/x;
    return me;
}

Output

Código:

O maximo do arquivo e: 45.000000
O minimo do arquivo e: 1.000000
A media do arquivo e: 99.800003

RUN SUCCESSFUL (total time: 11ms)

[cotolon]

Puede ser que el error esté en el .txt
Porque tu tienes los números así:

45 32 332 89 1

Tal vez funcione si los tienes así:

45
32
332
89
1

Fíjate en eso, si no es eso entonces no lo sé, aún no veo ficheros xd

[Heimish2000]

Depura y vete viendo que valores te lee del archivo.

[ecfisa]

Hola.

Creo que te estas complicando mucho, el código puede ser mucho más simple:

Código C:

#include <stdio.h>
#define MAX 99999
 
int main(){
  int v[MAX], i=0;
  float ma=-MAX, mi=MAX, md=0;
  FILE *fp = fopen("doc.txt", "rt");
 
  if(fp==NULL){
    printf("Error abriendo el archivo.\n");
    exit(1);
  }
  while(!feof(fp)){
    fscanf(fp, "%d", &v[i]);
    if(v[i] > ma) ma = v[i];
    if(v[i] < mi) mi = v[i];
    md += v[i++];
  }
  fclose(fp);
  printf("Numero mayor: %8.3f\n", ma);
  printf("Numero menor: %8.3f\n", mi);
  printf("Promedio    : %8.3f\n", md/i);
  return 0;
}
 

Saludos.

[rodrigoemece]

A ver, creo que el problema creo que está en que metí el cálculo de las funciones del main dentro del bucle. El programa que dice ecfisa es idóneo, lo que pasa es que tengo la obligación de crearlo con tres funciones que calculen los tres valores. Muchas gracias a los tres :) reposteo el código cuando lo solucione!

[rodrigoemece]

pfff nadie ve qué problema hay en la función max? :/

[rodrigoemece]

Así me funciona el máximo pero no el mínimo, pero no entiendo por qué :S (Cambios en líneas 23 y 26)

Código C:

#include <stdio.h>
#include <stdlib.h>
#define LNX 99999
 
float mx=0, mn=0, me=0; 
 
float max(float);
float min(float);
float med(float,int,int);
 
main(){
int v[LNX], i=0, z=1;
float ma=0, mi=0, md=0; 
 
FILE *fp;
fp = fopen("doc.txt","rt");
if(fp==NULL){
    printf("Erro na apertura do arquivo.\n");
    exit(1);
}
 
else{
mx=v[0];
 do{
   fscanf(fp,"%d",&v[i]);
   mn=v[0]; 
 
   if (feof(fp)!=0 || i>=LNX) z=i+1;
   ma=max(v[i]);
   mi=min(v[i]);
   md=med(v[i],i,z);
   i++;  
    }while(feof(fp)==0 && i<LNX);
          
    printf("O maximo do arquivo e: %f\n", ma);
    printf("O minimo do arquivo e: %f\n", mi);
    printf("A media do arquivo e: %f\n", md);
     }
  fclose(fp);
      }
 
 
float max(float vec)
{
    if (vec>mx) mx = vec;              
    return mx;
}
 
float min(float vec)
{
    if (vec<mn) mn = vec;              
    return mn;    
}
 
float med(float vec, int a, int x)
{
    me = (me + vec)/x;
    return me;
}
 

[ecfisa]

Hola rodrigoemece.

Proba de este modo:

Código C:

#include <stdio.h>
#include <stdlib.h>
 
#define MAX 99999
 
float max(int *, int);
float min(int *, int);
float med(int *, int);
 
void main(){
  int v[MAX], i=0;
 
  FILE *fp;
  fp = fopen("C:\\TMP\\doc.txt","rt");
  if(fp==NULL){
    printf("Error abriendo el archivo.\n");
    exit(1);
  }
  while(!feof(fp)) fscanf(fp, "%d", &v[i++]);
  fclose(fp);
  printf("Numero mayor: %8.3f\n", max(v,i));
  printf("Numero menor: %8.3f\n", min(v,i));
  printf("Promedio    : %8.3f\n", med(v,i));
  getchar();
}
 
float max(int *v, int n) {
  int i,mx=0;
  for(i=0;i<n;i++)
    if (v[i]> mx) mx = v[i];
  return mx;
}
 
float min(int *v, int n) {
  int i, mi=MAX;
  for(i=0;i<n;i++)
    if (v[i]<mi) mi = v[i];
  return mi;
}
 
float med(int *v, int n) {
  int i, me=0;
  for(i=0;i<n;i++) me += v[i];
  return (float)me/n;
}
 

Saludos

[rodrigoemece]

Perfecto :) pero te importaría mucho explicar mi error?

[ecfisa]

Perfecto :) pero te importaría mucho explicar mi error?

No, en absoluto.
Uno de los motivos es que inicializas variables globales dentro del ciclo do/while.

Así me funciona el máximo pero no el mínimo, pero no entiendo por qué

Funciona el máximo por que quitaste la variable mx desde dentro del do/while, y no funciona el mínimo por que lo estabas incializando a cero y luego no encontraba valor menor para efectuar el reemplazo. La variable mn, tiene que ser inicializada a un valor mayor al mayor de los datos, para que al compararlo, pueda tomar por menor a qualquiera de ellos. Del mismo modo la variable mx debe tomar un valor menor al menor valor esperado.

Por otro lado, aunque no afecta el funcionamiento, el arreglo de enteros v es innecesario, al igual que el tercer parámetro de la función med.

El codigo podria quedar así:

Código C:

#include <stdio.h>
 
#define LNX 99999
 
float mx=0, mn=LNX, me=0;
 
float max(float);
float min(float);
float med(float,int);
 
main(){
  int v, i=0, z=1;
  float ma=0, mi=0, md=0;
  FILE *fp;
 
  fp = fopen("doc.txt","rt");
  if(fp==NULL) {
    printf("Erro na apertura do arquivo.\n");
    exit(1);
  } // El operador [B]else[/B] es innecesario, sale por exit
  do{
    fscanf(fp,"%d",&v);
    if (feof(fp)!=0 || i>=LNX) z=i+1;
    ma=max(v);
    mi=min(v);
    md=med(v,z);
    i++;
  }while(feof(fp)==0 && i<LNX);
  printf("O maximo do arquivo e: %f\n", ma);
  printf("O minimo do arquivo e: %f\n", mi);
  printf("A media do arquivo e: %f\n", md);
  fclose(fp);
  getchar();
}
 
float max(float vec) {
  if (vec>mx) mx = vec;
    return mx;
}
 
float min(float vec) {
  if (vec<mn) mn = vec;
    return mn;    
}
 
float med(float vec, int x) {
  me = (me + vec)/x;
    return me;
}
 

Para detectar los fallos es muy útil hacer una prueba de escritorio sobre el código.

Saludos.

[rodrigoemece]

Iniciado por ecfisa

No, en absoluto.
Uno de los motivos es que inicializas variables globales dentro del ciclo do/while.


Funciona el máximo por que quitaste la variable mx desde dentro del do/while, y no funciona el mínimo por que lo estabas incializando a cero y luego no encontraba valor menor para efectuar el reemplazo. La variable mn, tiene que ser inicializada a un valor mayor al mayor de los datos, para que al compararlo, pueda tomar por menor a qualquiera de ellos. Del mismo modo la variable mx debe tomar un valor menor al menor valor esperado.

Por otro lado, aunque no afecta el funcionamiento, el arreglo de enteros v es innecesario, al igual que el tercer parámetro de la función med.

El codigo podria quedar así:

Código C:

#include <stdio.h>
 
#define LNX 99999
 
float mx=0, mn=LNX, me=0;
 
float max(float);
float min(float);
float med(float,int);
 
main(){
  int v, i=0, z=1;
  float ma=0, mi=0, md=0;
  FILE *fp;
 
  fp = fopen("doc.txt","rt");
  if(fp==NULL) {
    printf("Erro na apertura do arquivo.\n");
    exit(1);
  } // El operador [B]else[/B] es innecesario, sale por exit
  do{
    fscanf(fp,"%d",&v);
    if (feof(fp)!=0 || i>=LNX) z=i+1;
    ma=max(v);
    mi=min(v);
    md=med(v,z);
    i++;
  }while(feof(fp)==0 && i<LNX);
  printf("O maximo do arquivo e: %f\n", ma);
  printf("O minimo do arquivo e: %f\n", mi);
  printf("A media do arquivo e: %f\n", md);
  fclose(fp);
  getchar();
}
 
float max(float vec) {
  if (vec>mx) mx = vec;
    return mx;
}
 
float min(float vec) {
  if (vec<mn) mn = vec;
    return mn;    
}
 
float med(float vec, int x) {
  me = (me + vec)/x;
    return me;
}
 

Para detectar los fallos es muy útil hacer una prueba de escritorio sobre el código.

Saludos.

Muchísimas gracias por explicarlo :)