[SOLUCIONADO] problema con funcion en C ejercicio de ficheros

[crash_16109]

hola buen dia tengo un problema con esta funcion, necesito que la funcion detecte todas las letras mayusculas dentro de un texto que tengo guardado.
pero la funcion que hize mayus no da resultado
El if que tengo dentro del main si funciona pero la idea es que en el main no haya nada de eso, tengo que hacer funciones aparte y llamarlas desde el main no se cual sea mi error o si de plano todo esta mal no he trabajado mucho con funciones

Código C:

#include <stdio.h>
#include <stdlib.h>
#include <windows.h>
 
FILE *arch;
char c;
 
    int mayus(FILE *txt)
    {
        int c, cont_may = 0;
 
            while (!feof (arch))
            {
                c = fgetc(arch);
                if(c >64 && c <91)
                {
                cont_may ++;
 
                }
            }
    return c;
    }
 
    int main (){
 
    int cont_min = 0;
 
        arch=fopen("hola.txt", "r");
        system ("cls");
 
        while (!feof (arch))
        {
            c = fgetc(arch);
            printf("%c",c);
  
 //Este if lo puse para verificar que funcione y si lo hace pero si lo quiero poner fuera del main como una funcion, no se como hacerlo
        if(c >64 && c <91)
            {
                cont_may++;
 
            }
 
 
        }
        char a;
        mayus(&a);
        printf("cantidad de mayusculas es:%i",a);
 
        printf("suma:%i\n",cont_min);
 
 
        fclose(arch);
 
    }
 

[aguml]

lo estas orientando mal. Lo primero raro que veo es que la funcion tiene como parametro un puntero de tipo FILE el cual no se usa y que en su lugar usas otro global. Eso es algo muy feo y poco reutilizable. Despues en la funcion main usas un bucle while para recorrer todo el fichero y dentro de ese bucle llamas a tu funcion pero dentro de tu funcion tienes el mismo bucle con lo que hay dos problemas, el primero es que el bucle del main no sirve para nada ya que al salir de tu funcion, al leer otro caracter obtendrias eof o incluso algo raro. El segundo es que el return siempre devolveria el ultimo caracter y no el resto. Hay varias formas de afrontar esto. Puedes dejar el bucle del main y quitar el de la funcion, o puedes hacer uso de la variable que cuenta las mayusculas en tu funcion, y que no usas, para allocar memoria con malloc y en vez de retornar un char pues retornas un char*. A ver si mañana con tiempo puedo verlo y te pongo algo que funcione porque tienes un cacao muy gordo. ¿Que quieres retornar en la funcion realmente? ¿El caracter? ¿El numero de mayusculas?

[crash_16109]

Iniciado por aguml

lo estas orientando mal. Lo primero raro que veo es que la funcion tiene como parametro un puntero de tipo FILE el cual no se usa y que en su lugar usas otro global. Eso es algo muy feo y poco reutilizable. Despues en la funcion main usas un bucle while para recorrer todo el fichero y dentro de ese bucle llamas a tu funcion pero dentro de tu funcion tienes el mismo bucle con lo que hay dos problemas, el primero es que el bucle del main no sirve para nada ya que al salir de tu funcion, al leer otro caracter obtendrias eof o incluso algo raro. El segundo es que el return siempre devolveria el ultimo caracter y no el resto. Hay varias formas de afrontar esto. Puedes dejar el bucle del main y quitar el de la funcion, o puedes hacer uso de la variable que cuenta las mayusculas en tu funcion, y que no usas, para allocar memoria con malloc y en vez de retornar un char pues retornas un char*. A ver si mañana con tiempo puedo verlo y te pongo algo que funcione porque tienes un cacao muy gordo. ¿Que quieres retornar en la funcion realmente? ¿El caracter? ¿El numero de mayusculas?

si quiero que retorne el numero de mayusculas declarar mi funcion fuera del main y ya desde el main solo llamarla pero no tengo muy claro como hacer funciones, necesito que el programa haga mas cosas pero ocupo hacer que una funcion trabaje bien y ya con eso ya puedo guiarme para hacer las de mas

[vangodp]

El código comentado.

Código C:

#include <stdio.h>
#include <stdlib.h>
 
int mayus ( FILE * file ) {
    int c;
    int mayusculas = 0;    
    
    //Hasta llegar al final del fichero... 
    while ( !feof ( file ) ) {
        //...leerá letra por letra...
        c = fgetc ( file );
        
        //...imprimiendo y contando solo las mayusculas.
        if ( c > 64 && c < 91 ) {
            printf ( " %c", c );
            mayusculas++;
        }    
 
    }
    
    //de no haber letras en el archivo retorna 0
    if ( mayusculas == 0 ){
        return 0;
        
    //Si hay letras en el fichero retorna la cantidad.
    }else{
        return mayusculas;
    }
}
 
int main () {
    FILE * arch;  // <- Aqui se crea el puntero. Evite crear globales el maximo pocible
    
    //Muy importante hacer la comprobacion para saber si existe el fichero o no. De no haber ninguno no hay nada que leer y se debe interrumpir aqui por lo sano.
    if ( !(arch = fopen ( "prueba.txt", "r" )) ){
        printf("No existe ningun archivo llamado prueba.txt");
        return 1;
    }
    
    //mayus devuelve un entero que es el numero de mayusculas. Ese printf puedes poner dentro de la misma funcion si quisieras.
    printf ( "\n\ncantidad de mayusculas es:%d\n", mayus(arch) ); //LLamamos a mayus pasando el archivo
    
    //Al final cierra el archivo
    fclose ( arch );
    //hacemos una pausa
    getchar();
    //retornamos cero que en nuestro caso quiere decir "todo correcto".
    return 0;
}
 

Debes practicar y mucho con los archivos.

En esta pagina puedes encontrar una buena explicación y también muchos ejemplos sobre ficheros: http://www.c.conclase.net/ficheros/

Ahora si copias y pegas el código sin haber entendido nada apenas no lograrás gran cosa, y te estarás engañando a ti mismo.

busca en internet sobre funciones, ficheros etc etc. La pagina que te pase es muy buena.

Suerte

[aguml]

Amigo tu codigo no está mal pero la parte donde retornas el numero de mayusculas con el if...else sobra, simplemente pon esto:

Código C:

return mayusculas;
 

[vangodp]

ui O_O lleva razón. De primera hora la había echo así:

Código C:

#include <stdio.h>
#include <stdlib.h>
 
int mayus ( FILE * file, int mayusculas = 0 ) {
    int c = fgetc ( file );
    
    if ( c > 64 && c < 91 ) {
        printf ( " %c", c );
        mayusculas++;
    }    
 
    if ( mayusculas == 0 ){
        return 0;
       
    }else{
        return mayusculas;
    }
}
 
int main () {
    FILE * arch;
    int mayusculas = 0;
    if ( !(arch = fopen ( "prueba.txt", "r" )) ){
        printf("No existe ningun archivo llamado prueba.txt");
        return 1;
    }
    
    while ( !feof ( arch ) ) {
        mayusculas += mayus(arch);
    }
 
    printf ( "\n\ncantidad de mayusculas es:%d\n", mayusculas );
    
    fclose ( arch );
    getchar();
    return 0;
}
 

Se me escapo quitar eso después de adaptar todo dentro de la función. Si llega a ser para mi hasta el printf va dentro XD.

Efectivamente tiene el mismo efecto quitar if else y poner solo el return, más cortito se queda el código XDD. Gracias por avisar pero no se si puedo editar a esas alturas.

[aguml]

Yo lo dejaría así:

Código C:

#include <stdio.h>
#include <stdlib.h>
 
//Prototipos de funciones
int mayus ( FILE * file );
 
int main () {
   FILE * arch;  // <- Aqui se crea el puntero. Evite crear globales el maximo pocible
   int retval;
   //Muy importante hacer la comprobacion para saber si existe el fichero o no. De no haber ninguno no hay nada que leer y se debe interrumpir aqui por lo sano.
   arch = fopen ( "prueba.txt", "r" );
   if ( !arch ){
      printf("\n\nNo existe ningun archivo llamado prueba.txt\n\n");
      retval = -1;
   }else{
      retval = mayus(arch);
      //mayus devuelve un entero que es el numero de mayusculas. Ese printf puedes poner dentro de la misma funcion si quisieras.
      printf ( "\n\ncantidad de mayusculas es:%d\n\n", retval ); //LLamamos a mayus pasando el archivo
   }
 
   //Al final cierra el archivo
   fclose ( arch );
   //hacemos una pausa
   system("PAUSE");
   //retornamos cero que en nuestro caso quiere decir "todo correcto".
   return retval;
}
 
int mayus ( FILE * file ) {
   int c;
   int mayusculas = 0;
 
   //Hasta llegar al final del fichero...
   while ( !feof ( file ) ) {
      //...leerá letra por letra...
      c = fgetc ( file );
 
      //...imprimiendo y contando solo las mayusculas.
      if ( c >= 'A' && c <= 'Z' ) {
         printf ( " %c", c );
         mayusculas++;
      }
   }
   return mayusculas;
}
 

[vangodp]

Ha quedado más pro así, muy bien pues. XD
Ya tiene crash_16109 su respuesta sobrada.
Suerte.

[lareto]

Un par de comentarios.

while ( !feof ( file ) ) {

no está bien.
Entre otras cosas, porque puede suceder un error de lectura, y en ese caso jamás se saldrá del ciclo.
Y

system("PAUSE");

es una de esas prácticas tan difundidas que ya ni nos llama la atención. Yo creo que programar en C para hacer una cosa como system("PAUSE") es absurdo.
(Sugiero http://www.cplusplus.com/forum/articles/11153/ )

[vangodp]

y con que quieres parar el bucle? Se mas especifico con los errores que dices :/

[lareto]

https://www.google.co.uk/search?q=%2...+!feof+%28+%22

[aguml]

no estoy de acuerdo, en el caso de system, forma parte del estandar y funciona perfectamente y ademas el que con system se pueda lanzar un proceso con privilegios system no es cierto, al menos desde Xp sp3. En el caso de feof quizas seria mejor hacer algo como do[ c=fgetc(arch); if(ferror) break; ]while(!ferror(arch)); Estoy desde el movil y no me deja poner saltos de linea y tampoco corchetes

[crash_16109]

Iniciado por aguml

lo estas orientando mal. Lo primero raro que veo es que la funcion tiene como parametro un puntero de tipo FILE el cual no se usa y que en su lugar usas otro global. Eso es algo muy feo y poco reutilizable. Despues en la funcion main usas un bucle while para recorrer todo el fichero y dentro de ese bucle llamas a tu funcion pero dentro de tu funcion tienes el mismo bucle con lo que hay dos problemas, el primero es que el bucle del main no sirve para nada ya que al salir de tu funcion, al leer otro caracter obtendrias eof o incluso algo raro. El segundo es que el return siempre devolveria el ultimo caracter y no el resto. Hay varias formas de afrontar esto. Puedes dejar el bucle del main y quitar el de la funcion, o puedes hacer uso de la variable que cuenta las mayusculas en tu funcion, y que no usas, para allocar memoria con malloc y en vez de retornar un char pues retornas un char*. A ver si mañana con tiempo puedo verlo y te pongo algo que funcione porque tienes un cacao muy gordo. ¿Que quieres retornar en la funcion realmente? ¿El caracter? ¿El numero de mayusculas?

si quiero que retorne el numero de mayusculas declarar mi funcion fuera del main y ya desde el main solo llamarla pero no tengo muy claro como hacer funciones

gracias a ambos si me enrede en el acomodo ya estoy por teminar el programa completo