Lo primero es lo primero.
En un código C moderno no debería estar bien visto usar los tipos clásicos int
, short
, etc, ya que su rango de valores depende de la máquina en la que estemos trabajando. En su lugar sería más conveniente usar los tipos dados por la librería stdint.h
. Los tipos definidos en esta librería presentan la ventaja de que su rango de valores es fijo para cualquier máquina, lo cual evita sorpresas desagradables al diseñar aplicaciones a compilar para múltiples dispositivos.
Dicho esto una primera aproximación podría hacerse usando tipos con rango fijo:
void componeInt( char compuesto[14], int16_t a, int16_t b)
O dejar la firma con los tipos clásicos y hacer un cast dentro de la función, aunque en este caso puede que acabemos truncando el valor y perdiendo dígitos:
void componeInt( char compuesto[14], int a, int b)
{
sprintf( compuesto, "%d/%d", (int16_t )a, (int16_t )b );
}
Una tercera opción pasa por no usar sprintf
, función todo sea dicho de paso, catalogada como insegura porque no es capaz de controlar el desbordamiento de buffer. En su lugar es más seguro usar snprintf
. Esta función recibe, además de un buffer donde almacenar la salida, el tamaño máximo de almacenamiento. Esto impide escribir fuera del buffer a costa de sacrificar el final de la cadena:
void componeInt( char compuesto[14], int a, int b)
{
snprintf( compuesto, 14, "%d/%d", a, b );
}
Lo malo que tiene esta última solución es que podemos perder el último int
completamente. Si es un requisito mostrar los dos valores, aunque sea parcialmente, la mejor solución pasa por conjugar las soluciones que hemos visto en una sola. Bueno, realmente una o dos, dependiendo de los requisitos:
Truncando los valores dentro de la función
void componeInt( char compuesto[14], int a, int b)
{
snprintf( compuesto, 14, "%d/%d", (int16_t )a, (int16_t )b );
}
Limitando el tamaño de las variables directamente en la firma
void componeInt( char compuesto[14], int16_t a, int16_t b)
{
snprintf( compuesto, 14, "%d/%d", a, b );
}
Lo bueno que tienen estas dos soluciones es que hacen uso de una función segura snprintf
en vez de una insegura, sprintf
. Quizás entre estas dos posibles soluciones la más elegante sea la segunda, ya que al que vaya a hacer uso de la función le queda claro directamente en la firma que la función únicamente va a trabajar con valores de 16 bits.
EDITO:
Si se usan los tipos definidos en stdin.h
y no queremos llevarnos sorpresas al imprimir éstas variables tenemos que usar las macros declaradas en inttypes.h
. Estas macros garantizan que el valor se imprima de acuerdo a su rango concreto.
Bien es cierto que estas macros pueden complicar la lectura del código, pero es sólo un poco y en seguida se acostumbra uno a la nueva sintaxis:
void componeInt( char compuesto[14], int16_t a, int16_t b)
{
snprintf( compuesto, 14, "%" PRId16 "/%" PRId16, a, b );
}