Discussion :

[CyrIng]

Code :

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

1
2
3
4
5
6
7
8
9
int main(int argc, char *argv[])
{
char *p="Hello", a[]="Zello";

a[0]='H';
*(p+0)='Z';

return(0);
}

Ok, ça m'a l'air basique et le retourner dans les sens, j'ai une erreur de segmentation sur l'instruction en rouge

[gulain]

Salut,
c'est normal, tu ne peux pas modifier "Hello" (le comportement est non défini).

6.4.5 String literals
Syntax
string-literal:
" s-char-sequenceopt "

If the program attempts to modify such an array, the behavior is undefined.

p est un pointeur vers une chaine de caractères (un string literal ici), alors que a est un tableau de caractères que tu initialises avec "Hello".
Tu peux modifier le contenu de a, mais pas la chaine pointé par p.

[Bktero]

Pour compléter ce que vient de dire gulain, il faut savoir que quand tu fais ça :
char *p="Hello", aucun espace mémoire n'est réservé sur le tas comme ce serait le cas avec malloc, ou sur la pile comme ce serait le cas avec le tableau a. La chaine "Hello" est mise dans une zone spécifique de la mémoire où sont placées les constantes. Tu ne peux donc pas modifier les données pointées par p, bien que tu puisses modifier p lui-même.

Mon échange avec gangsoleil dans cette discussion pourra t'intéresser : http://www.developpez.net/forums/d12...n/#post6770934

PS : char *p="Hello" est une allocation statique : http://fr.wikipedia.org/wiki/Allocation_de_m%C3%A9moire

[CyrIng]

Merci Messieurs pour ces réponses qui sont fondées.
Je sentais bien l'effet "Constante"

Aussi, je dois revoir un aspect d'un plus gros programme dans lequel un tableau de pointeurs de char est initialisé avec des chaînes littérales.
Certaines de ces chaînes sont dynamiquement modifiées.

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
#define LEVELS  10

#define MENU_PROMPT_ROOT "[E]xit    [H]elp       [S]how         [D]ump ___________________"
...
#define MENU_PROMPT_MEMY "[<]Back   [A]rray ____________________________"

#define LV_ROOT 0
...
#define LV_ATTY 8
#define LV_STRY 9


char *menuPrompt[LEVELS]=
{
MENU_PROMPT_ROOT,
...
MENU_PROMPT_MEMY,
MENU_PROMPT_MEMY
};

menuPrompt[LV_ATTY][23]='[';

Bon pour du malloc, dommage je l'aimais bien cette souplesse d'assignation

[CyrIng]

Résolu par :

Code :

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

1
2
3
4
5
6
7
char menuPrompt[LEVELS][66]=
{
MENU_PROMPT_ROOT,
...
MENU_PROMPT_MEMY,
MENU_PROMPT_MEMY
};

En revanche, cela implique une longueur = 66 char pour toutes les littérales, voire > 66 avec warning du compilateur.
Et < à 66, ça tappe dans l'inconnu

[Bktero]

Cela implique une longueur inférieure ou égale à 65 (66 avec le caractère de fin de chaine), plutôt ?

[CyrIng]

Pour compléter ce que vient de dire gulain, il faut savoir que quand tu fais ça :
char *p="Hello", aucun espace mémoire n'est réservé sur le tas comme ce serait le cas avec malloc, ou sur la pile comme ce serait le cas avec le tableau a. La chaine "Hello" est mise dans une zone spécifique de la mémoire où sont placées les constantes. Tu ne peux donc pas modifier les données pointées par p, bien que tu puisses modifier p lui-même.

Mon échange avec gangsoleil dans cette discussion pourra t'intéresser : http://www.developpez.net/forums/d12...n/#post6770934

PS : char *p="Hello" est une allocation statique : http://fr.wikipedia.org/wiki/Allocation_de_m%C3%A9moire

J'ai lu cette discussion, merci ; et sans vouloir insister cela fut il toujours le cas ?
En effet, j'ai souvenir du compilateur Turbo C qui autorisait ce genre de réécriture !

[gulain]

J'ai lu cette discussion, merci ; et sans vouloir insister cela fut il toujours le cas ?
En effet, j'ai souvenir du compilateur Turbo C qui autorisait ce genre de réécriture !

Le comportement n'est pas défini, ça veut dire que le compilateur fait ce qu'il veut, en particulier il peut faire en sorte que tu puisses modifier ces chaînes de caractères (c'était peut-être le cas de Turbo C, je n'en sais rien).

[pythéas]

Sinon il y a strdup de la bibliothèque string.h qui fait à la fois l'allocation mémoire et l'assignation

Si tu remplaces

Code :

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

char *p="Hello"

par

Code :

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

char *p=strdup("Hello")

tu n'auras plus ton erreur de segmentation

Cette fonction n'est pas standard, mais malgré tout assez portable puisque POSIX.1.