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.

[CyrIng]

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.

Merci, je la garde sous le coude; mais depuis les heures ont coulé et par rage j'ai tout changé pour des tableaux à taille fixe
En fait je souhaitais définir dans le header file, une matrice de hauteur et de largeur résolues par le compilateur, avec surtout pour chaque ligne, un nombre variable de colonnes.

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
#include <stdio.h>
int main(int argc, char *argv[])
{
    int  x=0, y=0;
 
    char *M[]=
    {
        "1234567890$",
        "123$",
        "abcdefgh$",
        "%"
    };
 
    while(M[y][x]!='%')
    {
        while(M[y][x]!='$')
        {
            printf("%c",M[y][x]);
            x++;
        }
        printf("\n");
        x=0;
        y++;
    }
    return(0);
}

Où '$' et '%' sont les délimiteurs scannés par les boucles.

[CyrIng]

Merci, je la garde sous le coude; mais depuis les heures ont coulé et par rage j'ai tout changé pour des tableaux à taille fixe
En fait je souhaitais définir dans le header file, une matrice de hauteur et de largeur résolues par le compilateur, avec surtout pour chaque ligne, un nombre variable de colonnes.

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
#include <stdio.h>
int main(int argc, char *argv[])
{
    int  x=0, y=0;
 
    char *M[]=
    {
        "1234567890$",
        "123$",
        "abcdefgh$",
        "%"
    };
 
    while(M[y][x]!='%')
    {
        while(M[y][x]!='$')
        {
            printf("%c",M[y][x]);
            x++;
        }
        printf("\n");
        x=0;
        y++;
    }
    return(0);
}

Où '$' et '%' sont les délimiteurs scannés par les boucles.

Maintenant on y ajoute la réécriture et " Segmentation fault "

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
#include <stdio.h>
int main(int argc, char *argv[])
{
    int  x=0, y=0;

    char *M[]=
    {
        "1234567890$",
        "123$",
        "abcdefgh$",
        "%"
    };

    while(M[y][x]!='%')
    {
        while(M[y][x]!='$')
        {
            printf("%c",M[y][x]);
            x++;
        }
        printf("\n");
        x=0;
        y++;
    }
    M[2][3]='C';
    return(0);
}

[stroke]

C'est le même problème sauf que dans ce cas au lieu d'avoir un pointeur vers une chaine de caractères, tu as un tableau de pointeurs vers des chaines de caractères.
Tu essaies donc toujours de modifier des chaines constantes.

[CyrIng]

J'ai bien compris. Thanks

J'ai adapté en conséquence pour achever mon soft CoreMod : lecture des registres du Core i7 et de la DMI BIOS

Les sources sont dans mon Wiki et des copies d'écran dans le blog

Maintenant au dodo