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.