[SOLUCIONADO] Error al definir mi propia función

[Verraco]

Hola amigos!! Soy nuevo en el foro y me he registrado para poder pedir ayuda con todas mis dudas sobre la programación en C++ y si puedo, poder ayudar yo tambien.

Esta vez mi problema es el siguiente, me piden que haga un programa que pida por teclado una instrucción hexadecimal y que muestre a que clase pertenece(movimiento, lógico-aritméticas, salto incondicional o salto condicional).
Este es el código:

#include <stdio.h>

int obtclase (unsigned short a);

int main()
{
int aux;
unsigned short ins;
int clase;

printf ("EJERCICIO 6, 7 y 8.\n\n");

printf ("Dame una instruccion en hexadecimal: ");
fflush (stdin);
scanf ("%x", &aux);

ins=aux;
clase=(ins>>14)&3;

printf ("La clase de la instruccion %4.4X es %d\n", ins, clase);
return 0;
}


Hasta ahí todo correcto, pero en el siguiente apartado me dicen que haga mi propia función llamada obtclase que sustituye en el código a "(ins>>14)&3".
El digo de este apartado es este:

#include <stdio.h>

int obtclase (unsigned short a);

int main()
{
int aux;
unsigned short ins;
int clase;

printf ("EJERCICIO 6, 7 y 8.\n\n");

printf ("Dame una instruccion en hexadecimal: ");
fflush (stdin);
scanf ("%x", &aux);

ins=aux;
clase=obtclase(ins);

printf ("La clase de la instruccion %4.4X es %d\n", ins, clase);
return 0;
}

int obtclase (unsigned short a)
{
int clase;
unsigned short ins;
clase == (ins>>14)&3;

return clase;
}

El problema es que me he equivocado en algo ya que en el segundo programa sale que la clase de la instrucción es siempre 0. Mientras que en el primer programa iba de 0 a 3

Si alguien sabe en que me he equivocado me ayudaría muchísimo

Saludos y gracias :)

[guzzano]

Buenas.

Código C:

clase == (ins>>14)&3; >> Debería ser: clase = (a>>14)&3;
 

La variable ins, que está en la función obtclase, no es la misma que está en main, por lo tanto, le estabas pasando una variable vacía para que haga del desplazamiento. Error mío, edito, verdad que estás pidiendo un hexadecimal.

Saludos.

[Verraco]

Quedaría así? lo siento por mi torpeza, pero esq me sigue pasando lo mismo :/

Muchas gracias :)

Código:

#include <stdio.h>

int obtclase (unsigned short a);

int main()
{
int aux;
unsigned short ins;
int clase;

printf ("EJERCICIO 6, 7 y 8.\n\n");

printf ("Dame una instruccion en hexadecimal: ");
fflush (stdin);
scanf ("%x", &aux);

ins=aux;
clase=obtclase(ins);

printf ("La clase de la instruccion %4.4X es %d\n", ins, clase);
return 0;
}

int obtclase (unsigned short a)
{
int clase;
unsigned short a;
clase = (a>>14)&3;

return clase;
}

[guzzano]

Iniciado por Verraco

Quedaría así? lo siento por mi torpeza, pero esq me sigue pasando lo mismo :/

Muchas gracias :)

Código:

#include <stdio.h>

int obtclase (unsigned short a);

int main()
{
int aux;
unsigned short ins;
int clase;

printf ("EJERCICIO 6, 7 y 8.\n\n");

printf ("Dame una instruccion en hexadecimal: ");
fflush (stdin);
scanf ("%x", &aux);

ins=aux;
clase=obtclase(ins);

printf ("La clase de la instruccion %4.4X es %d\n", ins, clase);
return 0;
}

int obtclase (unsigned short a)
{
int clase;
unsigned short a;
clase = (a>>14)&3;

return clase;
}

Buenas, con:
int obtclase (unsigned short a) ya no te hace falta definir otra vez abajo el unsigned short a.

Código C:

#include <stdio.h>
 
unsigned short 
obtclase (unsigned short a)
{
  static int aux = 0;
  aux = (a >> 14) & 3;
 
  return aux;
}
 
int main()
{
  int aux;
  unsigned short ins;
  
  printf("Dame una instrucción en hexadecimal: ");
  scanf("%x", &aux);
 
  ins = aux;
 
  printf ("La clase de la instruccion %4.4X es %d\n", ins, obtclase(ins));
  return 0;
}
 

Así debería funcionar, de casualidad, ¿qué instrucción hexadecimal le das para que te de como resultado un número diferente a 0?

[Verraco]

MUCHÍSIMAS GRACIAS GUZZANO, me has ayudado enormemente :D:D:D:D