Discussion :

[binome-x]

Bonjour,
la compilation de mon code ne pose pas de problème et j'ai une erreur de segmentation à l’exécution.
Je voudrais simplement modifier un caractère dans une chaîne.
Voilà mon code :

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
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
int clr(char**);
 
int main()
{
	char* t="azerty";
	printf("%s\n",t);
	printf("%d\n",clr(&t));
	return 0;
}
 
int clr(char** ch)
{
	char* str;
	str=*ch;
	str[1] = 'C';
	return 0;
}

Quelqu'un pourrait m'éclairer ?
Merci.

[droggo]

Noe,

La question est : pourquoi passer un type char **, qui est un tableau de chaînes, alors qu'ailleurs tu utilises correctement le classique char* ?

[Bktero]

Le passage d'un pointeur sur pointeur sur char (légèrement différent d'un tableau de pointeur) n'a en effet pas d'intérêt, car tu modifies la zone pointée et non le pointeur lui-même.

Pour une erreur de segmentation ? Car t pointe vers une zone en mémoire constante et que tu ne peux pas y écrire. Remplacer ta déclaration par les deux lignes suivantes pour voir la différence :

Code :

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

1
2
    char* t= malloc(100);
    strcpy(t, "azerty");

[moins1]

Remplacer ta déclaration par les deux lignes suivantes pour voir la différence :

Code :

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

1
2
    char* t= malloc(100);
    strcpy(t, "azerty");

Ou plus simplement par:

Code :

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

char t[] = "azerty";

EDIT: Sans ta fonction avec **

[grim7reaper]

Remplacer ta déclaration par les deux lignes suivantes pour voir la différence :

Code :

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

1
2
    char* t= malloc(100);
    strcpy(t, "azerty");

Voire :

Code C :

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

char t[] ="azerty";

S‘il peut utiliser un tableau à la place de son pointeur.
S‘il a absolument besoin d‘un pointeur, il faut rester sur ta solution.

Au passage, le prototype du main n‘est pas standard (en C++ ok, en C non) :

Code :

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

1
2
3
toto.c:7:5: attention : function declaration isn’t a prototype [-Wstrict-prototypes]
toto.c: In function ‘main’:
toto.c:7:5: attention : old-style function definition [-Wold-style-definition]

Ça devrait être

Code :

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

int main(void)

pas

Code :

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

int main()

En C, une fonction qui prend void en paramètre c‘est TRÈS différent d‘une fonction avec une liste d’arguments vide…

[Médinoc]

Le tableau étant implicitement convertible en pointeur, ceci marcherait:

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
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
int ChangeSecondCaractere(char*);
 
int main(void)
{
	char t[] ="azerty";
	printf("%s\n",t);
	printf("%d\n", ChangeSecondCaractere(t));
	printf("%s\n",t);
	return 0;
}
 
int ChangeSecondCaractere(char* str)
{
	str[1] = 'C';
	return 0;
}

[Bktero]

Si j'ai mis :

Code :

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

1
2
char* t= malloc(100);
    strcpy(t, "azerty");

et non :

Code :

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

char t[] ="azerty";

c'est qu'il y a une grande différence entre les deux. Le premier donne un pointeur vers une zone constante (ou se trouve la chaine littérale) ; le second crée un tableau (donc une zone modifiable) et l'initialiser avec la chaine littérale. Ici, on voulait bien montrer le problème sur un pointeur.

[Médinoc]

Euh, les deux exemples de code de ton message (#7) créent une zone mémoire modifiable initialisée avec "azerty"... La seule différence est que le premier code crée une zone de 100 octets alors que le second code en crée une juste à la taille de la chaîne...

(et bien sûr la différence pile/tas, qu'il faut faire un free() sur le premier pour éviter une fuite de mémoire)

[Bktero]

Oui, bien sûr

Je trouvais juste que ça se rapprochait plus du cas original. Mais je préfère ton code !

[binome-x]

Désolé, un problème matériel m'a empêché de me connecter jusqu'à ce matin.
Je viens de lire tous les messages, je comprends le problème (je dois encore approfondir la question de la gestion de la mémoire). Merci à tous.