Discussion :

[muad'dib]

Bonjour à tous,

N'étant pas surchargé de boulot en ce moment, je suis allé lire quelques articles du dailywtf (nda: des histoires de bourdes réalisées dans le domaine de l'informatique et de la programmtion en général). Et je suis tombé sur cet article:
http://thedailywtf.com/Comments/Mister-Fixit.aspx

Pour résumer, il apparait que la bourde a été de changer le mode d'initialisation des variables. Ce qui auparavant était comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
    int main (int argc, char **argv) 
    { 
      int x;
      char data_string[15];
      ...
      x = 2;
      strcpy(data_string,"data data data");
      ...
    }

a été changé pour ceci

Code :

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

1
2
3
4
5
6
    int main (int argc, char **argv) 
    {
      int x = 2;
      char data_string[15] = "data data data";
      ...
    }

En gros, une initialisation des variables lors de la déclaration.

Je ne vois pas de problème pour ce qui est de l'entier. En revanche, il me semble que la façon de déclarer le tableau de char

Code :

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

char data_string[15] = "data data data";

peut poser problème car alors l'adresse de ce tableau n'est pas mémorisé, et la variable devrait être déclarée en tant que constante et ne devrait pas être modifiée par la suite. (je tiens ça du bouquin "C Unleashed" si ma mémoire est bonne)

Serait-ce donc cela la bourde qui a été commise ?

Voilà si comme moi vous avez un moment dites ce que vous en pensez

[Médinoc]

Pour moi, la bourde, c'est juste d'avoir passé son temps sur des futilités au lieu de chercher la fuite mémoire. Pour le tableau de char, je ne vois pas de problème (et rien ne dit que le tableau ne sera pas modifié par la suite, donc n'ajoutons pas "const" avant d'avoir vu le reste du code).

NOTE: Un intérêt de la seconde écriture, bien sûr, est de pouvoir utiliser const là où il peut être mis. Le problème est juste que ce n'était pas la priorité du jour, selon moi...

[diogene]

Je ne vois pas de problème pour ce qui est de l'entier. En revanche, il me semble que la façon de déclarer le tableau de char

Code :

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

char data_string[15] = "data data data";

peut poser problème car alors l'adresse de ce tableau n'est pas mémorisé, et la variable devrait être déclarée en tant que constante et ne devrait pas être modifiée par la suite. (je tiens ça du bouquin "C Unleashed" si ma mémoire est bonne)

- L'adresse du tableau est la valeur de data_string (dans la majorité des contextes),
- data_string est non modifiable (on ne peut pas écrire data_string =...)
- le tableau est modifiable (comme le dit Médinoc)

Peut être y a t-il confusion avec

Code :

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

char  * data_string = "data data data";

- l'adresse du tableau est dans data_string.
- data_string est modifiable
- le tableau est non modifiable ce qui fait préférer l'écriture :

Code :

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

char const * data_string = "data data data";