Foro de elhacker.net

Hola,

Si son tan amables de decirme como puedo pasar estos valores al siguiente código para ver que devuelve 06015653

CRC16.cpp

#pragma hdrstop
#include "crc16.h"
 
//----- No check stack overflow para uso desde comunicaciones
#pragma option -N-
 
 
const unsigned short Crc16::crc16table[256] =
{
	0x0000, 0x1021, 0x2042, 0x3063, 0x4084, 0x50a5, 0x60c6, 0x70e7,
	0x8108, 0x9129, 0xa14a, 0xb16b, 0xc18c, 0xd1ad, 0xe1ce, 0xf1ef,
	0x1231, 0x0210, 0x3273, 0x2252, 0x52b5, 0x4294, 0x72f7, 0x62d6,
	0x9339, 0x8318, 0xb37b, 0xa35a, 0xd3bd, 0xc39c, 0xf3ff, 0xe3de,
	0x2462, 0x3443, 0x0420, 0x1401, 0x64e6, 0x74c7, 0x44a4, 0x5485,
	0xa56a, 0xb54b, 0x8528, 0x9509, 0xe5ee, 0xf5cf, 0xc5ac, 0xd58d,
	0x3653, 0x2672, 0x1611, 0x0630, 0x76d7, 0x66f6, 0x5695, 0x46b4,
	0xb75b, 0xa77a, 0x9719, 0x8738, 0xf7df, 0xe7fe, 0xd79d, 0xc7bc,
	0x48c4, 0x58e5, 0x6886, 0x78a7, 0x0840, 0x1861, 0x2802, 0x3823,
	0xc9cc, 0xd9ed, 0xe98e, 0xf9af, 0x8948, 0x9969, 0xa90a, 0xb92b,
	0x5af5, 0x4ad4, 0x7ab7, 0x6a96, 0x1a71, 0x0a50, 0x3a33, 0x2a12,
	0xdbfd, 0xcbdc, 0xfbbf, 0xeb9e, 0x9b79, 0x8b58, 0xbb3b, 0xab1a,
	0x6ca6, 0x7c87, 0x4ce4, 0x5cc5, 0x2c22, 0x3c03, 0x0c60, 0x1c41,
	0xedae, 0xfd8f, 0xcdec, 0xddcd, 0xad2a, 0xbd0b, 0x8d68, 0x9d49,
	0x7e97, 0x6eb6, 0x5ed5, 0x4ef4, 0x3e13, 0x2e32, 0x1e51, 0x0e70,
	0xff9f, 0xefbe, 0xdfdd, 0xcffc, 0xbf1b, 0xaf3a, 0x9f59, 0x8f78,
	0x9188, 0x81a9, 0xb1ca, 0xa1eb, 0xd10c, 0xc12d, 0xf14e, 0xe16f,
	0x1080, 0x00a1, 0x30c2, 0x20e3, 0x5004, 0x4025, 0x7046, 0x6067,
	0x83b9, 0x9398, 0xa3fb, 0xb3da, 0xc33d, 0xd31c, 0xe37f, 0xf35e,
	0x02b1, 0x1290, 0x22f3, 0x32d2, 0x4235, 0x5214, 0x6277, 0x7256,
	0xb5ea, 0xa5cb, 0x95a8, 0x8589, 0xf56e, 0xe54f, 0xd52c, 0xc50d,
	0x34e2, 0x24c3, 0x14a0, 0x0481, 0x7466, 0x6447, 0x5424, 0x4405,
	0xa7db, 0xb7fa, 0x8799, 0x97b8, 0xe75f, 0xf77e, 0xc71d, 0xd73c,
	0x26d3, 0x36f2, 0x0691, 0x16b0, 0x6657, 0x7676, 0x4615, 0x5634,
	0xd94c, 0xc96d, 0xf90e, 0xe92f, 0x99c8, 0x89e9, 0xb98a, 0xa9ab,
	0x5844, 0x4865, 0x7806, 0x6827, 0x18c0, 0x08e1, 0x3882, 0x28a3,
	0xcb7d, 0xdb5c, 0xeb3f, 0xfb1e, 0x8bf9, 0x9bd8, 0xabbb, 0xbb9a,
	0x4a75, 0x5a54, 0x6a37, 0x7a16, 0x0af1, 0x1ad0, 0x2ab3, 0x3a92,
	0xfd2e, 0xed0f, 0xdd6c, 0xcd4d, 0xbdaa, 0xad8b, 0x9de8, 0x8dc9,
	0x7c26, 0x6c07, 0x5c64, 0x4c45, 0x3ca2, 0x2c83, 0x1ce0, 0x0cc1,
	0xef1f, 0xff3e, 0xcf5d, 0xdf7c, 0xaf9b, 0xbfba, 0x8fd9, 0x9ff8,
	0x6e17, 0x7e36, 0x4e55, 0x5e74, 0x2e93, 0x3eb2, 0x0ed1, 0x1ef0
};
 
 
/**************************************************************************
Funcion: add
Params : valor a incluir
Return : nada
 
Incluye valor parametro en calculo de CRC
**************************************************************************/
 
void Crc16::add (unsigned short c)
{
	crcval = crc16table [(crcval >> 8) & 0xff] ^ (crcval << 8) ^ c;
}

El CRC16.h

/**** HEADER. Gestion de CRC polinomico de 16 bits                      ****/
/***************************************************************************/
 
#ifndef CRC16_H
#define CRC16_H
 
 
 
/**************************************************************************
***************************************************************************
** 
**  Clase 'Crc16', implementa calculo de CRC de 16 bits
** 
***************************************************************************
**************************************************************************/
 
 
class Crc16 {
   static const unsigned short crc16table[256];
   unsigned short crcval;
 
public:
   Crc16 (void)                           { crcval = 0; }
   void init (void)                       { crcval = 0; }
   void set (unsigned short valor)	      { crcval = valor; }
   unsigned short value (void)	         { return crcval; }
   void add (unsigned short c);
};
 
#endif

Estoy tratando de correrlo en https://www.onlinegdb.com/online_c++_compiler (https://www.onlinegdb.com/online_c++_compiler)

Pero como no se C++ (prometo que aprenderé lo básico) necesito poder de momento echarlo andar.

Se agradece cualquier ayuda de antemano.


Saludos

El código que pasaste no hace nada por si solo.
Eso es tan sólo una clase, es decir, un tipo de dato inventado que permite crear "variables" de ese tipo y cada variable tendrá:
- su propio estado: la variable `crcval`
- una serie de habilidades o cosas que sabe hacer: las funciones `init()`, `set()`, `value()` y `add()`

Para ejecutar un programa en C++ necesitas de una función `main()` que es lo que realmente se ejecuta y ahí es donde tienes que usar estas otras partes auxiliares.
Entiendo que lo que quieres hacer es algo como esto:

#include "crc16.h"
#include <iostream>
 
using namespace std;
 
int main(int argc, char *argv[]) {
    Crc16 crc16;
    crc16.add(06015653);
 
    cout << "Salida: " << crc16.value() << endl;
} 

Problemas de todo esto:
- El número `06015653` (en notación octal porque empieza por 0) equivale al número: 1579947 (en notación decimal)
- La función `add()` recibe un argumento de tipo `unsigned short` cuyo rango de valores permitidos es: [0, 65535]
- No puedes pasar ese número a la función porque no va a funcionar correctamente, deberías cambiar el parámetro de `unsigned short` a `int` para aumentar ese rango de valores permitidos

Hola,

Gracias por tu respuesta.

El # es en expresión hexadecimal.

Yo entiendo que se le pasan estos valores y me genera uno final que sería el CRC16.


Saludos

No termino de entender qué expresión es hexadecimal.
El número que indicaste en el mensaje original es: 06015653 y ese número no es hexadecimal, es octal.

Quiero entender que son varios números hexadecimales por separado lo que quieres pasar pero no puedo adivinar si quieres:
- 0x06 + 0x01 + 0x56 + 0x53
- 0x0601 + 0x5653
- 0x06015653 <-- este espero que no porque ya te adelanto que también se pasa del rango permitido

Tienes razón mil disculpas, quiero pasar en hexadecimal a esa función

0x06 0x01 0x56 0x53

Para haga el cálculo y me entregué el CRC16 de esos números, creo que el a medida que lo calcula utiliza el resultado de cada 1, no lo tengo bien claro.


Saludos

Sí, el resultado es dependiente de los valores anteriores.
Lo que tienes que hacer es lo mismo que ya te puse pero pasando varios valores antes de mostrar el resultado.

#include "crc16.h"
#include <iostream>
 
using namespace std;
 
int main(int argc, char *argv[]) {
    Crc16 crc16;
    crc16.add(0x06);
    crc16.add(0x01);
    crc16.add(0x56);
    crc16.add(0x53);
 
    cout << "Salida: " << crc16.value() << endl;
} 

Para ejecutar esto en un compilador online como OnlineGDB (https://www.onlinegdb.com/) tienes que crear varios ficheros (en la parte de arriba a la izquierda del botón de "Run").
Cada fichero tiene que tener el nombre adecuado: "crc16.h", "crc16.cpp" y en el que pongas la función `main()` que por defecto se llamará "main.cpp"
Copia el contenido correspondiente a cada fichero y con darle al botón de "Run" ya verás el resultado

Excelente amigo, lo pruebo y te dejo saber.


Saludos

Muchísimas gracias, solucionado!!!

Saludos

Hola compi,

Me ha surgido algo curioso y es que el programa del proveedor que utiliza el CRC16 publicado aquí para los siguientes valores:

Hex         Dec
0x0B --> 11
0x01 --> 1
0x56 --> 86
0x50 --> 80

Genera una salida de C561 --> 50529

Y a mi 9A8B --> 39563

Me llama la atención que usa un número mayor que 9 donde ya cambia en hexadecimal pero no se si eso tiene algo que ver, incluso probé en ves de 11 17 que es B en hex y nada.


Tendrás alguna idea de que puede ser?


Saludos

Por un lado te preguntaría si estás 100% seguro de que el programa recibe los siguientes valores, en el orden indicado y ninguno más: Tanto si se le están pasando valores anteriores a esos como si el orden es diferente, el resultado también lo será.

Y por otro lado:
No entiendo qué fue lo que te resultó extraño aquí. La B en hexadecimal es un carácter válido y corresponde al 11 en decimal. No entiendo qué tiene ver el 17 en todo esto.

Si puedes borrar mis mensajes anteriores y dejar tu respuesta nada más, sucedía exactamente lo que dices, parámetros incompletos, llevaba varios días en ello y tuve lapsus mental no entendía nada.

Ya resuelto.

Saludos