Foro de elhacker.net

Buenas, tengo el siguiente programa que me pide un valor hexadecimal y lo transforma en decimal, sin importar que sea letras mayusculas o minúsculas, es código es el siguiente:

#include <stdio.h>

int main ()
{

char car1;
char car2;
char valor1;
char valor2;
int valor;
printf("Introduzca un valor hexadecimal:\n");
scanf("\n%c%c", &car1, &car2);

if('0'<=car1 && car1<='9')
valor1=car1-'0';
else if ('A'<=car1 && car1<='F')
valor1=car1-'A'+10;
else if ('a'<=car1 && car1<='f')
valor1=car1-'a'+10;

if('0'<=car2 && car2<='9')
valor2=car2-'0';
else if ('A'<=car2 && car2<='F')
valor2=car2-'A'+ 10;
else if ('a'<=car2 && car2<='f')
valor2=car2-'a'+10;

valor= 16*valor1 + valor2;

printf("EL valor decimal de 0x%c%c es %d\n",car1,car2,valor);
return 0;
}


Pero me han pedido que proteja el programa de tal forma que detecte si alguno de los carácteres es inválido, indicándolo con un mensaje de error y no procesándolos.

Lo he intentado poniendo otro if else, pero no sé como sacarlo, ¿alguna ayuda?

Gracias y un saludo

Hola

¿Es esto lo que pides??

#include <stdio.h>
 
int main ()
{
    char car1;
    char car2;
    char valor1;
    char valor2;
    int valor;
 
    printf("Introduzca un valor hexadecimal:\n");
    scanf("\n%c%c", &car1, &car2);
 
    if('0'<=car1 && car1<='9')
        valor1=car1-'0';
    else if ('A'<=car1 && car1<='F')
        valor1=car1-'A'+10;
    else if ('a'<=car1 && car1<='f')
        valor1=car1-'a'+10;
    else
        {
        printf("Valor erroneo");
        while(getchar()!='\n');
        getchar();
        return 0;
        }
 
    if('0'<=car2 && car2<='9')
        valor2=car2-'0';
    else if ('A'<=car2 && car2<='F')
        valor2=car2-'A'+ 10;
    else if ('a'<=car2 && car2<='f')
        valor2=car2-'a'+10;
    else
        {
        printf("Valor erroneo");
        while(getchar()!='\n');
        getchar();
        return 0;
        }
 
    valor= 16*valor1 + valor2;
 
    printf("EL valor decimal de 0x%c%c es %d\n",car1,car2,valor);
 
    while(getchar()!='\n');
    getchar();
    return 0;
}
 

Saludos

Gracias por la ayuda, creo que eso es lo que me piden pero, ¿podrías explicarme que hace lo que has añadido?

Saludos!

Hola

else
        {
        printf("Valor erroneo");
        while(getchar()!='\n');
        getchar();
        return 0;
        }
 

He añadido esto cada vez que comprobamos un carácter de los introducidos. Tú compruebas si son caracteres válidos con los if, pues si no son válidos entra en el else e imprime un mensaje por pantalla. El while lo que hace es limpiar el buffer de entrada y el getchar de detrás del while espera a que pulses una tecla, y el return para finalizar el programa. Al final de todo pasa lo mismo, limpiamos el buffer de entrada y el programa espera a que se pulse una tecla.

Espero que lo hayas entendido. Hay gente que para pausar el programa utiliza un system("pause"); pero eso solo funciona en windows, el getchar es estandar.

Saludos

Gracias por la aclaración.

Ahora me piden que convierta numeros hexadecimales a decimales hasta que introduzca la secuencia qq para salir del programa, ¿como se podría hacer?

Saludos

Hola

Inténtalo por lo menos, si no te sale te ayudo

Saludos

He pensado poner una estructura do while, pero la verdad es que no sé si lo estoy haciendo bien, he hecho esto:

#include <stdio.h>
 
int main ()
{
    char car1;
    char car2;
    char valor1;
    char valor2;
    int valor;
 do 
    {printf("Introduzca un valor hexadecimal:\n");
    scanf("\n%c%c", &car1, &car2);
 
    if('0'<=car1 && car1<='9')
        valor1=car1-'0';
    else if ('A'<=car1 && car1<='F')
        valor1=car1-'A'+10;
    else if ('a'<=car1 && car1<='f')
        valor1=car1-'a'+10;
    else
        {
        printf("Valor erroneo");
        while(getchar()!='\n');
        getchar();
        return 0;
        }
 
    if('0'<=car2 && car2<='9')
        valor2=car2-'0';
    else if ('A'<=car2 && car2<='F')
        valor2=car2-'A'+ 10;
    else if ('a'<=car2 && car2<='f')
        valor2=car2-'a'+10;
    else
        {
        printf("Valor erroneo");
        while(getchar()!='\n');
        getchar();
        return 0;
        }
 
    valor= 16*valor1 + valor2;
 
    printf("EL valor decimal de 0x%c%c es %d\n",car1,car2,valor);
 
    while(getchar()!='\n');
    getchar();
    }
while (car1== 'q' && car2=='q');
    return 0;
}

Hola

La solución podría ser un do-while pero no lo planteas del todo bien, te pongo el código.

He utilizado la variable "flag" para imprimir el resultado sólo cuando el valor es correcto, y si no entiendes algo ya sabes.

#include <stdio.h>
 
int main ()
{
    char car1;
    char car2;
    char valor1;
    char valor2;
    int valor, flag;
 
    do
        {
        flag = 0;
        printf("Introduzca un valor hexadecimal:\n");
        scanf("\n%c%c", &car1, &car2);
 
        if(car1 == 'q' && car2 == 'q')
            {
            printf("\nPresione Enter para salir.....");
            while(getchar()!='\n');
            getchar();
            return 0;
            }
        else
            {
            if('0'<=car1 && car1<='9')
                valor1=car1-'0';
            else if ('A'<=car1 && car1<='F')
                valor1=car1-'A'+10;
            else if ('a'<=car1 && car1<='f')
                valor1=car1-'a'+10;
            else
                {
                flag = 1;
                printf("Valor erroneo\n\n");
                }
 
            if('0'<=car2 && car2<='9')
                valor2=car2-'0';
            else if ('A'<=car2 && car2<='F')
                valor2=car2-'A'+ 10;
            else if ('a'<=car2 && car2<='f')
                valor2=car2-'a'+10;
            else
                {
                flag = 1;
                printf("Valor erroneo\n\n");
                }
 
            if(flag == 0)
                {
                valor= 16*valor1 + valor2;
                printf("EL valor decimal de 0x%c%c es %d\n\n",car1,car2,valor);
                }
            }
        }
    while(1);
 
    return 0;
}
 

Saludos

Gracias, aunque, ¿me podrías aclarar el uso de flag en el programa?, es que me lo han explicado pero no logro entenderlo bien la verdad

Saludos!

Hola

Pues flag, como su propio nombre indica es una bandera, es decir, una variable que dependiendo de su valor el programa hace una cosa u otra.

En el programa, en cada iteración se pone flag a 0 (es lo primero justo después del do). Cuando se introduce un valor erroneo se muestra un mensaje de error y se pone el flag a 1 para que no se calcule su valor decimal y no se muestre el resultado. Si el valor es correcto, flag sigue valiendo 0 y se calcula su valor decimal y se muestra.

No se si te habrá quedado claro.

Saludos

Vale, más o menos entiendo lo que es la bandera, pero por ejemplo, por que en el último while pones: while (1); ¿porqué (1)?

Hola

He puesto while(1) para que se esté ejecutando siempre.

Si se introduce "qq" el programa finaliza (return 0) sin ejecutar lo que hay detrás, aunque también se podría poner en el while la condición de salida y sería exactamente igual.

Saludos

Ok, muchísimas gracias ;)

Saludos

Hola yo primero idente el codigo (es que me gusta asi :p )

Y le hize un pequeño cambio al code, le quite el primer else. Y solo mostre el mensaje de error al final.

#include <stdio.h>
 
int main ()
{
    char car1;
    char car2;
    char valor1;
    char valor2;
    int valor,flag;
 
    do
    {
        flag = 0;
        printf("Introduzca un valor hexadecimal:\n");
        scanf("\n%c%c", &car1, &car2);
        if(car1 == 'q' && car2 == 'q')
        {
            printf("\nPresione Enter para salir.....");
            fflush(stdin);
            getchar();
            return 0;
        }
        if('0'<=car1 && car1<='9')
            valor1=car1-'0';
        else
            if ('A'<=car1 && car1<='F')
                valor1=car1-'A'+10;
            else
                if ('a'<=car1 && car1<='f')
                    valor1=car1-'a'+10;
                else
                    flag = 1;
        if('0'<=car2 && car2<='9')
            valor2=car2-'0';
        else
            if ('A'<=car2 && car2<='F')
                valor2=car2-'A'+ 10;
            else
                if ('a'<=car2 && car2<='f')
                    valor2=car2-'a'+10;
                else
                    flag = 1;
        if(flag == 0)
        {
            valor= 16*valor1 + valor2;
            printf("\nEL valor decimal de 0x%c%c es %d\n\n",car1,car2,valor);
        }
        else
            printf("\nValor erroneo\n\n");
    }
    while(1);
    return 0;
}
 

Una pequeña observacion, es que una buena practica de programación es hacer uso de una logica positiva... eso quiere decir que las "ramificaciones" deben estar cargadas por el lado de los if y no por el lado de los else.

Mientras escribia mire el codigo y decidi editarlo un poco de otra forma y quedo asi

#include <stdio.h>
 
int main ()
{
    char car1;
    char car2;
    char valor1;
    char valor2;
    int valor,flag;
 
    do
    {
        flag = 0;
        printf("Introduzca un valor hexadecimal:\n");
        scanf("\n%c%c", &car1, &car2);
        if(car1 == 'q' && car2 == 'q')
        {
            printf("\nPresione Enter para salir.....");
            fflush(stdin);
            getchar();
            return 0;
        }
        if('0'<=car1 && car1<='9')
            valor1=car1-'0';
        if ('A'<=car1 && car1<='F')
            valor1=car1-'A'+10;
        if ('a'<=car1 && car1<='f')
            valor1=car1-'a'+10;
        if('0'<=car2 && car2<='9')
            valor2=car2-'0';
        if ('A'<=car2 && car2<='F')
            valor2=car2-'A'+ 10;
        if ('a'<=car2 && car2<='f')
            valor2=car2-'a'+10;
        if(('0'<=car1 && car1<='9' || 'A'<=car1 && car1<='F' || 'a'<=car1 && car1<='f') && ('0'<=car2 && car2<='9' || 'A'<=car2 && car2<='F' || 'a'<=car2 && car2<='f'))
            flag=1;
        if(flag == 1)
        {
            valor= 16*valor1 + valor2;
            printf("\nEL valor decimal de 0x%c%c es %d\n\n",car1,car2,valor);
        }
        else
            printf("\nValor erroneo\n\n");
    }
    while(1);
    return 0;
}
 

Alternativas a hacer resolver un problema de diferentes maneras es lo que hay
 ::)

salu2 ;)

Gracias por la sugerencia :D

Saludos!