[char * ptr] peut modifer ou peut pas modifier

**Biosox** · 01/12/2006, 14h14

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 :

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

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 :

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

string2[0] = 'A';

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

Code :

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

*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?

**malves** · 01/12/2006, 14h20

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 :

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

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'

**Thierry Chappuis** · 01/12/2006, 14h20

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 :

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

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

**Gruik** · 01/12/2006, 14h22

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 *".

**Thierry Chappuis** · 01/12/2006, 14h24

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

**Biosox** · 01/12/2006, 14h25

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

(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

)

**Thierry Chappuis** · 01/12/2006, 14h33

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

(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

)

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 :

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

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

**Emmanuel Delahaye** · 01/12/2006, 15h07

Envoyé par mujigka

Code :

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

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 :

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

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...

**Emmanuel Delahaye** · 01/12/2006, 14h26

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 :

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: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 :

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

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 :

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

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...

**Médinoc** · 01/12/2006, 15h07

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...