Problema al crear una función a partir de un programa

[jamatbar]

Buenas, tengo el siguiente programa, que hace lo siguiente;

Un programa que lea dos caracteres hexadecimales y dé como salida su valor en decimal, en el que se permiten el uso de minusculas y además, que si se introduce un caracter invalido te lo indicará con un mensaje de error, y por ultimo, que si escribes la sentencia qq el programa se termine.

El codigo del programa es el siguiente (ya hace todo lo que se pide);

Código (c) [Seleccionar] Expandir

#include <stdio.h>

int main ()
{
 // Declaramos las variables
 char car1;
 char car2;
 char valor1;
 char valor2;
 int valor;
 int flag; // Declaramos la variable de control
   
do { // Iniciamos el bucle do-while
   flag=0;
   printf("Introduzca un valor hexadecimal:\n");
   scanf("\n%c%c", &car1, &car2);
   
   if (car1 == 'q' && car2 == 'q') // Con esta sentencia hacemos que el programa finalice al escribir qq
       {
       printf("Presione enter para salir\n");
       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",car1,car2,valor);
         }
    }
   
}  
   while(1);
   
   return 0;
}

Y ahora me piden lo siguiente: Modificar el programa de tal forma que, además de la función main, exista una función que tome como parámetros dos caracteres hexadecimales y devuelva su valor decimal: Esto supone: declarar la nueva función, definir la nueva función y modificar la función main para que use la nueva funcion.

Pues bien, haciendo algunos cambios he llegado a esto:

Código (c) [Seleccionar] Expandir

#include <stdio.h>

int hexadecimal (char car1, char car2);
int main ()

{

 char car1;
 char car2;
 char valor1;
 char valor2;
 int valor;
 int flag;
   

   flag=0;
   printf("Introduzca un valor hexadecimal:\n");
   scanf("\n%c%c", &car1, &car2);
   
   valor=hexadecimal(car1,car2);
   
   int hexadecimal (char car1, char car2)
   
   {
   
   if (car1 == 'q' && car2 == 'q')
       {
       printf("Presione enter para salir\n");
       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",car1,car2,valor);
         }
    }
    }
   

   
   return 0;
}

Pero me da una serie de errores al compilar, ¿alguna ayuda o sugerencia?

Gracias de antemano y saludos!

[jamatbar]

Bueno, he seguido intentándolo y he llegado a compilar sin errores, el problema es que es como si no usara la función, ya que pongo dos valores y me dice que su valor en hexadecimal es 0, y eso con cualquiera :S

Código (c) [Seleccionar] Expandir


#include <stdio.h>

int hexadecimal (char car1, char car2);
int main ()

{

  char car1;
  char car2;
  int valor;
  int flag;
 
   

    flag=0;
while ((car1!='q') && (car2!='q'))
{
    printf("Introduzca un valor hexadecimal:\n");
    scanf("\n%c%c", &car1, &car2);
    valor=hexadecimal(car1, car2);
    if ((car1!='q') && (car2!='q')){
            printf("EL valor decimal de 0x%c%c es %d\n",car1,car2,valor);
    }
    }
    return 0;
   
    }
   
   
    int hexadecimal (char car1, char car2)
   
    {
    char valor1;
    char valor2;
    int flag;
    int valor;
    if (car1 == 'q' && car2 == 'q')
        {
        printf("Presione enter para salir\n");
        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",car1,car2,valor);
          }
     }
     return valor;
     }
   


[rir3760]

Esa funcion la debes hacer de forma tal que solo indique si la conversion no es valida (por ejemplo devolviendo -1), quien la llama (la funcion principal en tu caso) se debe hacer cargo de hacer lo necesario (imprimir el mensaje de error).

Mas o menos asi:

Código (c) [Seleccionar] Expandir

#include <stdio.h>

int hexadecimal (char car1, char car2);

int main(void)
{
   char car1;
   char car2;
   int valor;
   int flag;
   
   while (1){
      puts("Introduzca un valor hexadecimal:");
      scanf(" %c%c", &car1, &car2);
      if (car1 == 'q' && car2 == 'q'){
         printf ("Presione enter para salir\n");
         while (getchar() != '\n')
            ;
         getchar();
         break;
      }else if ((valor = hexadecimal (car1, car2)) == -1)
         printf ("Valor erroneo\n\n");
      else
         printf("0x%c%c == %d\n", car1, car2, valor);
   }
   
   return 0;
}

int hexadecimal(char car1, char car2)
{
   char valor1;
   char valor2;
   
   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
      return -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
      return -1;
   
   return valor1 * 16 + valor2;
}

Un saludo

[jamatbar]

Gracias por la aclaración, ahora lo he ampliado para que convierta 4 digitos hexadecimales en un numero decimal, pero el orden del resultado sale alterado, por ejemplo, si pongo 25BC me sale que en decimal es 48165, cuando este número en hexadecimal es el BC25, es decir, justo al reves, este es el codigo:

EDITO POR QUE LO ACABO DE SOLUCIONAR

Un saludo!