[char * ptr] peut modifer ou peut pas modifier

bonjour.

Voici que je tombe sur un problème cocasse. Pour l'illustrer, j'ai réduit le code au strict minimum. Voici mon code, qui compile sans problèmes sur Ms Visual C++ 6.0

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

#include <string.h>   int main() { char * string1 = "un deux trois quatre"; char string2[21]; strcpy(string2,string1);     string2[0] = 'A'; *string1 = 'A';   return 0; }

---

Mon problème est le suivant: Lors de l'éxécution:
à la ligne:

Code:

---

string2[0] = 'A';

---

ça marche (string2 pointe alors vers une chaîne qui contient "An deux trois quatre"
mais à la ligne:

Code:

---

*string1 = 'A';

---

Le programme plante. ("access violation")

J'aimerais bien comprendre pourquoi?
Est-ce que lorse que j'initialise une chaîne directement "codée en dur" elle va se mettre dans une zone de la mémoire dans laquelle mon programme n'a pas les droits en écriture?

Oui tu es en lecture seule car tu pointes vers la zone de segment data de ton appli au même titre que les constantes par exemple.

Code:

---

1 2 3 4

char string2[21]; strcpy(string2,string1); string2[0] = 'A';

---

Ici tu copies string1 dans string2 qui se trouve dans la pile donc pas de soucis pour écrire 'A'

Citation:

---

Envoyé par Biosox

J'aimerais bien comprendre pourquoi?
Est-ce que lorse que j'initialise une chaîne directement "codée en dur" elle va se mettre dans une zone de la mémoire dans laquelle mon programme n'a pas les droits en écriture?

---

Normal, string1 pointe sur une constante chaine de caractère que tu ne peux pas modifier. C'est préférable de définir string1 de la manière suivante:

Code:

---

const char *string1 = "un deux trois quatre";

---

Par ailleurs, je préfère de manière générale utiliser strncpy au lieu de strcpy. Dans un code aussi simple que celui là où la tailles des chaînes manipulées est connue, cela n'a pas d'importance, mais autant prendre de bonnes habitudes.

Thierry

D'ailleurs, une option de gcc permet de dire que les chaines litterales sont de type "const char []" et non "char []", ce qui t'empeche de "mettre" une chaine litterale dans "char *".

Citation:

---

Envoyé par Gruik

D'ailleurs, une option de gcc permet de dire que les chaines litterales sont de type "const char []" et non "char []", ce qui t'empeche de "mettre" une chaine litterale dans "char *".

---

-Wwrite-strings

Thierry

Et bien merci pour ces précisions!

En effet, strncpy est préférable, mais en ecrivant un petit code à la va-vite, j'ai hlas encore bien des mauvaises habitudes:oops:

(moi qui était tout fièr d'avoir mis un "int main" plutot qu'un "void main" pour pas me faire taper sur les doigts:aie: )

Citation:

---

Envoyé par Biosox

Est-ce que lorse que j'initialise une chaîne directement "codée en dur" elle va se mettre dans une zone de la mémoire dans laquelle mon programme n'a pas les droits en écriture?

---

Oui, c'est possible.

http://emmanuel-delahaye.developpez....es.htm#tabchar

C'est pourquoi je recommande d'utiliser const avec les pointeurs sur chaines littérales.

Avec mon compilateur, ton code est douteux :

Code:

---

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:4: warning: function declaration isn't a prototype main.c: In function `main': main.c:5: warning: initialization discards qualifiers from pointer target type Linking console executable: C:\dev\forums2\console.exe Process terminated with status 0 (0 minutes, 1 seconds) 0 errors, 2 warnings

---

Parce qu'il est réglé pour considérer les chaines comme non modifiables.

http://emmanuel-delahaye.developpez....tm#cfg_compilo

Une fois modifié (const) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

#include <string.h>   int main (void) { char const *string1 = "un deux trois quatre"; char string2[21];   strcpy (string2, string1);   string2[0] = 'A'; *string1 = 'A';   return 0; }

---

l'erreur est détectée :

Code:

---

1 2 3 4 5 6

Compiling: main.c main.c: In function `main': main.c:11: error: assignment of read-only location Process terminated with status 1 (0 minutes, 0 seconds) 1 errors, 0 warnings

---

Je pense que VC++6 en 'warning level 4' devrait voir ça, sinon, voir si il y a des options de compilation 'manuelles' à placer sur la ligne de commande à la main...

Citation:

---

Envoyé par Biosox

Et bien merci pour ces précisions!

En effet, strncpy est préférable, mais en ecrivant un petit code à la va-vite, j'ai hlas encore bien des mauvaises habitudes:oops:

(moi qui était tout fièr d'avoir mis un "int main" plutot qu'un "void main" pour pas me faire taper sur les doigts:aie: )

---

Il s'agit juste de prendre de bonnes habitudes, il n'y a aucun problème avec strcpy dans ce court exemple. L'utilisation de strncpy ne prend pas plus temps... Attention toutefois au fait que si string1 est plus longue que string1, strncpy ne mettra pas '\0' à la fin de string2 (je me suis laissé avoir dernièrement):

Code:

---

1 2 3 4

const char *string1 = "un deux trois quatre"; char string2[10]; strncpy(string2, string1, sizeof(string2)); string2[sizeof(string2)-1] = '\0';

---

Thierry

Citation:

---

Envoyé par mujigka

Code:

---

1 2 3 4

const char *string1 = "un deux trois quatre"; char string2[10]; strncpy(string2, string1, sizeof(string2)); string2[sizeof(string2)-1] = '\0';

---

---

Compliqué...

Code:

---

1 2 3 4

const char *string1 = "un deux trois quatre"; char string2[10]; *string2 = 0; strncat(string2, string1, sizeof string2 -1);

---

Ne pas utiliser strncpy() du tout. Trop dangereux. Le remède est pire que le mal...

Citation:

---

Envoyé par Emmanuel Delahaye

Je pense que VC++6 en 'warning level 4' devrait voir ça, sinon, voir si il y a des options de compilation 'manuelles' à placer sur la ligne de commande à la main...

---

Pas à ma connaissance, et j'ai épluché les warnings de Visual pour tenter de trouver une telle option, en vain...

Citation:

---

Envoyé par Médinoc

Pas à ma connaissance, et j'ai épluché les warnings de Visual pour tenter de trouver une telle option, en vain...

---

Alors passer son code à gcc avant VC++6 est une option... (en mode vérification seule, je ne sais plus quel est le flag)

Citation:

---

Envoyé par Emmanuel Delahaye

Compliqué...

Code:

---

1 2 3 4

const char *string1 = "un deux trois quatre"; char string2[10]; *string2 = 0; strncat(string2, string1, sizeof string2 -1);

---

Ne pas utiliser strncpy() du tout. Trop dangereux. Le remède est pire que le mal...

---

Archivé dans mon guide des bonnes pratiques. Merci Emmanuel!

Thierry