funcion recibe dos parametros

Iniciado por ALONSOQ, 22 Agosto 2012, 16:53 PM

0 Miembros y 1 Visitante están viendo este tema.

avesudra

#10
Fire544: en la función verifica cadena tuya no recorres la cadena, por lo que no sirve(según creo yo pues si el usuario mete una cadena alfanumérica ¿como la tratarías?) , además tienes un pequeño error en el if pues el operador lógico and no es & , si no &&.Por lo demás bien.

ALONSOQ:, tu función verificaCadena también tiene un error (en mi opinión)pues solo comprobais (los dos tú y Fire) el primer carácter y dependiendo de que sea retornais , que en principio está bien , pero si el usuario mete aaaa1452345 por ejemplo ¿se tomaría como cadena o como número?

Arriba dejé unas funciones no sé si las habréis visto.
Regístrate en

ALONSOQ

Cita de: avesudra en 31 Agosto 2012, 20:16 PM
Fire544: en la función verifica cadena tuya no recorres la cadena, por lo que no sirve(según creo yo pues si el usuario mete una cadena alfanumérica ¿como la tratarías?) , además tienes un pequeño error en el if pues el operador lógico and no es & , si no &&.Por lo demás bien.

ALONSOQ:, tu función verificaCadena también tiene un error (en mi opinión)pues solo comprobais (los dos tú y Fire) el primer carácter y dependiendo de que sea retornais , que en principio está bien , pero si el usuario mete aaaa1452345 por ejemplo ¿se tomaría como cadena o como número?

Arriba dejé unas funciones no sé si las habréis visto.

Gracias Avesudra por tu aportación;

Por seguir ya en el mismo código que he enviado si en la cadena , en lugar de poner del 1 al 3, ponemos por ejemplo del 0 al 100, es decir
for (i=0;i<3;100++){
    if(cadena[i]>=65&cadena[i]<=122)
    return 1;
    else
    return 0;   


comprobaría los 100 primeros caracteres y entonces estaría bien, ¿ no es así?

gracias

avesudra

#12
Cita de: ALONSOQ en 31 Agosto 2012, 21:40 PM
comprobaría los 100 primeros caracteres y entonces estaría bien, ¿ no es así?

gracias
Ese código está un poco mal.

  • Lo primero no sabes si la posición 100 existe y te puede dar un error de SEGMENTATION_FAULT , ese famoso error del 0xC000000005.
  • Segundo , no llegará a recorrer la cadena aunque sepas cuán larga es la cadena pues en cuanto compares el primer carácter sea falsa o verdadera tu condición la cual está mal por que el operador lógico and es && no &, siempre retornará ¿entiendes?.
  • El operador and en la comparación está mal porque es tontería , un mismo valor , no puede cumplir dos comparaciones distintas.

Así que esa función medianamente bien sería así si quieres hacerlo por fuerza sin punteros , tienes que pasarle el largo que tiene , porque si nó da un SEGMENTATION_FAULT como una casa( si quieres mirarlo pruébalo).

int verificaCadena(char cadena[],int largo)
{
 int i = 0;
 while(i!=largo)
 {
   if(cadena[i]>=65 || cadena[i]<=122)
   ++i;
   else
   return 0;  
 }
}
}

¡Un saludo!
Regístrate en

ALONSOQ

#13
Creo queme estoy haciendo un lío;

Entiendo que la función verificacadena, en la funcíón principal, analiza    argv[1] y argv[2], sin embargo parece que no recorre todo el contenido, es decir tengo la duda de como recorre todos los caracteres de ambas posiciones para determinar si es número o cadena.

en tu código entiendo avesudra que recorres toda la cadena, creo que lo que haces, dentro del bucle for,  lo podríamos arreglar tambien con strlen(cadena).

Lo que sí he comprobado es que si el en el segundo argumento introducimos a2 (habiendo introducido en el primero un número), devuelve que es correcto, cuando no debería.

Avesudra ¿ como integrarías tu funcíon dentro del código que tengo hecho?  te importaría incluirla?

saludos

saludos

avesudra

Cita de: ALONSOQ en  1 Septiembre 2012, 00:16 AM
Creo queme estoy haciendo un lío;

Entiendo que la función verificacadena, en la funcíón principal, analiza    argv[1] y argv[2], sin embargo parece que no recorre todo el contenido, es decir tengo la duda de como recorre todos los caracteres de ambas posiciones para determinar si es número o cadena.

en tu código entiendo avesudra que recorres toda la cadena, creo que lo que haces, dentro del bucle for,  lo podríamos arreglar tambien con strlen(cadena).

Lo que sí he comprobado es que si el en el segundo argumento introducimos a2 (habiendo introducido en el primero un número), devuelve que es correcto, cuando no debería.

Avesudra ¿ como integrarías tu funcíon dentro del código que tengo hecho?  te importaría incluirla?

saludos

saludos
Por los líos no te preocupes , tranquilo que se resuelve todo  :D , claro exactamente puedes usar strlen pero como se te vé verde no te he dicho nada de eso.Te integro mi función en tu codigo :
#include <stdio.h>
/****************************************/
/** Retornan 0 si es verdad y -1 si no **/
/****************************************/
int verificaSiEsCadena(char *stream);
int verificaSiEsNumero(char *stream);

int verificaSiEsCadena(char *stream)
{
    if(verificaSiEsNumero(stream) == -1)
    {
        return 0;//Si no es numero , retorna 0 porque es una cadena.
    }
    return -1; //Si es numero retorna -1 porque no es una cadena , es un numero.
}
int verificaSiEsNumero(char *stream)
{
    while(*stream!='\0')
    {
        if(*stream<48 || *stream>57)
        {
            return -1;//Si no es numero retorna -1
        }
        ++stream;
    }
    return 0;//Si es numero retorna 0
}
int main(int argc, char *argv[])
{

    int i;


  printf("Numero de argumentos: %i\n", argc);
  printf("Nombre del programa: %s\n", argv[0]);

  for(i=1;i<(argc); i++)
   {
      printf("Argumento numero %i: %s\n", i, argv[i]);
   }

    if(argv[1]!=0&&argv[2]!=0)
    {
                           if(verificaSiEsCadena(argv[2])==0 && verificaSiEsNumero(argv[1])==0)//Si el segundo argumento es una cadena y el segundo es un numero.
                           {
                           printf("Son Correctos (Numero y Cadena)!!");
                           }
                           else
                           printf("No son Correctos");
   }
   else
   printf("No son Correctos");

}

Regístrate en

ALONSOQ

Avesudra, el programa funciona perfectamente, pero hay que modificar una pequeña cosa para que de el resultado buscado, porque si no mantiene el error de 2 c2 correctos.

En verificaSi es Numero:
int verificaSiEsNumero(char *stream)
{
    while(*stream!='\0')
    {
        if(*stream>=48 && *stream<=57)// si strema está entr 0  y 9//
        {
            return 0;//Si no es numero retorna -1
        }
        ++stream;
    }
    return -1;


Muchas gracias por tu aportación


avesudra

Está igual ¿no? Lo único es que ahora retorna al revés  :xD
Regístrate en

Fire544

#17
Avesudra: Claro tienes razon, pero fijate en el siguiente post, donde recorro todos los elementos, de un array que ya he concatenado. Y no estas correcto cuando dices que no recorro los argv[]; ten en cuenta que estas accediendo a punteros de caracteres no a caracteres en un array de chars, pero como quiera fijate donde hago referencia a lo que dices.

Cita de: Fire544 en 26 Agosto 2012, 20:30 PM
Mira ALONSOQ puedes optar por concatenar los elementos de argv[] a variables chars de cualquier tamaño y luego verificar que cada posicion de este vector este en un rango de 65 >= 97 que son las posiciones del alfabeto en la tabla ASCII !!!

Mira algo como esto !!!


char arrays[10];

strcpy(array, argv[1]);
for(int i=0; i<argc; i++)
{
if(argv[1]!=0)
{
if(array[i]>=65&array[i]<=97)
{
printf("Es una cadena\n");
}
}
}


A si puedes hacer con la posicion 2 de argv[] y listo, no he probado el codigo pero creo que no tendras problemas !!

Suerte !!  :xD
"Si enseñas a pezcar a un niño lo ayudas para toda la vida, si pezcas para alimentarlo lo ayudas por un momento".

xiruko

#18
dejo aqui mi pequeño aporte por si sirve de ayuda. solo verifica que el primer argumento sea un numero, ya que el segundo al ser cualquier cadena esta tambien puede contener numeros o ser todo numeros no? si no es asi, y el segundo parametro tiene que contener solo las letras del abecedario dimelo y modificare el codigo en un momento.


#include <stdio.h>

int main(int argc, char** argv) {

int i;

if (argc != 3) {
printf("Numero de parametros incorrecto.\n");
return 1;
}

//primer parametro un numero
for (i=0; argv[1][i]!='\0'; i++) {
if (argv[1][i] < 48 || argv[1][i] > 57) {
printf("Primer parametro no es un numero.\n");
return 2;
}
}

printf("Parametros correctos.\n");
return 0;
}


Citar
Entrada: ./main 123a
Salida: Numero de parametros incorrecto.

Entrada: ./main 123a hola
Salida: Primer parametro no es un numero.

Entrada: ./main 123 hola
Salida: Parametros correctos.

un saludo!