Discussion :

[psgman113]

Bonjour,
je fais un ptit programme qui convertit un entier décimal en une chaine de caractès "binaire"

ex: 10 sera convertit en 0000 1010

Voici mon code

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
 
 
 
char  * octet;
char * binaire;
 char * conversion (int decimal);
 
 
int main ()
{
binaire=conversion(10); // je met dans la chaine binaire le résultat de la conversion du 10
}
 
 
 char * conversion (int decimal)
{
// je remplis ici ma chaine octet  en faisant des octet[i]='0'; ou octet[j]='1'; et j'oublie pas //le octet[8]='\0'; pour le dernier caractère
 
return octet // je retourne ma chaine octet

Mais j'ai une erreur segmentation fault..

Merci

[Trap D]

Si tu montrais le code en détail de cette fonction, ce serait beaucoup mieux. Je soupçonne l'utilisation d'un bufer interne à la fonction ...
PS J'ai mal lu, ta chaîne globale octet (ça n'est déjà pas beau) n'est pas initialisée, (c'est encore pire).

[psgman113]

voici mon code

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
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
 
 
 
#include <stdlib.h>
#include <stdio.h>
#include <time.h>
 
int reste;
char  * octet;
char * chaine;
char * conversion (int decimal);
 
 
int main ()
{
chaine=conversion(10);
}
 
 
 char * conversion (int decimal)
{
   int i,j,l;
 
 
    l=128;// l définit le premier seuil 128
 
 
       for (i=0;i<8;i++) // initialisation de la chaine de caractere octet
          {
              octet[i]='0';
          }  
	octet[8]='\0';
 
 
if (decimal != 0)
{
 
 
   reste=decimal%2;
 
  if (reste ==1)
{
      octet[7]='1';
  decimal=decimal-1;
}
 
   else octet[7]='0';   
 
    i=0;
 
 
while ( decimal!=0)
{
      if (decimal<l)
      {
          octet[i]='0';
          l=l/2;
          i++;
      }
 
        if (decimal>l)
      {
          octet[i]='1';
	   decimal=decimal -l;
          l=l/2;
          i++;
      }
 
             if (decimal==l)
      {
          octet[i]='1';
          i++;
          for (j=i;j<8;j++)
          {
              octet[j]='0';
          }    
		octet[8]='\0';
 
                   decimal=0;
      }
}         
 
 
}
 
return octet;
}

[Emmanuel Delahaye]

voici mon code

Ne compile pas...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:7: warning: function declaration isn't a prototype
main.c: In function `conversion':
main.c:31: error: `reste' undeclared (first use in this function)
main.c:31: error: (Each undeclared identifier is reported only once
main.c:31: error: for each function it appears in.)
Process terminated with status 1 (0 minutes, 2 seconds)
3 errors, 1 warnings

Sinon, ce code me parait atrocement compliqué...

Je vais te révéler un Grand Secret : "En informatique, les données sont codées en binaire".

Donc, avec une combinaison astucieuse de décalages (>>, <<) et de masquage (&), tu devrais déterminer si les bits valent 0 ou 1 et afficher '0' ou '1' en conséquence...

[psgman113]

voilà j'ai modifié mes ptits oublis dans ma copie
Ok pour le codage en binaire qui doit être plus facile mais comme j'ai passé un peu de temps,
prenons mon code


En fait à la base je voudrais retourner ce tableau ou cette chaine de caractère à partir d'une fonction,
J'ai utilisé une chaine de caractères car je n'arrive pas à retourner un tableau d'entiers.

merci

[thewho]

Bonjour,

Les variables globales, ce n'est pas très bon.

Où sont les réservations mémoires pour tes pointeurs ?

Sans oublier leur libération, ensuite.

Et comme te l'a dit Emmanuel, côté algorithme, c'est plutôt compliqué.

[Emmanuel Delahaye]

Ok pour le codage en binaire qui doit être plus facile mais comme j'ai passé un peu de temps,
prenons mon code

Pas d'accord. L'essence de l'informatique, c'est pas le codage, qui un jour ou l'autre sera totalement automatisé (ou fait par des indiens, voire des chinois...), mais c'est la recherche du meilleur algorithme. Alors concentre toi plutôt là dessus, parce que c'est la base et c'est là qu'on développe ses connexions neuronales.

Il est inutile de perdre du temps à coder un algorithme faux ou inefficace.

[psgman113]

Merci pour l'aide , j'ai trouvé