buscar deuda mayor

[minette1988]

He modificado lo de double deuda_mayor = 0.0 y he puesto int deuda_mayor=0, pero me da estos errores:
En la función 'deuda_mayor':
deuda_mayor.c:113: error: el valor indicado por el subíndice no es ni matriz ni puntero
deuda_mayor.c:113: error: el valor indicado por el subíndice no es ni matriz ni puntero
deuda_mayor.c:117: error: el valor indicado por el subíndice no es ni matriz ni puntero
deuda_mayor.c:117: error: el valor indicado por el subíndice no es ni matriz ni puntero

[nicolas_cof]

Código [Seleccionar] Expandir

int main()
{
void crear_fich();
void deuda_mayor();
...

Las declaraciones de las funciones se hacen fuera de la funcion main()

En la función 'deuda_mayor':
deuda_mayor.c:113: error: el valor indicado por el subíndice no es ni matriz ni puntero
deuda_mayor.c:113: error: el valor indicado por el subíndice no es ni matriz ni puntero
deuda_mayor.c:117: error: el valor indicado por el subíndice no es ni matriz ni puntero
deuda_mayor.c:117: error: el valor indicado por el subíndice no es ni matriz ni puntero

Esos errores son porque intentas acceder a cli como si fuera un arreglo de estructura y no lo tenes declarado como tal.

Código [Seleccionar] Expandir

struct cliente cli;
...
cli[i].deuda
...

Otro error que te veo es que nunca inicializas la variable num_clientes sin embargo realizas una comparacion i < num_clientes

Salu10.

[minette1988]

Acabo de modificarlo, corriguiendo los errores que me has dicho, pero cuando le doy a la opción 2 que busca la deuda más alta y la muestra por pantalla no hace nada.

[nicolas_cof]

Sigue sin funcionarme. Ayudadme, por favor.

Hola minette1988, la verdad yo te recomendaria plantearte bien que queres hacer y cual seria la mejor manera de hacerlo.

A lo largo de varias correcciones que se le han hecho a tu codigo, se nota que careces de conocimiento de muchos conceptos basicos.

Siendote sincero y espero no ofenderte, tu codigo es un lindo lio.

Cuando haces un mal trabajo y nadie te lo dice quiere decir que se han rendido contigo. Cuando alguien te esta corrigiendo lo hacen porque quieren que lo hagas mejor.

No dudes en consultar cualquier duda que tengas, que para eso estamos...

Salu10.

[minette1988]

Hola, en la función deuda_mayor lo primero que hago es definir la estructura:
     struct cliente cli;
luego como me interesa buscar la deuda más alta declaro una variable llamada deuda_mayor de tipo int que será la posición en la que se encuentra. Hasta aquí bien ¿no?. La duda que tengo es esta: La variable deuda que tengo definida en el struct cliente, ¿la tengo que comparar con la variable deuda_mayor que se encuentra en la función deuda_mayor?, ¿la variable deuda tengo que volver a definirla en este módulo o no hace falta?

[minette1988]

He hecho la siguiente modificación :

Código (c) [Seleccionar] Expandir

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

#define LEE_CHAR(c)\
c=getchar();\
while((c=='\n') || (c==' ') || (c=='\t'))\
c=getchar();

#define LEE_CAD(c,n) { int i=0; \
                     c[i]=getchar();\
     while((c[i]=='\n') || (c[i]=='\t')) c[i]=getchar();\
     while ((c[i]!='\n') && (i < (n-1))){ \
        i++;\
c[i]=getchar(); }\
     c[i]='\0'; }

#define TAM_NOMBRE 100


struct cliente{
char nombre[TAM_NOMBRE];
        double deuda;
};
int main(){
  void alta();
  void mostrar_todo();
  void deuda_mayor();
  char op;
  FILE * pf;

  if((pf = fopen("datos.dat", "wb")) == NULL) {/*"rb+"*/
           /* Si no existe, ejecuta el módulo que lo cree */
   alta();

      if((pf = fopen("clientes.dat", "rb")) == NULL) {
perror("Fichero no accesible");
                exit(1);
      }
  }
 
  do{
printf("MENÚ\n----\n");

printf("1). Dar de alta un cliente.\n");
                printf("2). Mostrar todo.\n");
                printf("3). Deuda mayor.\n");
                printf("4). salir y eliminar.\n");
                printf("0). Salir\n\nOpción(0-4): ");

                do{
LEE_CHAR(op);
}while( (op < '0') || (op >'4') );

switch(op){
                    case '1': alta(pf); break;
                    case '2': mostrar_todo(pf); break;
                    case '3': deuda_mayor(pf); break;
                    case '4':{ fclose(pf); pf=NULL; remove("clientes.dat"); op= '0';
break;}
}
  }while(op != '0');

exit(0);
}//main()
//////////////////////////////////////////////////////////           
void alta(FILE *pf){
      struct cliente cli;
      int fin;

  if((pf = fopen("clientes.dat", "ab")) == NULL) {
printf("Error al crear el fitxer\n");
}
else {
fin = 1; /* Termina con fin == 0 */


while ( fin ) {
printf ("Nombre del cliente: ");
LEE_CAD(cli.nombre,TAM_NOMBRE);
                        printf ("Deuda: ");
                        scanf("%lf",&cli.deuda);
fwrite(&cli,1,sizeof cli,pf);
printf("Otro (0--> NO, 1--> SI)? ");
scanf("%d",&fin);
}
if (fclose(pf))
printf("Error al guardar el fichero\n");
}
}
/////////////////////////////////////////////
void mostrar_todo(FILE *pf){
       struct cliente cli;

       if((pf = fopen("clientes.dat", "rb")) == NULL) {
printf("Error al abrir el fichero clientes.dat\n");
       }
       else{

fread(&cli, 1, sizeof cli, pf);
        while (!feof(pf)){
printf("%s  %.2lf\n", cli.nombre, cli.deuda);
fread(&cli,1, sizeof cli,pf);
        }
        fclose(pf);
  }
}
//////////////////////////////////
void deuda_mayor(FILE * pf){
        struct cliente max,cli;

        if((pf = fopen("clientes.dat", "rb")) == NULL) {
printf("Error al abrir el fichero clientes.dat\n");
        }
        else{
             fread(&max,1,sizeof cli,pf);/*supongo que el primero es el mayor*/
             while(!feof(pf)){
                if(cli.deuda > max.deuda)/*si es mayor que el actual lo cambia*/
                   max=cli;
                fread(&cli,1,sizeof cli,pf);
             }
        }
        fclose(pf);
        printf("%s %.2lf\n",cli.nombre,cli.deuda);
}

Cuando lo compilo introduzco los datos de 2 clientes: Si la deuda del primero es mayor que la del segundo me muestra la del segundo que es la más pequeña, sin embargo si la deuda del segundo es mayor que la del primero me muestra ésta última. ¿Dónde está el fallo? Ayudadme, por favor, el martes tengo recuperación. Gracias.