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.

[diogene]

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.

C'est normal, on ne peut pas retourner un tableau.

Il faut SUPPRIMER les variables globales qui sont des ennemies perfides et dangereuses. Le code devrait être du style (indépendamment des problèmes d'algo dans la fonction)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void conversion (int decimal, char * octet );
int main ()
{
 char Octet[9];
 conversion(10, Octet);
}
 
 void conversion (int decimal, char * octet )
{
  int reste;
...

[Emmanuel Delahaye]

Le code devrait être du style (indépendamment des problèmes d'algo dans la fonction)

Comme je dis souvent (un peu trop...) 'inutilement complexe' et 'il manque 'l'essentiel'...

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
 
int conversion (char * octet, size_t size, int valeur)
{
  int err;
  int reste;
 
<...>
 
  return err;
}
 
int main (void)
{
   char chaine[9];
   conversion (chaine, sizeof chaine, 10);
}
 
...

[psgman113]

int conversion (char * octet, size_t size, int valeur)
{
int err;
int reste;

<...>

return err;
}

Pour toi, la fonction conversion retourne un entier??
Elle devrait plutot retourner un pointeur sur une chaine de caractères


et je dois mettre quelque chose devant

conversion (chaine, sizeof chaine, 10);

car conversion retourne quelque chose

Je devrais plutot faire un truk du genre

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
 
 
char * conversion (char * octet, size_t size, int valeur)
{
  char * err;
  int reste;
 
<...>
 
  return err;
}
 
int main (void)
{
char * chaine_finie;
 
 char chaine[9];
chaine_finie=   conversion (chaine, sizeof chaine, 10);
printf("%d",chaine_finie);
}

Après c'est vrai qu'il faut allouer de l'espace

est ce mieux?
Merci

[coca25]

Code :

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

1
2
 
int conversion (char * octet, size_t size, int valeur)

octet est un pointeur vers une chaine de caractere (enfin vers un caratere), en modifiant la valeur pointé, la chaine passe en parametre est modifiee aussi (chaine).

le retour, c'est juste pour indiquer si le traitement s'est bien passé et sinon quelle est l'erreur

[psgman113]

Et comment tu l'affiches la chaine de caractère dans le main?
le int devant la fonction me choque

[Emmanuel Delahaye]

Pour toi, la fonction conversion retourne un entier??

A ton avis 'err', ça veut dire quoi ?

Elle devrait plutot retourner un pointeur sur une chaine de caractères

C'est un choix, mais il conduit à de mauvaises pratiques.

Code :

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

   printf ("%s\n", converstion(chaine, sizeof chaine, -1));

Si il vaut NULL, tu fais quoi ?

Code :

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

Je devrais plutot faire un truk du genre

Oui, mais il faut tester chaine_finie avant de l'utiliser. Et "%d" pour une chaine, ça va pas trop le faire... Manque un '\n'...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int main (void)
{
   char chaine[9];
   char *chaine_finie =  conversion (chaine, sizeof chaine, 10);
   if (chaine_finie != NULL)
   {
      printf ("%s\n", chaine_finie);
   }
   return 0;
}

Il est plus clair de séparer le rôle de chacun et considérer le retour comme un code d'erreur 0=OK 1=ERR.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int main (void)
{
   char chaine[9];
   int err =  conversion (chaine, sizeof chaine, 10);
   if (!err)
   {
      printf ("%s\n", chaine);
   }
   return 0;
}

Comme souvent, les solutions les plus simples sont les meilleures...

[psgman113]

Merci pour l'aide , j'ai trouvé

[Pouic]

qui un jour ou l'autre sera totalement automatisé (ou fait par des indiens, voire des chinois...)

C'est pas la même chose?

[Emmanuel Delahaye]

C'est pas la même chose?

Les indiens et les chinois ? Non. Les indiens sont plus 'software' et les chinois plus 'hardware' (mais ça pourrait changer).

Ils devraient de partager le monde de la production informatique dans les 10 ans qui viennent (avec quelques grosses miettes pour le Brésil).

Pour survivre, les EU et l'Europe doit s'accrocher à leur savoir faire, c'est à dire la définition et la conception.

Le gourou à parlé, vous pouvez reprendre une activité normale...

[Pouic]

Les indiens et les chinois ?
Non. Les indiens sont plus 'software' et les chinois plus 'hardware' (mais ça pourrait changer).

Ils devraient de partager le monde de la production informatique dans les 10 ans qui viennent (avec quelques grosses miettes pour le Brésil).

Pour survivre, les EU et l'Europe doit s'accrocher à leur savoir faire, c'est à dire la définition et la conception.

Le gourou à parlé, vous pouvez reprendre une activité normale...

Nan, je voulais dire "automatiser la chose" et "faire développer par des indiens ou des chinois"

[Emmanuel Delahaye]

Nan, je voulais dire "automatiser la chose" et "faire développer par des indiens ou des chinois"

Je ne me permet pas de comparer les indiens et les chinois à des machines. (Des Cylons Eeek !)