conversion implicite de int a ushort

Version imprimable

Bonjour

je tente de créer une classe permettant le calcul d'un CRC16, copie d'une fonction dont je dispose en C (le but final est de faire communiquer mon pc avec un micro-controleur programmé en C)

Je but sur une erreur

Citation:

Impossible de convertir implicitement le type 'int' en 'ushort'. Une conversion explicite existe (un cast est-il manquant*?)

sur la ligne

Code:

crc = (crc >> 8) ^ CRC16Table[(crc ^ ((ushort)bytes[i])) & 0x00FF];

de cette classe

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace USB_GUI
{
    public class CRC16
    {
 
 
        public ushort ComputeChecksum(byte[] bytes, int taille)
        {
            ushort[] CRC16Table =
            {
            0x0000, 0xc0c1, 0xc181, 0x0140, 0xc301, 0x03c0, 0x0280, 0xc241,
            0xc601, 0x06c0, 0x0780, 0xc741, 0x0500, 0xc5c1, 0xc481, 0x0440,
            0xcc01, 0x0cc0, 0x0d80, 0xcd41, 0x0f00, 0xcfc1, 0xce81, 0x0e40,
            0x0a00, 0xcac1, 0xcb81, 0x0b40, 0xc901, 0x09c0, 0x0880, 0xc841,
            0xd801, 0x18c0, 0x1980, 0xd941, 0x1b00, 0xdbc1, 0xda81, 0x1a40,
            0x1e00, 0xdec1, 0xdf81, 0x1f40, 0xdd01, 0x1dc0, 0x1c80, 0xdc41,
            0x1400, 0xd4c1, 0xd581, 0x1540, 0xd701, 0x17c0, 0x1680, 0xd641,
            0xd201, 0x12c0, 0x1380, 0xd341, 0x1100, 0xd1c1, 0xd081, 0x1040,
            0xf001, 0x30c0, 0x3180, 0xf141, 0x3300, 0xf3c1, 0xf281, 0x3240,
            0x3600, 0xf6c1, 0xf781, 0x3740, 0xf501, 0x35c0, 0x3480, 0xf441,
            0x3c00, 0xfcc1, 0xfd81, 0x3d40, 0xff01, 0x3fc0, 0x3e80, 0xfe41,
            0xfa01, 0x3ac0, 0x3b80, 0xfb41, 0x3900, 0xf9c1, 0xf881, 0x3840,
            0x2800, 0xe8c1, 0xe981, 0x2940, 0xeb01, 0x2bc0, 0x2a80, 0xea41,
            0xee01, 0x2ec0, 0x2f80, 0xef41, 0x2d00, 0xedc1, 0xec81, 0x2c40,
            0xe401, 0x24c0, 0x2580, 0xe541, 0x2700, 0xe7c1, 0xe681, 0x2640,
            0x2200, 0xe2c1, 0xe381, 0x2340, 0xe101, 0x21c0, 0x2080, 0xe041,
            0xa001, 0x60c0, 0x6180, 0xa141, 0x6300, 0xa3c1, 0xa281, 0x6240,
            0x6600, 0xa6c1, 0xa781, 0x6740, 0xa501, 0x65c0, 0x6480, 0xa441,
            0x6c00, 0xacc1, 0xad81, 0x6d40, 0xaf01, 0x6fc0, 0x6e80, 0xae41,
            0xaa01, 0x6ac0, 0x6b80, 0xab41, 0x6900, 0xa9c1, 0xa881, 0x6840,
            0x7800, 0xb8c1, 0xb981, 0x7940, 0xbb01, 0x7bc0, 0x7a80, 0xba41,
            0xbe01, 0x7ec0, 0x7f80, 0xbf41, 0x7d00, 0xbdc1, 0xbc81, 0x7c40,
            0xb401, 0x74c0, 0x7580, 0xb541, 0x7700, 0xb7c1, 0xb681, 0x7640,
            0x7200, 0xb2c1, 0xb381, 0x7340, 0xb101, 0x71c0, 0x7080, 0xb041,
            0x5000, 0x90c1, 0x9181, 0x5140, 0x9301, 0x53c0, 0x5280, 0x9241,
            0x9601, 0x56c0, 0x5780, 0x9741, 0x5500, 0x95c1, 0x9481, 0x5440,
            0x9c01, 0x5cc0, 0x5d80, 0x9d41, 0x5f00, 0x9fc1, 0x9e81, 0x5e40,
            0x5a00, 0x9ac1, 0x9b81, 0x5b40, 0x9901, 0x59c0, 0x5880, 0x9841,
            0x8801, 0x48c0, 0x4980, 0x8941, 0x4b00, 0x8bc1, 0x8a81, 0x4a40,
            0x4e00, 0x8ec1, 0x8f81, 0x4f40, 0x8d01, 0x4dc0, 0x4c80, 0x8c41,
            0x4400, 0x84c1, 0x8581, 0x4540, 0x8701, 0x47c0, 0x4680, 0x8641,
            0x8201, 0x42c0, 0x4380, 0x8341, 0x4100, 0x81c1, 0x8081, 0x4040
            };
 
            ushort crc = 0xFFFF; //valeur initiale
 
            // Boucle de calcul du CRC16
	        for (int i = 0 ; i < taille ; i++ )
	        {
                crc = (crc >> 8) ^ CRC16Table[(crc ^ ((ushort)bytes[i])) & 0x00FF];
 	        }
            return crc;
        }
 
        public byte[] ComputeChecksumBytes(byte[] bytes, int taille)
        {
            ushort crc = ComputeChecksum(bytes,taille);
            return BitConverter.GetBytes(crc);
        }
 
    }
}

J'en suis a essayer de mettre des (ushort) de partout, ce qui illustre bien le fait que je ne comprends plus rien. Pour moi j'ai des int nullepart !

Merci par avance pour votre aide

05/09/2011, 23h21
petitours

je m'en suis "sorti" en remplaçant tous les "ushort" par des "int"... :oops:

Je suis quand même preneur de l'explication, histoire de me coucher moins idiot !
05/09/2011, 23h25
petitours

en fait non.... ca me retourne pas les bonnes valeurs !

Surement à cause des décalages de bit avec les types de données qui ne sont plus les même.

Au cas où j'ai codé un barbarisme incompréhensible en c#, voici la fonction C que je cherche à reproduire

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
U16 CRC_crc(U8 *Adresse, U8 Taille)
{
volatile unsigned char *lPtr ;
unsigned short crc, lSize ;
unsigned short PoidFort, PoidFaible ;
 
	crc = 0xFFFF ;// Initialisation de la valeur de calcul
 
// Boucle de calcul du CRC16
	for (lPtr = Adresse, lSize = Taille ; lSize-- != 0 ; )
	{
		crc = (crc >> 8) ^ CRC16Table[(crc ^ *lPtr++) & 0x00FF];
 	}
 
	return crc ;
}

06/09/2011, 10h05
Morgan7469

La conversion implicite n'est pas possible de int à ushort.
Et si je ne m'abuse, le ushort c'est un UInt16.

Essaye ceci pour ta conversion :

Code:

Convert.ToUInt16(toto);

Par contre tu risques d'avoir des erreurs si le int à convertir est supérieur à 65535 (valeur max du UInt16).
06/09/2011, 18h21
petitours

Merci beaucoup mais ca n'a rien changé.

J'ai vu que le message d'erreur me signalait le bug en colonne 23, ca m'a mis la puce à l'oreille et j'ai donc ecrit cela

Code:

crc = (ushort)((crc >> 8) ^ CRC16Table[(crc ^ (ushort)(bytes[i])) & 0x00FF]) ;

Je n'ai plus le message d'erreur !

Mais ca ne me donne pas le même resultat que ma fonction en C ! :roll::cry:
1 ligne, ca parait pourtant pas insurmontable...

Déja une étape pour rendre le truc plus lisible et compilable

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
  public class CRC16
  {
    static ushort[] CRC16Table =
            {
            0x0000, 0xc0c1, 0xc181, 0x0140, 0xc301, 0x03c0, 0x0280, 0xc241,
            0xc601, 0x06c0, 0x0780, 0xc741, 0x0500, 0xc5c1, 0xc481, 0x0440,
            0xcc01, 0x0cc0, 0x0d80, 0xcd41, 0x0f00, 0xcfc1, 0xce81, 0x0e40,
            0x0a00, 0xcac1, 0xcb81, 0x0b40, 0xc901, 0x09c0, 0x0880, 0xc841,
            0xd801, 0x18c0, 0x1980, 0xd941, 0x1b00, 0xdbc1, 0xda81, 0x1a40,
            0x1e00, 0xdec1, 0xdf81, 0x1f40, 0xdd01, 0x1dc0, 0x1c80, 0xdc41,
            0x1400, 0xd4c1, 0xd581, 0x1540, 0xd701, 0x17c0, 0x1680, 0xd641,
            0xd201, 0x12c0, 0x1380, 0xd341, 0x1100, 0xd1c1, 0xd081, 0x1040,
            0xf001, 0x30c0, 0x3180, 0xf141, 0x3300, 0xf3c1, 0xf281, 0x3240,
            0x3600, 0xf6c1, 0xf781, 0x3740, 0xf501, 0x35c0, 0x3480, 0xf441,
            0x3c00, 0xfcc1, 0xfd81, 0x3d40, 0xff01, 0x3fc0, 0x3e80, 0xfe41,
            0xfa01, 0x3ac0, 0x3b80, 0xfb41, 0x3900, 0xf9c1, 0xf881, 0x3840,
            0x2800, 0xe8c1, 0xe981, 0x2940, 0xeb01, 0x2bc0, 0x2a80, 0xea41,
            0xee01, 0x2ec0, 0x2f80, 0xef41, 0x2d00, 0xedc1, 0xec81, 0x2c40,
            0xe401, 0x24c0, 0x2580, 0xe541, 0x2700, 0xe7c1, 0xe681, 0x2640,
            0x2200, 0xe2c1, 0xe381, 0x2340, 0xe101, 0x21c0, 0x2080, 0xe041,
            0xa001, 0x60c0, 0x6180, 0xa141, 0x6300, 0xa3c1, 0xa281, 0x6240,
            0x6600, 0xa6c1, 0xa781, 0x6740, 0xa501, 0x65c0, 0x6480, 0xa441,
            0x6c00, 0xacc1, 0xad81, 0x6d40, 0xaf01, 0x6fc0, 0x6e80, 0xae41,
            0xaa01, 0x6ac0, 0x6b80, 0xab41, 0x6900, 0xa9c1, 0xa881, 0x6840,
            0x7800, 0xb8c1, 0xb981, 0x7940, 0xbb01, 0x7bc0, 0x7a80, 0xba41,
            0xbe01, 0x7ec0, 0x7f80, 0xbf41, 0x7d00, 0xbdc1, 0xbc81, 0x7c40,
            0xb401, 0x74c0, 0x7580, 0xb541, 0x7700, 0xb7c1, 0xb681, 0x7640,
            0x7200, 0xb2c1, 0xb381, 0x7340, 0xb101, 0x71c0, 0x7080, 0xb041,
            0x5000, 0x90c1, 0x9181, 0x5140, 0x9301, 0x53c0, 0x5280, 0x9241,
            0x9601, 0x56c0, 0x5780, 0x9741, 0x5500, 0x95c1, 0x9481, 0x5440,
            0x9c01, 0x5cc0, 0x5d80, 0x9d41, 0x5f00, 0x9fc1, 0x9e81, 0x5e40,
            0x5a00, 0x9ac1, 0x9b81, 0x5b40, 0x9901, 0x59c0, 0x5880, 0x9841,
            0x8801, 0x48c0, 0x4980, 0x8941, 0x4b00, 0x8bc1, 0x8a81, 0x4a40,
            0x4e00, 0x8ec1, 0x8f81, 0x4f40, 0x8d01, 0x4dc0, 0x4c80, 0x8c41,
            0x4400, 0x84c1, 0x8581, 0x4540, 0x8701, 0x47c0, 0x4680, 0x8641,
            0x8201, 0x42c0, 0x4380, 0x8341, 0x4100, 0x81c1, 0x8081, 0x4040
            };
 
 
    public ushort ComputeChecksum(byte[] bytes, int taille)
    {
 
      UInt16 crc = 0xFFFF; //valeur initiale
 
      // Boucle de calcul du CRC16
      for (int i = 0; i < taille; i++)
      {
        int x = (crc ^ ((ushort)bytes[i])) & 0x00FF;
        UInt16 HH = (UInt16)(crc >> 8);
        crc = (UInt16) (HH ^ CRC16Table[x]);
      }
      return crc;
    }
 
    public byte[] ComputeChecksumBytes(byte[] bytes, int taille)
    {
      ushort crc = ComputeChecksum(bytes, taille);
      return BitConverter.GetBytes(crc);
    }
 
  }

06/09/2011, 21h10
petitours

c'est plus propre, merci

par contre ca me donne toujours le même résultat...

une piste :
J'ai tenté de reproduire ce code C# dans mon programme C (µ controleur) et surprise, il donne bien la même chose qu'avant !

Le même code en c# et en C ne donne pas le même résultat !8O

Pour moi, ce code marche:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
void Main()
{
	var tab = Encoding.ASCII.GetBytes("Bonjour");
	var result = CRC16.ComputeChecksum(tab, tab.Length).ToString("X");
}
	  public class CRC16
	  {
		static ushort[] CRC16Table =
				{
				0x0000, 0xc0c1, 0xc181, 0x0140, 0xc301, 0x03c0, 0x0280, 0xc241,
				0xc601, 0x06c0, 0x0780, 0xc741, 0x0500, 0xc5c1, 0xc481, 0x0440,
				0xcc01, 0x0cc0, 0x0d80, 0xcd41, 0x0f00, 0xcfc1, 0xce81, 0x0e40,
				0x0a00, 0xcac1, 0xcb81, 0x0b40, 0xc901, 0x09c0, 0x0880, 0xc841,
				0xd801, 0x18c0, 0x1980, 0xd941, 0x1b00, 0xdbc1, 0xda81, 0x1a40,
				0x1e00, 0xdec1, 0xdf81, 0x1f40, 0xdd01, 0x1dc0, 0x1c80, 0xdc41,
				0x1400, 0xd4c1, 0xd581, 0x1540, 0xd701, 0x17c0, 0x1680, 0xd641,
				0xd201, 0x12c0, 0x1380, 0xd341, 0x1100, 0xd1c1, 0xd081, 0x1040,
				0xf001, 0x30c0, 0x3180, 0xf141, 0x3300, 0xf3c1, 0xf281, 0x3240,
				0x3600, 0xf6c1, 0xf781, 0x3740, 0xf501, 0x35c0, 0x3480, 0xf441,
				0x3c00, 0xfcc1, 0xfd81, 0x3d40, 0xff01, 0x3fc0, 0x3e80, 0xfe41,
				0xfa01, 0x3ac0, 0x3b80, 0xfb41, 0x3900, 0xf9c1, 0xf881, 0x3840,
				0x2800, 0xe8c1, 0xe981, 0x2940, 0xeb01, 0x2bc0, 0x2a80, 0xea41,
				0xee01, 0x2ec0, 0x2f80, 0xef41, 0x2d00, 0xedc1, 0xec81, 0x2c40,
				0xe401, 0x24c0, 0x2580, 0xe541, 0x2700, 0xe7c1, 0xe681, 0x2640,
				0x2200, 0xe2c1, 0xe381, 0x2340, 0xe101, 0x21c0, 0x2080, 0xe041,
				0xa001, 0x60c0, 0x6180, 0xa141, 0x6300, 0xa3c1, 0xa281, 0x6240,
				0x6600, 0xa6c1, 0xa781, 0x6740, 0xa501, 0x65c0, 0x6480, 0xa441,
				0x6c00, 0xacc1, 0xad81, 0x6d40, 0xaf01, 0x6fc0, 0x6e80, 0xae41,
				0xaa01, 0x6ac0, 0x6b80, 0xab41, 0x6900, 0xa9c1, 0xa881, 0x6840,
				0x7800, 0xb8c1, 0xb981, 0x7940, 0xbb01, 0x7bc0, 0x7a80, 0xba41,
				0xbe01, 0x7ec0, 0x7f80, 0xbf41, 0x7d00, 0xbdc1, 0xbc81, 0x7c40,
				0xb401, 0x74c0, 0x7580, 0xb541, 0x7700, 0xb7c1, 0xb681, 0x7640,
				0x7200, 0xb2c1, 0xb381, 0x7340, 0xb101, 0x71c0, 0x7080, 0xb041,
				0x5000, 0x90c1, 0x9181, 0x5140, 0x9301, 0x53c0, 0x5280, 0x9241,
				0x9601, 0x56c0, 0x5780, 0x9741, 0x5500, 0x95c1, 0x9481, 0x5440,
				0x9c01, 0x5cc0, 0x5d80, 0x9d41, 0x5f00, 0x9fc1, 0x9e81, 0x5e40,
				0x5a00, 0x9ac1, 0x9b81, 0x5b40, 0x9901, 0x59c0, 0x5880, 0x9841,
				0x8801, 0x48c0, 0x4980, 0x8941, 0x4b00, 0x8bc1, 0x8a81, 0x4a40,
				0x4e00, 0x8ec1, 0x8f81, 0x4f40, 0x8d01, 0x4dc0, 0x4c80, 0x8c41,
				0x4400, 0x84c1, 0x8581, 0x4540, 0x8701, 0x47c0, 0x4680, 0x8641,
				0x8201, 0x42c0, 0x4380, 0x8341, 0x4100, 0x81c1, 0x8081, 0x4040
				};
 
 
		public static ushort ComputeChecksum(byte[] bytes, int taille)
		{ 
		  int crc = 0xFFFF; //valeur initiale
 
		  // Boucle de calcul du CRC16
		  for (int i = 0; i < taille; i++)
		  {
			int x = (crc ^ bytes[i]) & 0x00FF;
			int HH = (crc >> 8);
			crc = (HH ^ CRC16Table[x]);
		  }
		  return Convert.ToUInt16(crc);
		}
 
		public static byte[] ComputeChecksumBytes(byte[] bytes, int taille)
		{
		  ushort crc = ComputeChecksum(bytes, taille);
		  return BitConverter.GetBytes(crc);
		}
 
	  }

J'obtiens E212, soit la même chose que dans le calculateur suivant: http://www.lammertbies.nl/comm/info/...lculation.html

06/09/2011, 22h46
petitours

:ccool:

J'étais sur de mon code C pour le modbus, du coup cette affirmation que c'est bon également là m'a obligé à voir l’évidence : je suis un boulet !
Je ne calculais pas sur le bon buffer d'un coté !!!!! :oops:
3 jours !!

Merci beaucoup