Foro de elhacker.net

buenas señores necesito qe me aclaren una duda con esta funcion estoy creando un programa en el que ingrese una cadena de caracteres y me bote lo siguientes:
"asd39n"       345     34.455     A345
cadena        entero    flotante     identificador

pero no se algo esta mal que no me da lo qu e yo quiero a ver denlñe una ojeada  mi code :D gracias

//------------------------------------------------------//
//------------------------------------------------------//
#include <iostream>
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <ctype.h>
#include <iomanip.h>
using std::cout;
using std::endl;
#include <cstring>
int ident=0;
//------------------------------------------------------//
//------------------------------------------------------//
void cadena_caracteres(char *sub_cadena );
void numero_entero(char *sub_cadena);
void numero_flotante(char *sub_cadena);
void identificador (char *sub_cadena);
//------------------------------------------------------//
//------------------------------------------------------//
int main()
{
  char frase[100];
  char *sub_cadena=NULL;
 cout<<"ingrese la frase a evaluar    ";
 gets(frase);
  sub_cadena = strtok(frase," ");
  while(sub_cadena!= NULL)
    {
      cout << sub_cadena << " ------> ";
//------------------------------------------------------//
      if(sub_cadena[0]=='"')
      {
        cadena_caracteres(sub_cadena);
 
      }
//------------------------------------------------------//
     for(int j=0;j<strlen(sub_cadena);j++)
      {
              if(sub_cadena[j]=='.')
              {
                 numero_flotante(sub_cadena);
                 ident=j;
                 break;
              }
      }
//------------------------------------------------------//
 
  if( isdigit(sub_cadena[0])!=0 && isdigit(sub_cadena[ident])!=0)
      {
            numero_entero(sub_cadena);
 
      }
//------------------------------------------------------//
if( isdigit(sub_cadena[0])!=0)
      {
            identificador(sub_cadena);
 
      }
//------------------------------------------------------//
      sub_cadena = strtok(NULL, " ");
    }
  system("pause");
}
 
//------------------------------------------------------//
//------------------------------------------------------//
 
void cadena_caracteres(char *sub_cadena)
{
 int n=strlen(sub_cadena);
 if(sub_cadena[n-1]=='"')
 {
    cout<<endl;
    cout<<setw (25)<<"ES UNA CADENA DE CARACTERES"<<endl;  
 }  
// else { cout<<endl;cout<<setw (25)<<"PARECIA  UNA CADENA DE CARACTERES PERO NO LO ES "<<endl;  }  
}
//------------------------------------------------------//
//------------------------------------------------------//
void numero_entero(char *sub_cadena)
{
   int x=0;   
    for(int i=0;i<strlen(sub_cadena);i++)  
    {
     if(isdigit(sub_cadena[i]))
       {
         x++;
       }
    }  
 
    if(x==strlen(sub_cadena))  
    {
      cout<<endl;cout<<setw (25)<<"ES UN NUMERO ENTERO "<<endl;
    }
    //else {cout<<endl;cout<<setw (25)<<"ESTO NO ES UN NUMERO ENTERO "<<endl;}
 
}
 
//------------------------------------------------------//
//------------------------------------------------------//
 
void numero_flotante(char *sub_cadena)
{
     int x=0,y=0;
 for(int i=0;i<strlen(sub_cadena);i++)  
    {
     if(isdigit(sub_cadena[i]))
       {
         x++;
       }
       if(sub_cadena[i]=='.')
              {
               y++;
              }
    }  
 
    if(x==strlen(sub_cadena)-1 &&  y==1)
    {
     cout<<endl;cout<<setw (25)<<"ES UN NUMERO FLOTANTE "<<endl;
    }
   // else {cout<<endl;cout<<setw (25)<<"ESTO NO ES UN NUMERO FLOTANTE "<<endl;}
}
//------------------------------------------------------//
//------------------------------------------------------//
 
void identificador (char *sub_cadena)
{
     int x=0,y=0;
 
 for(int i=0;i<strlen(sub_cadena);i++)  
    {
         if(isdigit(sub_cadena[i])!=0)
         {
          x++;
         }
          if(isdigit(sub_cadena[i])==0)
         {
          x++;
         }
     }
 
         if(isdigit(sub_cadena[0])==0 && x+y==strlen(sub_cadena))
         {cout<<endl;cout<<setw (25)<<"ESTO ES UN IDENTIFICADOR  "<<endl;}
}
 
 

Tu planteamiento diría que está mal. Por otro lado, creo que te conviene revisar la referencia de isdigit() (http://www.cplusplus.com/reference/clibrary/cctype/isdigit/)

Veamos, si lo he entendido bien:
si tiene números y minúsculas --> cadena
números únicamente: número decimal
número y signo de puntuación: flotante
Empieza por mayúscula y va seguido de números: Identificador

No te voy a dar el trabajo hecho, pero te daré un consejo (según lo que he entendido yo del problema):
En tu caso, o al menos así se me ocurre a mi, puede resultar más sencillo si mirás el problema como un flujo lógico

Un entero no es un flotante porque no tiene el punto, ni es un identificador porque no empieza por una mayúscula, ni es una cadena porque no tiene letras.

O un flotante no es un entero porque tiene punto, tampoco es un identificador porque no tiene una mayúscula, y no es una cadena porque no tiene minúsculas.

No se si me sigues. Por otro lado, no te estoy sugiriendo que hagas un programa que por fuerza bruta compruebe los 4 casos del tirón es decir, algo de este estilo:

Para nada hagas algo así, es perder el tiempo, creo yo. O bueno, no. Si de esa forma te resulta más intuitivo, hazlo así, y luego intenta pasarlo a una comprobación al vuelo que es más eficiente dado que no compruebas todos los casos.

(En el mejor de los casos, si lanzases pongamos 4 funciones y la primera te diese que si, obtendrías el resultado y ya no necesitarías lanzar las otras, pero en caso que lo que te han pasado fuese la última, haces el trabajo 4 veces)

no te entiendo a ver explicate "!"!"

Creo que antes no me he podido expresar mejor, te conviene revisar qué hace isdigit, y añado, qué clase de caracteres se encuentran en la tabla ASCII, porque isdigit solo te dice si pertenece a un tipo o no.

Tu problema (o uno de ellos):

 for(int i=0;i<strlen(sub_cadena);i++)  
    {
         if(isdigit(sub_cadena[i])!=0)
         {
          x++;
         }
          if(isdigit(sub_cadena[i])==0)
         {
          x++;
         }
     }
 

Si es un numero, incrementas x, sino, también, boom goes the dynamite.
De esta forma, directamente recorres la cadena llamando a isdigit 2 veces por cada carácter y ya, sin diferenciar si es una letra, un numero, o un signo de puntuación, o cualquier otro carácter especial.

Creo que aquí prefieras usar isalnum (http://www.cplusplus.com/reference/clibrary/cctype/isalnum/)

Por otro lado: las cadenas, las introduces como (ejemplo) "asd53" o como asd53 ? porque si es el segundo caso, problema.

Además, te repito, tu código comprueba los 4 casos (caden,a entero, float, identificador) SIEMPRE, y eso no es muy eficiente. Si ya has visto que es una cadena, por ejemplo, para qué hacer el resto?