[SOLUCIONADO] como sustituir la funcion fgets y gets en c

[Drewermerc]

hola

bueno quisiera saber si me podria ayudar con este programa en c que es para validar si lo que se ingresa son numeros o letras y es que uso la funcion fgets(numero, 20, stdin); y solo funciona corectamente el programa si introdusco los 20 caracteres y si uso la funcion gets no me deja compilar gcc espero que me puedan ayudar este es el codigo.

2 ‪#‎include‬ <stdio.h>
3 #include <stdlib.h>
4 #include <string.h>
5 #include <ctype.h>
6
7 int validar_num(char numero[50]);
8 int validar_cadena(char nombre[50]);
9
10 int main (void)
11 {
12 int sw;
13 char numero[50], nombre[50];
14
15 fflush(stdin);
16 printf("ingrese un numero: ");
17 fgets(numero, 50, stdin);
18
19 sw = validar_num(numero);
20 if (sw == 0)
21 {
22 printf("el numero es valido\n");
23 }
24 else
25 {
26 printf("el numero no es valido\n");
27 }
28
29 fflush(stdin);
30 printf("introduce un nombre\n");
31 fgets(nombre, 50, stdin);
32 sw = validar_cadena(nombre);
33 if (sw == 0)
34 {
35 printf("el nombre es correcto\n");
36 }
37 else
38 {
39 printf("el nombre es incorrecto\n");
40 }
41 return 0;
42 }
43
44 int validar_num(char numero[50])
45 {
46 int i = 0 , sw = 0, j ;
j = strlen(numero);
48 while (i<j && sw==0)
49 {
50 if (isdigit(numero[i]) !=0)
51 {
52 i ++;
53 }
54 else
55 {
56 sw = 1;
57 }
58 }
59 return 0;
60 }
61 int validar_cadena(char nombre[50])
62 {
63 int i = 0, sw = 0, j;
64 j = strlen(nombre);
65 while (i<j && sw == 0 )
66 {
67 if (isalpha(nombre[1]) != 0)
68 {
69 i++;
70
71 }
72 else
73 {
74 sw = 1;
75 }
76 }
77 return 0;
78 }

[vangodp]

int isalpha ( int c ); //Compruebe si el carácter es alfabético, retorna un numero positivo si c es una letra del alfabeto, caso contrario retorna cero(falso):http://www.cplusplus.com/reference/cctype/isalpha/

int isdigit ( int c ); //Compruebe si el carácter es el dígito decimal, retorna un numero (verdadero) si es decimal (0,1,2,3,4,5,6,7,8,9) caso contrario retorna 0 (falso):http://www.cplusplus.com/reference/cctype/isdigit/

int isalnum ( int c ); //Compruebe si el carácter es alfanumérico, retorna un numero verdadero si lo es, caso contrario retorna cero (falso):http://www.cplusplus.com/reference/cctype/isalnum/

tienes que hacer el include de #include <ctype.h>
Hay ejemplo en los enlaces que te pase.

[Drewermerc]

si eso ya lo ise al igual uso la funciones isdigit y isalpha pero es que es detalle es que cuando ingreso los numero o letras y las capturo con fgets no se cumple la validacion si no pongo los caracteres conpletos especificados en fgets espero que me puedas ayudar y gracias
a por cierto tambien incluyo la libreria ctype.h

[vangodp]

por que usas fgets?
Puedes usar getline(cin, nombre);
:D

[Drewermerc]

hola vangodp
oye pero no getline es de c++ o tambien se puede usar en c
y es que cuando lo uso me dice esto el compilador


validacion.c:17: aviso: se pasa el argumento 1 de ‘getline’ desde un tipo de puntero incompatible
/usr/include/stdio.h:673: nota: se esperaba ‘char ** __restrict__’ pero el argumento es de tipo ‘char *’
validacion.c:17: aviso: el paso del argumento 2 de ‘getline’ crea un puntero desde un entero sin una conversión
/usr/include/stdio.h:673: nota: se esperaba ‘size_t * __restrict__’ pero el argumento es de tipo ‘int’
validacion.c:17: error: faltan argumentos para la función ‘getline’

o sera que usa mal getline por que la use asi
getline(numero, 20);

[vangodp]

Pero dado que estas usando cosas como i++ que también es de cpp, pensé que no tendrías problemas.
Necesitas poner iostream para que ande.

[guzzano]

getline y variable++, no son de C++, la función getline, la estás usando mal.

Código C:

#include <stdio.h>
#include <string.h>
 
int main ( void )
{
  size_t size_str;
  char *str;
  
  size_str = 20;
  str = NULL;
  
  getline(&str, &size_str, stdin);
  
  printf("str_text: %s", str);
  
  free(str);
}
 

A getline, no le puedes pasar una variable en la memoria stack, tienes que pasarle un puntero ya que el reservará memoria para lo que esté en stdin, claro, siempre y cuando éste no pase el máximo. El prototipo de la función es

Código C:

ssize_t getline(char **lineptr, size_t *n, FILE *stream);
 

Pero no es para nada necesario que cambies tu fgets, por getline, función, que me parece de paso, algo absurda. Tu problema aquí, son las funciones de validación, siempre estás devolviendo 0.

Saludos

[Drewermerc]

hola guzzano.
gracias por tu respuesta pero sigo con lo mismo si no introdusco todos los numero que le hacegne a la variable size_t size_str; no me da los resultados correctos y si pongo todos los numeros de la variable size_t size_str; si me da los resultados correctos otra cosa que me pudieras sugerir porfavor y gracias por tu respuesta.

[vangodp]

Bueno...
Te lo voy explicar por que creo que solo te sale la cadena completa.
Después me dirás si estoy en lo cierto. ;)
cuando creas una cadena o como lo llames, de esta manera char cadena[10]; tienes como si fuera 10 variables seguidas cadena0, cadena1, cadena2.... hasta llegar a cadena9¿Correcto?
Bien..
se supone que estas variables tienen en ellas cualquier valor residuo de otros programas.
Son variables salvajes por que no sabemos lo que contiene hasta inicialisarlas.
Normalmente le damos el valor de 0 a todas, cosa que tu no lo haces en ningún momento.
como se ve eso ya es un grande problema, por que a la hora de mirar todas las casas, si no ocupa la palabra toda la cadena ya sabes que contiene 0 caso ayas inicialisado, caso contrario ni sabes lo que contiene esas casas.
Eso se puede comprobar muy fácilmente.
Por ejemplo si la cadena es como la de arriba:
char cadena[10];
le metemos algo mas corto no se algo como "LOL" XD
La cadena quedaria asi:
cadena[0] = 'L'
cadena[1] = 'O'
cadena[2] = 'L'
cadena[3] = ? <---¿pero aqui como le queda?
cadena[4] = ? <---¿y aqui?
cadena[5] = ?
cadena[6]= ?
cadena[7]= ?
cadena[8]= ?
cadena[9]= ?

Bueno, si la habíamos inicialisado ya sabemos que debería haber '0' en todas estas casas.
cadena[0] = 'L'
cadena[1] = 'O'
cadena[2] = 'L'
cadena[3] = '0'
cadena[4] = '0'
cadena[5] = '0'
cadena[6] = '0'
cadena[7] = '0'
cadena[8] = '0'
cadena[9] = '0'
Creo que eso ya es un error por que 0 no es alpha.
Si no inicialisamos la cadena peor aun por que no sabemos ni lo que hay en estas casas.
Otra cosa a destacar.
Si la palabra es mas corta que la cadena[10] como en el caso anterior "LOL" después de la ultima letra siempre va un \0 ('0') que indica que la cadena termina ahí.
Cuando escribes lol y tienes 10 huecos cuando pulsas enter al final se escribe un cero.
Puedes comprobar todo eso que digo imprimiendo una letra después de la ultima y veras lo que contiene.
Es mas, mejor imprimes todos las casillas para hacer comprobaciones, solo así tendrás control sobre tu código.
Siempre puedes hacer comprobaciones imprimiendo la cadena para ver el por que pasa lo que pasa.
Sabiendo eso creo que ya puedes afrontar el problema mejor.
isalpha mira letra por letra, si la palabra es mas corta seguro hay un cero al final de tu frase y por eso te tira error.
Comprueba eso y me lo dices ;)
El cero indica en final de la la palabra y es invisible para nosotros, pero esta ahí y se puede imprimir para verlo.
Leer la parte 5.2 de esta pagina
http://fismat.umich.mx/mn1/manual/node6.html

[vangodp]

Código C:

//NULL_XRAY.c
#include <stdio.h>
 
void isaNULL(char cadena[100] );
void imprimenulo( char cadena[100] );
void imprimea_asc2( char cadena[100] );
int main(){
    char cadena[100] = "prueba texto";
    isaNULL(cadena);
    printf("Imprimimos como chars, no se veran nulos\n");
    imprimenulo(cadena);
    printf("Imprimimos como caracteres de la tabla asc2 para poder ver los ceros\n");
    imprimea_asc2( cadena );
}
 
void isaNULL(char cadena[100] ){
    int i = 0;   
    while (cadena[i]){
        printf("%c",cadena[i] );
        i++;
        //Esto imprime todas las letas
    }
    
    printf("\nDespues de la ultima letra '%c' hay '%i' segun la tabla asc2 \n\n",cadena[i-1],cadena[i] );
    printf("Si hay un nulo al final y lo imprimes como char no te aparece: '%c'<---Lo ves?\n",cadena[i] );
}
 
void imprimenulo( char cadena[100] ){
    int i = 0;   
    while (i<100){
        printf("cadena[%2d] = '%c'\n",i,cadena[i] );
        i++;
    }
}
void imprimea_asc2( char cadena[100] ){
    int i = 0;   
    while (i<100){
        printf("cadena[%2d] = '%i'\n",i,cadena[i] );
        i++;
    }
}
 

Este programa te hace de chivato, puedes usar las funciones para ver lo que hay en tu código.
Si hay un cero y estas queriendo validar letras eso puede ser la causa de tu error.
NULL es un carácter no imprimible, puedes ver en la tabla asc2.
Espero que con esto puedas arreglarlo.
Una idea seria validar esos datos por la tabla asc2 ;)

[Drewermerc]

hola vangod.
oye segun lo que te entendi es que tengo que inicializarlas variables en este caso numero y nombre de mi codigo pero si las inicializo con "NULL o 0" no me deja compilar dise que no es un valor soportado por el tipo de puntero.
si no es asi espero que me pueda ayudar con otra respuesta o a depurar mi codigo aun que ya estoy un poco cansado no se si alomejor no entendi vien pero mañan lo leo de nuevo aber si logro hacerlo funcionar y gracias de nuevo.

[vangodp]

Código C:

#include <stdio.h>
void print_char(char cadena[100]);
 
int main(){
    char cadena[100]={'\0'}; //De esta forma limpias el buffer antes de usar
    print_char( cadena );
}
 
void print_char( char cadena[100] ){
    int i = 0;   
    while (i<100){
        printf("cadena[%2d] = '%c'\n",i,cadena[i] );
        i++;
    }
}
 

Así puedes limpiar el buffer.

Pero lo que te quiero decir es que la funcion isalpha() te retorna un valor verdadero CASO el valor sea a,b,c,d...
de lo contrario te va devolver nulo, y el '\0' no es una letra.
¿Que vas hacer si van mezcladas en este caso?
En todo caso tendrias que saber cuantas letras debes mirar.
Si por ejemplo quieres mirar la palabra "Hola" son solo 4 letras, si miras la 5º letra ahi estara la '\0' que te va tirar error ¿comprendes?

Eso se puede hacer asi:

while ( cadena[i] != '\0' )

Código C:

#include <stdio.h>
void print_char(char cadena[100]);
 
int main(){
    char cadena[100]= "hola";
    print_char( cadena );
    
}
 
void print_char( char cadena[100] ){
    int i = 0;
    
    //Mientras no aya NULL hacer las comparaciones necesarias   
    while ( cadena[i] != '\0' ){
        printf("cadena[%2d] = '%c'\n",i,cadena[i] );
        i++;
    }
}
 

En este caso no hace falta inicializar ya que sabemos que la primera letra después de la palabra sera siempre el dichoso null que dice que no vamos utilizar esos 100 espacios.
Pero recuerda que inicializar una variable es muy buena practica y te puede evitar mas de un dolor de cabeza.
Si tienes problemas imprime las tablas una y otra vez para ver por que tienes problemas.

Aquí te explica como puedes limpiar la memoria antes de usar.
Esta un función muy interesante llamada memset() y un par de sistemas mas ;)
http://funcion-linkgl.blogspot.com.e...e-memoria.html

[leosansan]

El origen del problema radica en que fgets introduce un '\n' al final de la cadena, por lo que has de tomar la longitud menos el retorno.

Funcionando:

Código C++:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
 
 int validar_num(char numero[50]);
 int validar_cadena(char nombre[50]);
 
int main (void)
{
  char numero[50], nombre[50];
  int sw;
  printf("ingrese un numero: ");
  fgets(numero, 50, stdin);
  sw = validar_num(numero);
  if (sw == 0)
    printf("el numero es valido\n");
  else
    printf("el numero no es valido\n");
  printf("introduce un nombre\n");
  fgets(nombre, 50, stdin);
  sw = validar_cadena(nombre);
  if (sw == 0)
  printf("el nombre es correcto\n");
  else
    printf("el nombre es incorrecto\n");
 return 0;
 }
 
 int validar_num(char numero[50])
 {
 int i = 0 ;
 for (i=0;i<strlen(numero)-1;i++)
   if (isdigit(numero[i]) ==0)
    return 1;
 return 0;
 }
 
int validar_cadena(char nombre[50])
 {
   int i = 0;
   for (i=0;i<strlen(nombre)-1;i++)
     if (isalpha(nombre[i]) == 0)
        return 1;
   return 0;
 }
 

Por cierto, a ver si nos acostumbramos indentar o sangrar o tabular los códigos si no se hacen ilegibles.

Y otra cosita, cuando un for, else o similar se reduce a una sola instrucción, no hace falta ponerles llaves, así no recargas el código.

Te he cambiado los while por for porque en este caso me parece más razonable conociendo el valor inicial y final.

[vangodp]

Bien señalado :D
El fgets guarda la el enter también ¿no? O sea (\n) que tampoco es alpha.
Pues hay que parar antes o seguro te va decir que no es una letra ni creo tampoco un numero.
Bug asegurado ><.

[Drewermerc]

gracias vangod por la informacion y tu consejo de iniciar variables lo tomare en cuenta gracias por responder.

gracias leosansan por la explicacion de \n no sabia que tan bien lo guardaba fgets y gracias por los consejos los pondre en practica y por corregir mi codigo gracias de nuevo.

[guzzano]

Iniciado por Drewermerc

gracias vangod por la informacion y tu consejo de iniciar variables lo tomare en cuenta gracias por responder.

gracias leosansan por la explicacion de \n no sabia que tan bien lo guardaba fgets y gracias por los consejos los pondre en practica y por corregir mi codigo gracias de nuevo.

Cómo simple duda, cuando te referías a "no me da los resultados correctos", a qué te referías

Saludos.

[vangodp]

Mi pregunta es... ¿Guarda '\n', las funciones scanf() getline() y otras funciones al pulsar enter, o solo fgets lo hace?

[leosansan]

Iniciado por vangodp

Mi pregunta es... ¿Guarda '\n', las funciones scanf() getline() y otras funciones al pulsar enter, o solo fgets lo hace?

Sólo fgets, que yo sepa con seguridad, pero la gracia, por llamarlo de alguna manera, es que forma parte de la cadena. No es como en el scanf que se queda en ocasiones en el buffer y con limpiarlo basta.

[vangodp]

¿Y eso que quiere decir?¿Que si lo imprimo hace el salto de linea automatico?

[Drewermerc]

hola guzzano.
bueno mira cuando dije que no me daba los resultados correctos es por que cuando yo pedia introducir los numeros y ponia letras me decia que estaba correcto y al igual que cuando pedia letras e introducia numeros pro eso decia que no me daba los resultados correctos.

hola vangodp
y si haci es si imprimes la variable de fgets si da el salto de linea pero tambien andube investigando y si quieres usar fgets en lugar de scanf gets etc.. esto del salto de linea lo puedes solucionar con la funcion strchr asi que pues eso no presenta mucho problema por que tambien lo puedes arreglar bueno asi lo veo yo y bueno aqi les dejo un link de como funciona la fncion por si lo nesesitan.
http://c.conclase.net/librerias/?ansifun=strchr

un ejemplo aqui

Código C:

#include<cstdio>
#include<cstdlib>
#include<cstring>
 
int main (void)
{
     struct pp  {
           char genero[50];
      };  
    struct pp temporal;
    char *pch;
    printf("texto\n");
    fgets(temporal.genero, 50, stdin);
    pch = strchr(temporal.genero, 10);
   
    if (pch != NULL)
         *pch = '\0';
 
   printf("%s", temporal.genero);
   return 0;
}
 

bueno no se si me podrian explicar no entendi todo el codigo , tambien no se si estoy bien pero eso de unir con un punto dos variables sellama concatenar, y es que tambien nose que es eso de struc segun yo eso es codigo de c++ y si no es asi corrijanme porfavor.

gracias a todos por sus respuestas me sirvieron mucho

[guzzano]

Si te referías a eso, la cuestión no estaba, si tu línea tenía un salto de línea o no. Es cuestión como te dije arriba, que tus variables devuelven siempre 0. Mira tus líneas 56 y 74. Que tengas una variable definida en main y otra en tu función con el mismo nombre, no quiere decir que cambiará la de main. Claramente puedes, pero ya seria con el uso de punteros.

El código de arriba, la estructura creada, no sé que hace ahí, es patética y inservible. Puedes hacerlo con una simple variable. Aquí te adjunto una explicación.

Código C:

#include <stdio.h>
#include <string.h>
 
int 
main (void)
{
  char str[256];
  char *strp;
  
  fgets(str, sizeof str, stdin);
  
  /* La función strchr, busca la
   * primera aparición cáracter 
   * que se le pasó. En tu caso, 
   * pasaste 10, que seria \n
   * según la tabla ASCII.
   *
   * Esta función, devolverá un 
   * puntero, que tendrá la
   * dirección donde se encontró.
   * Si no se encontró, devolverá
   * un valor NULO.
  */
  
  if (strp = strchr(str, '\n'))
    *strp = '\0';
  
  /* Esta condición, mira si el
   * resultado no es NULO, y 
   * con al asterisco accederemos
   * a la dirección y cambiaremos
   * su valor a un \0, que es un
   * fin de la cadena.
  */
  
  puts(str);
  
  return 0;
}
 

Saludos.

[Drewermerc]

hola guazzano.
gracias por tu respuesta ya me quedo mas claro claro con tu explicacion, y cierto ya noo recordaba que para trabar con variables iguales y en main como en funciones se debe trabajar con punteros pero gracias por tu respuesta mcuhas gracias