Cambios estéticos para facilitar la explicación
Permíteme poder hacer unos cuantos cambios estéticos que me permitan trabajar con tus datos. No recomiendo usar struct action
siempre, sino hacer un typedef
que permita usar esta estructura como un único tipo. La llamaremos, por facilidad, action
.
También pondré los namespaces
, ya que es una mala práctica de programación demasiado extendida usar using namespace std;
en alguna cabecera.
Y por último alinearé los tipos de datos por un lado y las variables por otro para placer visual. Todo esto mejorará notablemente tu forma de trabajar leyendo código, que es realmente el trabajo que vas a realizar más tiempo.
Tu tipo de dato de origen pasará a ser:
typedef struct s_action
{
std::string name ;
int type ;
std::vector < unsigned char > data ;
int value ;
unsigned char source ;
unsigned char destination;
} action;
El poner los ;
alineados ayudará muchísimo de cara a un futuro cuando tengas que editar todos los datos secuencialmente, ya que con un editor que permita copiar en vertical, tendrás gran parte del trabajo hecho.
Posteriormente veo que haces un typedef
, pero es que podías haberlo hecho desde antes (si es código heredado, haz baby steps: introduce el typedef, compila, y comprueba quién llama al struct
con los errores de compilación, haces los cambios uno por uno quitando struct
y listo)
typedef struct s_action_wpr
{
char * name ;
int type ;
key_wpr auth_key ;
unsigned char * data ;
int value ;
unsigned char source ;
unsigned char destination;
} action_wpr;
Por amor a los debugguers, no pongas el mismo nombre al typedef
que al struct
.
Habrás visto también que al colocar el struct de esta manera, queda el tipo de datos unívocamente definido. Esto nos ayudará en la siguiente fase, que consiste en aprender hacer TDD en C para la migración de los datos de origen a los datos de destino.
TDD en C para la migración de datos
Lo primero que tienes que testar es la migración en sí. No puedes picar un código sin pruebas, si lo haces, corres el riesgo de que acabes desarrollando Skynet y tu código tome conciencia de sí mismo.
Así que, para la migración, deberás crear una función que te permita introducir un action
y devuelva un action_wpr
. De ahí te darás cuenta que tendrás que hacer una función para migrar cada parte que no se migre de forma directa.
Migración de action
en action_wpr
Con un editor que permite la edición en vertical, puedes ver que los campos no son iguales, te sobra en action_wpr
un campo (que nos has omitido, imagino, a propósito)
typedef struct s_action | typedef struct s_action_wpr
{ | {
std::string name ; | char * name ;
int type ; | int type ;
std::vector < unsigned char > data ; | key_wpr auth_key ;
int value ; | unsigned char * data ;
unsigned char source ; | int value ;
unsigned char destination; | unsigned char source ;
} action; | unsigned char destination;
} action_wpr;
Así que voy a quitarlo a propósito, porque no hay forma de hacer el traspaso fácilmente:
typedef struct s_action | typedef struct s_action_wpr
{ | {
std::string name ; | char * name ;
int type ; | int type ;
std::vector < unsigned char > data ; | unsigned char * data ;
int value ; | int value ;
unsigned char source ; | unsigned char source ;
unsigned char destination; | unsigned char destination;
} action; | } action_wpr;
Mucho mejor.
¿Qué vemos que podemos traspasar sin hacer cambios? Pues los unsigned char
, los int
.
Nota: ¿Has visto que haya puesto una sola línea de código? No, a esto se le llama pensar antes de actuar, te acostumbrarás cuando te pase mil veces, así que asume siempre que lo haces mal. Cuando seas capaz de cambiar tu forma de hacer las cosas habrás pasado al siguiente nivel. No es un ataque, es un consejo.
¿Qué datos necesitas para terminar la migración que aún no hemos hecho? std::string
y std::vector < unsigned char >
en, respectivamente, char *
y unsigned char *
Migrar std::string
en char *
¿Conoces las propiedades de std::string
? ¿Y las de char *
? Viendo tu código, asumo que, efectivamente, no.
- No son la misma "cosa"
std::string
no es una forma de guardar caracteres uno detrás de otro, guardas cadenas de caracteres. Está indicado para guardar trozos de texto, y puedes realizarle operaciones de manipuación de cadenas de caracteres de forma directa con sus operadores.
char *
es un puntero a un "vector" de una dimensión de elementos de char
, igual que cuando estudias en matemáticas los vectores, van unos detrás de otros. Alguno de ellos puede ser \0
, 0xFF
o incluso 0xDE 0xAD 0xBE 0xEF
, que puede quedar como 0xDEADBEEF
, una referencia bastante habiutal en variables hexadecimales por su significado en inglés (vaca muerta).
Entendiendo que no son la misma cosa, ahora tienes que hacer una función a la que le pases el tipo de dato de un lado y devuelva el del otro con el contenido de entrada.
Ejemplos de tests que vas a tener que hacer:
#include <stdio.h>
#include <string.h>
#include <string>
char * funcion_migracion (std::string);
void test_harness_01 (void)
{
/** Datos con los que voy a probar **/
char * dato_char;
char comparador_base[] = "test de prueba 01";
/** Ejecución de la prueba **/
dato_char = funcion_migracion ( std::string ( comparador_base ) ); // Al hacer esto, haces la conversión explícita de `char[]` a `std::string`, aunque no haría falta porque `funcion_migracion` sólo acepta `std::string` y lo convertiría internamente por ti
/** Comprobar resultados esperados **/
if ( dato_char == NULL )
{
printf("dato_char no puede ser NULL\r\n");
return;
}
if ( strcmp(dato_char, comparador_base) != 0 )
{
printf("no se obtiene el restulado esperado\r\n");
}
/** Limpieza de los resultados obtenidos **/
if ( dato_char != NULL )
{
free ( dato_char );
}
}
Ahora pones diferentes resultados copiando y pegando la función test_harness_01
y modificando comparador_base
para ir viendo que te sale lo que tiene que salir. Mientras no salgan las pruebas, quiere decir que tu código no está bien hecho.
¿Qué obtienes con esto? Aprendes a hacer pruebas, sabes exactamente qué quieres y siempre que ejecutes los tests, volverás a comprobar que no has roto nada en el camino. Como habrás imaginado, estas pruebas no se ejecutan en tu programa real, se ejecutan en un lugar separado de tu programa, al que vamos a llamar "espacio de desarrollo". Te ayuda a que, cuando tengas que hacer o romper algo, tengas presente qué rompes y qué no, y mejora tu forma de realizar código a largo plazo. Dado que parece que estás aprendiendo a programar, tener esta forma de realizar programas es una herramienta de lo más útil para tu día a día.
Sabiendo esto, ya puedes, como ejercicio, saber cómo migrar un std::vector
de unsigned char
s en un unsigned char *
.
Migrar std::vector < unsigned char >
en unsigned char *
¿Por qué no es tan fácil la migración y te has quedado atascado? Efectivamente, porque te has olvidado de un detallito sin importancia que los tests nos enseñarán llegado el momento.
Voy a plantearte el siguiente entorno de tests para que lo veas por ti mismo, y veas qué ocurre.
#include <stdio.h>
#include <string.h>
#include <vector>
char * funcion_migracion_vector (std::vector < unsigned char >);
void test_harness_vector_01 (void)
{
/** Datos con los que voy a probar **/
char * dato_char;
std::vector < unsigned char > comparador_base;
comparador_base.push_back('\0');
/** Ejecución de la prueba **/
dato_char = funcion_migracion_vector ( comparador_base );
/*
* ¿Qué demonios tiene que ocurrir aquí?
*/
}
Acabas de descubrir que, por construcción, ¡un std::vector
de unsigned char
s permite insertar \0
en cualquier parte! ¿Y cómo podemos procesar los datos que hay detrás de \0
? ¿Uso magia vudú?
Necesitarás, inevitablemente, un contador de todos los elementos de ese vector
.
Y así es como se averiguan bugs de base que tienes en tu código y que desconocías. Igual que desconozco si eres profesional o no, sólo así salen estos errores, cuando haces pruebas (si las automatizas, directamente triunfas).
Una vez sabes cómo migrar los datos "difíciles" de migrar, hay que hacer el test de integración, que es migrar de una estructura a otra.
Migrar desde action
hasta action_wpr
En TDD se usa, llegado a este punto, una cosa que llamaremos mock
s, que permiten engañar al sistema para no hacer todas las llamadas internas (por ejemplo, engañar al sistema para que no llame a una base de datos, que es una operación lenta y costosa), como esto me huelo que va para wiki de comunidad, te lo pondré fácil y seguiremos con las pruebas sencillas, ahora tus estructuras pasarán, con el bug que mencioné antes, a esto:
typedef struct s_action | typedef struct s_action_wpr
{ | {
std::string name ; | char * name ;
int type ; | int type ;
std::vector < unsigned char > data ; | unsigned char * data ;
| unsigned int n_data_elms;
int value ; | int value ;
unsigned char source ; | unsigned char source ;
unsigned char destination; | unsigned char destination;
} action; | } action_wpr;
Aquí tendrías el código con el que tendrías que hacer la prueba para la migración entre estructuras:
#include <stdio.h>
#include <string.h>
#include <vector>
#include <string>
typedef struct s_action
{
std::string name ;
int type ;
std::vector < unsigned char > data ;
int value ;
unsigned char source ;
unsigned char destination;
} action;
typedef struct s_action_wpr
{
char * name ;
int type ;
unsigned char * data ;
unsigned int n_data_elms;
int value ;
unsigned char source ;
unsigned char destination;
} action_wpr;
action_wpr * funcion_migracion_structs (action);
void test_harness_migration_structs_01 (void)
{
/** Datos con los que voy a probar **/
action_wpr * dato_char;
action base;
base.name("namestring prueba 01");
base.type = 0xDEADBEEF;
base.data.push_back('h');
base.data.push_back('\0');
base.data.push_back('l');
base.data.push_back('a');
base.data.push_back(' ');
base.data.push_back('m');
base.data.push_back('u');
base.data.push_back('n');
base.data.push_back('d');
base.data.push_back('o');
base.value = -35;
base.source = 0x33;
base.destination = 0xCC;
/** Ejecución de la prueba **/
dato_char = funcion_migracion_vector ( base );
/*
* ¿Qué demonios tiene que ocurrir aquí?
*/
}
Como has visto, he dejado sin completar. ¿Qué faltaría?
Pues, con esto, puedes hacer la migración entre elemento y elemento. Tu siguiente y último paso, ahora que ya sabes cómo se testa en escalera, directamente será testar cómo se migra un "vector" de elementos de un tipo a otro.
Qué has aprendido
Has aprendido con esto que uno no se pone a programar a lo loco. Si tienes que migrar una estructura, haz pruebas para que dicha migración (migración como sinónimo de traspaso, de traductor, de conversor) esté siempre controlada.
Hay montones de frameworks que funcionan estupendamente en C y C++ para este fin. Y dado que los compiladores de C++ admiten código en C (usando extern 'C'), no tendrías problemas de ningún tipo en usar un framework de testing en C++, por ejemplo.
Mi respuesta no te hace código por ti. Ni pretendo que lo haga. Lo que hago es que veas qué problemática tienes a la hora de realizar código y una posible solución a corto y medio plazo que te ayudará.