Discussion :

[algerino77]

Bonjour à tous!
Je viens de découvrir se Forum par le biais d'un camarade qui a sollicité votre aide et a obtenu une réponse rapide et très pertinente!

Je vous explique: Je code une calculatrice avec le Lycée.
J'en suis arrivé a convertir du décimal en binaire. La conversion marche parfaitement lorsque je passe du décimal 32 bits au Binaire 16 ou 8 bits.
Par contre lorsque j'essaye de passer du décimal 32 bits au binaire 32 bits j'ai des valeurs fausses..

Précisions:
-Variable bits initialisée a 32.

Voila le code de ma fonction de conversion Décimal vers Binaire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
void DecVersBin()
{
int a=(int)Form1->Edit1->Text.ToDouble();
int i, masque;
 
Form1->Edit1->Clear();
 
        for(i=0;i<bits;i++)
        {
                if(bits==32)
                {
                masque=0x80000000;
                }
                if(bits==16)
                {
                masque=0x8000;
                }
                if(bits==8)
                {
                masque=0x80;
                }
 
                if((a&(masque>>i))==0)
                {
                Form1->Edit1->Text=Form1->Edit1->Text+'0';
                }
                else
                {
                Form1->Edit1->Text=Form1->Edit1->Text+'1';
                }
        }
 
}

Voila le code de ma ComboBox pour choisir les bits:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
void __fastcall TForm1::ComboBox3Change(TObject *Sender)
//OnChange de la ComboBox 32/16/8 bits
{
 
        if(Form1->ComboBox3->Text=="32 bits")
        {
                bits=32;
        }
        if(Form1->ComboBox3->Text=="16 bits")
        {
                bits=16;
        }
        if(Form1->ComboBox3->Text=="8 bits")
        {
                bits=8;
        }
 
 
        if (ComboBox2->Text=="Héxadécimal" && ComboBox3->Text=="32 bits")
        {
                if(bits==16)
                {
                Form1->Edit1->Text="0000"+Form1->Edit1->Text.SubString(1,4);
                bits=32;
                }
                else if(bits==8)
                {
                Form1->Edit1->Text="000000"+Form1->Edit1->Text.SubString(1,2);
                bits=32;
                }
        }
 
        if (ComboBox2->Text=="Héxadécimal" && ComboBox3->Text=="16 bits")
        {
                if(bits==32)
                {
                Form1->Edit1->Text=Form1->Edit1->Text.SubString(5,4);
                bits=16;
                }
                else if(bits==8)
                {
                Form1->Edit1->Text="00"+Form1->Edit1->Text.SubString(1,2);
                bits=16;
                }
        }
 
        if (ComboBox2->Text=="Héxadécimal" && ComboBox3->Text=="8 bits")
        {
                if(bits==32)
                {
                Form1->Edit1->Text=Form1->Edit1->Text.SubString(7,2);
                bits=8;
                }
                else if(bits==16)
                {
                Form1->Edit1->Text=Form1->Edit1->Text.SubString(3,2);
                bits=8;
                }
        }
 
}

Voila le code de ma ComboBox pour choisir Décimal / Hexa / Bin:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
void __fastcall TForm1::ComboBox2Change(TObject *Sender)
//OnChange de la ComboBox Déc/Hex
{
        if (ComboBox2->Text=="Décimal" && base=='H')
        {
                HexVersDec();
                base='D';
        }
 
        /*if (ComboBox2->Text=="Décimal" && base=='B')
        {
                BinVersDec();
                base='D';
        }*/
 
        if (ComboBox2->Text=="Héxadécimal" && base=='D')
        {
                DecVersHex();
                base='H';
        }
 
        /*if (ComboBox2->Text=="Héxadécimal" && base=='B')
        {
                BinVersHex();
                base='H';
        }*/
 
        if (ComboBox2->Text=="Binaire" && base=='D')
        {
                DecVersBin();
                base='B';
        }
 
        /*if (ComboBox2->Text=="Binaire" && base=='H')
        {
                HexVersBin();
                base='B';
        }*/
}

Par exemple pour convertir 9 en binaire j'obtient:
en 32 bits: 00000000000000000000000000001111
en 16 bits: 0000000000001001
en 8 bits: 00001001

Petit Sceen d'ensemble:

Merci d'avance pour votre aide précieuse.

[koala01]

Salut, et bienvenue sur le forum

Désolé, mais sans le code de la fonction DecVersHex, on ne pourras pas grand chose pour toi

Au passage :
Si tu es tout à fait débutant, je te conseillerais de commencer par coder des choses "simples" en console, plutôt que d'utiliser directement la vcl de borland.

C'est peut etre moins sexy, je te l'accorde, mais cela te permettra de prendre directement de bonnes habitudes et de te familiariser avec le C++

C'est d'autant plus vrai que, dés que tu utilises une IHM, elle vient généralement avec sa propre implémentation de tout un ensemble de concepts (à commencer par les chaines de caractères et les différentes collections comme les tableaux) qui ont parfois un comportement assez éloigné de ce qui est utilisable en C++ "pur".

Sinon, il t'intéressera peut être de savoir que les composants de type combobox ont généralement une fonction nommée seletedIndex (ou parfois simplement index!) qui permet d'obtenir une valeur numérique correspondant à la ligne sélectionnée (généralement, la première ligne est représentée par l'index 0, la deuxième par la ligne 1 et ainsi de suite... l'absence de sélection étant représentée par l'index -1)

C'est une valeur numérique, donc cela se compare beaucoup plus facilement, et tu peux même utiliser la syntax des switch...case que je vais te laisser chercher par toi même (où serait le plaisir autrement )

[alyma]

Bonjour,

Alors, quand tu es sur le 32bits, le masque vaut 0x80000000.
Or, quand tu fais le décalage, le compilateur va mettre des 1 devant, car il considère que le nombre est négatif ; c'est pourquoi ton résultat est faux.
Ainsi, il faut que tu déclares la variable masque en non signé.

[grim7reaper]

Or, quand tu fais le décalage, le compilateur va mettre des 1 devant, car il considère que le nombre est négatif

Ou pas.
C’est implementation-defined

The value of E1 >> E2 is E1 right-shifted E2 bit positions. If E1 has an unsigned type or if E1 has a signed type and a non-negative value, the value of the result is the integral part of the quotient of E1/2E2 . If E1 has a signed type and a negative value, the resulting value is implementation-defined.

Ainsi, il faut que tu déclares la variable masque en non signé.

Oui, ça serait déjà un bon point.

[algerino77]

Bonjour,

@koala01

Merci de ta réponse!
Justement, je début en c++ avec l'école sur un projet calculatrice qui utilise le logiciel que tu as cité

@all
int ce n'est pas un type de variable non signé ?
j'ai testé le code d'un ami, ce n'est pas ma fonction DecVersBin qui pause problème car son code ne marchais pas aussi sur mon projet, mais marche sur le siens..

Voila le code de ma fonction DecVersHex

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
void DecVersHex()
{
        Form1->Edit1->Text=Form1->Edit1->Text.IntToHex((int)Form1->Edit1->Text.ToDouble(),8);
}

Voila le code de ma fonction HexVersDec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
void HexVersDec()
{
int i, nb32=0, L=8, intcar;
char car;
        for(i=0;i<8;i++)
        {
                if(Form1->Edit1->Text.c_str()[L-i-1]>=0)
                {
                car=Form1->Edit1->Text.c_str()[L-i-1];
                }
                else
                {
                car='0';
                }
                        if(car<'A')
                        {
                        car=car-'0';
                        }
                        else
                        {
                                if(car<'a')
                                {
                                car=car-'A'+10;
                                }
                                else
                                {
                                car=car-'a'+10;
                                }
                        }
                intcar=car;
                nb32=nb32+(intcar<<(4*i));
        }
        Form1->Edit1->Text=nb32;
}

thx all!

[alyma]

As tu essayé en déclarant ta variable "masque" en unsigned int?

[algerino77]

Merci beaucoup à tous!

C’était bien le unsigned int..
Bizarre car chez mes camarades ça marche sans le unsigned..

Encore un grand merci!

[grim7reaper]

C’était bien le unsigned int..
Bizarre car chez mes camarades ça marche sans le unsigned..

Pas surprenant.
Le résultat est implementation-defined, donc ça va dépendre du compilateur utilisé, éventuellement de sa version et de la plateforme sous-jacente.

[algerino77]

Bon a savoir