Tableau de unsigned char vers DWORD

Version imprimable

Bonjour mon problème de conversion de la journée est le suivant !
Code:

1 2 3 4 5 6 7 8 9 unsigned char aby_user_id[4]; aby_user_id[0] = 0; aby_user_id[1] = 0; aby_user_id[2] = 9; aby_user_id[3] = 9; // pour le convertir en DWORD comme cela DWORD dw_test = 99;
Je vous remercie pour votre aide !

:koi: c'est ultra trivial :koi:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
    unsigned int value;
    unsigned char array_len, digit, ten;
 
// ... array_len = 4
 
    value = 0;
    ten   = 1;
 
    for(digit = array_len; digit > 0; --digit, ten*=10) {
        value += (array[array_len - digit] * ten);
    }
 
    DWORD dvalue;
 
    dvalue = (DWORD) value ; // perte de données
    dvalue = (value & 0x00FFFF); // ou bien

19/12/2016, 13h59
phoenixgreg

Salut foetus,

j'ai un peu de mal à comprendre ton code à vrai dire :s !
20/12/2016, 09h34
foetus

Déroule ma boucle :mouarf: :mouarf:
Tu pars de la dernière case (la 4ième), value vaut 0 et ten vaut 1

1ière itération: value = value + 9 * 1 = 0 + 9 * 1 = 9
On multiplie par 10 ten -> ten = 10

2ième itération: 3ième case

value = value + 9 * 10 = 9 + 9 * 10 = 99
On multiplie par 10 ten -> ten = 100

3ième itération: 2ième case

value = value + 0 * 100 = 99 + 0 * 100 = 99
On multiplie par 10 ten -> ten = 1000

3ième itération: 1ière case

value = value + 0 * 1000 = 99 + 0 * 1000 = 99
On multiplie par 10 ten -> ten = 10000

fin de l'itération -> value 99
20/12/2016, 10h31
phoenixgreg

D'accord c'est plus clair en effet ^^ je remplace "array" par mon tableau de byte si j'ai bien pigé donc ?

Code:

1 2 3 4 5 for(digit = array_len; digit > 0; --digit, ten*=10) { value += (m_aby_user_id[array_len - digit] * ten); } value = 2988 // oups
à partir de la troisième itération mon "ten" = 232, alors qu'on doit attendre 1000.

20/12/2016, 14h30
phoenixgreg
c'est bon, en fait tu partais du principe que le tableau de byte était rempli comme cela :
Code:

1 2 3 4 5 m_aby_user_id[0] = 9; m_aby_user_id[1] = 9; m_aby_user_id[2] = 0; m_aby_user_id[3] = 0;
alors que je le rempli comme ceci :
Code:

1 2 3 4 5 m_aby_user_id[0] = 0; m_aby_user_id[1] = 0; m_aby_user_id[2] = 9; m_aby_user_id[3] = 9;
et en effet je trouve bien value = 99, un grand merci foetus !

Bonjour
Quelque précision: Un DWORD est un entier non signé de 32 bits dont la plage de valeur décimale va de 0 à 4294967295. Et vu que DWORD est t’un type non signé, son premier bit (le bit le plus significatif appelé MSB) n'est pas réservé pour la signature en clair pour les nombres négatifs.

Citation:

Envoyé par foetus

:koi: c'est ultra trivial :koi:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
    unsigned int value;
    unsigned char array_len, digit, ten;
 
// ... array_len = 4
 
    value = 0;
    ten   = 1;
 
    for(digit = array_len; digit > 0; --digit, ten*=10) {
        value += (array[array_len - digit] * ten);
    }
 
    DWORD dvalue;
 
    dvalue = (DWORD) value ; // perte de données
    dvalue = (value & 0x00FFFF); // ou bien

Personnellement, le code que vous avez proposé n’est pas vraiment trivial il est beaucoup trop lourd je propose les instructions ci-dessous qui me semblent beaucoup plus simples à mon sens suivant le résultat souhaité.

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
 
/*
 ============================================================================
 Name        : Untitled.c
 Author      : SAMBIA39
 Version     : 0.1
 Copyright   : Copyright (c) 27/11/2016 SAMBIA39
 Description : Ansi-style
 ============================================================================
 */
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
 
typedef unsigned char BYTE;  // BYTE  = unsigned char 
typedef unsigned long DWORD; // DWORD = unsigned long
 
int main( void ){
 
	int ret_value = 0;
	DWORD convert = 0;
 
	BYTE t_byte[5] = { '0', '0', '9', '9','\0' };   // Tableau unsigned char = BYTE
	sscanf((char*)t_byte, "%ld", &convert );	// Conversion Cas 1 en DWORD
	sscanf((char*)t_byte, "%d", &ret_value);	// Conversion Cas 2 en INT 
	fprintf( stdout, "(Cas 1) -> %ld\n(Cas 2) -> %d\n", 
		convert, ret_value );
 
	return EXIT_SUCCESS;
}

20/12/2016, 16h36
phoenixgreg

Merci pour cette précision sambia39, ton code pourra me servir ! ;)
21/12/2016, 15h22
Médinoc

Attention, ce code est faux car il utilise sscanf() sur des chaînes non-terminées.
21/12/2016, 16h51
fearyourself

Et personne a vu que le bug du code de foetus était que ten était un charactére et non un entier?

:)
21/12/2016, 17h02
sambia39

Bonjour;
Effectivement il y a un oubli de terminaison de chaînes de caractères ‘\0’ ce qui peut causer un débordement. Hormis cet oubli on quoi l’ensemble des instructions sont elle fausse . Pourriez-vous m’éclairer ?
à bientôt
21/12/2016, 17h09
foetus

Citation:

Envoyé par fearyourself

Et personne a vu que le bug du code de foetus était que ten était un charactére et non un entier?

Effectivement :oops: :oops:, mais ce n'est pas un bug parce qu'un char est aussi un entier (j'ai déclaré là où il y avait déjà une variable sans vérifier le type)

Mais il est limité entre 0 et 255 (non signé) ou -127 +128 (signé) (à 1 près, je ne sais jamais).
Donc seulement 10 ou 100 sont valides. Après on a des débordements/ troncatures. :mrgreen:

Édit: On peut garder le char :mrgreen:, mais comme multiplicateur de 10.
21/12/2016, 17h38
fearyourself

C'est quand même un bug ;-) Ou alors une sacrée feature :)